Received-SPF: pass (google.com: domain of linux-kernel+bounces-60085-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
Message-ID: <59944211-d34a-4ba3-a1de-095822c0b3f0@intel.com>
Date: Fri, 9 Feb 2024 14:16:33 -0800
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v15-RFC 0/8] 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>
CC: "Yu, Fenghua" <fenghua.yu@intel.com>, Peter Newman
	<peternewman@google.com>, Jonathan Corbet <corbet@lwn.net>, Shuah Khan
	<skhan@linuxfoundation.org>, "x86@kernel.org" <x86@kernel.org>, Shaopeng Tan
	<tan.shaopeng@fujitsu.com>, James Morse <james.morse@arm.com>, Jamie Iles
	<quic_jiles@quicinc.com>, Randy Dunlap <rdunlap@infradead.org>, Drew Fustini
	<dfustini@baylibre.com>, "linux-kernel@vger.kernel.org"
	<linux-kernel@vger.kernel.org>, "linux-doc@vger.kernel.org"
	<linux-doc@vger.kernel.org>, "patches@lists.linux.dev"
	<patches@lists.linux.dev>
References: <20240126223837.21835-1-tony.luck@intel.com>
 <20240130222034.37181-1-tony.luck@intel.com>
 <91bd281e-e21f-4b60-9e73-2f14fcbec316@amd.com>
 <ZcZvguSNel-z3Xe5@agluck-desk3>
 <f4d4ff97-7d40-4915-943a-a7371e24dae5@amd.com>
 <65e1d936-0e1d-4d6c-81de-0b2b1d3256fd@intel.com>
 <SJ1PR11MB6083D539920A888F93902954FC4B2@SJ1PR11MB6083.namprd11.prod.outlook.com>
From: Reinette Chatre <reinette.chatre@intel.com>
In-Reply-To: <SJ1PR11MB6083D539920A888F93902954FC4B2@SJ1PR11MB6083.namprd11.prod.outlook.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?UE9Lc2N3eWZRRHBYK3B6blVjeXNUVm9jeXdWOVhvYUIzekdaL1J5bnVIYXRG?=
 =?utf-8?B?TEFyZHovbHd0eThUTFV1UlpJZHVrcmt4aEtoVnV3UFMvTkJwQXFKbVBPVHRm?=
 =?utf-8?B?WEdTWnhkWmMrK0NpZktZaXVTcmJ0cmdaQ2VlbXlMSDAwWXhwMzkxUkpVT2N4?=
 =?utf-8?B?VUJJUHFKcDNEWHJoaWpWOXJiT2NTQ3Z3SnBQcUtGR1pkQTRkRnVacTRHY2xL?=
 =?utf-8?B?MnFsbGxYQ3pQUW1hYkdWcHdnVk5zbFlzSEYzOG9kVGJxY1lSSEJMVDY2R2t4?=
 =?utf-8?B?alh1SUJmTWJ2cVZOc3dYQlBKb2kxbFZSZHpRZlBoV09tMi9qU05MWDJhdWNh?=
 =?utf-8?B?ZW5zMXAxYnROaVFMbXV0dFBIYXpsNEc0c2V1WHYwaDA0ajA5KzJQbXg5RHY1?=
 =?utf-8?B?amlSVkpXeGlxY3U0MWFzd0t5Q0REYllLTDFycUMyQnJXejRCVUJEdFZNMHRQ?=
 =?utf-8?B?NlVwOWtrdEpMaEpmOWFOWjlVTGZJWU1wWmZwMCtWM0xHdmNGQVhWdkFnYmRK?=
 =?utf-8?B?QThWM1ZRWTl2YXJ4U2tIYTczajJsS2ZybWIwMzVYMm5xb0ZCMi90Y1hMQVlz?=
 =?utf-8?B?UFdKMWNHZ1BheFBjZmwxQVpzZk5EQm5tZy9Vc0NwbDY5WGdHU3AvVEtNcGtt?=
 =?utf-8?B?aVZzUnA5TzAyclZNWmtlRVFFWk9lL3Bac2RTS2ljZnlvSW45dnVweTMzR2VF?=
 =?utf-8?B?czNEbHZlV3d4RGlXMjk4eTB1L1gzdy9jWkg5L05mYUNMSS9QK3I1MFFudnUv?=
 =?utf-8?B?Vy9IYmNUcE8vYVF5S0FodmQzb1dRV0djTjFNU0hUM3l3alZ0VEZkdzFxY1VW?=
 =?utf-8?B?UnBEVkpnWGpock1IUlZYUTR3N01CYmtHM2wvSzl2U0t2N1k0Rmp5dlcwbndE?=
 =?utf-8?B?SFpNMTg2d2hWWEtOZ1NZcGhXckUwM2dTc05Ba0pYSE94NloyUUROSTVCaFQ4?=
 =?utf-8?B?N29CbUxFT1dsN0RYUWVFV0dsQ01BcFg4cmZVekZ5KzBIWGhTOUZVWTA3Nlgy?=
 =?utf-8?B?L043WkwrampqMlZXUVEwaTBLVVJCMGRoOWVpMHpXdU9IRzJiTnFiUDVTUXNX?=
 =?utf-8?B?MWFmdHRxUUhxbGxkM0ZGSDVlci9kQUdwOURPaWZLSHRaVlI5NlJ1bTRvQXE0?=
 =?utf-8?B?VGtsRXNVRjVaSE1MUHVOWSt4eWNFSlFDcDhMR3J0T3BodHVEZFA0TVZ6R0xx?=
 =?utf-8?B?VUpNNWVZTjhhVXBHT1QvVWZRTndkcUNEYktyQkhNL09wSUNhSDRZMW5zbS9O?=
 =?utf-8?B?b0xqcEtTbGh4OVh3VVI0TElsbkpFTjhuL3UvOG9jU1RUaUMwQUtaR2hBeTlH?=
 =?utf-8?B?Wk9wdDZYclpHeDVpM1Vzb0UxNThXVTdSN0kzWVBtNzNXVkJrSHVNMngrN0Er?=
 =?utf-8?B?REZCQzNycFM3NkUxU3lSTkdXU0xUMDVLUUFKcEhmeUZCM3NmM3liUFlKUHdJ?=
 =?utf-8?B?SlVxK3hjTDMyS3FBQ1RnV1dxMTlzdmxkREZRUjI5T0JXZjc0U1pWUXl1cmhQ?=
 =?utf-8?B?ZjZsVkZNQUxubmRGTThybzh4WUhoQWhzbUlEVkQ2djc4Y2Y1dFpGZGZ1UXRk?=
 =?utf-8?B?aGUvNW9rbGxEWjhRK3FKTWdGQmx3emVBbjk3NXZaRDVWNklFMjJ3Rms3bzVh?=
 =?utf-8?B?SXpaRUZHRU1CWVN6VmlUM05jWDVoRGFVNm9XSjdBYmZxQTJ2T0p5K3hTU3B6?=
 =?utf-8?B?ZHQ5S0tBcHNINDFSSGNoMGVYMFBNZytoQkJwMUlNR1AxUlAwSHd1V3JCZkJp?=
 =?utf-8?B?RmVJSXFoQUVvenVWYUxZQzZzNzlSdGtQQi9iRDdPdlF5TTBoQVR6NnlTRWVF?=
 =?utf-8?B?TzRLVkhkUUJjaUtRM2w0U0RGSWN6eWpKT1ZuUW5UUG5MZXRYT3BRQ0lKYUFy?=
 =?utf-8?B?OHNyVTlLcmQ0blJSbjF1R1BWTHR6dFhpTXI1MDhGQ3pGMmo0bkVPaXl2ZWFV?=
 =?utf-8?B?RkE2SVFjcUtRVmJXaFIvV3drcUloNC9oTVU0OEVqbHlDK2JORnNFenBKRkh0?=
 =?utf-8?B?Q2N2NElFSlVYNE5BNVpJeWR6RWtrVHZqN2lWczVnUW0zVnJrWFlEQmxOd09v?=
 =?utf-8?B?Z1ZtZ0ZJaW4xWnF0N09vUTZtVW1xMXY4K0sxYXZoUmtuc0pyUmpXSitGTDg5?=
 =?utf-8?B?Uk9hcWRwSTYxZHBGRzRTSGU2WlF0RjRzb3c2WU1ZT1hJcllONzNZbmxNOVI4?=
 =?utf-8?B?T3c9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 685e1d73-f754-42b1-b5f1-08dc29bcc799
