Received-SPF: pass (google.com: domain of linux-kernel+bounces-48724-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) client-ip=2604:1380:40f1:3f00::1;
Message-ID: <78c88c6d-2e8d-42d1-a6f2-1c5ac38fb258@intel.com>
Date: Thu, 1 Feb 2024 10:46:45 -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: Tony Luck <tony.luck@intel.com>, Fenghua Yu <fenghua.yu@intel.com>, "Peter
 Newman" <peternewman@google.com>, Jonathan Corbet <corbet@lwn.net>, "Shuah
 Khan" <skhan@linuxfoundation.org>, <x86@kernel.org>
CC: Shaopeng Tan <tan.shaopeng@fujitsu.com>, James Morse
	<james.morse@arm.com>, Jamie Iles <quic_jiles@quicinc.com>, Babu Moger
	<babu.moger@amd.com>, Randy Dunlap <rdunlap@infradead.org>, Drew Fustini
	<dfustini@baylibre.com>, <linux-kernel@vger.kernel.org>,
	<linux-doc@vger.kernel.org>, <patches@lists.linux.dev>
References: <20240126223837.21835-1-tony.luck@intel.com>
 <20240130222034.37181-1-tony.luck@intel.com>
From: Reinette Chatre <reinette.chatre@intel.com>
In-Reply-To: <20240130222034.37181-1-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?ZlM4SGJUakI4b0ZPWGZBVTRQK3ZBdG8wTnMrTEFQR3kvWW9HdzI5ZEdXV0Vk?=
 =?utf-8?B?c05DYmNuWkl6UmFCS3F3Z052RmxLMGNKYWtpN0loZXBMSUZ0Z1BXaXFHcUNT?=
 =?utf-8?B?M205YXp2YlF1NzgxczFFUlgvcVZINy9TSWQ4MFJyQjl3L0UrbVc4MVU3S1VZ?=
 =?utf-8?B?UTVkaXNTRXFldE5VdzRFVk5xam5ScGZST1FCT3B4d2hxdithTHc2cExFWlFT?=
 =?utf-8?B?QTJOQmZBSy9hazBubWRmRWlrUnZvTHZjT01naHd2Yy9IT1RxVHZmWHFFdVRS?=
 =?utf-8?B?cHQzQndlRTJqcW1GOEFqR0p6TVg0MHd2N2IybWN6dzZXSkkzNVZVUkFhVjV2?=
 =?utf-8?B?VUhwemh3eHZBY1ZOQ2p1L1JTVURKZTdhcUlUblBPNG5qOExwQXU3dmgwb21t?=
 =?utf-8?B?WTVDbVBsRDREeDdXMk5NL1V3cXMxTWZ3NXhUQzI1Z09aUk5wYzRZVjBBUGg5?=
 =?utf-8?B?SjQvb1hjLzNVSURaaWk5d0E0RUNIMHhHa0tPdm9kaUo1YjBneklDa3Y3Mkhu?=
 =?utf-8?B?K3VhYVlJSjFWQXIzanFEWUdCbUNLaitsbGcrMmVHQ0V1ekQxWVNWMVhNN3k0?=
 =?utf-8?B?RnE5dkpJU05KV2ZRZisyS3NVSXhvSm8wTXRnb1BCMHVDZy9XV0JoODdZemdL?=
 =?utf-8?B?eEsrRHQzbVlobEZLSVZXZnkxM05QR04vU1VENVZNN3NHN1dta09QWGNkRWpM?=
 =?utf-8?B?dCsvMlJnOHJxaDRCQnhPWno2cmRYSUtkRkZxVE44bm9tQ1Z1blh5SVVoRFkr?=
 =?utf-8?B?dFRaa1dlRGozemN0ZnM2b3M2SnJISTVqMFpTc2xjL1FDNVRSWm1RRGFpQlhs?=
 =?utf-8?B?MFlEYXVTY2ZWQ2pXMkxIeFJYKzBLQzhNSmYwWnhNMjlEM1h5VzdqSi9yYUox?=
 =?utf-8?B?ckpHc1l3NlVLWXYreWxabHlZRDhLMTRqNHQ0YmMxOXZxTVRQN0ExR0hKQ0x2?=
 =?utf-8?B?cU1BbmZJTUlDN0xGSm85ZFdMYnA5SFo0VVNDK2d6aTRWdW9BZmRleW9JMlBy?=
 =?utf-8?B?aHAxeGEvNkQ0cEhWeW11bUZEQWJXMkdENkYzMXNnRVlFQndTaC9TSUE5cjRC?=
 =?utf-8?B?RXc5Mmk2aTlhMG1iVGdneUlOZXBCVGZ4V1J1ZzlKOGdVK2lzcHVLQkFFZDNa?=
 =?utf-8?B?UVF5R3BRcytka2R5UmJlNFN6Ly9XVXpyUzBlQVFjMWxoVml5WEtScGdCVi9L?=
 =?utf-8?B?S09iTGZVT1NFTmlTZ0ZjN1F5bkFCay9QR0hLTWZvV25tQitmZnk0YWZxVVhZ?=
 =?utf-8?B?QTR6VkZnUFo3amZMcWg3MFZ4Q1RxbDQvOWlJWEpYNThCamdXeWR6TDIxN2VD?=
 =?utf-8?B?eXRJTkdJc3NieHd4TlZDam03NnpieStmM0poWjVqc0tyNHdUdGtRV01ackZh?=
 =?utf-8?B?MlJiU2VuVHA3amlPQ1pBdnFxRWtuZXE4OVdLZVd2QU5mdEZlZDBHZkxEd0tO?=
 =?utf-8?B?am1XWHB0QktKY2xseU0vQnRuRDlvdTZzRHQxRHRReFh6UzI1QTRUVlZ3Qk9m?=
 =?utf-8?B?aFRSWTIwR0pldUNscnMzZENzSGJNRDZKZ3hISHRicS9ycGtxc1E4NEhpWHRF?=
 =?utf-8?B?QjVHNXVGT0g3c1BaQXBrQnRBay94RDhjVUExQm9kZk1FUnNTSzNKd1k0WDVv?=
 =?utf-8?B?MHhLSWhXUTZyZ3ZhNHN6ZGtDeVp2S1ZaVUxmTmcwQ2RJajNodWRQQld0MmlY?=
 =?utf-8?B?NnhJc0xHRHI5blhpb1d6dVJhak5CMFNRKzRlVXE0MHZsM1FVMkcwa0xOV0RW?=
 =?utf-8?B?VlRZUUlHeHJXdGdXT2ZsSUFyUUVnM1lIWG5IR1AzZUlvRU45cTdWR21iaE9O?=
 =?utf-8?B?bTZSL1dLcUozZ1pmUldwZjRGdDZ6V2E0bno1QTFkME1UQkN2aGF2dE1YT2FP?=
 =?utf-8?B?MGszOWdhZXJHRHlXK3NYeFUzSTZqZGxqbUZld0dNSGlvK3ZUN0YzUnhraHNk?=
 =?utf-8?B?RXBzdHQxRWdDVDhobzBhb2J0L29laFZQblhRQlQxc2R1RWxkNFVEQjhvbkJP?=
 =?utf-8?B?Q09mR0JLY29BNTEvQU5lWDFyMkVBbU1CbWVTRTcyTTJma2prZWpGZVRxODdJ?=
 =?utf-8?B?WkNsMUJ1VytIeGFTRXEyRzJSMGFEZ2ZNTlpGVHI3RzZlL0xlaHJJYmxVRnhj?=
 =?utf-8?B?aEh1UGhNTktNQnNmcHZvcENabFFUWlpEYU9MK0x1aHBsemVZakFiM1gxa1V0?=
 =?utf-8?B?S3c9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 147ec4f4-396d-48a1-3663-08dc235624c7
