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

[Reinette Chatre <reinette.chatre@intel.com>]

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@amd.com
>>>
>>> Correction:
>>>
>>> Tested-by: Babu Moger <babu.moger@amd.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