Re: [RFC PATCH v3 5/7] x86/mm: Introduce Remote Action Request

[Rik van Riel <riel@surriel.com>]

On Fri, 2025-06-06 at 01:45 +0300, Nadav Amit wrote:
> 
> However, I think perhaps the bigger issue is that you want to assume
> all cores have the same RAR settings, and right now it might be a bit
> inconsistent.

The documentation strongly suggests that RAR_INFO.TableMaxIndex
is architecturally determined, and always 64 on Sapphire Rapids.

I'm not sure we want code to handle a case the Intel documentation
seems to suggest cannot exist. Maybe somebody from Intel has
some firmer ideas here?

> 
> So you may want to do some initial checks on the BSP as for
> whether RAR is supported and what rar_max_payloads is (e.g., in
> bsp_init_intel() ). And then on each AP, in something like 
> init_intel() you’d call setup_clear_cpu_cap() to disable RAR if any
> CPU's max_payloads is different than the BSP.

Grabbing the value of rar_max_payloads from early_init_intel()
might make sense, since it is supposed to be a system-wide
value, and not a per CPU thing.

That seems like it would solve data race issues?

> 
> [ BTW: further regarding patch 4, it seems cleaner to call
>   rar_cpu_init() from Intel specific code like init_intel() ? ]
> 
Good idea, I'll move the call there.

> Just sharing my thoughts (and further clarifying them),
> 
I appreciate the comments and suggestions!

Now that the code seems to (finally) work reliably,
I should have a faster turnaround time incorporating
people's suggestions, too.

-- 
All Rights Reversed.