Re: [PATCH 2/2] cpu: intel, amd: mask cleared cpuid features

[Alan Cox <alan@lxorguk.ukuu.org.uk>]

> So actually, making it straightforward to disable CPUID feature bits
> just for every whim is the bug.

Sometimes its needed to make stuff work. Expecting user space to go
digging in odd places isn't good either but exposing *both* true/apparent
cpuid bits might not be a bad idea.

> I'd like to see a real valid reason why someone would even think that.
> Except virtualization folks who are crazy anyway, so that doesn't count :).

Which is a very large part of the x86 market. So they most definitely do
count. Virtualisation is somewhat different though. There you are trying
to define a subset of the features that all the systems in your
environment have so you can do migrations. Virtualisation you have rather
more different control of the cpuid and msrs anyway.

> Majority of users is majority of users no matter how you look at it!

That's not a good argument. The majority of users don't have SCSI,
certain processors and so on ...

> Right, and how is giving the user a heavy, well-oiled AK-47 to do that,
> user-friendly?

It's a point and click interface

> And this is exactly what I'm questioning: the usability, or rather, the
> mis-usability of such a feature.

What goes with that is "so how do you do it otherwise". Distros can
certainly add patches for such features if needed but that just makes it
even more fun to debug.
 
Does "bind mount your own cpuid file" cover this ?

Alan