Messages in this thread | | | Date | Mon, 9 Oct 2023 14:22:38 -0500 | Subject | Re: [PATCH v11 03/10] x86/resctrl: Rename rftype flags for consistency | From | "Moger, Babu" <> |
| |
Hi Boris,
On 10/9/23 12:19, Borislav Petkov wrote: > On Tue, Oct 03, 2023 at 06:54:23PM -0500, Babu Moger wrote: >> resctrl associates rftype flags with its files so that files can be chosen >> based on the resource, whether it is info or base, and if it is control >> or monitor type file. These flags use the RF_ as well as RFTYPE_ prefixes. >> >> Change the prefix to RFTYPE_ for all these flags to be consistent. >> >> Add RFTYPE_MON_BASE that will be used in a later patch. RFTYPE_MON_BASE >> complements existing RFTYPE_CTRL_BASE and represents files >> belonging to monitoring groups. > > First of all, the concept of "later patch" has no meaning when the > patches go into git, depending on the listing order. > > Second, why isn't this define added in the patch that's using it?
Ok. Will move RFTYPE_MON_BASE patch 10. Will update the text accordingly.
> > A patch needs to be a logically separate, possibly self-contained > change - remember that. > > Thx. >
-- Thanks Babu Moger
| |