Re: [PATCH] x86: introduce a new Linux defined feature flag for PATsupport

From: H. Peter Anvin
Date: Wed May 07 2008 - 22:29:42 EST


Linus Torvalds wrote:

*Certainly* I don't want anything like this crap:

- if (cpu_has_pat)
+ if (cpu_has_pat && cpu_has_pat_good)

This in fact is likely the best part of it.

Because that at least guarantees that we never say we have a good PAT when the hardware doesn't even report it.

As it is, we seem to just blindly override hardware. It may be correct for all the models we override, but still..


Yah, this is not good. We should mask out the bit, but never, ever, set it if it was clear to begin with (unless we have it on really, *really*, good authority.)

I'm embarrassed to have let that slink by.

-hpa
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/