Re: [PATCH v1 07/31] x86/resctrl: Wrap resctrl_arch_find_domain() around rdt_find_domain()

From: Dave Martin
Date: Thu Apr 11 2024 - 10:16:17 EST


On Mon, Apr 08, 2024 at 08:17:12PM -0700, Reinette Chatre wrote:
> Hi James,
>
> On 3/21/2024 9:50 AM, James Morse wrote:
> > rdt_find_domain() finds a domain given a resource and a cache-id.
> > It's not quite right for the resctrl arch API as it also returns the
> > position to insert a new domain, which is needed when bringing a
> > domain online in the arch code.
> >
> > Wrap rdt_find_domain() in a another function resctrl_arch_find_domain()
>
> "in a another" -> "in another"

Noted for James' attention.


> > so we avoid the unnecessary argument outside the arch code.
>
> Please do not impersonate code.

[...]

> Reinette

Noted.

Cheers
---Dave