Andi Kleen <andi@xxxxxxxxxxxxxx> writes: >> >>> Signed-off-by: Kan Liang <kan.liang@xxxxxxxxx> >> >> And here I thought that Andi was of the opinion that if you set CPUID to >> indicate a particular CPU you had better also handle all its MSRs. > > Yes, philosophically that would be the right way, > but we needed a short term fix to stop things from crashing, and that > was the simplest. I should add there is another reason for this patch now, and doing it in perf instead of somewhere else (this should probably go into the description). With PT on enabling LBR can #GP. So perf needs to handle this case without crashing. This can happen independently of any hypervisors. -Andi -- ak@xxxxxxxxxxxxxxx -- Speaking for myself only -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
