Re: [PATCH v1 27/31] x86/resctrl: Rename resctrl_sched_in() to begin resctrl_arch_

Dave Martin
2024-04-11T14:25:54Z

On Mon, Apr 08, 2024 at 08:26:04PM -0700, Reinette Chatre wrote:
> Hi James,
>
> In subject, did you intend to say "to begin with"?

I'd say the subject is likely to be abbreviated rather than incorrect,
but it certainly does no harm to clarify it. I'll make a note.

> On 3/21/2024 9:51 AM, James Morse wrote:
> > resctrl_sched_in() loads the architecture specific CPU MSRs with the
> > CLOSID and RMID values. This function was named before resctrl was
> > split to have architecture specific code, and generic filesystem code.
> >
> > This function is obviously architecture specific, but does not begin
> > with 'resctrl_arch_', making it the odd one out in the functions an
> > architecture needs to support to enable resctrl.
> >
> > Rename it for concistency. This is purely cosmetic.
>
> concistency -> consistency

Noted.

Cheers
---Dave