linux-kernel - Re: [PATCH v9 00/27] x86/resctrl: Move the resctrl filesystem code to /fs/resctrl

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] [day] [month] [year] [list]

Message-ID: <a949707f-6a46-44c8-ad4c-2aca949bbd5c@intel.com>
Date: Wed, 7 May 2025 13:36:26 -0700
From: Reinette Chatre <reinette.chatre@...el.com>
To: <babu.moger@....com>, James Morse <james.morse@....com>, <x86@...nel.org>,
	<linux-kernel@...r.kernel.org>, Dave Martin <dave.martin@....com>
CC: Thomas Gleixner <tglx@...utronix.de>, Ingo Molnar <mingo@...hat.com>,
	Borislav Petkov <bp@...en8.de>, H Peter Anvin <hpa@...or.com>,
	<shameerali.kolothum.thodi@...wei.com>, D Scott Phillips OS
	<scott@...amperecomputing.com>, <carl@...amperecomputing.com>,
	<lcherian@...vell.com>, <bobo.shaobowang@...wei.com>,
	<tan.shaopeng@...itsu.com>, <baolin.wang@...ux.alibaba.com>, Jamie Iles
	<quic_jiles@...cinc.com>, Xin Hao <xhao@...ux.alibaba.com>,
	<peternewman@...gle.com>, <dfustini@...libre.com>, <amitsinght@...vell.com>,
	David Hildenbrand <david@...hat.com>, Rex Nie <rex.nie@...uarmicro.com>,
	"Koba Ko" <kobak@...dia.com>, Shanker Donthineni <sdonthineni@...dia.com>,
	<fenghuay@...dia.com>
Subject: Re: [PATCH v9 00/27] x86/resctrl: Move the resctrl filesystem code to
 /fs/resctrl



On 5/7/25 1:27 PM, Moger, Babu wrote:
> On 5/7/25 11:49, James Morse wrote:
>> On 02/05/2025 17:45, Moger, Babu wrote:
>>> On 5/2/25 11:30, Reinette Chatre wrote:
>>>> On 5/2/25 9:04 AM, Moger, Babu wrote:
>>>>>
>>>>> I retested the patches on a couple of AMD systems, and everything looks good.
>>>>>
>>>>> Tested-by: Babu Moger babu.moger@....com
>>>
>>> Correction:
>>>
>>> Tested-by: Babu Moger <babu.moger@....com>
>>
>> Thanks for re-testing it!
>>
>>
>>>> Thank you very much.
>>>>
>>>>>
>>>>> It's very likely that these patches will be merged before my ABMC series [1].
>>>>>
>>>>> I'm currently working on v13 of the ABMC series and considering rebasing
>>>>> it on top of James' v9. That could potentially eliminate one review cycle
>>>>> during the merge process.
>>>>>
>>>>> What are your thoughts on this approach?
>>>>
>>>> Thanks for considering this. From my viewpoint this sounds like a good plan.
>>>
>>> Welcome.
>>>
>>>> I do not have insight into James's schedule to support this but from what I
>>>> can tell this work is close to being ready for inclusion.
>>>
>>> Sure. Lets wait for James or Martin's response.
>>
>> I've got a v9 to post, rebased on top of the cpumask changes. My only question is whether
>> to post it with the tail end of patches squashed together - to save Boris having to do it.
> 
> You have few comments to address in v9. I would say post v10 with all the
> comments addressed without squashing. Give the clear instructions about
> squashing.
> 
> Let the maintainer decide how they want to handle it. Pretty sure they
> have seen these cases already. Worst case you may have to spin another
> version. My 02 cents.


I agree. I also responded to same question with 
https://lore.kernel.org/lkml/b3e31246-52d7-42b8-97f7-35925c85ddb6@intel.com/

Reinette