2008-06-25 19:53:48

by Bernd Schubert

[permalink] [raw]
Subject: x86: enable pat on amd athlon mode=6

Hello,

on my AthlonXP 1700 PAT works fine (I tested it several days). Please accept one of the patches below, I don't know which you refer.


Signed-off-by: Bernd Schubert <[email protected]>

Patch to enable PAT specifically on model=6

diff --git a/arch/x86/kernel/cpu/addon_cpuid_features.c b/arch/x86/kernel/cpu/addon_cpuid_features.c
index c2e1ce3..61a5aa5 100644
--- a/arch/x86/kernel/cpu/addon_cpuid_features.c
+++ b/arch/x86/kernel/cpu/addon_cpuid_features.c
@@ -57,6 +57,8 @@ void __cpuinit validate_pat_support(struct cpuinfo_x86 *c)
case X86_VENDOR_AMD:
if (c->x86 >= 0xf && c->x86 <= 0x11)
return;
+ if (c->x86 == 0x6)
+ return;
break;
case X86_VENDOR_INTEL:
if (c->x86 == 0xF || (c->x86 == 6 && c->x86_model >= 15))


Or maybe PAT on all system >=6 is suitable?


diff --git a/arch/x86/kernel/cpu/addon_cpuid_features.c b/arch/x86/kernel/cpu/addon_cpuid_features.c
index c2e1ce3..f12c413 100644
--- a/arch/x86/kernel/cpu/addon_cpuid_features.c
+++ b/arch/x86/kernel/cpu/addon_cpuid_features.c
@@ -55,7 +55,7 @@ void __cpuinit validate_pat_support(struct cpuinfo_x86 *c)
{
switch (c->x86_vendor) {
case X86_VENDOR_AMD:
- if (c->x86 >= 0xf && c->x86 <= 0x11)
+ if (c->x86 >= 0x6 && c->x86 <= 0x11)
return;
break;
case X86_VENDOR_INTEL:


Thanks,
Bernd


2008-06-25 20:14:38

by Rene Herman

[permalink] [raw]
Subject: Re: x86: enable pat on amd athlon mode=6

On 25-06-08 21:53, Bernd Schubert wrote:

> on my AthlonXP 1700 PAT works fine (I tested it several days). Please
> accept one of the patches below, I don't know which you refer.
>
> Signed-off-by: Bernd Schubert <[email protected]>

Yes, PAT works equally well on my earlier model 6. The setup's already
changed in linux-next though and now blacklists some Intel models only:

http://git.kernel.org/?p=linux/kernel/git/sfr/linux-next.git;a=blob;f=arch/x86/kernel/cpu/addon_cpuid_features.c;h=84a8220a6072c0425b9da95fbb28b5a309cbfda1;hb=HEAD

Rene.

2008-06-25 20:20:14

by Rene Herman

[permalink] [raw]
Subject: Re: x86: enable pat on amd athlon mode=6

On 25-06-08 22:15, Rene Herman wrote:

> On 25-06-08 21:53, Bernd Schubert wrote:
>
>> on my AthlonXP 1700 PAT works fine (I tested it several days). Please
>> accept one of the patches below, I don't know which you refer.
>>
>> Signed-off-by: Bernd Schubert <[email protected]>
>
> Yes, PAT works equally well on my earlier model 6. The setup's already
> changed in linux-next though and now blacklists some Intel models only:
>
> http://git.kernel.org/?p=linux/kernel/git/sfr/linux-next.git;a=blob;f=arch/x86/kernel/cpu/addon_cpuid_features.c;h=84a8220a6072c0425b9da95fbb28b5a309cbfda1;hb=HEAD

Well, if I provide a link, I might as well look at it myself I guess...

Earlier patches on linux-kernel did the white->black list switching but
as you can see that in linux-next at least all of AMD is already
whitelisted.

Rene.

2008-06-25 20:44:47

by Bernd Schubert

[permalink] [raw]
Subject: Re: x86: enable pat on amd athlon mode=6

On Wednesday 25 June 2008, Rene Herman wrote:
> On 25-06-08 22:15, Rene Herman wrote:
> > On 25-06-08 21:53, Bernd Schubert wrote:
> >> on my AthlonXP 1700 PAT works fine (I tested it several days). Please
> >> accept one of the patches below, I don't know which you refer.
> >>
> >> Signed-off-by: Bernd Schubert <[email protected]>
> >
> > Yes, PAT works equally well on my earlier model 6. The setup's already
> > changed in linux-next though and now blacklists some Intel models only:
> >
> > http://git.kernel.org/?p=linux/kernel/git/sfr/linux-next.git;a=blob;f=arc
> >h/x86/kernel/cpu/addon_cpuid_features.c;h=84a8220a6072c0425b9da95fbb28b5a3
> >09cbfda1;hb=HEAD
>
> Well, if I provide a link, I might as well look at it myself I guess...
>
> Earlier patches on linux-kernel did the white->black list switching but
> as you can see that in linux-next at least all of AMD is already
> whitelisted.

Great, thanks!


Cheers,
Bernd