Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:7 as permitted sender) client-ip=2620:137:e000::3:7;
Message-ID: <532da5f5-f191-4abd-8452-ec7594275b6a@intel.com>
Date:   Mon, 20 Nov 2023 14:22:04 -0800
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v11 5/8] x86/resctrl: Add node-scope to the options for
 feature scope
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>,
        <linux-kernel@vger.kernel.org>, <linux-doc@vger.kernel.org>,
        <patches@lists.linux.dev>
References: <20231031211708.37390-1-tony.luck@intel.com>
 <20231109230915.73600-1-tony.luck@intel.com>
 <20231109230915.73600-6-tony.luck@intel.com>
From:   Reinette Chatre <reinette.chatre@intel.com>
In-Reply-To: <20231109230915.73600-6-tony.luck@intel.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?M2Q4NzEvejU2enZvRkxQWHFFSysrbGRVVWRJcnFIMEZrY3REemhHUkRKN01S?=
 =?utf-8?B?eGtzdW10a0tUNFpkSmo2SWEvUTViVWNPVmJpVnJ6UlFDeTNhaXkwWUdyRTMv?=
 =?utf-8?B?M3ptSlJ4elJsQWpYL3hxYkZXSDJucVg3TERMNURqMDcrWm1lSjkwSTMyVERY?=
 =?utf-8?B?REY3alJtczN6eU51TXVGb1FTNVRldTNpQTFpNzBNVVZmTFFaZVZKUS85TjFP?=
 =?utf-8?B?S2p3ak54bXpvN2lvam9HRVA3TEpZeGlTY3FTQm5VekN3U2kvZEczMGZ4TU1o?=
 =?utf-8?B?R2JGc3FESVcvY1FVSHZaZHlpNTlUaXNpeGZmMyt1Zmc3NjlvdlU2akJGbzVF?=
 =?utf-8?B?am1NcjF0Q09kY0pXWElMWU9WVUNrQktlK2lVcUxsOGFXV2M2RXBsSW5NNXBo?=
 =?utf-8?B?NkNQRjZTbWZpS1MrTkJkSm84UVhOaUpBckpTUHhnYXpXdWRUZ28xYUNUTFI3?=
 =?utf-8?B?RHhKVzdjOVdIU1RHUk1XSnF6d0pYcHhzOW1jb0piSk5TSzJBdGZYTlFCdWIy?=
 =?utf-8?B?dkxacXdUOFFsRi9VY3dZRWhzU3NTL2JxejAxRmtjWm9haUc1UXJWbTB4Wk1B?=
 =?utf-8?B?OGZxS0hUb2xRbmkzMFNwcTBSNDRwM0pQL3JVdWYrN0RTTzFEdmpWc25kQ3N0?=
 =?utf-8?B?ZlorMFV0akRBTWtUWjM0VUMyTDV4THd1bWc1eHZzTm1VU0NaSURMZmV6ZDVz?=
 =?utf-8?B?bUxOczAybk94WFdFZkY2M1NvcXhUNVB0bDMwWUpuQytpUTNwQlkrR1ZaU3E2?=
 =?utf-8?B?Y3lMdDZqaTZZVnJ4OU4rWkVXSkxIZzc0c1hvVS9CVzlUM2s1Ykg1TENjaFZs?=
 =?utf-8?B?aUZ4VTlGdjlLZXJIeXpnYklZYmxnSFNVYzFVWXQxTXQ5akVpZXAxQWF0N014?=
 =?utf-8?B?YUVuUG9TN3pxcDMrNGlYT0VyQTR2RGwvMy9TbzJlcEdCdEEwbmZUUE02UHRE?=
 =?utf-8?B?eWtYSy9rRVhzLzkwOXFLMW1CNVdtUjBzTTQzbGFyWEZrZ3dmcXpSZGJrbWpK?=
 =?utf-8?B?aVlFNDhOcDV4ZXFQNHFxMW9aZWhNTHY0Q1J4T2F0bm82UUt4ak9zdFRwZm5M?=
 =?utf-8?B?TUQ2ZXFPTkwvUUtzcTJrMnhxSTNSd1dHN3FDcTlCbjV2dmxPQldCN1J2Ulhu?=
 =?utf-8?B?cmpIUjJSL0d5SHNMaFpqZm9NSyswL3FwcEFsZkZVNG1CaHJYcFQ4WUY2WW94?=
 =?utf-8?B?bW1KMVVKTnpKeXFxM0xqZm9lMmRpWmFqVVZUbGlLNThoMG1BMnJacUQ4NU14?=
 =?utf-8?B?dHg4a0VyRERLV01xdkFJSHd2aUpVSGhXbkRqVVM0cGl1bGdxakVEWjRCYVBR?=
 =?utf-8?B?Vm16Mms4R3FMT3BFUERHWDdtUkhtbW5sbWc2L1Vka0gvY3EvaHdHS3hDVVMr?=
 =?utf-8?B?cVRLSGZmNFRXNTJYc2xvK3RwNFFQOW1uQmhrU0F3UnViNU56TjJVTUF1aWFm?=
 =?utf-8?B?TVRHRVlXQUxOTXM4Qlp4cDNUTXRvckJMeHBsUVZWZFZuU1UxaHRNM2MzTmVl?=
 =?utf-8?B?Y1lGSUFtd1pEZXp3VDh3QjVvSklDcjBCdll3MzdINmhUUFhVeTZKSWJnN3JR?=
 =?utf-8?B?a09nMzl4L25GTGJuU2txeU9ZbDFvWEk0V1gzYm1BRDg1YWRPaVlCTjZJNTh5?=
 =?utf-8?B?bVVHNnl5MUdKVGhPRnVSSXFwbDZYNG0rOXNlVTFmZk9lZkFXWXBndTlTVzZV?=
 =?utf-8?B?UWVDMGVOMk5xZUpoS0sxR2M4b3Z6NXVIMTg2MFlVZlF0T3FNQW1ta0d0R21i?=
 =?utf-8?B?dXN5U2hjeWZrM2w0RXFvanA4MWhpazZDNUVXUzdyTnFMcXpsNTg1Z2x2NXA5?=
 =?utf-8?B?WDJLbDIwVlcrZ2ZRZU03T3ZHcEZiODhrTW11UVJ4QXEvWGdLQjZIdXVQQU50?=
 =?utf-8?B?UnlCaFdLTFRtSW16SGpOTE1PSXpZaFlyRGNKTkl5QWYwc09DNHlvaFBrOFNh?=
 =?utf-8?B?SkZhTUE0K0Y1KzR3d1JReG5ORFRENCs4anZ6RWtMQ2Jpc1F1Q0krU3VnUXBz?=
 =?utf-8?B?NDRmczRseTJac3V6ZnV4TEJ0SnFKUms0TzNSUnIvU25JY3ZqcDA2KzdwZTB4?=
 =?utf-8?B?YktGbTdoWk93T1kyZEJ1aHpoMjZTdjJ2bndWUTRqdzdNRjIzZ2ZWaUdlMjVY?=
 =?utf-8?B?c3VZUWJsMklkanduQWwySE0zVTc0dTJQMWloYUsxdkQ0SHoyR0hLTWNHWlB5?=
 =?utf-8?B?MXc9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: d2a3a7b4-6207-4a27-f3f9-08dbea1720e5
