From owner-cvs-src@FreeBSD.ORG Thu Aug 2 21:17:59 2007 Return-Path: Delivered-To: cvs-src@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 2619316A417; Thu, 2 Aug 2007 21:17:59 +0000 (UTC) (envelope-from peter@FreeBSD.org) Received: from repoman.freebsd.org (repoman.freebsd.org [IPv6:2001:4f8:fff6::29]) by mx1.freebsd.org (Postfix) with ESMTP id 19D4413C45A; Thu, 2 Aug 2007 21:17:59 +0000 (UTC) (envelope-from peter@FreeBSD.org) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.14.1/8.14.1) with ESMTP id l72LHwld073314; Thu, 2 Aug 2007 21:17:58 GMT (envelope-from peter@repoman.freebsd.org) Received: (from peter@localhost) by repoman.freebsd.org (8.14.1/8.14.1/Submit) id l72LHwS9073313; Thu, 2 Aug 2007 21:17:58 GMT (envelope-from peter) Message-Id: <200708022117.l72LHwS9073313@repoman.freebsd.org> From: Peter Wemm Date: Thu, 2 Aug 2007 21:17:58 +0000 (UTC) To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org X-FreeBSD-CVS-Branch: HEAD Cc: Subject: cvs commit: src/sys/i386/i386 local_apic.c mp_machdep.c src/sys/amd64/amd64 local_apic.c mp_machdep.c X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 02 Aug 2007 21:17:59 -0000 peter 2007-08-02 21:17:58 UTC FreeBSD src repository Modified files: sys/i386/i386 local_apic.c mp_machdep.c sys/amd64/amd64 local_apic.c mp_machdep.c Log: Move mp_topology() from apic_init(i386) and apic_setup_local(amd64) to cpu_start_mp(). This is after we have read the cpuid registers to calculate the hyperthreading_cpus value for the sysctl that enables or disables hyperthread cores. Change mp_topology() to use that information rather than trying to do it itself. This solves the problem of ULE being incorrectly told that dual core Athlon64 X2 or Operton cpus are hyperthreading cores. At the very least, we now have a single piece of code to identify hyperthreading. Obtained from: jhb Approved by: re (kensmith) Revision Changes Path 1.41 +0 -4 src/sys/amd64/amd64/local_apic.c 1.287 +6 -14 src/sys/amd64/amd64/mp_machdep.c 1.43 +0 -4 src/sys/i386/i386/local_apic.c 1.281 +6 -14 src/sys/i386/i386/mp_machdep.c