2018-02-28 13:55 GMT+08:00 Tom Lendacky <[email protected]>: > On 2/27/2018 9:34 PM, Wanpeng Li wrote: >> From: Wanpeng Li <[email protected]> >> >> 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 <[email protected]> >> Cc: Paolo Bonzini <[email protected]> >> Cc: Radim Krčmář <[email protected]> >> Cc: Liran Alon <[email protected]> >> Cc: Nadav Amit <[email protected]> >> Cc: Borislav Petkov <[email protected]> >> Cc: Tom Lendacky <[email protected]> >> Signed-off-by: Wanpeng Li <[email protected]> >> --- >> 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
