On 05.09.2019 18:04, Roger Pau Monne wrote:
> --- a/xen/arch/x86/xstate.c
> +++ b/xen/arch/x86/xstate.c
> @@ -577,7 +577,11 @@ unsigned int xstate_ctxt_size(u64 xcr0)
> /* Collect the information of processor's extended state */
> void xstate_init(struct cpuinfo_x86 *c)
> {
> - static bool __initdata use_xsave = true;
> + /*
> + * NB: use_xsave cannot live in initdata because llvm might optimize
> + * reading it, see: https://bugs.llvm.org/show_bug.cgi?id=39707
> + */
> + static bool use_xsave = true;
> boolean_param("xsave", use_xsave);
>
> bool bsp = c == &boot_cpu_data;
I think we'd want to use __read_mostly then instead. Can be added
while committing of course, if you agree. With the addition
Acked-by: Jan Beulich <[email protected]>
Jan
_______________________________________________
Xen-devel mailing list
[email protected]
https://lists.xenproject.org/mailman/listinfo/xen-devel
