On 9/20/2017 4:13 PM, Paolo Bonzini wrote:
On 20/09/2017 08:35, Yu Zhang wrote:
2 reasons I did not choose to change kvm_cpuid(): 1> like Jim's
comments, kvm_cpuid() will eventually write the *eax - *edx no
matter a cpuid entry is found or not; 2> currently, return value of
kvm_cpuid() is either true when an entry is found or false otherwise.
We can change kvm_cpuid() to check the pointers of GPRs against NULL
and return false immediately. Then the false value would have 2
different meanings - entry not found, or invalid params.
Paolo, any suggestion? :-)
Radim, has already sent this version to Linus. :)
Got it. Thanks. :)
Yu
Paolo