linux-kernel - Re: [PATCH 0/4] cpumask: add missing API and simplify cpumask_any

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <240d0962-b381-479f-b3ed-4e6ce72d6570@intel.com>
Date: Wed, 23 Apr 2025 14:27:46 -0700
From: Reinette Chatre <reinette.chatre@...el.com>
To: Yury Norov <yury.norov@...il.com>, Tony Luck <tony.luck@...el.com>,
	"Thomas Gleixner" <tglx@...utronix.de>, Ingo Molnar <mingo@...hat.com>,
	"Borislav Petkov" <bp@...en8.de>, Dave Hansen <dave.hansen@...ux.intel.com>,
	"H. Peter Anvin" <hpa@...or.com>, Rasmus Villemoes
	<linux@...musvillemoes.dk>, <x86@...nel.org>, <linux-kernel@...r.kernel.org>,
	James Morse <james.morse@....com>
Subject: Re: [PATCH 0/4] cpumask: add missing API and simplify
 cpumask_any_housekeeping()

+James

Hi Yury,

On 4/7/25 8:38 AM, Yury Norov wrote:
> From: Yury Norov [NVIDIA] <yury.norov@...il.com>
> 
> cpumask library missed some flavors of cpumask_any_but(), which makes
> users to workaround it by using less efficient cpumask_nth() functions.
> 
> Yury Norov (4):
>   relax cpumask_any_but()
>   find: add find_first_andnot_bit()
>   cpumask_first_andnot
>   resctrl

(sidenote: above list of patch subjects do not match the series)

Thank you very much for doing this work. This simplifies resctrl code
significantly. I do have a couple of comments that you will find in
the individual patches. 

Regarding upstreaming I would like to propose that the upstreaming of
this work be split so that resctrl changes do not go upstream
via separate trees during this cycle. I am ok with delaying the resctrl
portion of this work for a cycle. This is because we hope to include a
huge change [1] to resctrl that includes the code modified in this series.
Having these two changes meet during merge window will be inconvenient
for maintainers involved. If you require a user to upstream these new
helpers then another possibility is to upstream this work via the tip repo
if that is ok with x86 maintainers so that that huge resctrl patch is created on
top if this work.

Reinette

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