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 C0BE5C44500 for ; Mon, 6 Jul 2026 16:07: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=kTxfUTzrDkQ4rb9tkp4K3XRYrFPm4KlMs6IsaSS0ik8=; b=ABVD5gjBPjICi2mQy7i5S9ZeT6 4pTScqtkaM1PfqmBnimiSMJbfTVhGDh/1sqYbD1lh/gcACEXqsCwHAJACbKTxuPfODDbSPNB8GSM/ pREbqAmOZTecDZ9zWVzehvs8t9EVV7/FtHTNJ/ugFBYaR7+6TPqdLwfXXwVGshIUQ7etErfeclAbv m6nvZF3IMbPS8d0E587PiWSTYM8cyLGrfdxKksg7ZKwMhJLM849bE3sxLbjIsVJ/oaKTYT8xWmyfq XSQEbioCqcxNU2aRDpeZ4v36CtfiP9+r3uoDpLMnjfhBk0ZT976R7nMq/oCgMfJVf6RhHA2E+VVt+ tJfcHLMw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1wglqW-0000000CzXu-1jYh; Mon, 06 Jul 2026 16:07:00 +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 1wglqR-0000000CzWR-469p for linux-arm-kernel@lists.infradead.org; Mon, 06 Jul 2026 16:06:59 +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 B6F0D1BCA; Mon, 6 Jul 2026 09:06:50 -0700 (PDT) Received: from e134344.cambridge.arm.com (e134344.arm.com [10.2.212.8]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPA id E0CF93F905; Mon, 6 Jul 2026 09:06:51 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=arm.com; s=foss; t=1783354015; bh=SgSLAmpEHjQVvI3w3XPeQkB8UDBA1FlyFSv8NkcsKME=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=DpV8Zz5yn6JaDRYQoZD9lVZFmu14vmLQbfXsXSfBSNGCFGS7luIjjP9Qqf8oqqyNB +KZLdtwCHikV+GXqIEBRDichY0xdV144CV7u0Prsx9zaM4XVKHTPY3sOwhTXpFXir6 Opqb7CZaY6zyrQ7bYmPaZhMdV6olelyxV/ypDIrA= From: Ben Horgan To: ben.horgan@arm.com Cc: james.morse@arm.com, reinette.chatre@intel.com, fenghuay@nvidia.com, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, tglx@linutronix.de, mingo@redhat.com, bp@alien8.de, dave.hansen@linux.intel.com, hpa@zytor.com, corbet@lwn.net, x86@kernel.org, linux-doc@vger.kernel.org, dave.martin@arm.com Subject: [PATCH v4 1/3] x86,fs/resctrl: Add resctrl_arch_preconvert_bw() Date: Mon, 6 Jul 2026 17:06:37 +0100 Message-ID: <20260706160639.2136674-2-ben.horgan@arm.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20260706160639.2136674-1-ben.horgan@arm.com> References: <20260706160639.2136674-1-ben.horgan@arm.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.9.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20260706_090658_292779_231FBB27 X-CRM114-Status: GOOD ( 17.29 ) 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 On MPAM systems the rounding behaviour of the MBA control would be improved if the rounding in the fs/resctrl code is removed but this is not the case for x86. To allow any rounding or conversion of the bandwidth value provided by the user to be specified by the arch code a new arch hook is required. Introduce resctrl_arch_preconvert_bw(), and add its x86 implementation. This is currently unused in resctrl but when plumbed in it will replace the call to roundup() in bw_validate(). Signed-off-by: Dave Martin Signed-off-by: Ben Horgan Reviewed-by: Reinette Chatre --- Changes since Dave's v2: Split from larger patch and add commit message Update kernel-doc (Reinette) Changes since v3: Swap parameter order (Reinette) Change summary prefix include fs/resctrl (Reinette) val -> @val Add Reinette's R-b --- arch/x86/kernel/cpu/resctrl/ctrlmondata.c | 6 ++++++ include/linux/resctrl.h | 19 +++++++++++++++++++ 2 files changed, 25 insertions(+) diff --git a/arch/x86/kernel/cpu/resctrl/ctrlmondata.c b/arch/x86/kernel/cpu/resctrl/ctrlmondata.c index b20e705606b8..81a08526d3d1 100644 --- a/arch/x86/kernel/cpu/resctrl/ctrlmondata.c +++ b/arch/x86/kernel/cpu/resctrl/ctrlmondata.c @@ -16,9 +16,15 @@ #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt #include +#include #include "internal.h" +u32 resctrl_arch_preconvert_bw(const struct rdt_resource *r, u32 val) +{ + return roundup(val, (unsigned long)r->membw.bw_gran); +} + int resctrl_arch_update_one(struct rdt_resource *r, struct rdt_ctrl_domain *d, u32 closid, enum resctrl_conf_type t, u32 cfg_val) { diff --git a/include/linux/resctrl.h b/include/linux/resctrl.h index 73ff522448a0..1ebd4b90043a 100644 --- a/include/linux/resctrl.h +++ b/include/linux/resctrl.h @@ -504,6 +504,25 @@ bool resctrl_arch_mbm_cntr_assign_enabled(struct rdt_resource *r); */ int resctrl_arch_mbm_cntr_assign_set(struct rdt_resource *r, bool enable); +/** + * resctrl_arch_preconvert_bw() - Prepare bandwidth control value for arch use. + * @r: Resource whose schema was written. + * @val: Bandwidth control value written to the schemata file by userspace. + * + * Convert the user provided bandwidth control value to an appropriate form for + * consumption by the hardware driver for resource @r. Converted value is stored + * in rdt_ctrl_domain::staged_config[] for later consumption by + * resctrl_arch_update_domains(). Is not called when MBA software controller is + * enabled. + * + * Architectures for which this pre-conversion hook is not useful should supply + * an implementation of this function that just returns @val unmodified. + * + * Return: + * The converted value. + */ +u32 resctrl_arch_preconvert_bw(const struct rdt_resource *r, u32 val); + /* * Update the ctrl_val and apply this config right now. * Must be called on one of the domain's CPUs. -- 2.43.0