Re: [Xen-devel] [PATCH v2 13/30] xen/x86: Clear dependent features when clearing a cpu cap

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 05.02.16 at 14:42, <andrew.cooper3@xxxxxxxxxx> wrote:
> --- a/xen/arch/x86/cpu/common.c
> +++ b/xen/arch/x86/cpu/common.c
> @@ -53,8 +53,24 @@ static unsigned int cleared_caps[NCAPINTS];
>  
>  void __init setup_clear_cpu_cap(unsigned int cap)
>  {
> +     const uint32_t *dfs;
> +     unsigned int i;
> +
> +     if ( test_bit(cap, cleared_caps) )
> +             return;
> +
>       __clear_bit(cap, boot_cpu_data.x86_capability);
>       __set_bit(cap, cleared_caps);
> +
> +     dfs = lookup_deep_deps(cap);
> +
> +     if ( !dfs )
> +             return;
> +
> +     for ( i = 0; i < FSCAPINTS; ++i ) {
> +             cleared_caps[i] |= dfs[i];
> +             boot_cpu_data.x86_capability[i] &= ~dfs[i];
> +     }
>  }

Oh, also - you're mixing styles here (note the stray blanks inside
for() and if()) ...

Jan


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