Received-SPF: pass (google.com: domain of linux-kernel+bounces-199465-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
Message-ID: <22569491-1b29-4b3d-8e9c-a5d10b73b6ab@intel.com>
Date: Mon, 3 Jun 2024 09:55:24 -0700
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH 0/3] Add and use get_cpu_cacheinfo_level()
To: Tony Luck <tony.luck@intel.com>, Greg Kroah-Hartman
	<gregkh@linuxfoundation.org>, Andrew Morton <akpm@linux-foundation.org>,
	Thomas Gleixner <tglx@linutronix.de>, Borislav Petkov <bp@alien8.de>
CC: Fenghua Yu <fenghua.yu@intel.com>, Maciej Wieczor-Retman
	<maciej.wieczor-retman@intel.com>, Peter Newman <peternewman@google.com>,
	James Morse <james.morse@arm.com>, Babu Moger <babu.moger@amd.com>, "Drew
 Fustini" <dfustini@baylibre.com>, Dave Martin <Dave.Martin@arm.com>,
	<linux-kernel@vger.kernel.org>, <patches@lists.linux.dev>
References: <20240531195720.232561-1-tony.luck@intel.com>
From: Reinette Chatre <reinette.chatre@intel.com>
Content-Language: en-US
In-Reply-To: <20240531195720.232561-1-tony.luck@intel.com>
Content-Type: text/plain; charset="UTF-8"; format=flowed
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?ZVZrUEdRZk9uRUVzZGZhM2RnWERTY1lrakhkTkxMcGZsemVrZnl2SmdYckF2?=
 =?utf-8?B?SU5mUnRxNVRtNGlrZUp6WFhtZWdRYlJXdkRkbTk0YW5xL0pERVJ5T1Z2R014?=
 =?utf-8?B?S2pSQm0zdDVIMkVGMXRFWXBRL2dlWGlZbWUzYTlzUHFXL2htWmo0WnZ3emp3?=
 =?utf-8?B?R3NXdVFQMjg4QUdtVVYzRGxlMFZSYldzUWcvZmFOVGovZ25pbVljRkhmQi9u?=
 =?utf-8?B?UEhMdTN1QnlLcnFPaGdXMUhFcW1vUkE1NFlRZzN3YXpZUlBiNkZTM0xQMGly?=
 =?utf-8?B?Y3dNWXZPOUZtV1Y3SG9vcFBURFg3Yno5WWI5ZEZtVlJiTVU4R3FlQmR0T0JB?=
 =?utf-8?B?dmV1OVpaS3daUW94MmhMaDZtZm5JWExydzRCcHBNYW11d0xIUWMvcUFBd0NT?=
 =?utf-8?B?cTVwUUYwcnZlVTdkZXJ0YlV2Skl4LzBEOFZGd2xRdXRLRU1ZWGxja3BYbmNy?=
 =?utf-8?B?cVA1RWVUNmk2WkUvMk9XRFRnR3NDcWF6UkVSM0dPMCtJMHVwL2crNkRDR3la?=
 =?utf-8?B?SXFDNFZMZjFFWHptOFVBeVdBdklpYytIOXVadEg5L0ZJT3lwNTJNYmxRdWRD?=
 =?utf-8?B?Q0R5SkF1L0ozbVovRjcvK2VONjJ3Nmpkdk55L0l0akNuZ2R3NTF4VVNtbVlX?=
 =?utf-8?B?OTRYWkU1SEdTTG54TkdvM2hiOVo2VDIvai82a2N4MEZrWFg4cmJqcGFjNERh?=
 =?utf-8?B?MGsvQWpZek1PMUpxSHJ1RTVNVWpHZFNNOU1CbGQwb0E0R3BON3I5NTdCUTR1?=
 =?utf-8?B?UW82M2UyeTNUbmxsYWplZzQ0RlB0Y1ZQZy9KdU1vUjRVV1kvOUZNTkJEcHF6?=
 =?utf-8?B?WEU4M0VDTVN1ZHBJUzJGalRGWlhLVjlBcjJvd05IMHJOREMrWFVSS0dIM0JK?=
 =?utf-8?B?TnFjaC9jQUtCYUFtdUd4dHVMOC84eXB2ZFdqVDFhdEhiaHFVSjhnbVp2dTNN?=
 =?utf-8?B?MU82amwzekRtL0Y5NGkydmR4ZFpBYzkvcElCZmExUkUzbzhhL2pjVXNwMlZv?=
 =?utf-8?B?MDQzSVFQRi9BVnY0UkM3UWtQSld4UDhQVUFrbG5IZ3V0djh4WmJpNTVsYllw?=
 =?utf-8?B?NVhGcHZaNEVWYTJVWFNiVWhxS2JITWpzTVlxaFlsNGJxUm00bHBsMkNOK2V5?=
 =?utf-8?B?bXNYVWlWY3lqK3cyQ1Y0WVZWWjIxSmxvZ1dGbkt3T1NnWXQzMFRQbHkrQ2JG?=
 =?utf-8?B?ZFhZMlQ5Y2hBZUw5YVRxUENqbXY2MFFpeWUvUHQyRVVna3BtM2xDLzEvWUda?=
 =?utf-8?B?U3lPWTZpZmlNMGp1VjFmNUtNZTl6R011WitOZlNSQitzdFVCNUNFN1Y0UGNn?=
 =?utf-8?B?Z1h1dHBSQ3hDaks3V1RJRno0a3lRanZEY2w1RVVhajMvOG9KZms4UVlkM0hy?=
 =?utf-8?B?RVlQNzl2SkpBR2Zzc1Vkd2J0V2krUEhTSVVMVk4wRmt6OU1SSlJ1VXFzcXh1?=
 =?utf-8?B?VXlIVllXcGtLUVF3SVJBdGtRM2V6YnBNWGR5ZHRkN210ZGR6aHlwQ2h5VHhX?=
 =?utf-8?B?bUZWQURELzVtTk4xMnBTWWxMcjB1eWVSRUJrbTRDdVNRTVdZdUhmKy8yR0V6?=
 =?utf-8?B?MkFXN1lQSmptc1h0MGJPeElFdmdPWUZQOEhnUUxoWXo1MkltcTJrbHBrSFIy?=
 =?utf-8?B?NEZHL25IMGxJdVdPck8rQWFmRWdDQ0t0eGZ3djhCTnNRN3NYcGVBWjZOakNr?=
 =?utf-8?B?MEV3Y0d4eUVDM28xVGsvbHpUOS9JMlNxdnJ6cUx6VUI0WHl5YkZmdGx2VEpE?=
 =?utf-8?B?ZnVCRUdYcE1KM0hRTTJpd3FpODk0Zno1c0RNMDN2SVF5dnd4N2F1eGNZeWVs?=
 =?utf-8?B?OTNJZXUzTDF1VmdaRlh2L21wcmRVQWZBdWdXY0cxWjRJYWI3ZHRXNVZJd3lB?=
 =?utf-8?B?b1BKU3A5QklPZ0FlOWtlVGhYdGJZVFg4eWg3ZWtQVEN3dForQjRSRFpGR285?=
 =?utf-8?B?ZU5FejJtT284MXIrV2lEWEVwODR2K2pjU0diZG1EQndFc1BWemVxdzRsbVFx?=
 =?utf-8?B?VFBJTjgwTnlPamI3YmdNSTNhUHMrbHZ4Q2I0VlNqelhjYW9DRFY1RjZPSFpl?=
 =?utf-8?B?eTJiYnp6L1dnS1FEc1kwMXp4WTd3a2ZUZUV1aUJvNVp0VXh0MU9mcUY3Z2Ix?=
 =?utf-8?B?eEZOYU9UaVlOQlhvMUVXaVVEa2w1Z085TXN6VWpZNXRHSUlJNEg3bllSbXVt?=
 =?utf-8?B?bWc9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: e39cd1b7-df1a-48ba-bd60-08dc83edf7c0
