Received-SPF: pass (google.com: domain of linux-kernel+bounces-59532-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: <91bd281e-e21f-4b60-9e73-2f14fcbec316@amd.com>
Date: Fri, 9 Feb 2024 09:27:56 -0600
User-Agent: Mozilla Thunderbird
Reply-To: babu.moger@amd.com
Subject: Re: [PATCH v15-RFC 0/8] Add support for Sub-NUMA cluster (SNC)
 systems
To: Tony Luck <tony.luck@intel.com>, Fenghua Yu <fenghua.yu@intel.com>,
 Reinette Chatre <reinette.chatre@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>,
 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>
Content-Language: en-US
From: "Moger, Babu" <babu.moger@amd.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?WGFTbzNsbVM4TmN4RUNXeE1CR2gzZ0haODNFZ2ZLMEdGU1padDdBSDcwTnNz?=
 =?utf-8?B?UzZkb05NWmpjaFlVQ09jakpPWGRUUWhNL0hUeW1HUEJ5amhJMGx4WVlrV2hu?=
 =?utf-8?B?eGU2VlcyUmJhTFpjZ21WSjRHLzUrWmkvUFJabWt0MkdnS2UvUG0reTRWdXly?=
 =?utf-8?B?MkNXU2ZjdHF6NFM5SXkvMGxZOTFGc3hWNXFGSHFSZkVPU0lkNUtFSC9hNXJa?=
 =?utf-8?B?bDhzT0U0V0JGQVo4UHhHenErU1BIb2x4eldUR0ladnpEbGJrSlNFZndFSExJ?=
 =?utf-8?B?cFBwZVFCV0dZNHNSd0dIbDlBckwvSkF1THBkRVJuNkJXOXFpSVI4SndVYng4?=
 =?utf-8?B?TlBGRnNsNHZoYjIxRWp0RXBMVlFLQ3RDQnMrWmFIQWQ1T2d2ZlhFOElCbjdp?=
 =?utf-8?B?YWVkNExvUGNDZ244TCtGbU9BOXhlSkRPODF1VUlPRjBkMVkvSkt1UXYrR3FL?=
 =?utf-8?B?S0pPa0VQWlYxR0pGcHJiZERhTitoKzJ2bkJVMnhPRWlxbmR0QXlRd2lNcTk2?=
 =?utf-8?B?c2haN3NzRUNlRGE2UHZFOHNvRERUaFY1WEwwSHFYdlNMbUttVmEvT2hMUXN2?=
 =?utf-8?B?QmlEYWdBMlQwd3h0bkRLaXA5LzdhUmRadzR1VXYyVzNxRkJ5R0pxQ3IwZVFL?=
 =?utf-8?B?aS9LdmthRUFzRjdidTg2dENETFFZak0zQTZzT2FaVjZzVEpVQWlDODVVcFN1?=
 =?utf-8?B?RElobW5OZS9qRWtsNGRQczdPQ1JXanFxQW1yMVZ0RVFYUGZoaFdaOXB3OFBJ?=
 =?utf-8?B?NEN1bUhQdEdEcGJROEF3YytINENDMDZTbkZuOHJiTkdKbVR6ajRxLzJyYmI2?=
 =?utf-8?B?VGZlVjMyZDRqYWt3SzFEWkRaYSt3R0huclNEdS9CcTR6QjAwMko1bkJ0aUp5?=
 =?utf-8?B?WHEzRlNlaXlRaWhYWFV6UE8ySDJIR0xNRFJOM1VOdkJDaHdvZzc3WU8wMGlq?=
 =?utf-8?B?dGNJNkZZTnNLaGVkcXVNejFYSE9HOE8rNWdHR2c0TW5MNm5sTFllcGhPbXNJ?=
 =?utf-8?B?czZ5eDZSTUZtZTQ3cUxraVFDYjdrdUZMUFRuZXhMUHBoNTFkOHZ5YmZJc3FI?=
 =?utf-8?B?UTlHemNiY1JiMC9NZ2xVTlNwcm43dmtVcjJsbWcvVkIrVyt3QmZUZWl1TW84?=
 =?utf-8?B?V3RtbW1HQklIRjZDK0pwQVYxQU1QbjQyQmtGNkhQNStSL1lkc1Q4SFk4blBo?=
 =?utf-8?B?VjJTRk9OQVQ0K2xXNnhYS1RGWUZCeVkwd1RZWWpSUTBmeVhzbU8xeC9TVkt0?=
 =?utf-8?B?QXZtNkNtY0xCL3RyVEZxNjJOV256ekpqTnpIWVpTd0dnK3dYeHNBWWpMWXFI?=
 =?utf-8?B?aFc4UVVXZXVOb09UalVhODRzRzJYOGVUZHJCTlZrV3JQR2ZwWHBJTUFwSGJi?=
 =?utf-8?B?T1NtbHVyVWNTYzE5dm5Kays2dWozVCtPc2h1azZUeTkvZWl0cSthRXY3OEpS?=
 =?utf-8?B?UHNEVzVPSnFzUXE1SHNxcERQWFlsNnVLR1BOZVRtd0RpSnM4SDFhK1gxcDJu?=
 =?utf-8?B?dTFWWTJUaWxNdDhiQ2lubVFMTVdKb2JPTFQvVFc4OUJPMURpRTNuemNyOVJL?=
 =?utf-8?B?K2oxMkVMWkpDVjZLM1FObUt0WVNyQjVXUWFWOGg4MC9JeWlNeXFpSkt1SU1q?=
 =?utf-8?B?WjVYb2FraUFGZmtjSFgyTW1KQVJNbVdDeE1wS3hOS3FrUk9LZDdRNTBJUHg0?=
 =?utf-8?B?cHMvTVB5NkVtUjRnRFZiUUdrOXBEZUhUQjdTZVBhaDFrTFJxd1NoRm5rQ3VB?=
 =?utf-8?B?RWFhTzlkcnhlUjZUcnBVRmZLbnVDU2hKTENyQ0JrZGMxWUJacHJXcFg1MndQ?=
 =?utf-8?B?VVluYThzdDhCSFhkYllUNHl0QmlEVWxMbE52cGFCSVlNdktKcjFZeHczdklo?=
 =?utf-8?B?aEo1ckt0UVR0clpFZ1NIWWl0d3ZTSGVwWksvTlpnbW9wOXR6MHFpa1pxNjBX?=
 =?utf-8?B?d1pnLzlPRmtzVGFhTU5LUTFuTmZjUC92QnFkc3NleXoxazh0SFloYUo4OWxZ?=
 =?utf-8?B?SjFTK3hiaWg1U3VWblV4MnhOYmNuT2VhM2NsQzNRc1czNlRKdS9wekxldTFI?=
 =?utf-8?B?OVUzYTZWUk50VndlOGxvWGVTbnVKZkZKL2RsWGlvMTQ3UUZ2d09aUlMwbFBs?=
 =?utf-8?Q?d6AA=3D?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 332de958-2d4f-441c-14da-08dc2983b290
X-MS-Exchange-CrossTenant-AuthSource: MW3PR12MB4553.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 09 Feb 2024 15:27:59.4349
 (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: LsLOOVMyNMSFBkYRhZels7HqFp4/e13PciPmnlIaFZkUDkpXvKtd6A8kqkAnJAvY
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM4PR12MB7503

Hi Tony,

On 1/30/24 16:20, 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.

To be honest, I like this series more than the previous series. I always
thought RDT_RESOURCE_L3_MON should have been a separate resource by itself.

You need to separate the domain lists for RDT_RESOURCE_L3 and
RDT_RESOURCE_L3_MON if you are going this route. I didn't see that in this
series. Also I have few other comments as well.

Thanks
Babu