Subject: Re: CVS commit: src/sys/arch/x86/x86
To: None <source-changes@NetBSD.org>
From: Joerg Sonnenberger <joerg@britannica.bec.de>
List: source-changes
Date: 12/21/2007 16:00:50
On Fri, Dec 21, 2007 at 09:47:40AM -0500, Greg Troxel wrote:
> Sorry - I didn't go read the code in question before commenting.  After
> looking at it, I don't understand if this is dealing with something that
> has a single temperature sensors for 2 cpus on the same chip, or a
> temperature sensor per cpu.

The sensor is per-core. The point I want to stress is that if you have a
Dual Xeon, you have two cores in two physical packages. We don't have
the logical mapping of cores to physical cpus yet and I believe that the
common entitiy of abstraction is still called "cpu" in NetBSD. For that
it is confusing to name them core0, core1, core2, core3 as it should be
"cpu0 core0", ..., " cpu1 core1" if anything, otherwise this is just a
distracting change.

Joerg