Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:4 as permitted sender) client-ip=2620:137:e000::3:4;
From:   "Luck, Tony" <tony.luck@intel.com>
To:     "Yu, Fenghua" <fenghua.yu@intel.com>,
        "Chatre, Reinette" <reinette.chatre@intel.com>,
        Peter Newman <peternewman@google.com>,
        "Jonathan Corbet" <corbet@lwn.net>,
        Shuah Khan <skhan@linuxfoundation.org>,
        "x86@kernel.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>,
        "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>
Subject: RE: [PATCH v7 0/8] Add support for Sub-NUMA cluster (SNC) systems
Thread-Topic: [PATCH v7 0/8] Add support for Sub-NUMA cluster (SNC) systems
Thread-Index: AQHZ9hPXprr0NDSkvUi/XWbinWWk4bA4PAyA
Date:   Tue, 3 Oct 2023 16:16:09 +0000
Message-ID: <SJ1PR11MB6083136387418886E2399B28FCC4A@SJ1PR11MB6083.namprd11.prod.outlook.com>
References: <20230928191350.205703-1-tony.luck@intel.com>
 <20231003160800.8601-1-tony.luck@intel.com>
In-Reply-To: <20231003160800.8601-1-tony.luck@intel.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?smPB/gTpPMUYHkG14240DLkOdZGZCkSXfk5ieXvB8octWryoAIOuUAKOovqG?=
 =?us-ascii?Q?iBD5omuGsWxgUlRs0tK6OgBrjL16i8sSPHIZR/+1Sgsypw9mV6liPg/mcFn4?=
 =?us-ascii?Q?uEJ9Bm3Ge8GF8+N7n/H8xg4h2munmI0M5l4zh4qZFy2nLbjtDlIYms1eUDTG?=
 =?us-ascii?Q?sIlB3Sp1HGl3VUcdxWOm3Lj4x6w8CD1HyKaDpjDhXPl3Q40aEop9ZxfRtASK?=
 =?us-ascii?Q?9fQVzr+opysrFr4c8rzG+02YS4UEXQbfbmspuBg1mUujMjGkA+PhY7RmQXM+?=
 =?us-ascii?Q?Vd6c+eQABgkCyI1VGFILTJU+Vd8aCe6K38ZNp5QrY44IQmDvi4dTWoPVn52I?=
 =?us-ascii?Q?fuZ7biY8HTswIVqr920OQEVsZIsKnnBta8dpbce/V9v/iTwa/2v7bPxciHeY?=
 =?us-ascii?Q?rRa73zkjC6zThPAPMHqJkQt59jErDAEFmM4yBluqbprq8o/a/CzliJHiEVJU?=
 =?us-ascii?Q?Evmp5fljjL6cSisvkukp/ZcxnnUeP7Dx5EU7lK6OvykTmm13n0r3W09zMhgF?=
 =?us-ascii?Q?nnEOTYqx0yIiiikxxsyojQcTWVHTLSn1FsfWvV5MvflYnJZvfkG0F/6YNWUM?=
 =?us-ascii?Q?cZQpNj4EB9/GjDuoldy1wT0TD/Tit6AzWdRKcvIbl4c3OhhTtvkxEV8ICl4/?=
 =?us-ascii?Q?2SmTvU5+lpMg6iQjCafLfwKIuU+wKZUUbfvtaG8CTapG52A+l/rp9JYELFjn?=
 =?us-ascii?Q?3bAFlTgGZCDEU7oHxN9FCDrLLVWyGfn6xm6jflZonYKGIDq0xShoIwqw6CJV?=
 =?us-ascii?Q?DbUvym229dX1+1JzpS0i90VgsrEmML1vpuwBaxAIr3hAuJ6ZXe5oEPMR+o+l?=
 =?us-ascii?Q?6qi62n8G9HzqHwTf78yPNwz9s2ug5nj/YkIJz/aId+9U4SPGyc2j3mUciTGe?=
 =?us-ascii?Q?UwbpDHvymCgc8C2Z9UWYEU8pPbY7YAQp2AhYiohm0vDsDzTtjs2bgFuBhJUB?=
 =?us-ascii?Q?7T+OQLf+AZtupnOA6OsU7jOMDxFXt2jstuxlOoAczSQW1YPin6oY6rrL5pLP?=
 =?us-ascii?Q?1YWESSuWcRzhlrlfeCuu44JSa5rN4QA1wBw7YjYJ6VFGi7RgPRoJXHneWXGv?=
 =?us-ascii?Q?KqPrBawb5sJ06rPwQLMHFrNdPJC7qRTWC/28eZI6TOD/eCYnL5zK016jTv3w?=
 =?us-ascii?Q?w+BuXo3oupPJoSmetVADnxyOCiPQhKBp3fFoZM4ZL3ISUWAAuNdBwm82wbtP?=
 =?us-ascii?Q?+EF6E84bKKhD0gUuOeq9g6DErLL2qCH2tjSWbgwP8UZq8WsbciqG87jzO9jS?=
 =?us-ascii?Q?QKDeXzgPqj7weEJnVJhYa2ar0j9Sx34BPaj4EuUXm1S+T7WDxO0tP67bpn5T?=
 =?us-ascii?Q?TXUgjoFJWWGN5/0vieTT5lMRxt33eJCUH98gT5BZxo7shl2GAjWrX/5Rs/Bw?=
 =?us-ascii?Q?0LcuylGN1KQww+oiv5Qpq7/+s9r0iUaif1z8XPl5n1rurYCWQ9H2/F+wsowx?=
 =?us-ascii?Q?DuJF3iv/R/oldE3MLMdYjO+2uenpd94WmvTt9rk1iUSQ2mJ4oZRnX1P4sw4L?=
 =?us-ascii?Q?YFMJjD7WajukK6BxCV2S9JGr1nkU83iZynHcFqZobZeXQQ6H2TZ6hCmd7KXE?=
 =?us-ascii?Q?qjO6RmTaHSntyxiAzR95/kc9yht7f6zxcfZ2JRX5?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: SJ1PR11MB6083.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 27a60a5d-baac-4766-c494-08dbc42c0e31
X-MS-Exchange-CrossTenant-originalarrivaltime: 03 Oct 2023 16:16:09.8365
 (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: qqXKsJ8zbQtlC5xZfVksPr5O4cmYuUalvTYhGZ5qzSqKf+Y6g4s0pehb7bgEqUtz7GqwRfZV6uDID9k0le/LOg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM4PR11MB6065
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,SPF_HELO_NONE,
        SPF_NONE autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (howler.vger.email [0.0.0.0]); Tue, 03 Oct 2023 09:17:45 -0700 (PDT)

> The Sub-NUMA cluster feature on some Intel processors partitions
> the CPUs that share an L3 cache into two or more sets. This plays
> havoc with the Resource Director Technology (RDT) monitoring features.
> Prior to this patch Intel has advised that SNC and RDT are incompatible.
>
> Some of these CPU support an MSR that can partition the RMID
> counters in the same way. This allows for monitoring features
> to be used (with the caveat that memory accesses between different
> SNC NUMA nodes may still not be counted accuratlely.
>
> Note that this patch series improves resctrl reporting considerably
> on systems with SNC enabled, but there will still be some anomalies
> for processes accessing memory from other sub-NUMA nodes.

Bother .. forgot to add the changes since last version summary
to the cover letter. I fixed all the issues called out by Peter Newman
in his review of v6 series. Specific details are included in each patch
(except for patch 0005 which is unchanged).

I added Peter's "Reviewed-by" to patches where he offered it AND
where I didn't make substantive changes (parts 4, 5, 6, 7)

-Tony