Re: [PATCH v5 00/10] x86/resctrl: Misc cleanup

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

Hi James, Borislav, and Thomas,

On 7/23/2020 11:31 AM, James Morse wrote:
> Hi Reinette,
> 
> On 08/07/2020 17:39, James Morse wrote:
>> These are the miscellaneous cleanup patches that floated to the top of
>> the MPAM tree.
> 
> Is there anything else I should do with this series?
> (Does patch 10 need an ACK from anyone else?)

James: From my perspective the series is ready for consideration by the
x86 maintainers and you already included the appropriate mailing list
and cc for this series to be queued for their consideration. My
"Reviewed-by" for patch 10 should have included a "#for resctrl". It
indeed needs more reviews for the cacheinfo.h changes but I expect that
to happen during review by x86 maintainers.

> 
> Would a branch or pull request make anyone's life easier?

Borislav and Thomas: Would this perhaps help you?

>> The only interesting thing are the patches to make the AMD/Intel
>> differences something resctrl understands, instead of just 'happening'
>> because of the different function pointers.
>> This will become more important once MPAM support is added. parse_bw()
>> and friends are what enforces resctrl's ABI. Allowing an
>> architecture/platform to provide a subtly different function here would
>> be bad for user-space.
>>
>> MPAM would set arch_has_sparse_bitmaps and arch_has_empty_bitmap, but
>> not arch_needs_linear.
> 
>> Nothing in this series should change any behaviour.

Thank you very much

Reinette