From owner-freebsd-current@FreeBSD.ORG Wed Sep 14 19:56:23 2011 Return-Path: Delivered-To: freebsd-current@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id BD5F8106564A for ; Wed, 14 Sep 2011 19:56:23 +0000 (UTC) (envelope-from avg@FreeBSD.org) Received: from citadel.icyb.net.ua (citadel.icyb.net.ua [212.40.38.140]) by mx1.freebsd.org (Postfix) with ESMTP id 137BB8FC17 for ; Wed, 14 Sep 2011 19:56:22 +0000 (UTC) Received: from porto.starpoint.kiev.ua (porto-e.starpoint.kiev.ua [212.40.38.100]) by citadel.icyb.net.ua (8.8.8p3/ICyb-2.3exp) with ESMTP id WAA00400; Wed, 14 Sep 2011 22:56:17 +0300 (EEST) (envelope-from avg@FreeBSD.org) Received: from localhost ([127.0.0.1]) by porto.starpoint.kiev.ua with esmtp (Exim 4.34 (FreeBSD)) id 1R3vZB-0001dr-C4; Wed, 14 Sep 2011 22:56:17 +0300 Message-ID: <4E7106DE.40707@FreeBSD.org> Date: Wed, 14 Sep 2011 22:56:14 +0300 From: Andriy Gapon User-Agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:6.0.2) Gecko/20110907 Thunderbird/6.0.2 MIME-Version: 1.0 To: Andrew Boyer References: <2C3C4570-C5F9-42F5-AA81-900151590DB8@averesystems.com> In-Reply-To: <2C3C4570-C5F9-42F5-AA81-900151590DB8@averesystems.com> X-Enigmail-Version: undefined Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: freebsd-current@FreeBSD.org Subject: Re: Identification of HTT cores on newer (CPUID leaf 11) Intel processors X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 14 Sep 2011 19:56:23 -0000 on 14/09/2011 20:59 Andrew Boyer said the following: > When FreeBSD examines the CPU topology using CPUID leaf 11 in > topo_probe_0xb(), it never sets hyperthreading_cpus. At the end of > topo_probe_0x4() it sets hyperthreading_cpus = cpu_logical. > > Adding that assignment to line 316 of sys/amd64/amd64/mp_machdep.c seems to > do the right thing on a system with two quad-core E5620 CPUs. The APIC IDs > that appear when SMT is enabled in the BIOS get marked AP/HT. > > Do you agree? I agree, but... But see this: http://thread.gmane.org/gmane.os.freebsd.devel.hackers/44007/focus=44024 Someone long ago has decided that new HTT is not the same as old HTT and that some rules that apply to old HTT should not apply to new HTT. Even the name. I think that that's not correct. But it doesn't seem that I am able to engage into a discussion the person who made that decision. Also I can not find any other interested developer either. Anyway, hyperthreading_cpus variable is useless beyond dmesg cosmetics. And I don't think that any of my changes affected the dmesg output. In my "avgBSD" I have different SMP topology code, but it's not ready yet to be submitted for merge into the main tree. -- Andriy Gapon