X-MS-Exchange-CrossTenant-AuthSource: SJ2PR11MB7573.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 03 Jun 2024 16:55:26.8073
 (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: FsSaFrJMs/ZFpdXgYLSpe+gq1+gfjcUJRdXcRkw6sDRhPUC+vNpbFHpeMIRv2mXoKHMghS6Y62vN9nBRtALW4mWjwyjkhETfskxtm51Ad0Y=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MW4PR11MB6863
X-OriginatorOrg: intel.com

Dear Maintainers,

On 5/31/24 12:57 PM, Tony Luck wrote:
> [get_maintainer.pl is vague about who owns <linux/cacheinfo.h>
>   to I've scattershooted some of the folks that committed changes
>   for this file]

Thank you to Andrew for picking up this series into the
mm-nonmm-unstable branch. I am not familiar with how patches flow
from this repo/branch but I expect that this inclusion will require some
high level coordination between the big repos since resctrl changes
usually flow upstream through x86/cache branch of tip and I
anticipate some conflicts and also needing to figure out how to deal
with this new dependency in planned resctrl changes.

As Tony mentioned upcoming resctrl changes depend on this new
include/linux/cacheinfo.h addition while some other planned
resctrl changes [2] will conflict with patch #2 and #3 of this series.

A previous change to include/linux/cacheinfo.h (commit
709c4362725a ("cacheinfo: Move resctrl's get_cache_id() to the
cacheinfo header file")) was able to accompany the resctrl
changes via x86/cache of tip. Could it be possible to repeat
such a patch flow?

Looking at mm-nonmm-unstable there is indeed one other pending
change that touches include/linux/cacheinfo.h ("cpumask: make
core headers including cpumask_types.h where possible"). I
confirmed that there is no conflict between it and patch #1
from this series, the two can be applied in any order. Of course,
this means nothing for any other changes to this file
that may arrive later during this cycle.

Could you please provide your guidance on how to achieve a smooth
upstream of this work?

Thank you

Reinette

[2] https://lore.kernel.org/lkml/20240321165106.31602-1-james.morse@arm.com/

> This helper function came up in discussion of the resctrl patches
> for Sub-NUMA Cluster (SNC) support. Reinette pointed out[1] that there
> are already two places where it would clean things up by avoiding
> open coding. The SNC patches will add two additional call sites.
> 
> So rather than have this jammed up as part of the SNC series, I'm
> posting it as a simple standalone cleanup.
> 
> Signed-off-by: Tony Luck <tony.luck@intel.com>
> 
> [1] https://lore.kernel.org/all/050c64b3-20b3-4db6-b782-f5124ebaab31@intel.com/
> 
> Tony Luck (3):
>    cacheinfo: Add function to get cacheinfo for a given (cpu, cachelevel)
>    x86/resctrl: Replace open code cacheinfo search in
>      pseudo_lock_region_init()
>    x86/resctrl: Replace open code cacheinfo search in
>      rdtgroup_cbm_to_size()
> 
>   include/linux/cacheinfo.h                 | 21 ++++++++++++++++-----
>   arch/x86/kernel/cpu/resctrl/pseudo_lock.c | 17 ++++++-----------
>   arch/x86/kernel/cpu/resctrl/rdtgroup.c    | 14 +++++---------
>   3 files changed, 27 insertions(+), 25 deletions(-)
> 
> 
> base-commit: 1613e604df0cd359cf2a7fbd9be7a0bcfacfabd0