lkml.org 
[lkml]   [2008]   [Jun]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 2/5] x86: PAT: fix ambiguous paranoia check in pat_init()
On 11-06-08 18:12, Andreas Herrmann wrote:

>> Again not wrong, or at least by design. Thomas Gleixner did it this way and
>> with that "paranoia check" explicitly bombing out only for SMP this
>> wouldn't have been by accident. He no doubt knows why he did so (and he's
>> in CC so if we're real lucky we might also now...)
>
> I guess at the time Thomas' patch was commited this was just fine.
>
> But with the recent Transmeta/Centaur patch, validate_pat_support()
> returns w/o disabling PAT even for such vendor's CPUs that don't
> support PAT,

In a sense that recent patch in the x86 tree could be consired the buggy
one as it fails to explicitly whitelist those models with functional PAT
while THAT was the setup of things here -- but yes, don't get me wrong,
I also think that setup wasn't particularly great.

Your followup patch turns the whitelist into a blacklist, blacklisting
those Intel models which weren't specifically whitelisted before, which
is a saner approach, so <shrug>. If things are ready for that, all the
better.

Rene.


\
 
 \ /
  Last update: 2008-06-12 06:35    [W:0.055 / U:0.004 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site