[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <c60e32da-7935-439e-9f24-fd637512757d@amd.com>
Date: Thu, 18 Dec 2025 15:56:24 -0600
From: "Moger, Babu" <bmoger@....com>
To: "Luck, Tony" <tony.luck@...el.com>, Aaron Tomlin <atomlin@...mlin.com>,
"Chatre, Reinette" <reinette.chatre@...el.com>,
"Dave.Martin@....com" <Dave.Martin@....com>,
"james.morse@....com" <james.morse@....com>,
"babu.moger@....com" <babu.moger@....com>,
"tglx@...utronix.de" <tglx@...utronix.de>,
"mingo@...hat.com" <mingo@...hat.com>, "bp@...en8.de" <bp@...en8.de>,
"dave.hansen@...ux.intel.com" <dave.hansen@...ux.intel.com>
Cc: "sean@...e.io" <sean@...e.io>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: RE: [PATCH v2 3/3] x86/resctrl: Add "*" shorthand to set minimum
io_alloc CBM for all domains
On 12/18/2025 3:49 PM, Luck, Tony wrote:
>> + if (id && *id == '*') {
>
> That will accept arbitrary chars after the "*".
>
> if (id && !strcmp(id, "*"))
oh.ok. Thanks
>
>> What do you think?
>
> Why is io_alloc special? The same simple change could apply to parse_line() to allow
> setting all domains in a resource to be set to the same value:
>
Yes. I am fine with that approach. Treat "*" equivalent for all the
resources not just io_alloc.
> # echo "L3:*=fff" > schemata
>
> # echo "MB:*=50" > schemata
>
> -Tony
>
>
Powered by blists - more mailing lists