cvs commit: src/sys/amd64/amd64 identcpu.c src/sys/i386/i386
jkim at FreeBSD.org
Mon Oct 17 11:17:53 PDT 2005
On Monday 17 October 2005 01:28 pm, David O'Brien wrote:
> On Mon, Oct 17, 2005 at 12:00:47PM -0400, Jung-uk Kim wrote:
> > On Sunday 16 October 2005 04:58 am, David E. O'Brien wrote:
> > > obrien 2005-10-16 08:58:27 UTC
> > >
> > > FreeBSD src repository
> > >
> > > Modified files:
> > > sys/amd64/amd64 identcpu.c
> > > sys/i386/i386 identcpu.c
> > > Log:
> > > For AMD processors, nullify CPUID.HTT. FreeBSD has no need
> > > for the information it conveys, and it is only confusing
> > > people. This fixes incorrect output in the previous commit.
> > >
> > > Revision Changes Path
> > > 1.138 +9 -0 src/sys/amd64/amd64/identcpu.c
> > > 1.147 +9 -0 src/sys/i386/i386/identcpu.c
> > Have you verified you are not breaking `logical_cpus' in
> > mp_machdep.c by doing this?
> > Show me the 'incorrect' output or please back it out.
> CPU: Dual Core AMD Opteron(tm) Processor 280 (2392.35-MHz
> K8-class CPU) ..
> Physical/Logical cores: 2/2
> FreeBSD/SMP: Multiprocessor System Detected: 4 CPUs
> cpu0 (BSP): APIC ID: 0
> cpu1 (AP): APIC ID: 1
> cpu2 (AP): APIC ID: 2
> cpu3 (AP): APIC ID: 3
> I sure as hell have 4 physical cores and no logical cores.
The information is per-CPU. You have two CPUs with dual cores, which
means the given information is correct. Please back it out.
> I do not think you fully understand what AMD dual-core CPU's.
More information about the cvs-all