Re: [PATCH v6 02/18] x86/resctrl: Add PLZA support to command-line options

[Babu Moger <babu.moger@amd.com>]

Hi Reinette,

On 10/15/26 09:17, Reinette Chatre wrote:
> Hi Babu,
> 
> On 10/1/26 8:10 AM, Babu Moger wrote:
>> Add a kernel command-line parameter to enable or disable the exposure of
>> the PLZA (Privilege Level Zero Association) feature to resctrl.
>>
>> This allows administrators to control PLZA visibility at boot time via the
>> rdt= option (e.g. rdt=plza or rdt=!plza).
>>
> 
> Apologies for not being clear in previous review. This changelog has a
> lot of repetition. Above two paragraphs are redundant, paragraph below is
> sufficient by itself. One possible change would be to spell out PLZA in
> the paragraph below. Spelling PLZA out may not be necessary since it is
> spelled out in the patch self, so up to you.

No worries. It will be:

Add "plza" to the rdt= command-line parameter so PLZA (Privilege Level 
Zero Association) exposure to resctrl can be enabled or disabled at 
boot. PLZA is exposed by default when supported; rdt=!plza suppresses it.



> 
>> Add "plza" to the rdt= command-line parameter so PLZA exposure to resctrl
>> can be enabled or disabled at boot. PLZA is exposed by default when
>> supported; rdt=!plza suppresses it.
>>
>> Signed-off-by: Babu Moger <babu.moger@amd.com>
>> ---
> 
> With first two paragraphs of changelog removed:
> | Reviewed-by: Reinette Chatre <reinette.chatre@intel.com>
> 
Thanks
Babu