Re: [Xen-devel] [PATCH] x86/amd: Protect set_cpuidmask() against #GP faults

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 05.06.14 at 13:20, <andrew.cooper3@xxxxxxxxxx> wrote:
> --- a/xen/arch/x86/cpu/amd.c
> +++ b/xen/arch/x86/cpu/amd.c
> @@ -151,7 +151,7 @@ static void __devinit set_cpuidmask(const struct 
> cpuinfo_x86 *c)
>       static unsigned int extfeat_ecx, extfeat_edx;
>       static unsigned int l7s0_eax, l7s0_ebx;
>       static unsigned int thermal_ecx;
> -     static bool_t skip_l7s0_eax_ebx, skip_thermal_ecx;
> +     static bool_t skip_feat, skip_extfeat, skip_l7s0, skip_thermal;

Please don't drop the _eax_ebx and _ecx suffixes - they're there
for a reason (leaf 7 likely also getting ECX and EDX a bitmap assigned
at some point, and the thermal leaf also having further unused
registers that may become subject to masking later on).

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel