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;
Message-ID: <cb925ffc-7c36-4a81-a61d-adc4d7e793e5@intel.com>
Date:   Thu, 30 Nov 2023 13:51:50 -0800
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v12 3/8] x86/resctrl: Prepare for different scope for
 control/monitor operations
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>,
        Shaopeng Tan <tan.shaopeng@jp.fujitsu.com>
References: <20231109230915.73600-1-tony.luck@intel.com>
 <20231130003418.89964-1-tony.luck@intel.com>
 <20231130003418.89964-4-tony.luck@intel.com>
From:   Reinette Chatre <reinette.chatre@intel.com>
In-Reply-To: <20231130003418.89964-4-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?WTNCQzR0L2svS2g5ZHBrREUvaW1adGZtdGROano0YTlHNVNhRTlJd256eWkr?=
 =?utf-8?B?UGVETzJEQWVjNGk0bVkrNmZtS29WQXd3OFFlamVoRnJoNG8vTVJNQkZPd2Rs?=
 =?utf-8?B?dGlrYm5GdWRyRjZmNVkxdnlTMFhQNCtXaGsvL3lrYWVZbnFzei8yOFFmRE5n?=
 =?utf-8?B?eERZTEp6UGtVWE52YnVFaTFJZkE4UVEzc2UzUndtZEdLK3BMRyt6SkZKam8v?=
 =?utf-8?B?K2xWOXo0NzBRTFBYbzQycGpISS9WSGNvYXFyRGJTK09POGdCQjNkMjhGeC94?=
 =?utf-8?B?SzFqdHZSNjlXWk9tNDU3SVhxYllXZ2xwcDB2aTY1amdRcmhwUmNsRmdtOXpx?=
 =?utf-8?B?YlF1dEkwd1ZBMDVlU010RE5vWnQrclBWTituanNibFZ2OUxhR3FQSWJNVDJ6?=
 =?utf-8?B?TGRTMUw5YUNaa0pLbk9uVnhOci82UXZsNUJtaXlCQXdSWEVjWGFGS1NsK0xV?=
 =?utf-8?B?RWxzUFZhY0x5OE9YQVh0VmtpZFBHcDhjK1dHVndsdWRKNlRiN1U5ejM5L1hM?=
 =?utf-8?B?VVg4aml6ZFdNTS9yK3NGaWpVbFhVZ3V6UVJIeGh6NU5XYTR1eHlSVExpaG5i?=
 =?utf-8?B?UVliV1VsdGc1dG1JeEFabDJEamlDQ1FHbjhkd3Y5L0dsbE4ybDJMeVd6dmlz?=
 =?utf-8?B?YnRIT0Q3QXd4NXN0TU1GLzFzQ0U2SjBuVTVhc1VyRXloU0p3UzJ3bCtLNzgz?=
 =?utf-8?B?NHAwaGFudFNoUEtuemVpcnJETm1OT010S1RYRGVoZmI3QkNlcVYzYUZSSEJI?=
 =?utf-8?B?ak5MaTM3dGdJWlNJV1h4OEJiTGJWS1NPUVNNWGRxb0NicS8zN2ZsQUh6WHIv?=
 =?utf-8?B?bFdnM2dPdDkvNFU4NDN4T0h2czM5UnpKSVl5VmloOU9iTU9BcE1aakRTQ2FS?=
 =?utf-8?B?YlpBbE1Lc1RjWmwrbEdiYWdPTU1LMURvellEMlE2cFlzdlB3OW1neXpJYkpz?=
 =?utf-8?B?Ty96WjNCSFppK05VMjBtTkk5TVVycDRaWWZ4cnFRTHZYODZ3SHFyRFFLQWM2?=
 =?utf-8?B?RlZCb24vb1l0RkZiaGFOSDdLaVlUdXI2d3VOV0NWQkhWRis3RnpVUlptVXRk?=
 =?utf-8?B?blhRTVRVd0tyQ2lUdWR2QU9jNnNDWVZJcERENmJzc3JrQjYrUkFlZGdzVk9Z?=
 =?utf-8?B?d2tTTzFwQVN4NmZyYzNSK2U4Y1VtVVJSYVhBTUVYbHA2YzBkeTNXVUhHMTJK?=
 =?utf-8?B?RGw3d1Nud0ZsMWJwZnF3dGQ1QklKd1hSUFBQM0ZpeWUyUVduYlZzR2hTOW9p?=
 =?utf-8?B?NitxaXNjZW5MUVIxUlZrR2NFclpNVXBsR2ZYN1dGU1FER083VEdXMmViTHBk?=
 =?utf-8?B?Y1VZb0k3Q2xhem91MHBRM3RVYTBlaE54MElKaTZoWm4xODg0UDlFUjJCbmw5?=
 =?utf-8?B?cTBTZC9iMVE3WWFJNFV5Y0R5UC8yOEZqQzJsbUM0ei9kRjBmVUxGYjl3ZzFk?=
 =?utf-8?B?N3dqSThjcXAzdG5EdjFlSFVqQVN5WTQ5WnNaazlsc0c2VE1aMjJoYXltRTFq?=
 =?utf-8?B?QXZhK3hlSzZnNklySWo0RFBsbUdEalZEVnNsQk1zN09RejBvUEFyeXhZakpF?=
 =?utf-8?B?cGxZZ0czRzJqekxZamF2eWhIUVVzUmUrbnRWbU9sNkVScG5VTVgyUkFhaHJ2?=
 =?utf-8?B?S1dGdkdrMVRxcVF5c3pHa3VXWEU4K2xpcG82NTFnVXZYYkJKUExYSDFMNWYy?=
 =?utf-8?B?b1F6dGZNVEU2bm5YMG9rQ2tiSVZsRVB2TEZrYjUzL0JXbTdyZndjbXRWQmh2?=
 =?utf-8?B?V3dvVHlXbDVIU29pM1JVWUJlOWpVdmdXdFp5b0JUSzVZNUdBWS9PWDVMRXhN?=
 =?utf-8?B?ZzdNUXZmK1UxUTI3WnlKSU4xZFl6Z0VOQ2pyaUpvS01PUEVEOHdHVVM0b2xS?=
 =?utf-8?B?azltTTdhQ1lWTmVXMGxtR1FPYlpzZGFZSXVSb1NLWUtQUTIveVUrOTRWUHZ0?=
 =?utf-8?B?SzlOQ0JSZWRoTkI0N0xFYkhZSGRnaTY4dHlpOHVjQ0dOanRrczFycm9pU0wv?=
 =?utf-8?B?R1Y3OThjc2xhWS8yQzFSbTRWVmt3ekhaNWpPRFplRmJQZWNPYVh1QlRQUFFz?=
 =?utf-8?B?ZGlLbnVtK2x6VHlZZUpHUzUxelRSMnR5VzFhUDdqa29LWThQOC83dUhFQkNa?=
 =?utf-8?B?QWh1UG1mZksyN2RzM2crRERXTXc3OUNRVVMxQ0lSakpQYXMzYk1qQjd0S09t?=
 =?utf-8?B?SHc9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 02e8b28b-7075-4a84-ab9c-08dbf1ee9259
