From patchwork Mon Feb 17 06:33:31 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Shuai Xue X-Patchwork-Id: 13977129 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id DA7F3C021A0 for ; Mon, 17 Feb 2025 06:33:47 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 071AA280031; Mon, 17 Feb 2025 01:33:47 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id F0640280030; Mon, 17 Feb 2025 01:33:46 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id DFFE2280032; Mon, 17 Feb 2025 01:33:46 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0017.hostedemail.com [216.40.44.17]) by kanga.kvack.org (Postfix) with ESMTP id C3788280030 for ; Mon, 17 Feb 2025 01:33:46 -0500 (EST) Received: from smtpin12.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay08.hostedemail.com (Postfix) with ESMTP id 50203141BA9 for ; Mon, 17 Feb 2025 06:33:46 +0000 (UTC) X-FDA: 83128470852.12.F6410B9 Received: from out30-119.freemail.mail.aliyun.com (out30-119.freemail.mail.aliyun.com [115.124.30.119]) by imf07.hostedemail.com (Postfix) with ESMTP id 3F49B40002 for ; Mon, 17 Feb 2025 06:33:42 +0000 (UTC) Authentication-Results: imf07.hostedemail.com; dkim=pass header.d=linux.alibaba.com header.s=default header.b=FlvGBTqh; spf=pass (imf07.hostedemail.com: domain of xueshuai@linux.alibaba.com designates 115.124.30.119 as permitted sender) smtp.mailfrom=xueshuai@linux.alibaba.com; dmarc=pass (policy=none) header.from=linux.alibaba.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1739774024; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=C//AWNBmkoQxyk9EmxC2ITDqwThmL3f05NW0jHto79U=; b=1NOjkSUF8L+EFeBhZNn7Vm3r2S13lV3g90qAU4U9ND0FwcgpOeoYvZ150Z+EHQ03fLpSLj K9OnZ/GJlYL8eza9KoarsUOnWDVETCZTFt5WSSzFSPRhU0GiyUO3KY4IrWnYaH0+vexswt /BqHvqsSq6n2a7Xpd+aAp80OpVed2dk= ARC-Authentication-Results: i=1; imf07.hostedemail.com; dkim=pass header.d=linux.alibaba.com header.s=default header.b=FlvGBTqh; spf=pass (imf07.hostedemail.com: domain of xueshuai@linux.alibaba.com designates 115.124.30.119 as permitted sender) smtp.mailfrom=xueshuai@linux.alibaba.com; dmarc=pass (policy=none) header.from=linux.alibaba.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1739774024; a=rsa-sha256; cv=none; b=HzAbXr2QY6jrnYEnZfIeAT8M8nCSP6b3Uqk+40a2SSmHnTHyNo2OyzpevuUDNG/Vuvx+bJ aub8w1ESjJYy1B3eybQqjYJwTD4IpJKkHYVhhb6vIrKr9/YZ0HlwhNmmyBcd47TxNMaj0z RgM4JU0EWqgQrURwTK3vycmvl/8p2XE= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.alibaba.com; s=default; t=1739774020; h=From:To:Subject:Date:Message-ID:MIME-Version; bh=C//AWNBmkoQxyk9EmxC2ITDqwThmL3f05NW0jHto79U=; b=FlvGBTqh6c9aFIDwnAtQaUDwxcIzWCEpU0Ryver0NEwZDfggbH0YPUZInCtpm2MUoc33F/x83R//Rzszv0S22curvnB+dRnFaXR1l9nVS1Fs30O3eD50gdm4VbWHswdrSzpbBf8qSXw+Wi6IMXuLOBI3bfss65oreJua855kFxE= Received: from localhost.localdomain(mailfrom:xueshuai@linux.alibaba.com fp:SMTPD_---0WPb1mOT_1739774017 cluster:ay36) by smtp.aliyun-inc.com; Mon, 17 Feb 2025 14:33:38 +0800 From: Shuai Xue To: tony.luck@intel.com, bp@alien8.de, nao.horiguchi@gmail.com Cc: tglx@linutronix.de, mingo@redhat.com, dave.hansen@linux.intel.com, x86@kernel.org, hpa@zytor.com, linmiaohe@huawei.com, akpm@linux-foundation.org, peterz@infradead.org, jpoimboe@kernel.org, linux-edac@vger.kernel.org, linux-kernel@vger.kernel.org, linux-mm@kvack.org, baolin.wang@linux.alibaba.com, tianruidong@linux.alibaba.com Subject: [PATCH v2 1/5] x86/mce: Collect error message for severities below MCE_PANIC_SEVERITY Date: Mon, 17 Feb 2025 14:33:31 +0800 Message-ID: <20250217063335.22257-2-xueshuai@linux.alibaba.com> X-Mailer: git-send-email 2.44.0 In-Reply-To: <20250217063335.22257-1-xueshuai@linux.alibaba.com> References: <20250217063335.22257-1-xueshuai@linux.alibaba.com> MIME-Version: 1.0 X-Rspam-User: X-Rspamd-Server: rspam11 X-Rspamd-Queue-Id: 3F49B40002 X-Stat-Signature: n74m5zr4658kmfq8rrynec7p19p6i3ek X-HE-Tag: 1739774022-968020 X-HE-Meta: U2FsdGVkX1+XjXbFfvjkuRF7rCUtI152e8yrLbMDIMqNCGzh+3d1Mu9mR7CgnWjrRvicIPc5HY4Zf6lik1iKTNKF4WULDfiWpMLZrzTfDgdqWQKryGne0kJDmwH3+q8tgc2R1PW1qLnYsZnXu7viY5Y+/nRZcxjexFoB9IVVjugHuHLo60cbLuqmQ77SOAEZ9/9Jrc6l2SAqueC0W3G/YNvFp/NstA8Xrfuzzhi+9TM66Mr9BS8QPEpteR4zBtuks2QQ61mNxDjGNz/rk09J9pnmpQ0JHLYGIAONxVszyIH51yylRq8D4mzD5TlYlq9KH4lhaVmD+c1xhvy9qByl3UAM8fxIgkvX6j5OswFTtHAYZzdrsAt1usVqaaHWklnnFcibo7zu4XnzgjaJRWnJgOsOWROnNdJ0EmoqzIu6OGUDuy2W8dvmXfs0Pdmxea5ViB41do9eGMFOxSh5NcMAWrlzuwLVuM+6Ci4rIBDx3YEtV/EgYY1L41/JYtqDd9hZUFTvPxAVxYJRjOApcKz6ExU3WZOTxddpvM6D9Ump7wSwvv1ERPKx7K7BnB6yudmgVYgAyDawX2VU1eiPAUc2WLfF+zB2yFroCrbUBgf/deBCgU0RiSxVUePlt70aZvwk4mCHYuG7rjGDctCIjMiBikCuZiHMgFMkDR3dupSbwo90q1b7pu85d6Peg46MnRljKNVvlpxcmoNTAV3LDa+112ttNr11EnVpOALADdt099JPkoB0jCNMVacRh1dCafv2duJoX9fmoTfvjhwCVg2oZEGQZjM5Bpx63+BTLky9tktT6seSIQckrZnrilPt2FRqIEO/v+32/Sp6DsiZ3v1wlhqomVz1uokkvdhjZaBB76G6E0MV/dQ95KLmUbT38RUCijvAnczjTaHfbjw/jEox484Sv9Mmg+eh4+uQkbD1Hs1krqjZZP3KhOvVffapqYeQc7TqHxEyRSmRQIwK6di 1lorGchM R2FaTdINLWTpMW0I1Ye9mxunRa+aBZZ8gzR2tL9p1wUd/4+J+U7CJKs0NpBj3iOZ2W35qCr6VnrLCP1n1RjTf1x1Wpw4RoRZEPxFY8yv+MS/vy2hwLgszGHVU91GPowGBxz8ixkbBUU3BXiXGv9ABGYLlt1oUjwbnKwg+wRRUeCw+dXnCIesrWpN8Ogjix7+GjUEDaWnaqRofB9ZUaaypR5e2aL8AmNaq6Gr74zZjSVPmLi4WV5cs0PYRJPHM3/KmejBmLttAe4lE75WVBZ/lUNKkO6t0mXOnq+KOPdxT4/6opvM4N9ae8aQYL1c5Ad7fHre6p2NRhbiv9FdGKbA+g9p2AcplbvxaY2ij X-Bogosity: Ham, tests=bogofilter, spamicity=0.004170, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: Currently, mce_no_way_out() only collects error messages when the error severity is equal to `MCE_PANIC_SEVERITY`. To improve diagnostics, modify the behavior to also collect error messages when the severity is less than `MCE_PANIC_SEVERITY`. Signed-off-by: Shuai Xue --- arch/x86/kernel/cpu/mce/core.c | 17 +++++++++++------ 1 file changed, 11 insertions(+), 6 deletions(-) diff --git a/arch/x86/kernel/cpu/mce/core.c b/arch/x86/kernel/cpu/mce/core.c index 0dc00c9894c7..f2e730d4acc5 100644 --- a/arch/x86/kernel/cpu/mce/core.c +++ b/arch/x86/kernel/cpu/mce/core.c @@ -925,12 +925,13 @@ static __always_inline void quirk_zen_ifu(int bank, struct mce *m, struct pt_reg * Do a quick check if any of the events requires a panic. * This decides if we keep the events around or clear them. */ -static __always_inline int mce_no_way_out(struct mce_hw_err *err, char **msg, unsigned long *validp, - struct pt_regs *regs) +static __always_inline bool mce_no_way_out(struct mce_hw_err *err, char **msg, + unsigned long *validp, + struct pt_regs *regs) { struct mce *m = &err->m; char *tmp = *msg; - int i; + int i, cur_sev = MCE_NO_SEVERITY, sev; for (i = 0; i < this_cpu_read(mce_num_banks); i++) { m->status = mce_rdmsrl(mca_msr_reg(i, MCA_STATUS)); @@ -945,13 +946,17 @@ static __always_inline int mce_no_way_out(struct mce_hw_err *err, char **msg, un quirk_zen_ifu(i, m, regs); m->bank = i; - if (mce_severity(m, regs, &tmp, true) >= MCE_PANIC_SEVERITY) { + sev = mce_severity(m, regs, &tmp, true); + if (sev >= cur_sev) { mce_read_aux(err, i); *msg = tmp; - return 1; + cur_sev = sev; } + + if (cur_sev == MCE_PANIC_SEVERITY) + return true; } - return 0; + return false; } /* From patchwork Mon Feb 17 06:33:32 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Shuai Xue X-Patchwork-Id: 13977130 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id C0E9CC021A6 for ; Mon, 17 Feb 2025 06:33:48 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 320FA280030; Mon, 17 Feb 2025 01:33:47 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 23633280033; Mon, 17 Feb 2025 01:33:47 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 06005280032; Mon, 17 Feb 2025 01:33:47 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0012.hostedemail.com [216.40.44.12]) by kanga.kvack.org (Postfix) with ESMTP id D81FB280031 for ; Mon, 17 Feb 2025 01:33:46 -0500 (EST) Received: from smtpin15.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay06.hostedemail.com (Postfix) with ESMTP id 5AD0AB2C87 for ; Mon, 17 Feb 2025 06:33:46 +0000 (UTC) X-FDA: 83128470852.15.EA18246 Received: from out30-118.freemail.mail.aliyun.com (out30-118.freemail.mail.aliyun.com [115.124.30.118]) by imf03.hostedemail.com (Postfix) with ESMTP id 25D1420002 for ; Mon, 17 Feb 2025 06:33:43 +0000 (UTC) Authentication-Results: imf03.hostedemail.com; dkim=pass header.d=linux.alibaba.com header.s=default header.b=ENRb7n9t; spf=pass (imf03.hostedemail.com: domain of xueshuai@linux.alibaba.com designates 115.124.30.118 as permitted sender) smtp.mailfrom=xueshuai@linux.alibaba.com; dmarc=pass (policy=none) header.from=linux.alibaba.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1739774024; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=bq0U27NFG8TdcE9f/ikRqDiyjfIWEAdUTwz5YbwhJro=; b=d15v/6fRIpnnfcLeFKfoM18N8ETp5hHKlxWdsLFBhmQ4a2hmufG0SjamfModZqUjVatIVT Qz9x9TGxjrrD6RysMQtcqUJ9QqVt3YZj71Kexafkv7UX/X1EvIlepM3vKvqYFvwGiGU3KL vL3f6Xu4DaWPZ2OhsaK4rqv7lQefCGU= ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1739774024; a=rsa-sha256; cv=none; b=z28HTE7NSp1wXohOT5B5Vxmfv+UD+tY3cnrvoe6tjgdQy7xXvxuGFMuNkPNsjSPgsnjtpu nq6D5Za6tbJsUVjVXmzsNjjYIzMEjTtqifFxgzvqmomOTSPpZQCg5Nj8R+XnyQ6FIdIgCV 2gZZ4Er9iyVO6Wsl0V08Jp4rXFpDZCM= ARC-Authentication-Results: i=1; imf03.hostedemail.com; dkim=pass header.d=linux.alibaba.com header.s=default header.b=ENRb7n9t; spf=pass (imf03.hostedemail.com: domain of xueshuai@linux.alibaba.com designates 115.124.30.118 as permitted sender) smtp.mailfrom=xueshuai@linux.alibaba.com; dmarc=pass (policy=none) header.from=linux.alibaba.com DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.alibaba.com; s=default; t=1739774020; h=From:To:Subject:Date:Message-ID:MIME-Version; bh=bq0U27NFG8TdcE9f/ikRqDiyjfIWEAdUTwz5YbwhJro=; b=ENRb7n9tPtV/73UpqK6T5i29B/2WzR1UDrClsvAX3orgiIDTyfPkBQaCMU+B/rOoe8CnGpvfzcCzqKX+MaOlNmhEwOHw2xVA3BTfHJzoNSBVeyScsyaGaS34p/WzbNEi+I1PaDI4N5ejcRUvw59zBbTCs+/sEiOZk5A1mnBpXKw= Received: from localhost.localdomain(mailfrom:xueshuai@linux.alibaba.com fp:SMTPD_---0WPb1mPM_1739774018 cluster:ay36) by smtp.aliyun-inc.com; Mon, 17 Feb 2025 14:33:39 +0800 From: Shuai Xue To: tony.luck@intel.com, bp@alien8.de, nao.horiguchi@gmail.com Cc: tglx@linutronix.de, mingo@redhat.com, dave.hansen@linux.intel.com, x86@kernel.org, hpa@zytor.com, linmiaohe@huawei.com, akpm@linux-foundation.org, peterz@infradead.org, jpoimboe@kernel.org, linux-edac@vger.kernel.org, linux-kernel@vger.kernel.org, linux-mm@kvack.org, baolin.wang@linux.alibaba.com, tianruidong@linux.alibaba.com Subject: [PATCH v2 2/5] x86/mce: dump error msg from severities Date: Mon, 17 Feb 2025 14:33:32 +0800 Message-ID: <20250217063335.22257-3-xueshuai@linux.alibaba.com> X-Mailer: git-send-email 2.44.0 In-Reply-To: <20250217063335.22257-1-xueshuai@linux.alibaba.com> References: <20250217063335.22257-1-xueshuai@linux.alibaba.com> MIME-Version: 1.0 X-Rspam-User: X-Rspamd-Queue-Id: 25D1420002 X-Rspamd-Server: rspam07 X-Stat-Signature: xm16c3wzpyfkykhgjzxaamhn9sp89m4x X-HE-Tag: 1739774023-496608 X-HE-Meta: U2FsdGVkX19fpqyiWyGInUO4Pc8LUSYT/4q4ScFWemBs91VOlZoUA3ZUgVvvEedOWo40piHIdMYbmOrjXKQhttUzy4IkphsBhDeq5cT1gU5w1kmHsoFDtY3hUd/S0G3TUzu1TejVp6SyJB4yv+jiG/w5AvO6SvKK+yggPuQ8Nzdggc9YcNsPHz5rknLLff/V8DenRnKDNInoURLFuyp3nqiS/faz29sRUzx2jlN07cTzhWcIwHXUp7dVp9vlytsGDXLLqaU/NKs+uYQknWV/cr3ZvXvpwz5If2wNWx7cokzmF5PK6gU1p7W4ir9DB9EFFIBqqA9ftr0eqIhUpOvMdY+6jzHsK+RPEQ04ZX6pPeWLXxVAWc8Qc/jUo061nip1OD8d8GI7wQEa50VXDyRkWWbcp8MPlATHO6VGyxw1LsFnZLcATQ4e4aEtOj3kq7PE28+OrZsWaqmVVrgtNS4AVcpV0u95V5ZVbxGL8YJv1Om69KKs0Gc9Q4OjfGFa3CAvsegD/wPJ0JeeRXHHTYDqoNs4lhBdV8UxrCqiwq7G7AIYzu0qW3TiPgpsITSkWFBQvyCtrQrtYB96lVVxIl4eUqbYWxN0zHdNuABqtBV+9NOMkU/oyuQdh33z3MOJ2peJRf4yZnR8Os9t2AaQ4qAjyON8fXM0jDGKcjSf6EHy83Lmb+ryqqcygiJ27pcoYU3RCcT+bEbg6nmbSbb1wp3K6jNYmApuLmsp0PSca29svzV1iqqQb7x2bBHMvX0PHv5rhFgjUngLzcVSX8zkkH58sow5LWzoK0G8Aq9AToevUWFcY3C741ddCKI8qbnrGJ/UiwX1wIojLsGUb6eXcrqZ/LftNyCVQuEJcnU2CB2CKeh2Fskqh0S9Dq0f8MB4Kbete2UUjp6Cl2k3rUe+yoxs/jgLlo1RQcoLBygELRHQrdVkmXScLQ/5xYQcSnyJMgGhCCROR9BB+wgPN+B3Zds 8dGwHeXr 5Aw+Df5kBgg4cwftdP1H7vMk3oDOtd9lTbChGmI70vfvhyTUKCbnc3br4axCqMQ4Am4XG4dvzxsP+Q64X2TdnXCZFxuGQGU2cFUd51PBwjOX0nfVV1mtNlsFLiT0yBNYIOP/DuUZMXnd8BiGV5ULG78zNQsmro3PFZiIOCSXZQWlCzRMc220Nm5PnyWBHXs0LXuQiKtngbq5vfJeJpt6ceKOUQ0luxhQ3rW2duusGHpMRlkXkz9EF2yvuweNlriyxtTm8c4O1vjsQY6DLTlXaL1eC0Z9koFTaC0w0o+gOQcpzSzs= X-Bogosity: Ham, tests=bogofilter, spamicity=0.000017, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: The message in severities is useful for identifying the type of MCE that has occurred; dump it if it is valid. Signed-off-by: Shuai Xue --- arch/x86/kernel/cpu/mce/core.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/arch/x86/kernel/cpu/mce/core.c b/arch/x86/kernel/cpu/mce/core.c index f2e730d4acc5..c1b945dbccdf 100644 --- a/arch/x86/kernel/cpu/mce/core.c +++ b/arch/x86/kernel/cpu/mce/core.c @@ -1456,6 +1456,8 @@ static void queue_task_work(struct mce_hw_err *err, char *msg, void (*func)(stru if (count > 1) return; + if (msg) + pr_err("%s\n", msg); task_work_add(current, ¤t->mce_kill_me, TWA_RESUME); } From patchwork Mon Feb 17 06:33:33 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Shuai Xue X-Patchwork-Id: 13977132 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 6D9E1C021A0 for ; Mon, 17 Feb 2025 06:33:53 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 766A4280034; Mon, 17 Feb 2025 01:33:49 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 7173C280032; Mon, 17 Feb 2025 01:33:49 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 5DFBB280034; Mon, 17 Feb 2025 01:33:49 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0012.hostedemail.com [216.40.44.12]) by kanga.kvack.org (Postfix) with ESMTP id 3E6DA280032 for ; Mon, 17 Feb 2025 01:33:49 -0500 (EST) Received: from smtpin09.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay03.hostedemail.com (Postfix) with ESMTP id D59B5A1B2A for ; Mon, 17 Feb 2025 06:33:48 +0000 (UTC) X-FDA: 83128470936.09.4A105C0 Received: from out30-132.freemail.mail.aliyun.com (out30-132.freemail.mail.aliyun.com [115.124.30.132]) by imf10.hostedemail.com (Postfix) with ESMTP id 8E0C4C0010 for ; Mon, 17 Feb 2025 06:33:46 +0000 (UTC) Authentication-Results: imf10.hostedemail.com; dkim=pass header.d=linux.alibaba.com header.s=default header.b=ceuTCBZS; spf=pass (imf10.hostedemail.com: domain of xueshuai@linux.alibaba.com designates 115.124.30.132 as permitted sender) smtp.mailfrom=xueshuai@linux.alibaba.com; dmarc=pass (policy=none) header.from=linux.alibaba.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1739774027; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=nVZg/Xi0EFgJwO78HVQQzmH5TmzN10gdAW21xjQG3r4=; b=DKD8KIJR6ziBX2AJuSyrcM+o6rKbddyvhHEk/G+Hz2p0VkkZ5DB6Ru85q3IOl/uX3xnUGa 5vmqdLbspdQoymWQzOE3hwzv+RT3xL04OBvDZBXwBm6t7loG/zTa51HKpE2CUqan7xVo6b j0gqJimyEdBV+bM9KVC2aZzT2101HEQ= ARC-Authentication-Results: i=1; imf10.hostedemail.com; dkim=pass header.d=linux.alibaba.com header.s=default header.b=ceuTCBZS; spf=pass (imf10.hostedemail.com: domain of xueshuai@linux.alibaba.com designates 115.124.30.132 as permitted sender) smtp.mailfrom=xueshuai@linux.alibaba.com; dmarc=pass (policy=none) header.from=linux.alibaba.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1739774027; a=rsa-sha256; cv=none; b=NnuTsluOmU4K1JuR57Jl9NMfZ/fZzylGtwhIFcHoS7Tez6j5QQ8CQmmls23VJULnx2j8b+ feq5GS10Ew2OZJshklQEnXtNC41zkBWhwS1lx/OOTYU25LKbI/45cYbp07H4zmg+Yydbg5 +k+juogpTuqTgycyhQEmknyTypKjwKk= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.alibaba.com; s=default; t=1739774022; h=From:To:Subject:Date:Message-ID:MIME-Version; bh=nVZg/Xi0EFgJwO78HVQQzmH5TmzN10gdAW21xjQG3r4=; b=ceuTCBZSvIcvKrK8/rvcF6MEAimkrZA/RXC5ONxzaRWFXxn5kctpxXX/oJz6KfYXH0PzHolPRxkt9F64OKdrQ5yyiNd+cBK41i6azKYcdHNFeXQsCcm0dGUM0TIEkaRyC0OfBvH8LPujGxNJjsfSpeSPoXzKfsrECP76Er8CbqM= Received: from localhost.localdomain(mailfrom:xueshuai@linux.alibaba.com fp:SMTPD_---0WPb1mQ2_1739774019 cluster:ay36) by smtp.aliyun-inc.com; Mon, 17 Feb 2025 14:33:40 +0800 From: Shuai Xue To: tony.luck@intel.com, bp@alien8.de, nao.horiguchi@gmail.com Cc: tglx@linutronix.de, mingo@redhat.com, dave.hansen@linux.intel.com, x86@kernel.org, hpa@zytor.com, linmiaohe@huawei.com, akpm@linux-foundation.org, peterz@infradead.org, jpoimboe@kernel.org, linux-edac@vger.kernel.org, linux-kernel@vger.kernel.org, linux-mm@kvack.org, baolin.wang@linux.alibaba.com, tianruidong@linux.alibaba.com Subject: [PATCH v2 3/5] x86/mce: add EX_TYPE_EFAULT_REG as in-kernel recovery context to fix copy-from-user operations regression Date: Mon, 17 Feb 2025 14:33:33 +0800 Message-ID: <20250217063335.22257-4-xueshuai@linux.alibaba.com> X-Mailer: git-send-email 2.44.0 In-Reply-To: <20250217063335.22257-1-xueshuai@linux.alibaba.com> References: <20250217063335.22257-1-xueshuai@linux.alibaba.com> MIME-Version: 1.0 X-Rspam-User: X-Stat-Signature: a77k6nq9feduqusnff1131q77ydz1wrz X-Rspamd-Server: rspam05 X-Rspamd-Queue-Id: 8E0C4C0010 X-HE-Tag: 1739774026-608705 X-HE-Meta: U2FsdGVkX19uUZtbdoP4fZTa5x/edT/tnF7MHAwYcamN+lCu6KL3/OCwiTvFzYw7KSJN31n7NOb97y3ZAP/+MXDhc00DgvnZN7zWFGEGI/3lLinoeu3CUrPeUpf0pdRh195AskIMER2IgT9X8LX5Oh3eZck3h+CyikblE6bdM4UuYqGc46ILQX4K5Ck42GwtXHybO5Cse+lCnBORU+Kda370IzaC99xilad8qcP/QRe2oxOe77opEwbkkDj1UEGZBzcovtaZBG/WEmHQymURq6rQ8cftCXhmDqpVCiBQr/8/Rmev9bXP8Q7ax8UzKX1dUgDu0L/FWvxhWQRG1JOtui3ceQoEZCSlVex5RazzawZNRoif+HFs5facBEQN53gqG3c7UQmHQnqoMCoypQ69gNrZVLweq+0Pdm5j95GRFZ5S93jF4L/dgN2QKTJdGg24DV1Mj+SvOB1kpn/HmovQM89Z3wbEpW9pFN4LIWqReq0dQ4Q5s3COW77khEHWlzp2gmObSbKAFgIlpOuRNcMHN/U4yo2hUVxx43fSGVwPh26dETPkwTm3zS3iPiW6Xlj9IhXw8k7vH3Gc9h09iYPPYyzs0nc/4BNHO3lYp+1PqHScaWKm01FxNZ0EpJILTDbNAnQqdRXGZfLux9+Hmerw6v8ljVzlb5pjH+u55Udf8BWhUlQVGMxDLkULfkBw1mSOfWY9FxLkzCXSXaSwgJdQGURmBmAp/upLleAghFEJh5nLEscD67uc1Kohp2PFcAyN20igMfqzUIqljXQAZfTOiAPM2schFccRo87fpP0Vly+BszeklC1zqGoZUVvrVGqqBLugWXXvrnBCGIcqBzuPnZSFJql2Pp3Bmj1CXrb4Z2/A6joVcvWpty25Iv78Ew8tBvzsuK0kwcGUNJnvNfwf1ep7vrZQpiDGt5cbFmXLQrA0laOKnA4xbdQRKUiPP9BMWv+9thczX0OPHL/vjuV BYzlODbG dL2LkEIT08kVKBLF+w7ldlBzTp9jtdfafCfpRBOSS+VyfYk+uyZ8UYo0kuGiUgtUYvgbWx09fKEo/L+k6StWkJrrE8BKht8y4bedqXVdQoFTKWUmk8wZSNLBf3MZm0YidOFartGC3FyCbwbYsQ/nl8gdQfGCs2EIUppjStLU9zuozB9+L283ypZ6p86pjiGGhnEcgCZ8UIRcDZ8UVuOXCknWSflDm0pkTkhJCKHauRlaaRtYjv0AoySRKnrZC/HuaMY83QXu0zBLKFq7/XoJaCzUSTVJuWJEr71YXFaQYUAxCNzuftArBk2lglcjp0PxURm0cP8Un639TaquTaVvX9iNVSDQvM2nMNOii0xrJ8Uh1deQM+4scrPSa1Q== X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: Commit 4c132d1d844a ("x86/futex: Remove .fixup usage") introduced a new extable fixup type, EX_TYPE_EFAULT_REG, and later patches updated the extable fixup type for copy-from-user operations, changing it from EX_TYPE_UACCESS to EX_TYPE_EFAULT_REG. Specifically, commit 99641e094d6c ("x86/uaccess: Remove .fixup usage") altered the extable fixup type for the get_user family, while commit 4c132d1d844a ("x86/futex: Remove .fixup usage") addressed the futex operations. This change inadvertently caused a regression where the error context for some copy-from-user operations no longer functions as an in-kernel recovery context, leading to kernel panics with the message: "Machine check: Data load in unrecoverable area of kernel." To fix the regression, add EX_TYPE_EFAULT_REG as a in-kernel recovery context for copy-from-user operations. Signed-off-by: Shuai Xue Fixes: 4c132d1d844a ("x86/futex: Remove .fixup usage") Cc: stable@vger.kernel.org --- arch/x86/kernel/cpu/mce/severity.c | 21 ++++++++++++++++----- 1 file changed, 16 insertions(+), 5 deletions(-) diff --git a/arch/x86/kernel/cpu/mce/severity.c b/arch/x86/kernel/cpu/mce/severity.c index dac4d64dfb2a..14c2d71c3ce1 100644 --- a/arch/x86/kernel/cpu/mce/severity.c +++ b/arch/x86/kernel/cpu/mce/severity.c @@ -16,6 +16,7 @@ #include #include #include +#include #include "internal.h" @@ -285,7 +286,8 @@ static bool is_copy_from_user(struct pt_regs *regs) */ static noinstr int error_context(struct mce *m, struct pt_regs *regs) { - int fixup_type; + const struct exception_table_entry *e; + int fixup_type, imm; bool copy_user; if ((m->cs & 3) == 3) @@ -294,9 +296,14 @@ static noinstr int error_context(struct mce *m, struct pt_regs *regs) if (!mc_recoverable(m->mcgstatus)) return IN_KERNEL; + e = search_exception_tables(m->ip); + if (!e) + return IN_KERNEL; + /* Allow instrumentation around external facilities usage. */ instrumentation_begin(); - fixup_type = ex_get_fixup_type(m->ip); + fixup_type = FIELD_GET(EX_DATA_TYPE_MASK, e->data); + imm = FIELD_GET(EX_DATA_IMM_MASK, e->data); copy_user = is_copy_from_user(regs); instrumentation_end(); @@ -304,9 +311,13 @@ static noinstr int error_context(struct mce *m, struct pt_regs *regs) case EX_TYPE_UACCESS: if (!copy_user) return IN_KERNEL; - m->kflags |= MCE_IN_KERNEL_COPYIN; - fallthrough; - + m->kflags |= MCE_IN_KERNEL_COPYIN | MCE_IN_KERNEL_RECOV; + return IN_KERNEL_RECOV; + case EX_TYPE_IMM_REG: + if (!copy_user || imm != -EFAULT) + return IN_KERNEL; + m->kflags |= MCE_IN_KERNEL_COPYIN | MCE_IN_KERNEL_RECOV; + return IN_KERNEL_RECOV; case EX_TYPE_FAULT_MCE_SAFE: case EX_TYPE_DEFAULT_MCE_SAFE: m->kflags |= MCE_IN_KERNEL_RECOV; From patchwork Mon Feb 17 06:33:34 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Shuai Xue X-Patchwork-Id: 13977134 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id C9A52C021A0 for ; Mon, 17 Feb 2025 06:33:58 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 93D65280036; Mon, 17 Feb 2025 01:33:51 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 8ECDA280032; Mon, 17 Feb 2025 01:33:51 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 718EC280036; Mon, 17 Feb 2025 01:33:51 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0010.hostedemail.com [216.40.44.10]) by kanga.kvack.org (Postfix) with ESMTP id 4E6AD280032 for ; Mon, 17 Feb 2025 01:33:51 -0500 (EST) Received: from smtpin26.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay05.hostedemail.com (Postfix) with ESMTP id 09A3C4C2DD for ; Mon, 17 Feb 2025 06:33:51 +0000 (UTC) X-FDA: 83128471062.26.014192E Received: from out30-110.freemail.mail.aliyun.com (out30-110.freemail.mail.aliyun.com [115.124.30.110]) by imf02.hostedemail.com (Postfix) with ESMTP id B594A80005 for ; Mon, 17 Feb 2025 06:33:48 +0000 (UTC) Authentication-Results: imf02.hostedemail.com; dkim=pass header.d=linux.alibaba.com header.s=default header.b=CUttw4BX; spf=pass (imf02.hostedemail.com: domain of xueshuai@linux.alibaba.com designates 115.124.30.110 as permitted sender) smtp.mailfrom=xueshuai@linux.alibaba.com; dmarc=pass (policy=none) header.from=linux.alibaba.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1739774029; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=vvMi8bcBgppidelzMl/TigrL5YZhFRFfYcEROd+lYRc=; b=TGHp++DM/9JU4Qs0VaxJUGc9I0UurCUFcIsn/OtFvcaVsJTB+aQKYSe64VuA3y17xzHKuN O6sYagGTpuQcgyvVBBxIWqONsvdMNLi0q4ulAV1Mb11LKvsdsRRXcbndiTjwXt2biSbur3 Wp3Rhhn9kaC6q//u03huUSYjzCcu8WQ= ARC-Authentication-Results: i=1; imf02.hostedemail.com; dkim=pass header.d=linux.alibaba.com header.s=default header.b=CUttw4BX; spf=pass (imf02.hostedemail.com: domain of xueshuai@linux.alibaba.com designates 115.124.30.110 as permitted sender) smtp.mailfrom=xueshuai@linux.alibaba.com; dmarc=pass (policy=none) header.from=linux.alibaba.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1739774029; a=rsa-sha256; cv=none; b=bRUeoc/rSY1hSZ+eYpLMs3l5vQg0XVy4/3zLNMkXAnGT9RE2yEFk0epBvAZoa0yXP2Cb01 jLEOu3GpUySnuKUxGF/f5bPCVACoiEx6jBqjRnAIv1+jFUqhc+4ySV5r2cg/q6oh946rL0 uy2FYmHYncdsUwzZx0zr/NjzmBo58Hw= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.alibaba.com; s=default; t=1739774023; h=From:To:Subject:Date:Message-ID:MIME-Version; bh=vvMi8bcBgppidelzMl/TigrL5YZhFRFfYcEROd+lYRc=; b=CUttw4BX2Tfeg8MVmM77B3AvtOGqbsysc202/jGw7+0lMGAs/ut0QlyEP+jfnWrmeztmtBTNvagnKqNgGnBzTxKM7/NIUykck9+jpTI8dzoxLVa61qXmKOV+Znyronry3e/R5qs2SxKm3uDc1UGQbABKeSOCumXvsy1iiuqTyPQ= Received: from localhost.localdomain(mailfrom:xueshuai@linux.alibaba.com fp:SMTPD_---0WPb1mQP_1739774021 cluster:ay36) by smtp.aliyun-inc.com; Mon, 17 Feb 2025 14:33:41 +0800 From: Shuai Xue To: tony.luck@intel.com, bp@alien8.de, nao.horiguchi@gmail.com Cc: tglx@linutronix.de, mingo@redhat.com, dave.hansen@linux.intel.com, x86@kernel.org, hpa@zytor.com, linmiaohe@huawei.com, akpm@linux-foundation.org, peterz@infradead.org, jpoimboe@kernel.org, linux-edac@vger.kernel.org, linux-kernel@vger.kernel.org, linux-mm@kvack.org, baolin.wang@linux.alibaba.com, tianruidong@linux.alibaba.com Subject: [PATCH v2 4/5] mm/hwpoison: Fix incorrect "not recovered" report for recovered clean pages Date: Mon, 17 Feb 2025 14:33:34 +0800 Message-ID: <20250217063335.22257-5-xueshuai@linux.alibaba.com> X-Mailer: git-send-email 2.44.0 In-Reply-To: <20250217063335.22257-1-xueshuai@linux.alibaba.com> References: <20250217063335.22257-1-xueshuai@linux.alibaba.com> MIME-Version: 1.0 X-Rspam-User: X-Rspamd-Server: rspam11 X-Rspamd-Queue-Id: B594A80005 X-Stat-Signature: fk5mqswz7teog37xahuade8nzbffgf16 X-HE-Tag: 1739774028-188827 X-HE-Meta: U2FsdGVkX18XoqYLmnAeO7c8FE1Io6VT4UydTx+3LcJlSgwt5GS1PonXV5kd7LUO4po+Y8SYPvy9dEG2pckLVCPQtbh4XsIit25hxrOvLRMJZ9n+cQQK9dY//Slgs37nEqjjjwCv3wpfo60RF7ugUMLtxyuA6UaaC9/mEoxP1ZcefeZhR9UFDOAMEg6L16oRRkMxk6fp75lbSSl9ox0K1w1zUj2UTP61FO6ToBBouoFBwGvrMau3o15EdlxPuqESVyjhNrLndoR4K402B809EhrSB+hKPUxb8+ed7QktIjxvTiv9s7iGTTv21QqrAwrsZTb0rCJJq1rDlPU2wjduTd6lvWDbK/oLCBjQi2j88ax65smyXG3P7ELYEEgh466BSs8XYz3+BgRfbtmzgYt8yHKkGOGEdC9q6LscUnQZ6xRGuqXMT8GnrYqXLSuNfQhf/7E7mwTFyt9QGMuKBVMG3G74BHAACFoRBT0TgsE1SdD/Lqah+6fEm+soetRC2/9f+9zxYC/bd4cr213rNNlQQI/bU8XTtU9F4HTlmAMiqIzvM5ZH9xy2UutF8m5Oh8lIdk3guwT34UOlfRJ1oxbF8U6FQp5jAS56j5GVCNyzZb5vi1cvEbaucNxMWb2W9mb7NL90wsHdokt/oC6Of0ELy7zT11lD2McrWi7X+7D3XOICZQwJ4KtNbR6t0D1tacGUOlTspmaPXNlE1XCF671Uu7/A8P9smRifQangoAqkD+0KMCPmhvnqrtevXg89936PKMgP/su3T8+mN6xdTWxylyrohVPdsDVFz1zE8UQ152aIqkkhBo6G3+dD6sjpxVhlL8yWE6XKyqZxmeD77dhU+FhXV3EwC2bHReH+CuJKSfw9XGQVFGDhvNu+GcwaB0a1sgBu5zxvAFlOCdg2nQFxxuS/c2B4FVM+GZfKBG4ngM17MF8UYRoLKNPqyGES69fDFznh/w6Ed4cSDpaUKqE fpa1InmM zavsZZDci/uVikFPIGrTRX5dvbiTEX8Hh/FvAkCZ9yCKpLNuKVRicMne77EeTTNjMXpMvhJVPaiKaA01dmrfaiBjAIcf2DPPjlRpDJGzYdFbzQRsCkavGphKyuiu3eZOgves4YtXC1KU8a3T1ApwUI9XDmyZGe/4ZKplziw+kbyFBA8Iyk0C6RP6rEFWoe8+QjBZb5UFoy4OjhPVtOReUG/wkcRwn9gs1ZF43gWcy+Gkx00gCvd7yshg3Xvv0aOVggzOqUjrVzA3yZgHbkskqG4a6EBWwJQmi+TVYA2i31g5WWhlGZ1F34CEgmd/pU+z8+LmNRRzSSUHixdq4+QomXgoCsboOCAO8BV79ypijsg7JdzAY/zSZZXNwboFyd4PkKFEH X-Bogosity: Ham, tests=bogofilter, spamicity=0.052083, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: When an uncorrected memory error is consumed there is a race between the CMCI from the memory controller reporting an uncorrected error with a UCNA signature, and the core reporting and SRAR signature machine check when the data is about to be consumed. If the CMCI wins that race, the page is marked poisoned when uc_decode_notifier() calls memory_failure(). For dirty pages, memory_failure() invokes try_to_unmap() with the TTU_HWPOISON flag, converting the PTE to a hwpoison entry. As a result, kill_accessing_process(): - call walk_page_range() and return 1 regardless of whether try_to_unmap() succeeds or fails, - call kill_proc() to make sure a SIGBUS is sent - return -EHWPOISON to indicate that SIGBUS is already sent to the process and kill_me_maybe() doesn't have to send it again. However, for clean pages, the TTU_HWPOISON flag is cleared, leaving the PTE unchanged and not converted to a hwpoison entry. Conversely, for clean pages where PTE entries are not marked as hwpoison, kill_accessing_process() returns -EFAULT, causing kill_me_maybe() to send a SIGBUS. Console log looks like this: Memory failure: 0x827ca68: corrupted page was clean: dropped without side effects Memory failure: 0x827ca68: recovery action for clean LRU page: Recovered Memory failure: 0x827ca68: already hardware poisoned mce: Memory error not recovered To fix it, return 0 for "corrupted page was clean", preventing an unnecessary SIGBUS. Fixes: 046545a661af ("mm/hwpoison: fix error page recovered but reported "not recovered"") Signed-off-by: Shuai Xue Cc: stable@vger.kernel.org --- mm/memory-failure.c | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/mm/memory-failure.c b/mm/memory-failure.c index 995a15eb67e2..b037952565be 100644 --- a/mm/memory-failure.c +++ b/mm/memory-failure.c @@ -881,12 +881,17 @@ static int kill_accessing_process(struct task_struct *p, unsigned long pfn, mmap_read_lock(p->mm); ret = walk_page_range(p->mm, 0, TASK_SIZE, &hwpoison_walk_ops, (void *)&priv); + /* + * ret = 1 when CMCI wins, regardless of whether try_to_unmap() + * succeeds or fails, then kill the process with SIGBUS. + * ret = 0 when poison page is a clean page and it's dropped, no + * SIGBUS is needed. + */ if (ret == 1 && priv.tk.addr) kill_proc(&priv.tk, pfn, flags); - else - ret = 0; mmap_read_unlock(p->mm); - return ret > 0 ? -EHWPOISON : -EFAULT; + + return ret > 0 ? -EHWPOISON : 0; } /* From patchwork Mon Feb 17 06:33:35 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Shuai Xue X-Patchwork-Id: 13977133 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 1F776C021A0 for ; Mon, 17 Feb 2025 06:33:56 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 0140D280035; Mon, 17 Feb 2025 01:33:51 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id F09DF280032; Mon, 17 Feb 2025 01:33:50 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id D808E280035; Mon, 17 Feb 2025 01:33:50 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0011.hostedemail.com [216.40.44.11]) by kanga.kvack.org (Postfix) with ESMTP id B4151280032 for ; Mon, 17 Feb 2025 01:33:50 -0500 (EST) Received: from smtpin27.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay05.hostedemail.com (Postfix) with ESMTP id 0BB334C4D0 for ; Mon, 17 Feb 2025 06:33:50 +0000 (UTC) X-FDA: 83128471020.27.5EEC76B Received: from out30-110.freemail.mail.aliyun.com (out30-110.freemail.mail.aliyun.com [115.124.30.110]) by imf11.hostedemail.com (Postfix) with ESMTP id 0F7CB4000C for ; Mon, 17 Feb 2025 06:33:47 +0000 (UTC) Authentication-Results: imf11.hostedemail.com; dkim=pass header.d=linux.alibaba.com header.s=default header.b=WSjrr6uD; spf=pass (imf11.hostedemail.com: domain of xueshuai@linux.alibaba.com designates 115.124.30.110 as permitted sender) smtp.mailfrom=xueshuai@linux.alibaba.com; dmarc=pass (policy=none) header.from=linux.alibaba.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1739774028; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=2HAfRf79L/SPv5Q+bga3YvpNIcG3LqJbC8lB0lpSzQc=; b=N18rgF5FzPQnd+IIE2oELivBOqesl8ZcXo0dKfdT7Fd+U6StOpF+Dyhgjj2sWpZbK/f4Gb oDhsWdARiDQrfbygBgzOJZMxA0vI/IV2MBPzz/YuSUT3+B1RyBw4w2EHG+LS4n26M6kyyH y8szwvCCm+ERuymEsu261aO+4dfs/6U= ARC-Authentication-Results: i=1; imf11.hostedemail.com; dkim=pass header.d=linux.alibaba.com header.s=default header.b=WSjrr6uD; spf=pass (imf11.hostedemail.com: domain of xueshuai@linux.alibaba.com designates 115.124.30.110 as permitted sender) smtp.mailfrom=xueshuai@linux.alibaba.com; dmarc=pass (policy=none) header.from=linux.alibaba.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1739774028; a=rsa-sha256; cv=none; b=7O5KsxpqK5uLcjkTlfQYREsinVg6p9mtoL4ownlr0gZb9DDl98f0RBhTe2CM+VRt635t0L tKcwn5sh+G7dvFKXrxIoJpiTgJ2MXc4vJKI2uwVxzqetzCUQzD9FDFUy4V5qvCiecXN9/z GS0q0fNoGc6+VhQSv9x8wsEZj2YTw8Y= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.alibaba.com; s=default; t=1739774024; h=From:To:Subject:Date:Message-ID:MIME-Version; bh=2HAfRf79L/SPv5Q+bga3YvpNIcG3LqJbC8lB0lpSzQc=; b=WSjrr6uDIeBpDzPLGd+hZDrC1F6HOTHsOniX4O9MlXYAHQK0NukIXxdeHPEFi15ZVZMVxvfwIqA/zOvie7RzO3K4qSAm4oPCfZyfkO/bjsYgeaDMY0Oj9c0aJ3OpV8w/ui6gJVVxzMgcbfQSwQL/2kDCnKPB8lFmzuMBGsozaWg= Received: from localhost.localdomain(mailfrom:xueshuai@linux.alibaba.com fp:SMTPD_---0WPb1mQt_1739774022 cluster:ay36) by smtp.aliyun-inc.com; Mon, 17 Feb 2025 14:33:43 +0800 From: Shuai Xue To: tony.luck@intel.com, bp@alien8.de, nao.horiguchi@gmail.com Cc: tglx@linutronix.de, mingo@redhat.com, dave.hansen@linux.intel.com, x86@kernel.org, hpa@zytor.com, linmiaohe@huawei.com, akpm@linux-foundation.org, peterz@infradead.org, jpoimboe@kernel.org, linux-edac@vger.kernel.org, linux-kernel@vger.kernel.org, linux-mm@kvack.org, baolin.wang@linux.alibaba.com, tianruidong@linux.alibaba.com Subject: [PATCH v2 5/5] mm: memory-failure: move return value documentation to function declaration Date: Mon, 17 Feb 2025 14:33:35 +0800 Message-ID: <20250217063335.22257-6-xueshuai@linux.alibaba.com> X-Mailer: git-send-email 2.44.0 In-Reply-To: <20250217063335.22257-1-xueshuai@linux.alibaba.com> References: <20250217063335.22257-1-xueshuai@linux.alibaba.com> MIME-Version: 1.0 X-Rspamd-Server: rspam04 X-Rspamd-Queue-Id: 0F7CB4000C X-Stat-Signature: d75wzyfrydx7uhfoqpcrnp574sfdkmwh X-Rspam-User: X-HE-Tag: 1739774027-790101 X-HE-Meta: U2FsdGVkX1/R9pxYC/kmU5qqkfdQ+mjvNutsP0/oroBe+8sIz3EnTQxInCeu54U4rTyczX3ei2e6BoD6blYaJ4F7uk6J872PjcyD8Y4/X8MiSzCbum1thGKTNHTjzJyc9pvgU32iOJvoozZlCViboZcQU8IzbRIxvDlsZnXbAACMiuemUYRHtIBKFRIiEYRzUGb9ZXkN/XY+8AgXnT5ImAWyRAGKCkHtpZY3lUQfqScrQkp4eb0oxIU1nH2hzu9kwUd32A9umDzSKWdcz8RJqEvbSn2HAgYrE78NRPO+G8E6mmS5oqgOmgnhOfeLZO2kozLNgOH5jA7qEE4He7SgtYhW7P4/fVp1rO+OYWzFI8G5FyJ4vOVVxj7v4w8NHo8CmPLE8Mb5/0HUiNKQJ8p8v8jCbBO4hmxYfzEZK8W7rfZqQcUdspWW6pwaenVTiWhaxiiZ1VKkSM/rVZ3BY0ibGa5lwZT+7BQH9vZ0ASYusogmUWuUsngCCxD57CqfpXQk9WZGbju6Q2nBF704voC1Tl4qrHjmZ0luPAVb3NtluBBX4n/ysJT3XzYolsE5HDl33kJ0OpxJWsVeYpy/mnXDtJOamplwcpyvw7hRo8wW+6P8cjfkXRAPGCcVdrojv+6PeHWDT7vq6c326D1Cc5hLGRKtWrcpnVy5gwj0i3oTYUhd0fNUe3KGnIYEXWk5lA447YNH2McqlrKrTT9CLDM4lT9lBK6GWCv4arn5XFsOpqV7dSKzZY+8zAEbmVeKT6C4UxxWlFjsocicXBhqAzq5voNPL4o4CRQ2eeoE0oMjVBecy4g0Eyblw+egr06FK46NOCR3LSjm53UEzOuZSVdJ0FHEJC+PeNsk2gf3004mPZnCFjO+FlkyChPtPWelEdw36wCnfXHkedUbkWCiP+4KaAowO9vMPd5E2uA8CjKR8fUR37UNUSwBYrbJEnbmoMx15yBWDzmAubijLVjEHcw HWs32i5P IluRFaez/wIBfigNCn6qBYLftp1NbxsnoH6du9JCB7d0RVA4cFY1xVv5cDaljevDbC/s3jOvfTI6xZWFY1+POLH6rzxSmOalkWh4LsRrRRRuZ2YPvWSvAUbERGgnrgtySFflnRtS0Jv/r/IZgxrUuj0pE17zRk9LX1KDcFzxUhZwmcyDne6C4KkH+qTRLecJt0cyOOIO5ltwI82ht5YL3SR/eOfG+xOmMOtki9yC+xfLRWdSsOQH+69JNagmXoj6MNhlF0gTsMNEqwdVoNkkXYbyUJMvefp92AK0PR6UWRbNGWdpCvT76G6OD0eYdB/Q31KhFbgvTUGypcxNw20ubkOGQpsEt87FGgoe6nnES6a67kdFJhdiyb7/jQfhnbU5tZQ3HWEJ0VX8eJuA= X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: Part of return value comments for memory_failure() were originally documented at the call site. Move those comments to the function declaration to improve code readability and to provide developers with immediate access to function usage and return information. Signed-off-by: Shuai Xue Reviewed-by: Jarkko Sakkinen Reviewed-by: Jonathan Cameron Reviewed-by: Yazen Ghannam Reviewed-by: Jane Chu Acked-by: Miaohe Lin --- arch/x86/kernel/cpu/mce/core.c | 7 ------- mm/memory-failure.c | 10 +++++++--- 2 files changed, 7 insertions(+), 10 deletions(-) diff --git a/arch/x86/kernel/cpu/mce/core.c b/arch/x86/kernel/cpu/mce/core.c index c1b945dbccdf..c81695f320bc 100644 --- a/arch/x86/kernel/cpu/mce/core.c +++ b/arch/x86/kernel/cpu/mce/core.c @@ -1403,13 +1403,6 @@ static void kill_me_maybe(struct callback_head *cb) return; } - /* - * -EHWPOISON from memory_failure() means that it already sent SIGBUS - * to the current process with the proper error info, - * -EOPNOTSUPP means hwpoison_filter() filtered the error event, - * - * In both cases, no further processing is required. - */ if (ret == -EHWPOISON || ret == -EOPNOTSUPP) return; diff --git a/mm/memory-failure.c b/mm/memory-failure.c index b037952565be..8649849bcdb4 100644 --- a/mm/memory-failure.c +++ b/mm/memory-failure.c @@ -2216,9 +2216,13 @@ static void kill_procs_now(struct page *p, unsigned long pfn, int flags, * Must run in process context (e.g. a work queue) with interrupts * enabled and no spinlocks held. * - * Return: 0 for successfully handled the memory error, - * -EOPNOTSUPP for hwpoison_filter() filtered the error event, - * < 0(except -EOPNOTSUPP) on failure. + * Return: + * 0 - success, + * -ENXIO - memory not managed by the kernel + * -EOPNOTSUPP - hwpoison_filter() filtered the error event, + * -EHWPOISON - the page was already poisoned, potentially + * kill process, + * other negative values - failure. */ int memory_failure(unsigned long pfn, int flags) {