From: "Moger, Babu" <bmoger@amd.com>
To: Tony Luck <tony.luck@intel.com>, "Moger, Babu" <babu.moger@amd.com>
Cc: "corbet@lwn.net" <corbet@lwn.net>,
"Yu, Fenghua" <fenghua.yu@intel.com>,
"Chatre, Reinette" <reinette.chatre@intel.com>,
"tglx@linutronix.de" <tglx@linutronix.de>,
"mingo@redhat.com" <mingo@redhat.com>,
"bp@alien8.de" <bp@alien8.de>,
"dave.hansen@linux.intel.com" <dave.hansen@linux.intel.com>,
"x86@kernel.org" <x86@kernel.org>,
"hpa@zytor.com" <hpa@zytor.com>,
"paulmck@kernel.org" <paulmck@kernel.org>,
"rdunlap@infradead.org" <rdunlap@infradead.org>,
"tj@kernel.org" <tj@kernel.org>,
"peterz@infradead.org" <peterz@infradead.org>,
"yanjiewtw@gmail.com" <yanjiewtw@gmail.com>,
"kim.phillips@amd.com" <kim.phillips@amd.com>,
"lukas.bulwahn@gmail.com" <lukas.bulwahn@gmail.com>,
"seanjc@google.com" <seanjc@google.com>,
"jmattson@google.com" <jmattson@google.com>,
"leitao@debian.org" <leitao@debian.org>,
"jpoimboe@kernel.org" <jpoimboe@kernel.org>,
"Edgecombe, Rick P" <rick.p.edgecombe@intel.com>,
"kirill.shutemov@linux.intel.com"
<kirill.shutemov@linux.intel.com>,
"Joseph, Jithu" <jithu.joseph@intel.com>,
"Huang, Kai" <kai.huang@intel.com>,
"kan.liang@linux.intel.com" <kan.liang@linux.intel.com>,
"daniel.sneddon@linux.intel.com" <daniel.sneddon@linux.intel.com>,
"pbonzini@redhat.com" <pbonzini@redhat.com>,
"sandipan.das@amd.com" <sandipan.das@amd.com>,
"ilpo.jarvinen@linux.intel.com" <ilpo.jarvinen@linux.intel.com>,
"peternewman@google.com" <peternewman@google.com>,
"Wieczor-Retman, Maciej" <maciej.wieczor-retman@intel.com>,
"linux-doc@vger.kernel.org" <linux-doc@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"Eranian, Stephane" <eranian@google.com>,
"james.morse@arm.com" <james.morse@arm.com>
Subject: Re: [PATCH v8 08/25] x86/resctrl: Introduce interface to display number of monitoring counters
Date: Fri, 11 Oct 2024 15:49:48 -0500 [thread overview]
Message-ID: <8af0ce3a-1327-3ffc-ac5c-e495f9cdf5d0@amd.com> (raw)
In-Reply-To: <Zwlj5TQxZphcuDSR@agluck-desk3.sc.intel.com>
Hi Tony,
On 10/11/2024 12:44 PM, Tony Luck wrote:
> On Thu, Oct 10, 2024 at 03:32:08PM -0500, Moger, Babu wrote:
>>> # cat /sys/fs/resctrl/info/L3_MON/mbm_assign_control
>>> //0=tl;1=tl;
>>>
>>> there aren't separate counts from each of domain 0 and domain 1.
>>
>> Yes. There is. Each domain has its own count. I am not sure about your config.
>
> I've been reading the code and see better now.
>
> There are a bunch (32) of counters per domain.
>
> But you have a system-wide allocator. So when making
> a group you may allocate counters 2 and 3 for total
> and local respectively. Then configure the local instance
> of counter 2 on each domain (recording that in the per-domain
> bitmap) for total bandwidth. Ditto for counter 3 instances
> on each domain.
Yes. That is correct.
>
> If the user updates the configuration to stop counting
> on domain 1. Then the per-domain bitmap is updated to
> show counters 2 and 3 are no longer in use on this domain.
> But those counters aren't freed (because domain 0 is still
> using them).
Yes. Correct.
>
> Is there some hardware limitation that would prevent
> re-using domain 1 counters 2 & 3 for some other group (RMID)?
>
> Or is this just a s/w implementation detail because
> you have a system wide allocator for counters?
>
There is no hardware limitation. It is how resctrl is designed.
In case of Intel(with two sockets, 16 CLOSIDs), You can only create 16
groups. Each group will have two domains(domain 0 for socket 0 and
domain 1 for socket 1).
# cat schemata
MB:0=100;1=100
L3:0=ffff;1=ffff;
We may have to think of addressing this sometime in the future.
--
- Babu Moger
next prev parent reply other threads:[~2024-10-11 20:49 UTC|newest]
Thread overview: 124+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-10-09 17:39 [PATCH v8 00/25] x86/resctrl : Support AMD Assignable Bandwidth Monitoring Counters (ABMC) Babu Moger
2024-10-09 17:39 ` [PATCH v8 01/25] x86/cpufeatures: Add support for " Babu Moger
2024-10-09 17:39 ` [PATCH v8 02/25] x86/resctrl: Add ABMC feature in the command line options Babu Moger
2024-10-16 3:06 ` Reinette Chatre
2024-10-09 17:39 ` [PATCH v8 03/25] x86/resctrl: Consolidate monitoring related data from rdt_resource Babu Moger
2024-10-09 17:39 ` [PATCH v8 04/25] x86/resctrl: Detect Assignable Bandwidth Monitoring feature details Babu Moger
2024-10-16 3:06 ` Reinette Chatre
2024-10-09 17:39 ` [PATCH v8 05/25] x86/resctrl: Introduce resctrl_file_fflags_init() to initialize fflags Babu Moger
2024-10-09 17:39 ` [PATCH v8 06/25] x86/resctrl: Add support to enable/disable AMD ABMC feature Babu Moger
2024-10-11 18:14 ` Tony Luck
2024-10-11 20:53 ` Moger, Babu
2024-10-16 3:07 ` Reinette Chatre
2024-10-09 17:39 ` [PATCH v8 07/25] x86/resctrl: Introduce the interface to display monitor mode Babu Moger
2024-10-09 22:42 ` Tony Luck
2024-10-10 14:54 ` Moger, Babu
2024-10-10 15:07 ` Luck, Tony
2024-10-10 15:30 ` Moger, Babu
2024-10-10 16:02 ` Luck, Tony
2024-10-11 22:24 ` Reinette Chatre
2024-10-14 15:16 ` Moger, Babu
2024-10-16 3:12 ` Reinette Chatre
2024-10-16 15:57 ` Moger, Babu
2024-10-16 16:25 ` Reinette Chatre
2024-10-09 17:39 ` [PATCH v8 08/25] x86/resctrl: Introduce interface to display number of monitoring counters Babu Moger
2024-10-09 22:49 ` Tony Luck
2024-10-10 15:12 ` Moger, Babu
2024-10-10 15:58 ` Luck, Tony
2024-10-10 16:57 ` Moger, Babu
2024-10-10 17:08 ` Luck, Tony
2024-10-10 18:36 ` Moger, Babu
2024-10-10 18:57 ` Luck, Tony
2024-10-10 20:32 ` Moger, Babu
2024-10-11 17:44 ` Tony Luck
2024-10-11 20:49 ` Moger, Babu [this message]
2024-10-11 21:36 ` Tony Luck
2024-10-14 16:46 ` Reinette Chatre
2024-10-14 17:20 ` Moger, Babu
2024-10-14 17:49 ` Luck, Tony
2024-10-14 19:21 ` Moger, Babu
2024-10-14 19:51 ` Luck, Tony
2024-10-14 20:05 ` Reinette Chatre
2024-10-14 20:32 ` Moger, Babu
2024-10-24 17:29 ` Moger, Babu
2024-10-24 17:37 ` Luck, Tony
2024-10-25 20:31 ` Moger, Babu
2024-10-14 16:59 ` Reinette Chatre
2024-10-14 19:23 ` Moger, Babu
2024-10-14 16:25 ` Reinette Chatre
2024-10-14 17:46 ` Moger, Babu
2024-10-14 18:30 ` Reinette Chatre
2024-10-14 18:51 ` Moger, Babu
2024-10-09 17:39 ` [PATCH v8 09/25] x86/resctrl: Add __init attribute to dom_data_init() Babu Moger
2024-10-16 3:13 ` Reinette Chatre
2024-10-16 17:32 ` Moger, Babu
2024-10-16 18:55 ` Reinette Chatre
2024-10-16 20:18 ` Moger, Babu
2024-10-09 17:39 ` [PATCH v8 10/25] x86/resctrl: Introduce bitmap mbm_cntr_free_map to track assignable counters Babu Moger
2024-10-16 3:14 ` Reinette Chatre
2024-10-17 16:55 ` Moger, Babu
2024-10-09 17:39 ` [PATCH v8 11/25] x86/resctrl: Introduce mbm_total_cfg and mbm_local_cfg in struct rdt_hw_mon_domain Babu Moger
2024-10-16 3:15 ` Reinette Chatre
2024-10-09 17:39 ` [PATCH v8 12/25] x86/resctrl: Remove MSR reading of event configuration value Babu Moger
2024-10-16 3:16 ` Reinette Chatre
2024-10-17 17:59 ` Moger, Babu
2024-10-09 17:39 ` [PATCH v8 13/25] x86/resctrl: Introduce mbm_cntr_map to track assignable counters at domain Babu Moger
2024-10-16 3:19 ` Reinette Chatre
2024-10-09 17:39 ` [PATCH v8 14/25] x86/resctrl: Add data structures and definitions for ABMC assignment Babu Moger
2024-10-16 3:21 ` Reinette Chatre
2024-10-17 18:52 ` Moger, Babu
2024-10-17 21:13 ` Reinette Chatre
2024-10-17 23:02 ` Moger, Babu
2024-10-09 17:39 ` [PATCH v8 15/25] x86/resctrl: Introduce cntr_id in mongroup for assignments Babu Moger
2024-10-16 3:22 ` Reinette Chatre
2024-10-17 19:19 ` Moger, Babu
2024-10-09 17:39 ` [PATCH v8 16/25] x86/resctrl: Implement resctrl_arch_config_cntr() to assign a counter with ABMC Babu Moger
2024-10-16 3:23 ` Reinette Chatre
2024-10-17 22:44 ` Moger, Babu
2024-10-09 17:39 ` [PATCH v8 17/25] x86/resctrl: Add the interface to assign/update counter assignment Babu Moger
2024-10-16 3:25 ` Reinette Chatre
2024-10-17 22:56 ` Moger, Babu
2024-10-18 15:59 ` Reinette Chatre
2024-10-21 14:40 ` Moger, Babu
2024-10-21 15:31 ` Reinette Chatre
2024-10-22 1:15 ` Moger, Babu
2024-10-09 17:39 ` [PATCH v8 18/25] x86/resctrl: Add the interface to unassign a MBM counter Babu Moger
2024-10-16 3:29 ` Reinette Chatre
2024-10-17 23:11 ` Moger, Babu
2024-10-18 16:06 ` Reinette Chatre
2024-10-09 17:39 ` [PATCH v8 19/25] x86/resctrl: Auto assign/unassign counters when mbm_cntr_assign is enabled Babu Moger
2024-10-11 17:17 ` Tony Luck
2024-10-11 21:17 ` Moger, Babu
2024-10-11 21:33 ` Luck, Tony
2024-10-14 15:43 ` Moger, Babu
2024-10-14 16:18 ` Luck, Tony
2024-10-14 16:35 ` Moger, Babu
2024-10-15 2:39 ` Reinette Chatre
2024-10-15 15:43 ` Moger, Babu
2024-10-15 16:57 ` Luck, Tony
2024-10-15 17:18 ` Reinette Chatre
2024-10-15 20:42 ` Moger, Babu
2024-10-16 3:30 ` Reinette Chatre
2024-10-18 14:22 ` Moger, Babu
2024-10-09 17:39 ` [PATCH v8 20/25] x86/resctrl: Report "Unassigned" for MBM events in mbm_cntr_assign mode Babu Moger
2024-10-11 17:23 ` Tony Luck
2024-10-11 21:21 ` Moger, Babu
2024-10-16 3:31 ` Reinette Chatre
2024-10-18 14:31 ` Moger, Babu
2024-10-09 17:39 ` [PATCH v8 21/25] x86/resctrl: Introduce the interface to switch between monitor modes Babu Moger
2024-10-16 3:36 ` Reinette Chatre
2024-10-18 15:13 ` Moger, Babu
2024-10-09 17:39 ` [PATCH v8 22/25] x86/resctrl: Configure mbm_cntr_assign mode if supported Babu Moger
2024-10-09 17:39 ` [PATCH v8 23/25] x86/resctrl: Update assignments on event configuration changes Babu Moger
2024-10-16 3:40 ` Reinette Chatre
2024-10-18 15:50 ` Moger, Babu
2024-10-09 17:39 ` [PATCH v8 24/25] x86/resctrl: Introduce interface to list assignment states of all the groups Babu Moger
2024-10-16 3:40 ` Reinette Chatre
2024-10-21 14:56 ` Moger, Babu
2024-10-09 17:39 ` [PATCH v8 25/25] x86/resctrl: Introduce interface to modify assignment states of " Babu Moger
2024-10-16 3:43 ` Reinette Chatre
2024-10-21 17:04 ` Moger, Babu
2024-10-21 17:20 ` Reinette Chatre
2024-10-22 1:12 ` Moger, Babu
2024-10-16 3:05 ` [PATCH v8 00/25] x86/resctrl : Support AMD Assignable Bandwidth Monitoring Counters (ABMC) Reinette Chatre
2024-10-21 17:09 ` Moger, Babu
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=8af0ce3a-1327-3ffc-ac5c-e495f9cdf5d0@amd.com \
--to=bmoger@amd.com \
--cc=babu.moger@amd.com \
--cc=bp@alien8.de \
--cc=corbet@lwn.net \
--cc=daniel.sneddon@linux.intel.com \
--cc=dave.hansen@linux.intel.com \
--cc=eranian@google.com \
--cc=fenghua.yu@intel.com \
--cc=hpa@zytor.com \
--cc=ilpo.jarvinen@linux.intel.com \
--cc=james.morse@arm.com \
--cc=jithu.joseph@intel.com \
--cc=jmattson@google.com \
--cc=jpoimboe@kernel.org \
--cc=kai.huang@intel.com \
--cc=kan.liang@linux.intel.com \
--cc=kim.phillips@amd.com \
--cc=kirill.shutemov@linux.intel.com \
--cc=leitao@debian.org \
--cc=linux-doc@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lukas.bulwahn@gmail.com \
--cc=maciej.wieczor-retman@intel.com \
--cc=mingo@redhat.com \
--cc=paulmck@kernel.org \
--cc=pbonzini@redhat.com \
--cc=peternewman@google.com \
--cc=peterz@infradead.org \
--cc=rdunlap@infradead.org \
--cc=reinette.chatre@intel.com \
--cc=rick.p.edgecombe@intel.com \
--cc=sandipan.das@amd.com \
--cc=seanjc@google.com \
--cc=tglx@linutronix.de \
--cc=tj@kernel.org \
--cc=tony.luck@intel.com \
--cc=x86@kernel.org \
--cc=yanjiewtw@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).