Received: by 2002:a05:6a10:206:0:0:0:0 with SMTP id 6csp3050835pxj; Mon, 14 Jun 2021 13:12:30 -0700 (PDT) X-Google-Smtp-Source: ABdhPJw5fmjcLBYDodIDdHbfFhPaKBMkePyAQpuD/RKlzdr1aNfzZg+w9xTnvs4x9nhAQGBxoBwT X-Received: by 2002:a17:907:c02:: with SMTP id ga2mr530275ejc.215.1623701550154; Mon, 14 Jun 2021 13:12:30 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1623701550; cv=none; d=google.com; s=arc-20160816; b=IiMzwNMxAsewhs/NlN+hWrQtkYq1N0l0C2XyXWRH9XoYSZWAWB4ISZXd+jTusa+Et5 6j+c3VSoGOigZRgwYWiSUd+Wyus0Tquyrg/PiTPEF7fDuQGHUrDfRg1FJVPb+Hw12Kbj tDWpJb6eX7F0BiX2oPcmuJlR3xAm/zY4oo5NUvYkC0DQ0B6eZWrcOEiNTcoNUg4wjGlJ AxKPY8QxZRA5lq/8xee6ru4YN5tv3QwvetASMz4QrJMcYadIpezXsRsa5THmaagLTBvD GCja9mxrgxY/N+KHxtlnKXby45Lpb3QFWYQ/ulx0AIJJWBpQnLZpHOpMdrfWYjSSwtEN 99oA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from; bh=yUMWt1T1MhWZgypeA5I2iNSp6Cmp7QKkTHPaVY02/EI=; b=P8qRmgoCOqv0orn85Xgi9uC0+Dqq7t/UHful8lh9dsY94Sd3XhFVx0czusXsQdfTpo P5A10skQ6xaZQa7NdZSD7Z1TX8sj4XPRx/Fs2suZ9WPh8SgMZ2DeOP5DPt6ozQYRTkc+ 68oMfjabv5nK0EIh92866Uv9qbs3r2qLFwdZlSQvz9Zf/tg9HWzcbxhMwvqB85Wws5bU PT2FhU5t8xzLVBSsCcqmHciRR3lWyPewq2EKshwBb9/1rvS7ykkeEfHD+tRYrB/dh7y0 dwCZ+i2b0pAQe70Clk83Kcs4Nw0DyPFdO10kBEHVVTNfEYhUjxkssGE5Of6+XjEj3+ee 5Ffw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=arm.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id w8si12219948ejs.658.2021.06.14.13.12.07; Mon, 14 Jun 2021 13:12:30 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=arm.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S235177AbhFNUMb (ORCPT + 99 others); Mon, 14 Jun 2021 16:12:31 -0400 Received: from foss.arm.com ([217.140.110.172]:45584 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S235016AbhFNUMU (ORCPT ); Mon, 14 Jun 2021 16:12:20 -0400 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 D759911D4; Mon, 14 Jun 2021 13:10:16 -0700 (PDT) Received: from merodach.members.linode.com (unknown [172.31.20.19]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id EF00A3F694; Mon, 14 Jun 2021 13:10:14 -0700 (PDT) From: James Morse To: x86@kernel.org, linux-kernel@vger.kernel.org Cc: Fenghua Yu , Reinette Chatre , Thomas Gleixner , Ingo Molnar , Borislav Petkov , H Peter Anvin , Babu Moger , James Morse , shameerali.kolothum.thodi@huawei.com, Jamie Iles , D Scott Phillips OS , lcherian@marvell.com Subject: [PATCH v4 08/24] x86/resctrl: Add resctrl_arch_get_num_closid() Date: Mon, 14 Jun 2021 20:09:25 +0000 Message-Id: <20210614200941.12383-9-james.morse@arm.com> X-Mailer: git-send-email 2.20.1 In-Reply-To: <20210614200941.12383-1-james.morse@arm.com> References: <20210614200941.12383-1-james.morse@arm.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org To initialise struct resctrl_schema's num_closid, schemata_list_create() reaches into the architectures private structure to retrieve num_closid from the struct rdt_hw_resource. The 'half the closids' behaviour should be part of the filesystem parts of resctrl that are the same on any architecture. struct resctrl_schema's num_closid should include any correction for CDP. Having two properties called num_closid is likely to be confusing when they have different values. Add a helper to read the resource's num_closid from the arch code. This should return the number of closid that the resource supports, regardless of whether CDP is in use. Once the CDP resources are merged, schemata_list_create() can apply the correction itself. Using a type with an obvious size for the arch helper means changing the type of num_closid to u32, which matches the type already used by struct rdtgroup. reset_all_ctrls() does not use resctrl_arch_get_num_closid(), even though it sets up a structure for modifying the hardware. This function will be part of the architecture code, the maximum closid should be the maximum value the hardware has, regardless of the way resctrl is using it. All the uses of num_closid in core.c are naturally part of the architecture specific code. Reviewed-by: Jamie Iles Signed-off-by: James Morse --- Changes since v3: * Fixed a spelling mistake, removed four spaces. Changes since v2: * Added comment in rdt_hw_resource * Shuffled commit message, Changes since v1: * Rewrote commit message * Whitespace fixes * num_closid becomes u32 in all occurrences to reduce surprises --- arch/x86/kernel/cpu/resctrl/core.c | 5 +++++ arch/x86/kernel/cpu/resctrl/internal.h | 8 ++++++-- arch/x86/kernel/cpu/resctrl/rdtgroup.c | 4 ++-- include/linux/resctrl.h | 6 +++++- 4 files changed, 18 insertions(+), 5 deletions(-) diff --git a/arch/x86/kernel/cpu/resctrl/core.c b/arch/x86/kernel/cpu/resctrl/core.c index 9ba12c3d175a..b406cca56ed4 100644 --- a/arch/x86/kernel/cpu/resctrl/core.c +++ b/arch/x86/kernel/cpu/resctrl/core.c @@ -450,6 +450,11 @@ struct rdt_domain *get_domain_from_cpu(int cpu, struct rdt_resource *r) return NULL; } +u32 resctrl_arch_get_num_closid(struct rdt_resource *r) +{ + return resctrl_to_arch_res(r)->num_closid; +} + void rdt_ctrl_update(void *arg) { struct msr_param *m = arg; diff --git a/arch/x86/kernel/cpu/resctrl/internal.h b/arch/x86/kernel/cpu/resctrl/internal.h index e800de110230..5e6bfe27513c 100644 --- a/arch/x86/kernel/cpu/resctrl/internal.h +++ b/arch/x86/kernel/cpu/resctrl/internal.h @@ -366,7 +366,11 @@ struct rdt_parse_data { * struct rdt_hw_resource - arch private attributes of a resctrl resource * @conf_type: The type that should be used when configuring. temporary * @resctrl: Attributes of the resource used directly by resctrl. - * @num_closid: Maximum number of closid this hardware can support. + * @num_closid: Maximum number of closid this hardware can support, + * regardless of CDP. This is exposed via + * resctrl_arch_get_num_closid() to avoid confusion + * with struct resctrl_schema's property of the same name, + * which has been corrected for features like CDP. * @msr_base: Base MSR address for CBMs * @msr_update: Function pointer to update QOS MSRs * @mon_scale: cqm counter * mon_scale = occupancy in bytes @@ -379,7 +383,7 @@ struct rdt_parse_data { struct rdt_hw_resource { enum resctrl_conf_type conf_type; struct rdt_resource resctrl; - int num_closid; + u32 num_closid; unsigned int msr_base; void (*msr_update) (struct rdt_domain *d, struct msr_param *m, struct rdt_resource *r); diff --git a/arch/x86/kernel/cpu/resctrl/rdtgroup.c b/arch/x86/kernel/cpu/resctrl/rdtgroup.c index da2e1c3a414e..39ec0344963e 100644 --- a/arch/x86/kernel/cpu/resctrl/rdtgroup.c +++ b/arch/x86/kernel/cpu/resctrl/rdtgroup.c @@ -104,7 +104,7 @@ int closids_supported(void) static void closid_init(void) { struct resctrl_schema *s; - int rdt_min_closid = 32; + u32 rdt_min_closid = 32; /* Compute rdt_min_closid across all resources */ list_for_each_entry(s, &resctrl_schema_all, list) @@ -2134,7 +2134,7 @@ static int schemata_list_create(void) s->res = r; s->conf_type = resctrl_to_arch_res(r)->conf_type; - s->num_closid = resctrl_to_arch_res(r)->num_closid; + s->num_closid = resctrl_arch_get_num_closid(r); INIT_LIST_HEAD(&s->list); list_add(&s->list, &resctrl_schema_all); diff --git a/include/linux/resctrl.h b/include/linux/resctrl.h index de0bef438e54..78c6dafd42c1 100644 --- a/include/linux/resctrl.h +++ b/include/linux/resctrl.h @@ -177,6 +177,10 @@ struct resctrl_schema { struct list_head list; enum resctrl_conf_type conf_type; struct rdt_resource *res; - int num_closid; + u32 num_closid; }; + +/* The number of closid supported by this resource regardless of CDP */ +u32 resctrl_arch_get_num_closid(struct rdt_resource *r); + #endif /* _RESCTRL_H */ -- 2.30.2