On 10.01.2025 14:28, Alejandro Vallejo wrote:
> --- a/xen/arch/x86/xstate.c
> +++ b/xen/arch/x86/xstate.c
> @@ -1024,9 +1024,10 @@ int handle_xsetbv(u32 index, u64 new_bv)
>
> uint64_t read_bndcfgu(void)
> {
> + uint64_t bndcfgu = 0;
> unsigned long cr0 = read_cr0();
> - struct xsave_struct *xstate
> - = idle_vcpu[smp_processor_id()]->arch.xsave_area;
> + struct vcpu *v = idle_vcpu[smp_processor_id()];
Question on this one remains: Can it be pointer-to-const (in the longer
run; certainly in can be right now)?
> + struct xsave_struct *xstate = VCPU_MAP_XSAVE_AREA(v);
I realize my similar remark on this one was actually wrong; the asm()s
clearly modify what is being pointed top.
Jan
