Received-SPF: pass (google.com: domain of linux-kernel+bounces-2083-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Message-ID: <1516e4ad-b0c6-4c03-922a-5b14522c8d0a@intel.com>
Date: Fri, 15 Dec 2023 21:02:01 -0800
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v8 21/24] x86/resctrl: Allow overflow/limbo handlers to be
 scheduled on any-but cpu
Content-Language: en-US
To: James Morse <james.morse@arm.com>, <x86@kernel.org>,
	<linux-kernel@vger.kernel.org>
CC: Fenghua Yu <fenghua.yu@intel.com>, Thomas Gleixner <tglx@linutronix.de>,
	Ingo Molnar <mingo@redhat.com>, Borislav Petkov <bp@alien8.de>, H Peter Anvin
	<hpa@zytor.com>, Babu Moger <Babu.Moger@amd.com>,
	<shameerali.kolothum.thodi@huawei.com>, D Scott Phillips OS
	<scott@os.amperecomputing.com>, <carl@os.amperecomputing.com>,
	<lcherian@marvell.com>, <bobo.shaobowang@huawei.com>,
	<tan.shaopeng@fujitsu.com>, <baolin.wang@linux.alibaba.com>, Jamie Iles
	<quic_jiles@quicinc.com>, Xin Hao <xhao@linux.alibaba.com>,
	<peternewman@google.com>, <dfustini@baylibre.com>, <amitsinght@marvell.com>
References: <20231215174343.13872-1-james.morse@arm.com>
 <20231215174343.13872-22-james.morse@arm.com>
