Subject: Re: PR/35302: Wrong Intel CPUs features display
To: None <port-amd64@NetBSD.org>
From: Nicolas Joly <firstname.lastname@example.org>
Date: 03/07/2007 12:03:29
On Wed, Mar 07, 2007 at 08:40:00AM +0100, Bernd Ernesti wrote:
> On Wed, Mar 07, 2007 at 12:00:51AM +0100, Nicolas Joly wrote:
> > On Tue, Mar 06, 2007 at 07:13:46PM +0100, Bernd Ernesti wrote:
> > > The only problem is that it changes amd64 and not i386 too.
> > i386 does not have the problem; it support Intel CPUs for quite a long
> > time, and display correct CPUs features.
> Hmm, ok not Intel CPUs, but AMD CPUs are still displayed differently on i386:
> cpu0: AMD Dual-Core Opteron or Athlon 64 X2 (686-class)
Ok, but that's another story. For now, i'm just fixing incorrect
display on amd64 port for some CPUs.
> > > IMHO we should move this function into x86.
> > I'm not sure for the whole function, but indeed some parts could be
> > shared.
> I can't talk about this function because I know to few about it, but
> IMHO all parts which would work on i386 and amd64 should be shared.
I had a look; and in this area, cpu_info structures are currently too
different to share some code. But this can change ...
uint32_t ci_signature; /* X86 cpuid type */
uint32_t ci_feature_flags;/* X86 %edx CPUID feature bits */
uint32_t ci_feature2_flags;/* X86 %ecx CPUID feature bits */
uint32_t ci_feature3_flags;/* X86 extended feature bits */
uint32_t ci_padlock_flags;/* VIA PadLock feature bits */
uint32_t ci_cpu_class; /* CPU class */
uint32_t ci_brand_id; /* Intel brand id */
uint32_t ci_vendor; /* vendor string */
uint32_t ci_cpu_serial; /* PIII serial number */
uint64_t ci_tsc_freq; /* cpu cycles/second */
Biological Software and Databanks.
Institut Pasteur, Paris.