Re: [PATCH v2 00/35] x86/resctrl: Move the resctrl filesystem code to /fs/resctrl

From: David Hildenbrand
Date: Thu May 23 2024 - 11:08:05 EST


On 23.05.24 17:01, Dave Martin wrote:
Hi Reinette,

Apologies for the slow response...

On Tue, May 14, 2024 at 08:56:41AM -0700, Reinette Chatre wrote:
Hi Dave,

On 5/14/2024 8:17 AM, Dave Martin wrote:
On Wed, May 08, 2024 at 08:41:37AM -0700, Reinette Chatre wrote:
On 4/26/2024 8:05 AM, Dave Martin wrote:
This is a respin of the resctrl refactoring series described below,
addressing review feedback. Many thanks to those to responded with
feedback on the v1 series [2].

See Notes and FYIs in the individual patches for details on the changes
and outstanding issues.

What are your expectations regarding this series while considering [1] and [2]?

Reinette

[1] https://lore.kernel.org/lkml/ZiE8%2foXyjBef2qTy@xxxxxxxxxxxxxxx/
[2] https://lore.kernel.org/lkml/acce93e6-4c8e-472d-a2bc-5cbad2950661@xxxxxxxxx/


I plan to propose a reworked version of the fflags and string parser
stuff, and take a look at the other more minor outstanding issues.

The series does need rebasing, but otherwise I don't anticipate much
change to the code from the Arm side unless there are further review
comments.

Do you have a preference on how this series should proceed?

I believe that I already answered this question in [2], hence my confusion
about this posting.

Reinette

I guess James and I need to tidy up the loose ends and repost this.

I'm hoping we can get a v3 out over the next few weeks or so.

Finally having more time to review this (I hope ...) does it make sense to review this v2 or wait for v3?

--
Cheers,

David / dhildenb