From: Reinette Chatre <reinette.chatre@intel.com>
In-Reply-To: <20231215174343.13872-22-james.morse@arm.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?THcwak5BMWJzdmVFVml1QVBzVGhKZ09KVGJ5UUNhdFBKZWYwNkRJSEd0LzFi?=
 =?utf-8?B?WmJRWHdqelZmVjBZMzFvSUZyZjBkbFd5VmNZL0ZzTmNCd1hZRG54dFBEaGhp?=
 =?utf-8?B?c0tRcFRrZ1VKK09GSUkwTkVBUnp4Qys5T2xsdTJLOU9iN1lNOHE4YUhCdG4y?=
 =?utf-8?B?UTNURE03VXpCd2ZTQkhoL3dGdC9xTXloVU5INnVuOXZJQmZqcllLM3FUa0RD?=
 =?utf-8?B?SkJEMytscklYb3NqQVBjRHVBcGZJVkh3UFpmUGp3enhNZEVtWjdMUjB2ekZ6?=
 =?utf-8?B?ZVZ6SkpxNlRWb0JqZ0FpUGxKQjFRUXlrS1YwZG1wd0lYTzllYWQvYlRWa2kx?=
 =?utf-8?B?TlFsT3JsRkx4RXc1UnJ4dXhKaUxjVUg4NGp2SlRsV1B0YU82Zm9zSmZBbzc0?=
 =?utf-8?B?bFJJaUZUSHJmRGNoV1EyaVhYRUZ1MFp1b1pRd3FDeXNLbkx0TEphYVByNFVP?=
 =?utf-8?B?ZkFra1Q2VW90Z0ZlT1ZTdmRYYVBTZk9xUU1QNmxFMEJpWjVVR3RFYjcwN0Jk?=
 =?utf-8?B?dERBSFJyKzMyaXU5Q0pHNVdlNEt3RmNmV21UR0JxQk1OamxNVElWdVlEY1Bw?=
 =?utf-8?B?R09POXhONEVWRkx5QjlpYnBHY1JzSGx3dFFjWGZLZGRqVlQ3Q2NjVG82UUcy?=
 =?utf-8?B?elU5eXVLSnFNTS80dWd6aVZvYVhINjI3WUZVUC9pcUk4WmVENERNelFVc0pL?=
 =?utf-8?B?d1RjOGNkZ0xrWVZlYVFpWVVFU2hvZ0VqTG1zZ3VCWmtxWGtxNk50SEFMb1BP?=
 =?utf-8?B?bXFuTHdGUjE5SDJSSHUxaEhYbTJPUDZiMFhxSWFUYkVvd2ozaUw3VXdEVU94?=
 =?utf-8?B?SXBuRGgyRTA1dTdFdDRwZ29MV2xKTVNYNzcrd2pJSkhHQk5mQVdMOEZIQWY1?=
 =?utf-8?B?YitHWXE2cjgyTzdqejdQUkF3KzlnM1F3aWJZQ0lvZ3pVUnZ5NU41MU1iMURD?=
 =?utf-8?B?aGlocWo5bnU1OENIWXNIVStnVXl4RGtxQWcvWTVkbVpxQzdKOTI5bzdOcis5?=
 =?utf-8?B?MHQwenNtY3doUEFSOXZwQkhPblBVUEtOek1zYjdocmpBUU02RnlpLzJIK1lq?=
 =?utf-8?B?REFPcThrNHR3MkUyeDRQM1lvVk5WMjdmYVZsTnRiOFVLMzRuOC9MUmY3YWJo?=
 =?utf-8?B?RTdxUE9ESmg3M2czM1duVFk3bGZNOSsxSWpqODhSL2JaQ2JHQ2N4T3RvclM5?=
 =?utf-8?B?VXIxdzBEVkRVRUtEUS9QMkZTM3NseVZyRlRXSXpYaXd0VHNvYWloWGxlYlZh?=
 =?utf-8?B?RCtBRUR3dDdnOVJGNXRXMWxYV1cvcXdrVzU0cXAvbWxHaUhMVGo5d1RLMnBk?=
 =?utf-8?B?d2dEMmVCRXBHdUpoNHgrSm5Ba21YWVMrc1NGOGJKbXZVWU5BZ1hlR3VLeDNr?=
 =?utf-8?B?SXFiOEtTMTMvU1hFRGN3T2xpRWx1M1krR1BaUE9Zek42OUpmVDNDQVdnZzM3?=
 =?utf-8?B?cEdYZHA4ZDU2SW9CL2RXTjNHVmJoeDRmWTQyOXpqaFdpa2hJSCswLzZZVElP?=
 =?utf-8?B?SFdhVGhhWmdnbFY1bTRWTnhCVzRWc1hieEltcmEyS1F4SEo0a1lVdEx4dERS?=
 =?utf-8?B?ajdmRzh3MXpTZTVqckZXM1hJb2w4THRRb0lGMWlvaEVUbzZkQ1owVk9jOHlj?=
 =?utf-8?B?YnJEZEhtcU83U2xsdmx3R1dIQjhBOThOUDNwcHBjM3dmdS8rOXdHWWwzTjZ6?=
 =?utf-8?B?QjRCUmhiRFp1QXIrcHRCNEJZNldZOWkxZmdjTDlBWXNlQzk4MmVEVitlMWtr?=
 =?utf-8?B?SHZIc2hXdGFQRUdKN2RqR25ueC9ROTg2c0hPdWxTNG9oaGc4S3pCcjhiTVdZ?=
 =?utf-8?B?dHVTcTdCc3EyeFk0SmxGZVc2blkyQURrZnBRdUk1YXBxLzlLVzFoVHZ0elll?=
 =?utf-8?B?VXJqNldEUXJzUUtibWRnOG03eGhNbi9mQzRwV1pIQlF0WldzRGZoaWxIR1hV?=
 =?utf-8?B?N2RNZnpIZmxrY2M3SllvYmwxVVRjd0xXWTlWVStqdkQ1TzhaQ0hNeGRtSHVZ?=
 =?utf-8?B?ajBZcFd0eVJxRzk4WG5hSVRpSm5aTjdzNGc3dmxCRVVoZ2h6Vnp5TzRNdTFv?=
 =?utf-8?B?cjNVN1gxUTFxeTh0bXZhSWlZSmtnNnM2dGI1MGRTaWVSNGxLZFlzQlN0S1pt?=
 =?utf-8?B?M0NCWWJuL3JYZnV2WC9YS0poejhzSFBGU0xjSzJlZUNQMEZYbHNvNi9JYWNR?=
 =?utf-8?B?ZXc9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 06ad0b24-ff93-4151-fb9e-08dbfdf42434
X-MS-Exchange-CrossTenant-AuthSource: SJ2PR11MB7573.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 Dec 2023 05:02:02.5401
 (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: es8u75eFDsQgy9+fnhGogOfbhK3BVZiHoChcoft4Eflt4ox1jA7FWvr9MmNgbmXahX6z53bd+9TvxluoQkcKer+IbCRgOfUC8UW13Fo5dQY=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MW4PR11MB5775
X-OriginatorOrg: intel.com

Hi James,

On 12/15/2023 9:43 AM, James Morse wrote:
> When a CPU is taken offline resctrl may need to move the overflow or
> limbo handlers to run on a different CPU.
> 
> Once the offline callbacks have been split, cqm_setup_limbo_handler()
> will be called while the CPU that is going offline is still present
> in the cpu_mask.
> 
> Pass the CPU to exclude to cqm_setup_limbo_handler() and
> mbm_setup_overflow_handler(). These functions can use a variant of
> cpumask_any_but() when selecting the CPU. -1 is used to indicate no CPUs
> need excluding.
> 
> Signed-off-by: James Morse <james.morse@arm.com>
> Tested-by: Shaopeng Tan <tan.shaopeng@fujitsu.com>
> Tested-by: Peter Newman <peternewman@google.com>
> Tested-by: Babu Moger <babu.moger@amd.com>
> Reviewed-by: Shaopeng Tan <tan.shaopeng@fujitsu.com>
> Reviewed-by: Babu Moger <babu.moger@amd.com>
> ---

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

Reinette