X-MS-Exchange-CrossTenant-AuthSource: SJ2PR11MB7573.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 09 Feb 2024 22:16:36.0255
 (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: 6bG6rP82Rl5ylpm9nqVs7UAPKToMbZtzEY+mEMcB4LVlTMuNYYrf39PDZd8d1U5WqJx0cPUSFXKtk39+so0AQIikTxZwdAOtoN4lq/dsbxo=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CO1PR11MB5011
X-OriginatorOrg: intel.com

Hi Tony,

On 2/9/2024 1:36 PM, Luck, Tony wrote:
>>> Reinette seem to have some concerns about this series. But, I am fine with
>>> both these approaches. I feel this is more clean approach.
>>
>> I questioned the motivation but never received a response. 
> 
> Reinette,
> 
> Sorry. My motivation was to reduce the amount of code churn that
> was done in the by the previous incarnation.
> 
>    9 files changed, 629 insertions(+), 282 deletions(-)
> 
> Vast amounts of that just added "_mon" or "_ctrl" to structure
> or variable names.

I actually had specific points that this response also ignores. 
Let me repeat and highlight the same points:

1) You claim that this series "removes the need for separate domain
   lists" ... but then this series does just that (create a separate
   domain list), but in an obfuscated way (duplicate the resource to
   have the monitoring domain list in there).
2) You claim this series "reduces amount of code churn", but this is
   because this series keeps using the same original data structures
   for separate monitoring and control usages. The previous series made 
   an effort to separate the structures for the different usages
   but this series does not. What makes it ok in this series to
   use the same data structures for different usages? 

Additionally:

Regarding "Vast amounts of that just added "_mon" or "_ctrl" to structure
or variable names." ... that is because the structures are actually split,
no? It is not just renaming for unnecessary churn.

What is the benefit of keeping the data structures to be shared
between monitor and control usages?
If there is a benefit to keeping these data structures, why not just
address this aspect in previous solution?

Reinette