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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 1919AFF8873 for ; Wed, 29 Apr 2026 14:15:07 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: MIME-Version:References:In-Reply-To:Message-ID:Date:Subject:Cc:To:From: Reply-To:Content-Type:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=KzuxTBhAKGpRNhDCaAbCDDTfw8b7Lf8OfGqqk7nXzaw=; b=XQ16PTQDqptQxn9rxBp0zUO1WV UsQ1PqeU08UT6n9fWotjG6iBG3n8iHi1y8btGBacORchtLxC33cS7+4IediDvfyWW06C6CgvGCC7e WKIvivDMpvHGz1tYc5Gx+NJewFaR8ebSO6jx/FTirzx4Q9q4PK7BspZihXvV2FDkJfSyVPqesYQkK /pOTwOWjykshPWnJpzfY34fbAvkxft2H90FExRX0fAwauOlO6zqt3bbHMCSgDBgayWDecXWutN2hZ +GVvmxZ4JQVw+8G6DdenqOGOFSmj3cmei9ox4frTWAgdTTe3jSy0nVcImjgn7ZuE+jQPT4QxMux0S CLf7evPA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1wI5gr-00000003k2A-00ML; Wed, 29 Apr 2026 14:15:01 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1wI5gj-00000003jvN-16hy for linux-arm-kernel@lists.infradead.org; Wed, 29 Apr 2026 14:14:54 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 370391C01; Wed, 29 Apr 2026 07:14:46 -0700 (PDT) Received: from e142021.munich.arm.com (e142021.arm.com [10.41.150.154]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 81DF53F62B; Wed, 29 Apr 2026 07:14:48 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=arm.com; s=foss; t=1777472091; bh=wnINJ4qqC5jjasrgT/cw8BNQc6KV8jan1/3jpxXMVaQ=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=OQoPfOC4B1H8Q4FAfavj/0IISrRjLH/8CILEmmydD3VI6qY+g8aKkk9RvnVzCR4HW QR3XejycOJ+zazznJcIGoEY88hym/1oeAAm5tHm5PB576kPegqVyVbEfqLZ1mwn5Y0 BLmH/MDR0g/1yBrMzRs4k65WelLIujcqfPwDzBJA= From: Andre Przywara To: Lorenzo Pieralisi , Hanjun Guo , Sudeep Holla , Catalin Marinas , Will Deacon , "Rafael J . Wysocki" , Len Brown , James Morse , Ben Horgan , Reinette Chatre , Fenghua Yu Cc: Jonathan Cameron , linux-acpi@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: [PATCH 4/5] arm_mpam: prevent MPAM-Fb accesses inside IRQ handler Date: Wed, 29 Apr 2026 16:13:38 +0200 Message-ID: <20260429141339.3171205-5-andre.przywara@arm.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20260429141339.3171205-1-andre.przywara@arm.com> References: <20260429141339.3171205-1-andre.przywara@arm.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20260429_071453_431654_121A0ECD X-CRM114-Status: GOOD ( 13.95 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org When an MPAM MSC gets into an error condition, it can trigger an error IRQ. We cannot really do much about those errors, but we at least query and log the error, then disable MPAM functionality. This error report relies on reading the MSC's error status register (ESR) in the IRQ handler, which is not possible for MPAM-Fb based MSC accesses, since they involve mailbox routines that might sleep. For simplicity just skip the ESR read when the MSC is not using direct MMIO accesses. We will wrap up MPAM functionality regardless, knowing the exact error value will not change that. Signed-off-by: Andre Przywara --- drivers/resctrl/mpam_devices.c | 30 +++++++++++++++++++----------- 1 file changed, 19 insertions(+), 11 deletions(-) diff --git a/drivers/resctrl/mpam_devices.c b/drivers/resctrl/mpam_devices.c index 90b49bfd5caa..62aa04cb6905 100644 --- a/drivers/resctrl/mpam_devices.c +++ b/drivers/resctrl/mpam_devices.c @@ -2472,7 +2472,7 @@ static int mpam_disable_msc_ecr(void *_msc) static irqreturn_t __mpam_irq_handler(int irq, struct mpam_msc *msc) { - u64 reg; + u64 reg = 0; u16 partid; u8 errcode, pmg, ris; @@ -2481,22 +2481,30 @@ static irqreturn_t __mpam_irq_handler(int irq, struct mpam_msc *msc) &msc->accessibility))) return IRQ_NONE; - reg = mpam_msc_read_esr(msc); + /* MPAM-Fb MSC accesses cannot be done in atomic context. */ + if (msc->iface == MPAM_IFACE_MMIO) { + reg = mpam_msc_read_esr(msc); - errcode = FIELD_GET(MPAMF_ESR_ERRCODE, reg); - if (!errcode) - return IRQ_NONE; + errcode = FIELD_GET(MPAMF_ESR_ERRCODE, reg); + if (!errcode) + return IRQ_NONE; + } /* Clear level triggered irq */ mpam_msc_clear_esr(msc); - partid = FIELD_GET(MPAMF_ESR_PARTID_MON, reg); - pmg = FIELD_GET(MPAMF_ESR_PMG, reg); - ris = FIELD_GET(MPAMF_ESR_RIS, reg); + if (reg) { + partid = FIELD_GET(MPAMF_ESR_PARTID_MON, reg); + pmg = FIELD_GET(MPAMF_ESR_PMG, reg); + ris = FIELD_GET(MPAMF_ESR_RIS, reg); + + pr_err_ratelimited("error irq from msc:%u '%s', partid:%u, pmg: %u, ris: %u\n", + msc->id, mpam_errcode_names[errcode], partid, + pmg, ris); + } else { + pr_err_ratelimited("unknown error irq from msc:%u\n", msc->id); + } - pr_err_ratelimited("error irq from msc:%u '%s', partid:%u, pmg: %u, ris: %u\n", - msc->id, mpam_errcode_names[errcode], partid, pmg, - ris); /* Disable this interrupt. */ mpam_disable_msc_ecr(msc); -- 2.43.0