Received-SPF: pass (google.com: domain of linux-kernel+bounces-215528-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: <00bc5d85-e5ae-9ddd-7c2f-699785e40289@amd.com>
Date: Fri, 14 Jun 2024 17:31:00 -0500
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101
 Thunderbird/91.5.0
Reply-To: babu.moger@amd.com
Subject: Re: [PATCH v20 00/18] Add support for Sub-NUMA cluster (SNC) systems
Content-Language: en-US
To: "Luck, Tony" <tony.luck@intel.com>,
 "babu.moger@amd.com" <babu.moger@amd.com>,
 "Chatre, Reinette" <reinette.chatre@intel.com>,
 "Yu, Fenghua" <fenghua.yu@intel.com>,
 "Wieczor-Retman, Maciej" <maciej.wieczor-retman@intel.com>,
 Peter Newman <peternewman@google.com>, James Morse <james.morse@arm.com>,
 Drew Fustini <dfustini@baylibre.com>, Dave Martin <Dave.Martin@arm.com>
Cc: "x86@kernel.org" <x86@kernel.org>,
 "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
 "patches@lists.linux.dev" <patches@lists.linux.dev>
References: <20240610183528.349198-1-tony.luck@intel.com>
 <658d9869-ef22-48a7-876a-5bbba4f134ff@amd.com>
 <bc52a051-4296-48ac-9a83-29139855553f@intel.com>
 <e17d3a79-9c07-44cd-b0ea-2d0545680342@amd.com>
 <f57e9cf2-35b6-401d-afc2-8d11b22836c2@intel.com>
 <a65739a4-1c0e-ab36-611d-e2da0bd1d00a@amd.com>
 <SJ1PR11MB6083C6F02ADFC43B5316F71EFCC22@SJ1PR11MB6083.namprd11.prod.outlook.com>
From: "Moger, Babu" <bmoger@amd.com>
In-Reply-To: <SJ1PR11MB6083C6F02ADFC43B5316F71EFCC22@SJ1PR11MB6083.namprd11.prod.outlook.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
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?d2pFUHpxRlRDa2NBYXZ3RER4UEN3KzR1V0FyeHhLZTlwU3VFSFNZR2VkNzBy?=
 =?utf-8?B?eHlvR0pHUG5uNDBSVVhVdFB5RjBya0JyUkRRUEhibDF1UEQvSzVoV084MVVY?=
 =?utf-8?B?Y1ZKaURjWEROVzJYeG9TMXlJRndrakJIRmFoa3QxTnd4Zk83ZHNaOU1CUWdz?=
 =?utf-8?B?bWlTbHhoWjBmQWZ3K3ZUR0tnSjl6QmxwTStYb0tsTU1mRkRrcnFvMkdQbVJi?=
 =?utf-8?B?dTIyRDVUZk1JNmNGNWZGbHh1T3VJRUxTQUtvM1pqdUZ3cVhEUzVHaUVUNmJr?=
 =?utf-8?B?WUZlTVl2VzFvaEUvN3ZmWFlNdG8xQm5QOE5vYkw3NkhzYjEvS0s2ajI2UlBS?=
 =?utf-8?B?MlJyVjFUZUErMlBwV1lGVlhQbnhSdCs1YngyVXVwUnd5eU10Qkp2bjVaMEth?=
 =?utf-8?B?T3lrOExaWk0yMnJ2UFpiWjVZR2txdDM0SDZNdXFyMTIyREF3aFRTRUsvOHdJ?=
 =?utf-8?B?S1d5eUxmdVRubWFlU1pNVmNmeVRjeWM3MWdxR1ZMaEhQNUpzeXRWODhaWWtw?=
 =?utf-8?B?RXVlTnZkVzlMVGozTnQ4NzJJcmRNM3phRWR0VEVxN0hGRGFxM3NCYlJFY21F?=
 =?utf-8?B?U3dzbDlGNEsrZlB2UmpVREZ2UEEyeGNpVW5TTnFMVnZkMlBkTDJ4Tnlrblcx?=
 =?utf-8?B?aEJQK0U5Z3Z5dzczM0J5ak9PRkZMMnExOFRZZnJ2bkpNeFpiMjM1RXNicVpF?=
 =?utf-8?B?Q2NpbUxucWpkS2JCekRUNTZ0d3V4U3U1dkRqcllLRS9QZ0t1N1hBN1RDdFN6?=
 =?utf-8?B?ZUhFOEMwVndieHE4TG1QNXlVRFdRNklRUXJqWEZuQ0FDM0pZZ1FOYS80SlVS?=
 =?utf-8?B?WGtobGJQZVNhMU1Va1I5OWRBd2pYMEtuL213L2ZMUGU2OVgrTlhCdkZUTEJR?=
 =?utf-8?B?ZXZmQ0pxU1p1ekhyanB1a0FrM3JNQXBLVFpWeG03WUVUQjZuQmFieEh4Ui9X?=
 =?utf-8?B?Q3JpNlFyYU5YT0hiTmN2aGVBbmFxVzE2Sm9EdUx1R1VOT2kvOVAwaWF4L3ZG?=
 =?utf-8?B?ODBiMnY2Z2dDdHdnK3ByQ2t2L09OeEdRbkZ3TFVZYVprd3FFUnRDL0xCL2do?=
 =?utf-8?B?OTFyaEFzM2t5cHhrV1Q1amd2aGFSUWl5YkpDNXJYbXBVWXZhQlBvTkhBc3FO?=
 =?utf-8?B?Tk1Fc2VYejdTYjg1VVp1R2VTcnZidXhpdno3L1Q4Tk9ndkJJSit1Uk01cE1M?=
 =?utf-8?B?RXJ1OWhML3FveUtFLzFIRTV1bnFudkloemhUckhRdVcyckpmdDJKakZxUXlG?=
 =?utf-8?B?b2hMNDg4VythOXBmQUNtRTNYalUxeThDZ0lPY0JDMXdJOCs5Z0VTQmd0d0ov?=
 =?utf-8?B?bU41SElJc0xHL1hPSWNEeGtPZkdCL24zQUJWYzErcHlKQ3hITnZBSHQ2alpX?=
 =?utf-8?B?WmpVYnU3cGt5MHl2djhjajhXbDdHTU9LUVFwTkp6ZjdLeVFFdlBkZVdZckRX?=
 =?utf-8?B?dG82UGtqZzNYVzcxb2h0OXVaUGlvZUZLV0ZMYlB3elZDSlpBam9YbXhqcUhr?=
 =?utf-8?B?OU1TQ09paEgvT2lLVmFlU3RIRkJsaUswZk1EVTkvenVYcmxGZ0txb0VEaHRL?=
 =?utf-8?B?RU9Cc0h1UkZBaDVvaitxKzVmaE55VWp3Syt2MmcydWdpWUJMNk1hZ29aUGpQ?=
 =?utf-8?B?VS9zRWJyUi8xL0VZVVNXaFZpREY4NWl6SkF3UlZuSStXRjF5UCtLeUdRSEdX?=
 =?utf-8?B?a3p1Z2l4cXllb0tLYUtSbkt0UktmeXhtbmNCMU11SEFaMGRTRlN6NitJNnVY?=
 =?utf-8?B?REhHY0FueXl3VjNQS2dyY2plaC9YejRyOFZOdGJRTEhMMThGN0RoZUNvb0pa?=
 =?utf-8?B?c1FtNFd5WWdKVzdzNWN2TzNGczEyZkwwR3VsbnRGVmJYaHpFVFRoTjByZTNi?=
 =?utf-8?B?YmE1cW1nVE11dWllWURiUFZTdWRtYXZCc3cvVWlzNFRESEYyR0lsNFZjMCtJ?=
 =?utf-8?B?c3Z1Z25UNzJEVWZQOGczeWpBQUN4UlRVcHNCRFRRS2l6aFlDcmh6TzJDQ3FH?=
 =?utf-8?B?QTlZS0FCQjZpWm4wSTk4S2pUcklJMUFvYndlRFpWeFBhNmJDd2tUMWlralQ1?=
 =?utf-8?B?U2VKVjBqZjdhWmVDeDhBalI3Q2ZSVDFCU3M0eEdiQW10VVVJMWdtVHB1aE16?=
 =?utf-8?Q?6HT3UH5VSuPmuzelX0xs0iqZM?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: a18578eb-9af6-4377-ec75-08dc8cc1acc4
X-MS-Exchange-CrossTenant-AuthSource: MW3PR12MB4553.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 14 Jun 2024 22:31:03.6421
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: HnpdVoDZvBUmeHsi5woMvsvlKBsHcdh6OFxVWR0yeIh4o/Jy/Za3SdqjPwuK9Hon
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH7PR12MB7872


On 6/14/2024 4:40 PM, Luck, Tony wrote:
>> We have ctrl_scope, mon_scope, ctrl_domains. mon_domains.  Only one
>> resource, RDT_RESOURCE_L3 is going to use these fields. Rest of the
>> resources don't need these fields. But these fields are part of all the
>> resources.
>>
>> I am not too worried about the size of the patch.  But, I don't foresee
>> these fields will be used anytime soon in these resources(MBA. L3.
>> SMBA). Why add it now? In future we may have to cleanup all these anyways.
> 
> Babu,
> 
> I mentioned yesterday that future patches could split struct rdt_resource. I was noodling
> at doing so back in February. Patches (messy, not finished or fit for consumption) are
> here (just to give you an idea where things might build from here):
> 
> git://git.kernel.org/pub/scm/linux/kernel/git/aegl/linux.git rdt_split_resource

I see, it is similar to splitting the resource (option a. v15-RFC) and 
little more. I think we should move to that direction eventually.
-- 
Babu Moger