On Tue, Apr 18, 2023 at 11:35:41AM +0200, Jan Beulich wrote:
> The function merely needs moving earlier in the file to avoid the need
> for a forward declaration. While moving it, also rename it following the
> recent folding of CPUID and MSR policies.
> 
> Signed-off-by: Jan Beulich <jbeul...@suse.com>

Acked-by: Roger Pau Monné <roger....@citrix.com>

We might also want to rename the hvm_function_table hook.

One minor comment below.

> 
> --- a/xen/arch/x86/domain.c
> +++ b/xen/arch/x86/domain.c
> @@ -288,6 +288,16 @@ void update_guest_memory_policy(struct v
>      }
>  }
>  
> +/*
> + * Called during vcpu construction, and each time the toolstack changes the
> + * CPUID configuration for the domain.

The comment also needs to be updated to contain CPUID/MSR or some
such now.

Thanks, Roger.

Reply via email to