[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <bc7340be-9728-4938-9196-8bc48d56c068@intel.com>
Date: Fri, 25 Jul 2025 16:36:24 -0700
From: Reinette Chatre <reinette.chatre@...el.com>
To: Tony Luck <tony.luck@...el.com>, Fenghua Yu <fenghuay@...dia.com>, "Maciej
Wieczor-Retman" <maciej.wieczor-retman@...el.com>, Peter Newman
<peternewman@...gle.com>, James Morse <james.morse@....com>, Babu Moger
<babu.moger@....com>, Drew Fustini <dfustini@...libre.com>, Dave Martin
<Dave.Martin@....com>, Chen Yu <yu.c.chen@...el.com>
CC: <x86@...nel.org>, <linux-kernel@...r.kernel.org>,
<patches@...ts.linux.dev>
Subject: Re: [PATCH v7 16/31] x86,fs/resctrl: Add and initialize rdt_resource
for package scope core monitor
Hi Tony,
In subject, should "core" be dropped?
On 7/11/25 4:53 PM, Tony Luck wrote:
> Add a new PERF_PKG resource and introduce package level scope for
> monitoring these events so that CPU hotplug notifiers can build domains
It is not obvious what "these events" refer to here.
> at the package granularity.
>
> Use the physical package ID available via topology_physical_package_id()
> to identify the monitoring domains with package level scope.
>
> This enables user space to use
> /sys/devices/system/cpu/cpuX/topology/physical_package_id
> to identify the monitoring domain a CPU is associated with.
Above two paragraphs can be merged?
>
> Signed-off-by: Tony Luck <tony.luck@...el.com>
> ---
Patch looks good.
Reinette
Powered by blists - more mailing lists