Re: [PATCH 11/33] arm_mpam: Add the class and component structures for firmware described ris

[Ben Horgan]

Hi Gavin,

I was a bit hasty on one of those changes.

On 11/12/25 16:39, Ben Horgan wrote:
> Hi Gavin,
> 
> On 11/9/25 00:07, Gavin Shan wrote:
>> Hi Ben,
>>
>> On 11/7/25 10:34 PM, Ben Horgan wrote:
>>> From: James Morse <james.morse@xxxxxxx>
>>>
>>> An MSC is a container of resources, each identified by their RIS index.
>>> Some RIS are described by firmware to provide their position in the
>>> system.
>>> Others are discovered when the driver probes the hardware.
>>>
>>> To configure a resource it needs to be found by its class, e.g. 'L2'.
>>> There are two kinds of grouping, a class is a set of components, which
>>> are visible to user-space as there are likely to be multiple instances
>>> of the L2 cache. (e.g. one per cluster or package)
>>>
>>> Add support for creating and destroying structures to allow a hierarchy
>>> of resources to be created.
>>>
>>> CC: Ben Horgan <ben.horgan@xxxxxxx>
>>> Tested-by: Fenghua Yu <fenghuay@xxxxxxxxxx>
>>> Tested-by: Shaopeng Tan <tan.shaopeng@xxxxxxxxxxxxxx>
>>> Tested-by: Peter Newman <peternewman@xxxxxxxxxx>
>>> Signed-off-by: James Morse <james.morse@xxxxxxx>
>>> Signed-off-by: Ben Horgan <ben.horgan@xxxxxxx>

>> This function is only used in mpam_devices.c and won't be exposed in the
>> future, we can make it 'static' and 'inline'.
> 
> Done

Gets used later in mpam_resctl.c so I'll keep as is.

Thanks,

Ben