Received-SPF: pass (google.com: domain of linux-kernel+bounces-156263-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
Message-ID: <a5029ad6-3e5e-46bf-881f-950a8a393956@intel.com>
Date: Tue, 23 Apr 2024 21:15:07 -0700
User-Agent: Mozilla Thunderbird
Subject: Re: [RFC PATCH v3 00/17] x86/resctrl : Support AMD Assignable
 Bandwidth Monitoring Counters (ABMC)
To: Dave Martin <Dave.Martin@arm.com>, "Moger, Babu" <babu.moger@amd.com>
CC: Peter Newman <peternewman@google.com>, <corbet@lwn.net>,
	<fenghua.yu@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>
References: <cover.1711674410.git.babu.moger@amd.com>
 <CALPaoCheW=Jz2R3gMKcgwQe6ONDrRqu3tUxeg=A3USg6BC8buA@mail.gmail.com>
 <7ccd59b8-9fe3-4d1f-82f5-f33d96dbf5ac@amd.com>
 <ZiaRewZJnLaDSx3m@e133380.arm.com>
 <004dcaeb-30ae-4b27-895a-4025a670fcbf@amd.com>
 <ZierjRNDMfg5swT8@e133380.arm.com>
Content-Language: en-US
From: Reinette Chatre <reinette.chatre@intel.com>
In-Reply-To: <ZierjRNDMfg5swT8@e133380.arm.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 8bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?SHRja3lFWG51alpLUzNLY2t4YWZqZ09SWDZONndvRVlva1d0RXQxeXBqcTRa?=
 =?utf-8?B?RFNsNEJ5MFZ2SWpJOGNZek54TWFLRElQTjlTWXc2MlZvZW1MUlVvM0ozL1ds?=
 =?utf-8?B?TnZZcm9qVGtNYWRtUnl6VU5HdllXTEZDT1YwSDlIODk2MUhzU1BDNmppbU9K?=
 =?utf-8?B?a0s2eG5jbyt6VkZsUFdhd2htQjlTUEZYUVozdDY3WksvWG9QY0plb2o3L0pu?=
 =?utf-8?B?RXQ0b2g0ZXNvZzgraFQyRU5NMk5tQUNkcndQVFNqdU1xZEVaNEUranoxTHdi?=
 =?utf-8?B?dnFpQmU4bVRuTlFFN0ZMYW9GaXVUcE5kazhHUkRDaTdZcnNTdWRYcldUK3Ru?=
 =?utf-8?B?MlNkNG9aT1M4ckpvVkwxd2N2dTYxMlFwOE53MmFnWEtEa0VHcFFQVXBoTTlQ?=
 =?utf-8?B?STZQTjNoZmJHKzM3Zkp4bkVBRy9ZUFdMckFFOHJRWWY4NU1tNW15cGc3WnMw?=
 =?utf-8?B?d0xQaG44eVZOOUxlUEdsYzNUd2ZEa3IxVU8xWi9ZaVVZMDRSZ2dGK25MTm9N?=
 =?utf-8?B?QnI3RU8wayt5STIrWHltUERPd0QvN2hHSUR6NzlzR1pvRmF2b2RxdE9WVjhB?=
 =?utf-8?B?R0ZaNnRWVW1lUlovK20rL1VXOHFhZjBDcmZzcEx6dWNBVE5BdGN4elEvaHdQ?=
 =?utf-8?B?VXdvRFY1cTVlOWtwV01keGh1Y25xdmdjVDBiVFQ3bE1sZ2Qvd1B5WFB3SElJ?=
 =?utf-8?B?czRKdjBxYzdhblJ2UUZPaWdnWmhIK0RPNTBya1BOUTlWemxLMTZDL1FFbGkx?=
 =?utf-8?B?aEJreElqS29UNVBiK1VJUjNCTEM2ak80SVVoWVYraWl2b1lWdFlTdUlFQU43?=
 =?utf-8?B?VWU2WVQybDc3Q3JlT1p3bytSdVUvTGo1WTNqbDMyZ2EwNWVvQzRBZWdMcjVP?=
 =?utf-8?B?NkowQ3VFUEppOGVvTUQ2NFJpZzJZdTUrT3ZNbEFLZm1OcVVkZVcvRHE3bXFm?=
 =?utf-8?B?N0dISTIzMlV6eUpzVGQ4MGtiSnFzb1B2UEwwVkNyaW1mUUMwN3EvU2VIbzdB?=
 =?utf-8?B?MDhlYmxuTm9pZG9hblcyWFVJS3ZxOTA5KzBPOWpyL1loS0M3dlYybzNVc3d4?=
 =?utf-8?B?bVMrbENWcVZSUjErS3VZZkpGK1NwekhoYkEzVFVZUzlaMjFrdWFGK3hmblc3?=
 =?utf-8?B?Qjd0cDNSMk0zV1dkZUk2WW81b3hVYmFYemVZRXJiZmNlSXRZaXRFS3FEWUpa?=
 =?utf-8?B?YVNpbjV6Z1RJZ3ZqMkpvVG5yZkFCZWVraTlqaEc3MkVrTHBERVpTdER3ZDdu?=
 =?utf-8?B?NUpQc1Z2Q1dFTm9GUFdRVVprdng3SkVib3h6TVNxdzBicmFRbTA5bzNhQWpI?=
 =?utf-8?B?QUZ6SEJNUHY2M1dIcmx0MVVwMUFpV2lQUG5DWko3SEJyUVhaRmF5RmxyeUJS?=
 =?utf-8?B?dGtVT053azF5RXVEakF5eGc5ajd2V2ViTGFWUVl4d0lmQXNoSENMaUtrS2Vj?=
 =?utf-8?B?OUdLL1VGWE5wSEVoWGMvRzlVdHpkMkZZRFdEQjFCc2NRL25KSXNpQjNOd2xo?=
 =?utf-8?B?ZlAxaUZlbmhBU1QzaHdxSUE3cVRMTUNiTXBXTGZQNGczWXA1WXdTb2tvd1lp?=
 =?utf-8?B?R0dJbTN4QnVRK0ZTTDZPdFRHeWRaZlgxRnlvemJRSFo4ZDlaMHZiTk9yS0RW?=
 =?utf-8?B?bXMvd0dqUHppbDBzS1VBS054eFdsWlpyU2ZEaTNhR0REdGJHczd2U3g2bWR6?=
 =?utf-8?B?dkoyOWRzSHQzMFNQUjVOWlg2eTUwUjd3Ly85OEtLTmZ1L3VhOUlhaWhuY0lR?=
 =?utf-8?B?NlZYcVlKcFZod0NWQkJsRlBtb1o4d2pOVnFJY2tnMzJieFNjTXNaUmhkd0dT?=
 =?utf-8?B?TFFXQ1A3NlEyYTljSlduUnQ2RitJN2VxZTRXdmFhY2xxbEhoS2VlSUFwdUJH?=
 =?utf-8?B?V1RUdnN0YXBwZlBhQjFFcVZWaytKSWdUYWR0SDI5WDY4SFVlUXNodjZFeWdX?=
 =?utf-8?B?d0FHTWlWZWVXY2UrMEF1ak9RREIvY2FGanlQWGhYbjBkWDFHTzRzOUQ4M3dm?=
 =?utf-8?B?cVpna3dWTXNwdXl1Qkc5WHlyQ1RRamg5RXJpSnRRV1JHeUFLTDR0bFlCY1Vp?=
 =?utf-8?B?MmtaOW5QNEdEQkdGa0dZM0ZFYUk3dHE4Q1RGcHRvZWwxbFRLVTVZTCtadVN1?=
 =?utf-8?B?bnRiWGFyaDlQdXl0ajdBWE5Qd3drcDRDOE50YlQzaHNuZ1kwanVGUGRiUzhv?=
 =?utf-8?B?cHc9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: adf108e0-4dfc-4795-312f-08dc64152260
X-MS-Exchange-CrossTenant-AuthSource: SJ2PR11MB7573.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 24 Apr 2024 04:15:11.4927
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: JT1DWkQCCl7KqS9ul3bJ7dOFaPIzi2ilyatNMhC1jrun7swyCNPqgbfO5Kr0ycUGRzCLXuzhRVNG43Fvka4vF+pJy/czTGYopt8OyH43Bqw=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS0PR11MB7927
X-OriginatorOrg: intel.com


On 4/23/2024 5:37 AM, Dave Martin wrote:
> On Mon, Apr 22, 2024 at 03:44:26PM -0500, Moger, Babu wrote:
>> Hi Dave,
>>
>> On 4/22/24 11:34, Dave Martin wrote:
>>> 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?
>>
>> We could. But I don't see the need for wildcard ("*") or ranges and
>> complexity that comes with that.
> 
> For "*", I mean that this would just stand for "all cpus", not a generic
> string match; apologies if I didn't make that clear.

(reading this by replacing "all cpus" with "all domains")

This sounds reasonable to me. It may indeed make the parsing simpler by
not needing the ugly checks Babu mentioned in [1].

Reinette

[1] https://lore.kernel.org/lkml/7ccd59b8-9fe3-4d1f-82f5-f33d96dbf5ac@amd.com/