X-MS-Exchange-CrossTenant-AuthSource: SJ2PR11MB7573.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 20 Nov 2023 22:22:06.1598
 (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: Dot9Ax6/5miglCJwcQ/jsUF2Idt+jWqHp4tki69rpkoAiN6IqAxt28TE6xrI2mAGofGLqeo7ILnJsYzNDzKfcNLkVTQD58WtnyMsnYzOmK4=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY8PR11MB7797
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,
        RCVD_IN_DNSWL_BLOCKED,SPF_HELO_NONE,SPF_NONE,T_SCC_BODY_TEXT_LINE
        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 (snail.vger.email [0.0.0.0]); Mon, 20 Nov 2023 14:22:19 -0800 (PST)

Hi Tony,

On 11/9/2023 3:09 PM, Tony Luck wrote:
> Currently supported resctrl features are all domain scoped the same as the
> scope of the L2 or L3 caches.
> 
> Add RESCTRL_NODE as a new option for features that are scoped at the
> same granularity as NUMA nodes. This is needed for Intel's Sub-NUMA
> Cluster (SNC) feature where monitoring features are node scoped.
> 
> Reviewed-by: Peter Newman <peternewman@google.com>
> Signed-off-by: Tony Luck <tony.luck@intel.com>
> ---

Please follow guidance in section "Ordering of commit tags" from
Documentation/process/maintainer-tip.rst

With that addressed (thus adding Reviewed-by tags after your 
author SOB):
Reviewed-by: Reinette Chatre <reinette.chatre@intel.com>

Reinette