Received-SPF: pass (google.com: domain of linux-kernel+bounces-153710-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161;
Date: Mon, 22 Apr 2024 17:34:03 +0100
From: Dave Martin <Dave.Martin@arm.com>
To: "Moger, Babu" <babu.moger@amd.com>
Cc: Peter Newman <peternewman@google.com>, corbet@lwn.net,
	fenghua.yu@intel.com, reinette.chatre@intel.com, tglx@linutronix.de,
	mingo@redhat.com, bp@alien8.de, dave.hansen@linux.intel.com,
	x86@kernel.org, hpa@zytor.com, paulmck@kernel.org,
	rdunlap@infradead.org, tj@kernel.org, peterz@infradead.org,
	yanjiewtw@gmail.com, kim.phillips@amd.com, lukas.bulwahn@gmail.com,
	seanjc@google.com, jmattson@google.com, leitao@debian.org,
	jpoimboe@kernel.org, rick.p.edgecombe@intel.com,
	kirill.shutemov@linux.intel.com, jithu.joseph@intel.com,
	kai.huang@intel.com, kan.liang@linux.intel.com,
	daniel.sneddon@linux.intel.com, pbonzini@redhat.com,
	sandipan.das@amd.com, ilpo.jarvinen@linux.intel.com,
	maciej.wieczor-retman@intel.com, linux-doc@vger.kernel.org,
	linux-kernel@vger.kernel.org, eranian@google.com,
	james.morse@arm.com
Subject: Re: [RFC PATCH v3 00/17] x86/resctrl : Support AMD Assignable
 Bandwidth Monitoring Counters (ABMC)
Message-ID: <ZiaRewZJnLaDSx3m@e133380.arm.com>
References: <cover.1711674410.git.babu.moger@amd.com>
 <CALPaoCheW=Jz2R3gMKcgwQe6ONDrRqu3tUxeg=A3USg6BC8buA@mail.gmail.com>
 <7ccd59b8-9fe3-4d1f-82f5-f33d96dbf5ac@amd.com>
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <7ccd59b8-9fe3-4d1f-82f5-f33d96dbf5ac@amd.com>

Hi Babu,

On Thu, Apr 04, 2024 at 03:02:45PM -0500, Moger, Babu wrote:
> Hi Peter,
> 
> 
> On 4/4/24 14:08, Peter Newman wrote:
> > Hi Babu,
> > 
> > On Thu, Mar 28, 2024 at 6:07 PM Babu Moger <babu.moger@amd.com> wrote:
> >>    The list follows the following format:
> >>
> >>        * Default CTRL_MON group:
> >>                "//<domain_id>=<assignment_flags>"
> >>
> >>        * Non-default CTRL_MON group:
> >>                "<CTRL_MON group>//<domain_id>=<assignment_flags>"
> >>
> >>        * Child MON group of default CTRL_MON group:
> >>                "/<MON group>/<domain_id>=<assignment_flags>"
> >>
> >>        * Child MON group of non-default CTRL_MON group:
> >>                "<CTRL_MON group>/<MON group>/<domain_id>=<assignment_flags>"
> >>
> >>        Assignment flags can be one of the following:
> >>
> >>         t  MBM total event is assigned
> >>         l  MBM local event is assigned
> >>         tl Both total and local MBM events are assigned
> >>         _  None of the MBM events are assigned
> >>
> >>         Examples:
> >>
> >>         # cat /sys/fs/resctrl/info/L3_MON/mbm_assign_control
> >>         non_defult_group//0=tl;1=tl;2=tl;3=tl;4=tl;5=tl;6=tl;7=tl;
> >>         non_defult_group/non_default_mon1/0=tl;1=tl;2=tl;3=tl;4=tl;5=tl;6=tl;7=tl;
> >>         //0=tl;1=tl;2=tl;3=tl;4=tl;5=tl;6=tl;7=tl;
> >>         /default_mon1/0=tl;1=tl;2=tl;3=tl;4=tl;5=tl;6=tl;7=tl;
> >>
> >>         There are four groups and all the groups have local and total event assigned.
> >>
> >>         "//" - This is a default CONTROL MON group
> >>
> >>         "non_defult_group//" - This is non default CONTROL MON group
> >>
> >>         "/default_mon1/"  - This is Child MON group of the defult group
> >>
> >>         "non_defult_group/non_default_mon1/" - This is child MON group of the non default group
> >>
> >>         =tl means both total and local events are assigned.
> > 
> > I recall there was supposed to be a way to perform the same update on
> > all domains together so that it isn't tedious to not do per-domain
> 
> Yes. Correct. Reinette suggested to have "no domains" means ALL the domains.

Would "*" be more intuitive?

Whatever is done here to describe the "wildcard node", would it be worth
having the node field parse the same way in the "schemata" files?

Is there any merit in having range match expressions, e.g. something like

	0-3,8-11=foo;4-7,12-*=bar

(The latter is obvious feature creep though, so a real use case for this
would be needed to justify it.  I don't have one right now...)

[...]

Cheers
---Dave