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 1B01FC43458 for ; Fri, 3 Jul 2026 08:38:50 +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: Content-Type:In-Reply-To:From:References:Cc:To:Subject:MIME-Version:Date: Message-ID:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=+Ydx47S+8hLe5C1ynBhLgl8bYfakXCGpIrz8h9G44Uk=; b=QRxNHW/4teMAMJjDYBjTQhQsfy onBUyMGUec4g0boZaNbljsuh+zZPM08Dyb5HIHxu+rdN0tvRfv02dOknxEgAiSFn6v5YZivX3ZYhs QU2JPvGeWbJcxjzBN3heRNjo7lBPDRPU/hGC+ipULjhrnlYp9LggjyKKQnrpNCExdv0ltOBsPCHEP SkK0xESuIJs/7h7TIIlG/661tjOcoeHPQqnOOwNWdxXcq85m/LgcFnmi7dDWOAnD6hjCRJsysFxq5 vVeToD++fFjseT4shuatQUKB5CBZxTjxsSj74xRPfvccDRZ/PyntMR+cgaFk9+SLg3JoVw87d8CGc PhPLeIiQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1wfZPv-00000006NFf-3UfT; Fri, 03 Jul 2026 08:38:35 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1wfZPr-00000006NEx-2JDw for linux-arm-kernel@lists.infradead.org; Fri, 03 Jul 2026 08:38:34 +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 E09051476; Fri, 3 Jul 2026 01:38:24 -0700 (PDT) Received: from [10.211.55.3] (unknown [10.57.73.238]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 796F73F905; Fri, 3 Jul 2026 01:38:25 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=arm.com; s=foss; t=1783067909; bh=KcfAyjvfIggeRPUgM5kQPKdIyZKtYOy5nlDTfsdQLwM=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=GVNmKlA2lu3JViMT/eyKZ0Fyi70W3QFFyZVhNf0Jiw+O7ybwekm2eWKH7waeumEgv WOvthIC+BYMk3E/wuZ/vqkdx+65WWp7KhMHvYo0nL652grpVgQ9lay7FkmEy3JhZbc FvNwCqN4CcZ4ae2nRYYOH9w3NHp4RfF19M9jqbdw= Message-ID: <029910c3-e90f-4a35-8b01-4d3ff548deea@arm.com> Date: Wed, 1 Jul 2026 20:56:53 +0100 MIME-Version: 1.0 User-Agent: Thunderbird Daily Subject: Re: [PATCH v2 02/15] arm_mpam: propagate MSC read errors for wrapper functions To: Andre Przywara , Lorenzo Pieralisi , Hanjun Guo , Sudeep Holla , Catalin Marinas , Will Deacon , "Rafael J . Wysocki" , Len Brown , James Morse , Reinette Chatre , Fenghua Yu Cc: Jonathan Cameron , Srivathsa L Rao , Ganapatrao Kulkarni , Trilok Soni , Srinivas Ramana , Niyas Sait , linux-acpi@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org References: <20260702162229.4008659-1-andre.przywara@arm.com> <20260702162229.4008659-3-andre.przywara@arm.com> Content-Language: en-US From: Ben Horgan In-Reply-To: <20260702162229.4008659-3-andre.przywara@arm.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.9.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20260703_013831_728749_AFC7B4D8 X-CRM114-Status: GOOD ( 20.11 ) 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 Hi Andre, On 7/2/26 17:22, Andre Przywara wrote: > Allow the wrapper functions for IDR and ESR accesses to return an > error, and propagate read errors from the lower level up. > > Signed-off-by: Andre Przywara > --- > drivers/resctrl/mpam_devices.c | 51 +++++++++++++++++++++++----------- > 1 file changed, 35 insertions(+), 16 deletions(-) > > diff --git a/drivers/resctrl/mpam_devices.c b/drivers/resctrl/mpam_devices.c > index df14b4513382..ce8738adb6ff 100644 > --- a/drivers/resctrl/mpam_devices.c > +++ b/drivers/resctrl/mpam_devices.c > @@ -247,27 +247,34 @@ static bool mpam_msc_check_aidr(struct mpam_msc *msc) > return true; > } > > -static u64 mpam_msc_read_idr(struct mpam_msc *msc) > +static int mpam_msc_read_idr(struct mpam_msc *msc, u64 *res) > { > u32 idr_high = 0, idr_low; > + int ret; > > lockdep_assert_held(&msc->part_sel_lock); > > - mpam_read_partsel_reg(msc, IDR, &idr_low); > - if (FIELD_GET(MPAMF_IDR_EXT, idr_low)) > - mpam_read_partsel_reg(msc, IDR + 4, &idr_high); > + ret = mpam_read_partsel_reg(msc, IDR, &idr_low); > + if (!ret && FIELD_GET(MPAMF_IDR_EXT, idr_low)) > + ret = mpam_read_partsel_reg(msc, IDR + 4, &idr_high); > > - return ((u64)idr_high << 32) | idr_low; > + if (!ret) > + *res = ((u64)idr_high << 32) | idr_low; > + > + return ret; > } Why does the pattern here, use of !ret rather than early return, differ from the one you've used in mpam_msc_read_esr()? Thanks, Ben > > -static void mpam_msc_clear_esr(struct mpam_msc *msc) > +static int mpam_msc_clear_esr(struct mpam_msc *msc) > { > u32 esr_low; > + int ret; > > - __mpam_read_reg(msc, MPAMF_ESR, &esr_low); > + ret = __mpam_read_reg(msc, MPAMF_ESR, &esr_low); > + if (ret) > + return ret; > > if (!esr_low) > - return; > + return 0; > > /* > * Clearing the high/low bits of MPAMF_ESR can not be atomic. > @@ -277,18 +284,30 @@ static void mpam_msc_clear_esr(struct mpam_msc *msc) > */ > if (msc->has_extd_esr) > __mpam_write_reg(msc, MPAMF_ESR + 4, 0); > + > __mpam_write_reg(msc, MPAMF_ESR, 0); > + > + return 0; > } > > -static u64 mpam_msc_read_esr(struct mpam_msc *msc) > +static int mpam_msc_read_esr(struct mpam_msc *msc, u64 *res) > { > u32 esr_high = 0, esr_low; > + int ret; > > - __mpam_read_reg(msc, MPAMF_ESR, &esr_low); > - if (msc->has_extd_esr) > - __mpam_read_reg(msc, MPAMF_ESR + 4, &esr_high); > + ret = __mpam_read_reg(msc, MPAMF_ESR, &esr_low); > + if (ret) > + return ret; > + > + if (msc->has_extd_esr) { > + ret = __mpam_read_reg(msc, MPAMF_ESR + 4, &esr_high); > + if (ret) > + return ret; > + } > > - return ((u64)esr_high << 32) | esr_low; > + *res = ((u64)esr_high << 32) | esr_low; > + > + return 0; > } > > static void __mpam_part_sel_raw(u32 partsel, struct mpam_msc *msc) > @@ -993,7 +1012,7 @@ static int mpam_msc_hw_probe(struct mpam_msc *msc) > > /* Grab an IDR value to find out how many RIS there are */ > mutex_lock(&msc->part_sel_lock); > - idr = mpam_msc_read_idr(msc); > + mpam_msc_read_idr(msc, &idr); > mpam_read_partsel_reg(msc, IIDR, &msc->iidr); > > mutex_unlock(&msc->part_sel_lock); > @@ -1009,7 +1028,7 @@ static int mpam_msc_hw_probe(struct mpam_msc *msc) > for (ris_idx = 0; ris_idx <= msc->ris_max; ris_idx++) { > mutex_lock(&msc->part_sel_lock); > __mpam_part_sel(ris_idx, 0, msc); > - idr = mpam_msc_read_idr(msc); > + mpam_msc_read_idr(msc, &idr); > mutex_unlock(&msc->part_sel_lock); > > partid_max = FIELD_GET(MPAMF_IDR_PARTID_MAX, idr); > @@ -2492,7 +2511,7 @@ static irqreturn_t __mpam_irq_handler(int irq, struct mpam_msc *msc) > &msc->accessibility))) > return IRQ_NONE; > > - reg = mpam_msc_read_esr(msc); > + mpam_msc_read_esr(msc, ®); > > errcode = FIELD_GET(MPAMF_ESR_ERRCODE, reg); > if (!errcode)