From mboxrd@z Thu Jan 1 00:00:00 1970 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 5A2C4D21262 for ; Thu, 17 Oct 2024 09:39:34 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id DDB846B0083; Thu, 17 Oct 2024 05:39:33 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id D984F6B0088; Thu, 17 Oct 2024 05:39:33 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id C529D6B0083; Thu, 17 Oct 2024 05:39:33 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0015.hostedemail.com [216.40.44.15]) by kanga.kvack.org (Postfix) with ESMTP id A88A86B007B for ; Thu, 17 Oct 2024 05:39:33 -0400 (EDT) Received: from smtpin26.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay05.hostedemail.com (Postfix) with ESMTP id D97E54056F for ; Thu, 17 Oct 2024 09:39:26 +0000 (UTC) X-FDA: 82682596332.26.ED1C2E7 Received: from frasgout.his.huawei.com (frasgout.his.huawei.com [185.176.79.56]) by imf17.hostedemail.com (Postfix) with ESMTP id 5628B40004 for ; Thu, 17 Oct 2024 09:39:23 +0000 (UTC) Authentication-Results: imf17.hostedemail.com; dkim=none; dmarc=pass (policy=quarantine) header.from=huawei.com; spf=pass (imf17.hostedemail.com: domain of jonathan.cameron@huawei.com designates 185.176.79.56 as permitted sender) smtp.mailfrom=jonathan.cameron@huawei.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1729157864; a=rsa-sha256; cv=none; b=mytUcyCuDe3hsL44vaZhrv59LbYL9vcU7YWsZFAIRUbtbdQvcIk88OSyzpKd0eZdepfgCw RskF1g5BRYc5P1/XP5fTiwOlyS+Z5mR6Mv/uaTc2ay7LBlnf2wMCnH3OHCVemS6+2+jhSX IyQjlbOb9PZnjSKZTFWyaBW+dyO+2oQ= ARC-Authentication-Results: i=1; imf17.hostedemail.com; dkim=none; dmarc=pass (policy=quarantine) header.from=huawei.com; spf=pass (imf17.hostedemail.com: domain of jonathan.cameron@huawei.com designates 185.176.79.56 as permitted sender) smtp.mailfrom=jonathan.cameron@huawei.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1729157864; 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-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=xRxiak0nTjLTdpjHLjJD+gM51MOdPvppJSGq4fwClW8=; b=gSBFhAUDK4oy8nvWZtq0y8q7FyxMIe/Jyoi9xxms0T6jcGmwv5Cvlli1KvF4lz5KzytEVR xYOCfuAOMiYkh7N4mCbWDUgjQf+mWpTirST7fQtASeMKQiiCanCaNQ4+gvHEELk5r1HfKh /Ptl4BmHZODmIc/rxggV13GX7rx4/5I= Received: from mail.maildlp.com (unknown [172.18.186.31]) by frasgout.his.huawei.com (SkyGuard) with ESMTP id 4XTjT32j0Bz6JBQb; Thu, 17 Oct 2024 17:38:47 +0800 (CST) Received: from frapeml500008.china.huawei.com (unknown [7.182.85.71]) by mail.maildlp.com (Postfix) with ESMTPS id 40E151400C9; Thu, 17 Oct 2024 17:39:27 +0800 (CST) Received: from localhost (10.126.174.164) by frapeml500008.china.huawei.com (7.182.85.71) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2507.39; Thu, 17 Oct 2024 11:39:25 +0200 Date: Thu, 17 Oct 2024 10:39:23 +0100 From: Jonathan Cameron To: Shuai Xue CC: , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , Subject: Re: [PATCH v14 1/3] ACPI: APEI: send SIGBUS to current task if synchronous memory error not recovered Message-ID: <20241017103923.00007033@Huawei.com> In-Reply-To: <20241014084240.18614-2-xueshuai@linux.alibaba.com> References: <20221027042445.60108-1-xueshuai@linux.alibaba.com> <20241014084240.18614-2-xueshuai@linux.alibaba.com> Organization: Huawei Technologies Research and Development (UK) Ltd. X-Mailer: Claws Mail 4.1.0 (GTK 3.24.33; x86_64-w64-mingw32) MIME-Version: 1.0 Content-Type: text/plain; charset="US-ASCII" Content-Transfer-Encoding: 7bit X-Originating-IP: [10.126.174.164] X-ClientProxiedBy: lhrpeml100003.china.huawei.com (7.191.160.210) To frapeml500008.china.huawei.com (7.182.85.71) X-Rspamd-Queue-Id: 5628B40004 X-Rspam-User: X-Rspamd-Server: rspam05 X-Stat-Signature: qbrq47hx6nauosf8nrfeyc6ynh9umgw5 X-HE-Tag: 1729157963-186016 X-HE-Meta: U2FsdGVkX1+gqoQW3o3c0X2acVgOJMxlyzKgG3YCsY+TsgRhXJCQYLVyd/X0EcXMR9mkmL+4KQm/vu4hipQn1YVAljQmXuhmIOtKE1p2Mj4m1E0Va9C6o4vEGwiQz1CBVx+JXS7kOToxJdCvqVDl9ibqan1amm+Cy4kHZYj9bpXPWkaRPgQjf3RswNKjrd7dVFzGFeUjJvdnHeyO5mO8iy2ptxMYv3Dupu8GM5LxZ+AtA+/i11dqfTAiWxytOSFFtGPMMpQQ1FWB1fS1izoKwnJZwRS621ArJoHyztAsPAmRaR7cJonbRkJ37YCGJJh+I/h4lOSnxJYcTcK9hG8KKQiOOA3S0rqC5D+/QrOgAhMctc+bMpzi1CMtRwS6V7oPKWfSdQTMMvFj9Q58buLdGzB2io6vzSOFv4la0aLeSHk11GWSH5i3l+muDVyBNVqJbnkRlzI9y7c9hY0n/eWVwv4JYprBVbJ4hchaydcC0nBns1yFzfYKBsH+RMjGynbyPMzfKfTG2S/tjHfN4dSsza7w/tYOkku0qA2HuWirNVSYsfvJ6ycfHvWlqWUCICnLMw/nLEZ1B9JHCwA2atZMp1ykcVeq5hnNY7WRHBj4v0MmjyXKlC97Vno1Ko3d0YW4qMicJa1AzfjMj55NBFcbpFahnZ3krQ86VkAM0lHs78DdpcDxO7MFHpyb0Fnk4qYuBGvszlL84Y9u0YsAg28NyTrWvk9GCXoOMKTY2quuhPPQ93OWsUDV2E4SMDGDa0F2/c042Z1jPT3qYtTknnGozv7nndBqU2tuVxjzlvwNlg0WCclMN2hxpquxVc5v6JNUKqwMyO0FdNP+9zBndIM8XpmSVb6S57Wk06bJrfHIJoirRtSHGGpxCilHKUgOu2iIuQQW+yabbA/TkzHwusXImEXNKa1kV48cE7wLYXuMwuFL0c4i9LYPV4fYmxzdqsvwuUoAs8E4+CcgTjod/Tv oE+qXyyX i+i41Y0gAUCqDxRgkdwrZjn+Kz+2RIpo0+xn3rV0y1F444cickKrrmio/3b0USBJlZMY3N4LjhYp3V6jZEfV+EiE1BDcez7lbYzkLz/uT3dky28/0ylMODw7kUnLRnlI7iR2+FjAbs2PwVYnqn5O2FAlBMEKJZ4mPPvmu1SEYfFvIG9MJBR1kPW7zEVm6rd/XYljEFVWiQNtNt0d5K+vckFlkhu0K0hf/ZvVlONP765Q9mZnReI2WV1/4wGWubLENAGCd3drE3b4aPJ7KDCxClF5xgQ== 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: On Mon, 14 Oct 2024 16:42:38 +0800 Shuai Xue wrote: > Synchronous error was detected as a result of user-space process accessing > a 2-bit uncorrected error. The CPU will take a synchronous error exception > such as Synchronous External Abort (SEA) on Arm64. The kernel will queue a > memory_failure() work which poisons the related page, unmaps the page, and > then sends a SIGBUS to the process, so that a system wide panic can be > avoided. > > However, no memory_failure() work will be queued when abnormal synchronous > errors occur. These errors can include situations such as invalid PA, > unexpected severity, no memory failure config support, invalid GUID > section, etc. In such case, the user-space process will trigger SEA again. > This loop can potentially exceed the platform firmware threshold or even > trigger a kernel hard lockup, leading to a system reboot. > > Fix it by performing a force kill if no memory_failure() work is queued > for synchronous errors. > > Signed-off-by: Shuai Xue > Reviewed-by: Jarkko Sakkinen The subtle cases in here are the various other forms of delayed handling buried in some of the record handling that don't set queued. I've been through them all and have convinced myself that either hey should never be synchronous or that there is no attempt to recover in kernel today (non memory things such as CXL protocol collapse, which might I guess be detected synchronously on a read - though I'd expect poison and a memory error first) so the correct thing to do is what you have here. Fiddly code though with a lot of paths, so more eyes welcome! Reviewed-by: Jonathan Cameron +CC linux-cxl for info. > --- > drivers/acpi/apei/ghes.c | 10 ++++++++++ > 1 file changed, 10 insertions(+) > > diff --git a/drivers/acpi/apei/ghes.c b/drivers/acpi/apei/ghes.c > index ada93cfde9ba..f2ee28c44d7a 100644 > --- a/drivers/acpi/apei/ghes.c > +++ b/drivers/acpi/apei/ghes.c > @@ -801,6 +801,16 @@ static bool ghes_do_proc(struct ghes *ghes, > } > } > > + /* > + * If no memory failure work is queued for abnormal synchronous > + * errors, do a force kill. > + */ > + if (sync && !queued) { > + pr_err("%s:%d: hardware memory corruption (SIGBUS)\n", > + current->comm, task_pid_nr(current)); > + force_sig(SIGBUS); > + } > + > return queued; > } >