X-MS-Exchange-CrossTenant-AuthSource: SJ2PR11MB7573.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 01 Feb 2024 18:46:47.2525
 (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: RKTOlvPRasK39rwq6Kg09RezcKZOVBBhn6zAumwS2ogfBeR8mlLhEmyX9KYaa1x/x0BHbkDcriDEiVT1GJhc5EhtM4wIM9RrDUDD7qOCeBM=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MW5PR11MB5811
X-OriginatorOrg: intel.com

Hi Tony,

On 1/30/2024 2:20 PM, Tony Luck wrote:
> This is the re-worked version of this series that I promised to post
> yesterday. Check that e-mail for the arguments for this alternate
> approach.
> 
> https://lore.kernel.org/all/ZbhLRDvZrxBZDv2j@agluck-desk3/
> 
> Apologies to Drew Fustini who I'd somehow dropped from later versions
> of this series. Drew: you had made a comment at one point that having
> different scopes within a single resource may be useful on RISC-V.
> Version 14 included that, but it's gone here. Maybe multiple resctrl
> "struct resource" for a single h/w entity like L3 as I'm doing in this
> version could work for you too?
> 
> Patches 1-5 are almost completely rewritten based around the new
> idea to give CMT and MBM their own "resource" instead of sharing
> one with L3 CAT. This removes the need for separate domain lists,
> and thus most of the churn of the previous version of this series.

I do not see it as removing the need for separate domain lists but
instead keeping the idea of separate domain lists but in this case
duplicating the resource in order to host the second domain
list. This solution also keeps the same structures for control and
monitoring that previous version cleaned up [1].
To me this thus seems like a similar solution as v14 but with
additional duplication due to an extra resource and without the cleanup.

Reinette

[1] https://lore.kernel.org/lkml/20240126223837.21835-5-tony.luck@intel.com/