Re: [PATCH v2 17/35] x86/resctrl: Move mbm_cfg_mask to struct rdt_resource

[James Morse]

Hi Dave,

On 26/04/2024 16:08, Dave Martin wrote:
> From: James Morse <james.morse@xxxxxxx>
> 
> The mbm_cfg_mask field lists the bits that user-space can set when
> configuring an event. This value is output via the last_cmd_status
> file.
> 
> Once the filesystem parts of resctrl are moved to live in /fs/, the
> struct rdt_hw_resource is inaccessible to the filesystem code. Because
> this value is output to user-space, it has to be accessible to the
> filesystem code.
> 
> Move it to struct rdt_resource.


>  * Reword comments to avoid being overly arch-specific.
> 
>    Is the _content_ of mbm_cfg_mask arch-independent?

This comes from the mbm_total_bytes_config, nothing in Documentation/arch/x86/resctrl.rst
says this is AMD specific, so it can be assumed to be defined by resctrl, but depending on
some hardware feature.

I'm not sure we can make this retroactively platform specific.


Thanks,

James