Re: [PATCH] perf/x86: check ucode before disabling PEBS on SandyBridge

H. Peter Anvin
2012-06-13T15:47:18Z

On 06/13/2012 08:37 AM, Peter Zijlstra wrote:
>>
>> perf might, but I have seen such frankensystems in the field and they
>> work (although I haven't seen that particular combo, I have to admit.)
>
> Thing is, if we boot on the wsm, perf could fault the system with
> nonexistent MSR accesses, if we boot on the nhm worst is wrong numbers I
> think.
>
> I'm not really inclined to 'fix' this.. the best I'm willing to accept
> is a patch that would detect this situation and simply disable all
> hardware perf support for the platform.
>

BIOS is *supposed* to pick the least capable CPU as the BSP. BIOS
being, well, BIOS is known to not always do that, but on the other hand
it is apparently common enough that at least *some* BIOSes are known to
pay attention and do so.

Anyway, I'm not suggesting we should go out of our way to make perf work
on these systems. Most likely, as you point out, it is either "just
going to work" or totally fail.

What I don't want to see is creating a new facility and encouraging
people to use it, when it is known to be in error. I'm trying to find
out at the moment just where the practical cutoff is.

-hpa