From owner-freebsd-stable@FreeBSD.ORG  Mon Sep  6 16:33:26 2010
Return-Path: <owner-freebsd-stable@FreeBSD.ORG>
Delivered-To: freebsd-stable@freebsd.org
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id 7C13F10656CB;
	Mon,  6 Sep 2010 16:33:26 +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 733D38FC18;
	Mon,  6 Sep 2010 16:33:25 +0000 (UTC)
Received: from odyssey.starpoint.kiev.ua (alpha-e.starpoint.kiev.ua
	[212.40.38.101])
	by citadel.icyb.net.ua (8.8.8p3/ICyb-2.3exp) with ESMTP id TAA05715;
	Mon, 06 Sep 2010 19:33:21 +0300 (EEST)
	(envelope-from avg@freebsd.org)
Message-ID: <4C8517D1.3050603@freebsd.org>
Date: Mon, 06 Sep 2010 19:33:21 +0300
From: Andriy Gapon <avg@freebsd.org>
User-Agent: Mozilla/5.0 (X11; U; FreeBSD amd64; en-US;
	rv:1.9.2.8) Gecko/20100823 Lightning/1.0b2 Thunderbird/3.1.2
MIME-Version: 1.0
To: Jeremy Chadwick <freebsd@jdc.parodius.com>
References: <AANLkTinYUz0V+2nSWBMYLf2fL8HnUQ-fvXT0q-5WY4bb@mail.gmail.com>
	<4C782D3B.6020407@icyb.net.ua>
	<201008271743.29393.jkim@FreeBSD.org>
	<4C7835E6.6070309@icyb.net.ua> <4C7A2782.5040009@freebsd.org>
	<4C84DBE6.5010809@freebsd.org>
	<20100906122325.GA78727@icarus.home.lan>
	<4C84E4E1.8010709@freebsd.org>
	<20100906131203.GA80251@icarus.home.lan>
	<4C84EC62.6020205@freebsd.org>
	<20100906162230.GA1444@icarus.home.lan>
In-Reply-To: <20100906162230.GA1444@icarus.home.lan>
X-Enigmail-Version: 1.1.2
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit
Cc: freebsd-current@freebsd.org, freebsd-stable@freebsd.org
Subject: Re: patch for topology detection of Intel CPUs
X-BeenThere: freebsd-stable@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: Production branch of FreeBSD source code <freebsd-stable.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-stable>, 
	<mailto:freebsd-stable-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-stable>
List-Post: <mailto:freebsd-stable@freebsd.org>
List-Help: <mailto:freebsd-stable-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-stable>,
	<mailto:freebsd-stable-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Mon, 06 Sep 2010 16:33:26 -0000

on 06/09/2010 19:22 Jeremy Chadwick said the following:
> On Mon, Sep 06, 2010 at 04:28:02PM +0300, Andriy Gapon wrote:
>> on 06/09/2010 16:12 Jeremy Chadwick said the following:
>>> Great, thanks!  I'll be testing this out on two separate systems, both
>>> RELENG_8:
>>>
>>> - Supermicro X7SBA     + Intel C2D E8400 (stepping 10)
>>> - Supermicro X7SBL-LN2 + Intel C2D E6600 (stepping 6)
>>>
>>> I'll make sure to provide what the topology looks like before and after.
>>> Is CPU-relevant dmesg output sufficient?
>>
>> If you mean something like the below, then yes.  Thanks!
>> [...]
> 
> All done.  Good news (I think): there's no difference in the CPU-related
> topology on either system with your patch, aside from kernel build date.
> The topologies are still detected correctly.  In case you want them:
> 

Thanks a lot for the test!

[test results snipped]

> All other systems I have are C2D and C2Q-based, but I can't easily test
> on those given their production roles.  If there's a particular Intel
> processor family/model you're interested in, let me know and I can dig
> around to see if I have access to one.

No particular models in mind.
If you have systems with more complex topologies, like multiple physical packages
or HTT enabled, I will be interested in seeing test results for those.
Thanks again.
-- 
Andriy Gapon