X-MS-Exchange-CrossTenant-AuthSource: SJ2PR11MB7573.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 30 Nov 2023 21:51:56.4703
 (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: /kPOm7G5XKF9JBtF3boHuDpvPeC8gQjrFngQwFKKiLtQeKEUmfmWOK92ryt7vcCqqbE8zaBYwoQkXzgZB5SguGzD8gemAwxT1/JfT80XtLA=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH0PR11MB5315
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-0.9 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED,
	DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,
	MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE
	autolearn=unavailable autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on howler.vger.email
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]); Thu, 30 Nov 2023 13:52:28 -0800 (PST)

Hi Tony,

On 11/29/2023 4:34 PM, Tony Luck wrote:
> Resctrl assumes that control and monitor operations on a resource are
> performed at the same scope.
> 
> Prepare for systems that use different scope (specifically Intel needs
> to split the RDT_RESOURCE_L3 resource to use L3 scope for cache control
> and NODE scope for cache occupancy and memory bandwidth monitoring).
> 
> Create separate domain lists for control and monitor operations.
> 
> Note that errors during initialization of either control or monitor
> functions on a domain would previously result in that domain being
> excluded from both control and monitor operations. Now the domains are
> allocated independently it is no longer required to disable both control
> and monitor operations if either fail.
> 
> Signed-off-by: Tony Luck <tony.luck@intel.com>
> Reviewed-by: Shaopeng Tan <tan.shaopeng@jp.fujitsu.com>
> Tested-by: Shaopeng Tan <tan.shaopeng@jp.fujitsu.com>
> ---

According to Documentation/process/maintainer-tip.rst the
"Tested-by" tag should go before the "Reviewed-by" tag. Please
check the whole series.

Reviewed-by: Reinette Chatre <reinette.chatre@intel.com>

Reinette