Received-SPF: pass (google.com: domain of linux-kernel+bounces-186722-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
Message-ID: <7887abff-8118-4064-8cb6-3f4af8cc4b2a@intel.com>
Date: Wed, 22 May 2024 14:12:20 -0700
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v18 07/17] x86/resctrl: Prepare for new Sub-NUMA (SNC)
 cluster monitor files
To: Tony Luck <tony.luck@intel.com>, Fenghua Yu <fenghua.yu@intel.com>, Maciej
 Wieczor-Retman <maciej.wieczor-retman@intel.com>, Peter Newman
	<peternewman@google.com>, James Morse <james.morse@arm.com>, Babu Moger
	<babu.moger@amd.com>, Drew Fustini <dfustini@baylibre.com>, Dave Martin
	<Dave.Martin@arm.com>
CC: <x86@kernel.org>, <linux-kernel@vger.kernel.org>,
	<patches@lists.linux.dev>
References: <20240515222326.74166-1-tony.luck@intel.com>
 <20240515222326.74166-8-tony.luck@intel.com>
Content-Language: en-US
From: Reinette Chatre <reinette.chatre@intel.com>
In-Reply-To: <20240515222326.74166-8-tony.luck@intel.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?RnJKSU11ejdPOHJTTTkwMXpoVkFydk92ZXFVLy80NlF0QWpLNXlqdDJBRWsv?=
 =?utf-8?B?VmdQOVlGVnFDZ1h4NTJQMWltdTRvc2NPL1p4cU56UTRlN0pYYzd6OVcza0Zl?=
 =?utf-8?B?WWM3RWFIS3JnYzlQTEhpTzFsOUhGSUNVcnA0RU9hMFNKNmptNDN5NTB4Z3hZ?=
 =?utf-8?B?eGc4RHAzc3RlRFNzb1R2SmwxcTY0cFVkTWZRNmw3b2FRVU44cGpBRDFwM1VZ?=
 =?utf-8?B?UExiK3hPcEw1NjFkTWRoYTdNb3V1WXlzaTJoMWswemZMT2Vnc3J6VWIwVk1k?=
 =?utf-8?B?RlQzV29LN21oS3l2TzV4djBUMFNzeDZrdTBRVVlkSmJLbkV5a1d1enZNQjJu?=
 =?utf-8?B?NVhETlNqanJVOWE2Uno3aVFnMGM5Mnl6L3VvSGNjTWlqMURlQ1ovVDllOTNU?=
 =?utf-8?B?K2orcU5MKzg3emxIdDlqZ09vSkdLNmtRNzZnMHFQem82NE9YVWo5TC9pbVJB?=
 =?utf-8?B?QnE0YmZPK09JSjVIYVF4dlY0NlJqOWlIMVlOWFJZaUoxRGJ0cE8vQkhYSWdD?=
 =?utf-8?B?MFNDYnRJRkRaNmxoam1DYkZwQ0hROTZ2ZEwxcVdLWVkvd2VVYTRFUnhrTkpC?=
 =?utf-8?B?alFYUDlEQkcyVW5NbW5JQTNaWWl0Qmd6N3dubW01cEVYNjk4Y0Y1TUlZWnlH?=
 =?utf-8?B?aFZLRVlIOFM5SFEzYmhOYXdDcExac0Z0THN2aE5scXMyWTAraHdHVnJoOGRk?=
 =?utf-8?B?UXJqYXBEV1JKMUdPUFpHZmY0TDJ2c25GQjBLazFubHNWQ3B6dnpVYkxmUmxD?=
 =?utf-8?B?N1ZuOCtOQWExZ1pZMDR3d1JHR3VRbllaWjJMaVVrTWF6WW1vOUZDaGw1aGU3?=
 =?utf-8?B?VlBzTDZyRTd4SUt4OEtwS25sOXNqc0ZqU2s4NXJkaCtIZ1ZzSlpyTFpwNFBL?=
 =?utf-8?B?dzFLNzYyM1lFYXpXdEozeHE0RytZWk9zM25KNW5vRHNlKy9KQ0hEL2g0Uzd0?=
 =?utf-8?B?RDROTm1lK2hMSlJ5TGVYbnpIN2VNVzhZMXBlR1RqNW1jbXQvaFF1MmdYVEMv?=
 =?utf-8?B?N0JCRWV2ZWo5aDNWV0laSGRwbXgrbEZTZDVlVnlyRDFEM3JSOE5OcVY3cTg2?=
 =?utf-8?B?M3h2bTNVS0t0VjNzVCttTVBCejk3cER3VFFVZ3BlSnl3VUpXbklDWjZMWkdj?=
 =?utf-8?B?T2o3bHpsVTdzUkppU3FTLzI1enVXRi9FT1ZlZW5iQnpyRmFNRWJIbitJa1V0?=
 =?utf-8?B?Z2g3N3BOV3FhaTBhRlJUOHdETkN3VlV4MU1Mcm9CS1V2bUdtVXI1Z201NEE5?=
 =?utf-8?B?K2RFUlgxYnNPUXhxZjErNStPUHhvRjZRTmF2MFlXQ0FTdVFEUkZLYlNhOC9R?=
 =?utf-8?B?cjErdmhMU1o3VkYySTdXbUh3S0grYjJSTlNwallpdWRXcE5oTmFUenk4NUQ1?=
 =?utf-8?B?RUZ2SHNaMnVMOHBUQ2NZckdvU2p0Rkk1OUxraFI4TTI4VlJJWjJHaUhIbWtR?=
 =?utf-8?B?OHc0VmdPRnJKZnBGeW96SkFHbHZ0elo4WUhMYlRVZVpvMjNmZlAvOUdjc2Zw?=
 =?utf-8?B?Wkg1SUNDdHBRKzEwZ251Uk1CVk1CRC9mcURtZWNYSzR2SzcxTHhVbG1nalNN?=
 =?utf-8?B?NHZDc0tPL2dXYzZIREVSRTc4K1ZQMW95eUVxNFBibzNEZEdBalFWcHpjNGVk?=
 =?utf-8?B?UnkydUZ5QkN6TStZN2ozUm5UY0o0L0tJY3c2blRBcHE5RnpUMU02UTRhb0lu?=
 =?utf-8?B?M1ZoNGNzdVYvbmxaVVRrbTdvR1VIYkZHVGVSSXVMdTBRYVVkMXR0dTFhNVBh?=
 =?utf-8?B?QWg2aG9xK2NXMGxSYW14LzZ0SWdrc3F0OHpsN0pMSUZyTkR0QTNhUGo3NTFj?=
 =?utf-8?B?OXhiaXMvV2l4K0Fab1p1cDNzUHRVb1hOM2JodEFtNW1YRVlOcU9oNENwYk1W?=
 =?utf-8?B?N0hhRWZUWitIRE8rZFIxQ0V4R3RQVmFBUFdsNDRxQXdtYmhYSmVUbkowWUg0?=
 =?utf-8?B?TFJtSGlJdVlzczFaNnl2ekVzM2hxOGJ3OHFxM3B1UUQ4ZHBKa29DNkg3WVVt?=
 =?utf-8?B?ZmU4WDBPK0pxUEFMTlJrQ3FGV3NtaUdGVW9lTnVxLzhxTzZLQlN2cFAvSWc1?=
 =?utf-8?B?UGJYUktxMndyaTJQWklwY05RbDM4M3pPQkhRSkJxSEZhOVk2Z0pEVW5mYXd6?=
 =?utf-8?B?T3dBNjl2d2c1UXJJU1hJSEZwSXFHeGpTNWwwSW0wM1phTy90T1lzVjBRS0VK?=
 =?utf-8?B?amc9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 809ae837-6f03-48ca-7355-08dc7aa3df2c
X-MS-Exchange-CrossTenant-AuthSource: SJ2PR11MB7573.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 22 May 2024 21:12:22.3941
 (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: PrE+xAWvR4hNTpWdl4IBivazxYVhJ1iZvylBWOBTxgVVCVVLIDjUovHlKyRE2T2NVVaPFsCOYuG9XCnBk7+PZgEktMus1hNlzA9SLSYwalg=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN7PR11MB6559
X-OriginatorOrg: intel.com

Hi Tony,

Please note in the subject (and many places in this series) the
text "Sub-NUMA (SNC)" is used. Is that intentional? Other places
it reads "Sub-NUMA Cluster (SNC)".

On 5/15/2024 3:23 PM, Tony Luck wrote:
> When SNC is enabled monitoring data is collected at the SNC node
> granularity, but must be reported at L3-cache granularity for
> backwards compatibility in addition to reporting at the node
> level.
> 
> Add a mon_display_scope field to the rdt_resource structure to track
> reporting scope. Default is for non-SNC systems where both scopes
> are the same.

Just to confirm ... the reporting scope needs to be superset of
the "collecting"(?) scope. Is this something that is implicitly enforced?

Reinette