From nobody Fri Dec 19 19:01:35 2025 Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.9]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id C2ABE3090C9 for ; Mon, 13 Oct 2025 22:34:13 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=198.175.65.9 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1760394856; cv=none; b=lI7w1XiwfWJo6jJdLdmh9cFSuUBbK3IurOhu+f/vccLmIib4z1aStVk1fEw52hHFeka067APYzhzPN8EiL0Qetny4tzU6ljfKRGskgMtW0er3QOd4WBu/8Pmd5kbkJLM9p8am7cw28SWKE0TqirNneJrFkHFB7h3sVrnXHMPlNk= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1760394856; c=relaxed/simple; bh=tkKjneMjkAKhQ2DMKRriXTuPPq69NgdK6JOt5IQgoK0=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=BNQ/H3pHUtX/Cz7LrJHFLqcg/X1WmfphMtdqgXyRCmZ36cX4XfedRQRAIOvopb0vEGkQ7tyly3khWt4RuPGgl7bqM8ep4JvGzVPoEVSKKTSTJFSznO1Ob5Nb0DGe92cVTbFNSSQMf0UnKYDjgDjshYlewd/VXvgC0VuVW3PAVB8= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=intel.com; spf=pass smtp.mailfrom=intel.com; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b=l2hRAgZc; arc=none smtp.client-ip=198.175.65.9 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=intel.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=intel.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b="l2hRAgZc" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1760394854; x=1791930854; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=tkKjneMjkAKhQ2DMKRriXTuPPq69NgdK6JOt5IQgoK0=; b=l2hRAgZcf2p9cig0Gwjrd+IdqitAIEBmCS5Xe2N3kvOarEEIfXDfw+u0 yku6ULVjgLem+dtxq6bXJobWatzLlqbpvEpBdGtXfGYgUW6pK7g7r1Sxp UsG70XxFbLIx6t6dqoZOu3Xa/evrnT12W0c1NDdq0sCSznrgFbbFtiIp1 tVS1TUHw4urSElmm+BBXImwPlnzM0pDTfqfW/bdPjiOdmtFr9V7t8XttL IhoLMgofNlvchGwkY0Ojwn3t+s2QuFdbRnoI2/DxZurkyU5dUwhcEBTMe 9/jazBqt9AhLFoRmuwJNBI1qrgwaUeVqWQScnk2u9cfiBBRyGrogjUOVp A==; X-CSE-ConnectionGUID: /yeDZa6WTbGAQC0v7BzAKg== X-CSE-MsgGUID: Fs5dzlt8Ro+DYvCRBjUzgA== X-IronPort-AV: E=McAfee;i="6800,10657,11581"; a="85168885" X-IronPort-AV: E=Sophos;i="6.19,226,1754982000"; d="scan'208";a="85168885" Received: from fmviesa008.fm.intel.com ([10.60.135.148]) by orvoesa101.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 13 Oct 2025 15:34:02 -0700 X-CSE-ConnectionGUID: GejNULltT5iMJLJ2b3l8BQ== X-CSE-MsgGUID: P4fsl1GlS0eRsMdCQFPjWQ== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.19,226,1754982000"; d="scan'208";a="182145313" Received: from sramkris-mobl1.amr.corp.intel.com (HELO agluck-desk3.intel.com) ([10.124.222.79]) by fmviesa008-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 13 Oct 2025 15:34:02 -0700 From: Tony Luck To: Fenghua Yu , Reinette Chatre , Maciej Wieczor-Retman , Peter Newman , James Morse , Babu Moger , Drew Fustini , Dave Martin , Chen Yu Cc: x86@kernel.org, linux-kernel@vger.kernel.org, patches@lists.linux.dev, Tony Luck Subject: [PATCH v12 20/31] fs/resctrl: Refactor mkdir_mondata_subdir() Date: Mon, 13 Oct 2025 15:33:34 -0700 Message-ID: <20251013223348.103390-21-tony.luck@intel.com> X-Mailer: git-send-email 2.51.0 In-Reply-To: <20251013223348.103390-1-tony.luck@intel.com> References: <20251013223348.103390-1-tony.luck@intel.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" mkdir_mondata_subdir() is unreasonably complicated because of the support f= or Sub-NUMA Cluster (SNC) mode. Split out the SNC code into a helper function to make it easier to add supp= ort for a new telemetry resource. Suggested-by: Reinette Chatre Signed-off-by: Tony Luck --- fs/resctrl/rdtgroup.c | 83 +++++++++++++++++++++++++++---------------- 1 file changed, 52 insertions(+), 31 deletions(-) diff --git a/fs/resctrl/rdtgroup.c b/fs/resctrl/rdtgroup.c index c9d2cc1fd8bf..b7eaa4388768 100644 --- a/fs/resctrl/rdtgroup.c +++ b/fs/resctrl/rdtgroup.c @@ -3196,22 +3196,16 @@ static void rmdir_mondata_subdir_allrdtgrp(struct r= dt_resource *r, =20 static int mon_add_all_files(struct kernfs_node *kn, struct rdt_domain_hdr= *hdr, struct rdt_resource *r, struct rdtgroup *prgrp, - bool do_sum) + int domid, bool do_sum) { struct rmid_read rr =3D {0}; - struct rdt_l3_mon_domain *d; struct mon_data *priv; struct mon_evt *mevt; - int ret, domid; - - if (!domain_header_is_valid(hdr, RESCTRL_MON_DOMAIN, RDT_RESOURCE_L3)) - return -EINVAL; + int ret; =20 - d =3D container_of(hdr, struct rdt_l3_mon_domain, hdr); for_each_mon_event(mevt) { if (mevt->rid !=3D r->rid || !mevt->enabled) continue; - domid =3D do_sum ? d->ci_id : d->hdr.id; priv =3D mon_get_kn_priv(r->rid, domid, mevt, do_sum); if (WARN_ON_ONCE(!priv)) return -EINVAL; @@ -3227,24 +3221,20 @@ static int mon_add_all_files(struct kernfs_node *kn= , struct rdt_domain_hdr *hdr, return 0; } =20 -static int mkdir_mondata_subdir(struct kernfs_node *parent_kn, - struct rdt_domain_hdr *hdr, - struct rdt_resource *r, struct rdtgroup *prgrp) +static int mkdir_mondata_subdir_snc(struct kernfs_node *parent_kn, + struct rdt_domain_hdr *hdr, + struct rdt_resource *r, struct rdtgroup *prgrp) { struct kernfs_node *kn, *ckn; struct rdt_l3_mon_domain *d; char name[32]; - bool snc_mode; int ret =3D 0; =20 - lockdep_assert_held(&rdtgroup_mutex); - if (!domain_header_is_valid(hdr, RESCTRL_MON_DOMAIN, RDT_RESOURCE_L3)) return -EINVAL; =20 d =3D container_of(hdr, struct rdt_l3_mon_domain, hdr); - snc_mode =3D r->mon_scope =3D=3D RESCTRL_L3_NODE; - sprintf(name, "mon_%s_%02d", r->name, snc_mode ? d->ci_id : d->hdr.id); + sprintf(name, "mon_%s_%02d", r->name, d->ci_id); kn =3D kernfs_find_and_get(parent_kn, name); if (kn) { /* @@ -3260,27 +3250,58 @@ static int mkdir_mondata_subdir(struct kernfs_node = *parent_kn, ret =3D rdtgroup_kn_set_ugid(kn); if (ret) goto out_destroy; - ret =3D mon_add_all_files(kn, hdr, r, prgrp, snc_mode); + ret =3D mon_add_all_files(kn, hdr, r, prgrp, d->ci_id, true); if (ret) goto out_destroy; } =20 - if (snc_mode) { - sprintf(name, "mon_sub_%s_%02d", r->name, hdr->id); - ckn =3D kernfs_create_dir(kn, name, parent_kn->mode, prgrp); - if (IS_ERR(ckn)) { - ret =3D -EINVAL; - goto out_destroy; - } + sprintf(name, "mon_sub_%s_%02d", r->name, hdr->id); + ckn =3D kernfs_create_dir(kn, name, parent_kn->mode, prgrp); + if (IS_ERR(ckn)) { + ret =3D -EINVAL; + goto out_destroy; + } =20 - ret =3D rdtgroup_kn_set_ugid(ckn); - if (ret) - goto out_destroy; + ret =3D rdtgroup_kn_set_ugid(ckn); + if (ret) + goto out_destroy; =20 - ret =3D mon_add_all_files(ckn, hdr, r, prgrp, false); - if (ret) - goto out_destroy; - } + ret =3D mon_add_all_files(ckn, hdr, r, prgrp, hdr->id, false); + if (ret) + goto out_destroy; + + kernfs_activate(kn); + return 0; + +out_destroy: + kernfs_remove(kn); + return ret; +} + +static int mkdir_mondata_subdir(struct kernfs_node *parent_kn, + struct rdt_domain_hdr *hdr, + struct rdt_resource *r, struct rdtgroup *prgrp) +{ + struct kernfs_node *kn; + char name[32]; + int ret =3D 0; + + lockdep_assert_held(&rdtgroup_mutex); + + if (r->mon_scope =3D=3D RESCTRL_L3_NODE) + return mkdir_mondata_subdir_snc(parent_kn, hdr, r, prgrp); + + sprintf(name, "mon_%s_%02d", r->name, hdr->id); + kn =3D kernfs_create_dir(parent_kn, name, parent_kn->mode, prgrp); + if (IS_ERR(kn)) + return PTR_ERR(kn); + + ret =3D rdtgroup_kn_set_ugid(kn); + if (ret) + goto out_destroy; + ret =3D mon_add_all_files(kn, hdr, r, prgrp, hdr->id, false); + if (ret) + goto out_destroy; =20 kernfs_activate(kn); return 0; --=20 2.51.0