On 22.06.2020 14:39, Andrew Cooper wrote:
> On 22/06/2020 13:09, Jan Beulich wrote:
>> Coverity validly complains that the new call from
>> tools/tests/cpu-policy/test-cpu-policy.c:test_cpuid_current() leaves
>> two fields uninitialized, yet they get then consumed by
>> x86_cpuid_copy_to_buffer(). (All other present callers of the function
>> pass a pointer to a static - and hence initialized - buffer.)
>>
>> Coverity-ID: 1464809
>> Fixes: c22ced93e167 ("tests/cpu-policy: Confirm that CPUID serialisation is
>> sorted")
>> Signed-off-by: Jan Beulich <jbeul...@suse.com>
>>
>> --- a/xen/lib/x86/cpuid.c
>> +++ b/xen/lib/x86/cpuid.c
>> @@ -176,6 +176,10 @@ void x86_cpuid_policy_fill_native(struct
>> ARRAY_SIZE(p->extd.raw) - 1); ++i )
>> cpuid_leaf(0x80000000 + i, &p->extd.raw[i]);
>>
>> + /* Don't report leaves from possible lower level hypervisor. */
>
> ", for now."
>
> This will change in the future.
I was pondering at that moment whether to add it, but then I didn't
think we'd want to let shine through lower level hypervisor info.
But yes, I've added it, because it won't be wrong to say "for now",
even if it end up being for much longer.
> With this, Reviewed-by: Andrew Cooper <andrew.coop...@citrix.com>
Thanks, Jan
