Re: [PATCH v12 02/26] x86/resctrl: Remove MSR reading of event configuration value

Moger, Babu
2025-04-14T15:57:27Z

Hi Reinette,

On 4/11/25 15:50, Reinette Chatre wrote:
> Hi Babu,
>
> On 4/3/25 5:18 PM, Babu Moger wrote:
>> The event configuration is domain specific and initialized during domain
>> initialization. The values are stored in struct rdt_hw_mon_domain.
>>
>> It is not required to read the configuration register every time user asks
>> for it. Use the value stored in struct rdt_hw_mon_domain instead.
>
> Storing and maintaining the event configuration creates confusion with
> the new event configurations introduced in the rest of this series. I
> think that it will be simpler to keep BMEC support as-is.

That is fine with me. I will remove first two patches from this series.
There are multiple things going on with resctrl. We can do these
optimizations later.
--
Thanks
Babu Moger