Re: [PATCH v4 05/19] x86/resctrl: Introduce resctrl_file_fflags_init to initialize fflags

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Reinette,

On 6/13/24 19:57, Reinette Chatre wrote:
Hi Babu,

Shortlog: "Introduce resctrl_file_fflags_init() ..."

Sure.



On 5/24/24 5:23 AM, Babu Moger wrote:
The functions thread_throttle_mode_init() and mbm_config_rftype_init()

You can drop "The functions".

Sure.


both initialize fflags for resctrl files. Adding the new files will

"Adding the new files" -> "Adding new files"? (this should be in new
paragraph)

Sure.


involve adding another function to initialize the fflags. All of this

Solution should be in new paragraph and written in imperative mood.

ok


can be simplified by adding a new function resctrl_file_fflags_init()
and passing the file name and flags to be initialized.

Consolidate multiple fflags initialization into resctrl_file_fflags_init()
and remove thread_throttle_mode_init() and mbm_config_rftype_init().

Signed-off-by: Babu Moger <babu.moger@xxxxxxx>
---

Patch looks good.

--
Thanks
Babu Moger




[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux