Re: [RFC PATCH v3 04/17] x86/resctrl: Introduce resctrl_file_fflags_init

Moger, Babu
2024-05-06T20:23:14Z

Hi Reinette,

On 5/3/24 18:26, Reinette Chatre wrote:
> Hi Babu,
>
> In shortlog, please use () to indicate function:
> resctrl_file_fflags_init().
>
> On 3/28/2024 6:06 PM, Babu Moger wrote:
>> Consolidate multiple fflags initialization into one function.
>>
>> Remove thread_throttle_mode_init, mbm_config_rftype_init and
>> consolidate them into resctrl_file_fflags_init.
>
> This changelog has no context and only describes what the code does,
> which can be learned from reading the patch. Could you please
> update changelog with context and motivation for this change?
How about this?

Consolidate multiple fflags initialization into one function.

The functions thread_throttle_mode_init() and mbm_config_rftype_init()
both initialize fflags for resctrl files. This can be simplified into one
function passing the file name and flags to be initialized. It also helps
code simplification of new flags initialized for ABMC feature.
--
Thanks
Babu Moger