Re: [PATCH] x86/resctrl: Don't try to free nonexistent RMIDs

From: Dave Martin
Date: Wed Jun 19 2024 - 08:54:46 EST


Hi,

On Tue, Jun 18, 2024 at 10:08:47AM -0700, Reinette Chatre wrote:
> Hi Dave,
>
> On 6/18/24 9:16 AM, Dave Martin wrote:
>
> >
> > Either way, I don't think this impacts the tested-ness of this patch,
> > but please shout if you have concerns.
> >
>
> I agree. The submission stated that it was tested on x86
> "for the monitors-present case". When booting with
> "rdt=!cmt,!mbmtotal,!mbmlocal" (as I did in my test) resctrl
> is not able to discover monitoring support and it
> thus tested the "monitors _not_ present case".
>
> Reinette

Ah, right. I missed the significance of the options.

Thanks again!
---Dave
>
>