From owner-cvs-src@FreeBSD.ORG Mon Oct 17 18:51:47 2005 Return-Path: X-Original-To: cvs-src@FreeBSD.org Delivered-To: cvs-src@FreeBSD.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id B186416A41F; Mon, 17 Oct 2005 18:51:47 +0000 (GMT) (envelope-from jkim@FreeBSD.org) Received: from anuket.mj.niksun.com (gwnew.niksun.com [65.115.46.162]) by mx1.FreeBSD.org (Postfix) with ESMTP id E74F043D53; Mon, 17 Oct 2005 18:51:46 +0000 (GMT) (envelope-from jkim@FreeBSD.org) Received: from niksun.com (anuket [10.70.0.5]) by anuket.mj.niksun.com (8.13.1/8.13.1) with ESMTP id j9HIxY34081089; Mon, 17 Oct 2005 14:59:34 -0400 (EDT) (envelope-from jkim@FreeBSD.org) From: Jung-uk Kim To: obrien@FreeBSD.org Date: Mon, 17 Oct 2005 14:51:31 -0400 User-Agent: KMail/1.6.2 References: <200510160858.j9G8wR9v045670@repoman.freebsd.org> <200510171417.38095.jkim@FreeBSD.org> <20051017183139.GA97488@dragon.NUXI.org> In-Reply-To: <20051017183139.GA97488@dragon.NUXI.org> MIME-Version: 1.0 Content-Disposition: inline Content-Type: text/plain; charset="euc-kr" Content-Transfer-Encoding: 7bit Message-Id: <200510171451.34478.jkim@FreeBSD.org> X-Virus-Scanned: ClamAV devel-20050919/1139/Sun Oct 16 20:30:04 2005 on anuket.mj.niksun.com X-Virus-Status: Clean Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/amd64/amd64 identcpu.c src/sys/i386/i386 identcpu.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: Mon, 17 Oct 2005 18:51:48 -0000 On Monday 17 October 2005 02:31 pm, David O'Brien wrote: > On Mon, Oct 17, 2005 at 02:17:33PM -0400, Jung-uk Kim wrote: > > 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. > > No, this given information is wrong. AMD does not have > HyperThreading. You are treating it as if it does. ??? Did I ever mention Hyper-Threading here? I actually replaced misleading 'Hyperthreading' with 'Physical/Logical cores.' > The rest of the kernel also should not be trying to treat AMD > dual-core as HyperThreading. Sigh... Sure it should not. That's why there is `hyperthreading_cpus' for. > The only thing that should care in the freebsd kernel right now > about AMD dual-core is PowerNow! support. It probably needs to > know what cores match physical cores so it can proper work in a > multi-socket system. > > Please point to what part of `logical_cpus' in mp_machdep.c you > feel is broken by this change. > > > > I do not think you fully understand what AMD dual-core CPU's. > > > > Oh, please... > > The code you committed suggest this... I will just shut up and code if you 'feel' that I am not capable. Jung-uk Kim