Re: Discrepancies between /proc/cpuinfo and Dave J's x86info

[Jordan Breeding]

Dave Jones wrote:
> 
> On Wed, 11 Jul 2001, Hugh Dickins wrote:
> 
> > As others have said, cpuid level 3 corresponds to Processor Serial
> > Number enabled.  I think what you have here is a machine on which
> > the BIOS has disabled PSN on the first CPU, but left it enabled on the
> > second CPU, and so the kernel has then disabled it on the second CPU.
> 
> I'll bet that's exactly what it is. Good work.
> 
> This patch (against 247pre6) should keep the cpuinfo in sync with the real
> state of the CPU..
> 
> regards,
> 
> Dave.
> 
> --
> | Dave Jones.        http://www.suse.de/~davej
> | SuSE Labs
> 
> diff -urN --exclude-from=/home/davej/.exclude linux-247pre7/arch/i386/kernel/setup.c linux-dj/arch/i386/kernel/setup.c
> --- linux-247pre7/arch/i386/kernel/setup.c      Wed Jul 11 13:16:10 2001
> +++ linux-dj/arch/i386/kernel/setup.c   Wed Jul 11 13:18:27 2001
> @@ -1994,6 +1994,7 @@
>                 wrmsr(0x119,lo,hi);
>                 printk(KERN_NOTICE "CPU serial number disabled.\n");
>                 clear_bit(X86_FEATURE_PN, &c->x86_capability);
> +               c->cpuid_level--;
>         }
>  }

It does keep everything in sync here.  Thanks for the help.

Jordan