2018-02-28 13:55 GMT+08:00 Tom Lendacky <thomas.lenda...@amd.com>: > On 2/27/2018 9:34 PM, Wanpeng Li wrote: >> From: Wanpeng Li <wanpen...@tencent.com> >> >> Linux (among the others) has checks to make sure that certain features >> aren't enabled on a certain family/model/stepping if the microcode version >> isn't greater than or equal to a known good version. >> >> By exposing the real microcode version, we're preventing buggy guests that >> don't check that they are running virtualized (i.e., they should trust the >> hypervisor) from disabling features that are effectively not buggy. >> >> Suggested-by: Filippo Sironi <sir...@amazon.de> >> Cc: Paolo Bonzini <pbonz...@redhat.com> >> Cc: Radim Krčmář <rkrc...@redhat.com> >> Cc: Liran Alon <liran.a...@oracle.com> >> Cc: Nadav Amit <nadav.a...@gmail.com> >> Cc: Borislav Petkov <b...@alien8.de> >> Cc: Tom Lendacky <thomas.lenda...@amd.com> >> Signed-off-by: Wanpeng Li <wanpen...@tencent.com> >> --- >> v4 -> v5: >> * microcode_version be u64 and initialized suitable, remote hte shifts >> v3 -> v4: >> * add the shifts back >> v2 -> v3: >> * remove the shifts >> * add the MSR_IA32_UCODE_REV version to the "feature MSRs" > > I think you lost this part from the patch series. I don't see where you > add MSR_IA32_UCODE_REV to the msr_based_features array.
I sent out a wrong version, will send again. Regards, Wanpeng Li
