From owner-freebsd-stable@FreeBSD.ORG  Thu Jul 15 20:02:05 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 6A15C106566C;
	Thu, 15 Jul 2010 20:02:05 +0000 (UTC)
	(envelope-from olli@lurza.secnetix.de)
Received: from lurza.secnetix.de (lurza.secnetix.de [IPv6:2a01:170:102f::2])
	by mx1.freebsd.org (Postfix) with ESMTP id E0A608FC1E;
	Thu, 15 Jul 2010 20:02:04 +0000 (UTC)
Received: from lurza.secnetix.de (localhost [127.0.0.1])
	by lurza.secnetix.de (8.14.3/8.14.3) with ESMTP id o6FK1mPk088946;
	Thu, 15 Jul 2010 22:02:03 +0200 (CEST)
	(envelope-from oliver.fromme@secnetix.de)
Received: (from olli@localhost)
	by lurza.secnetix.de (8.14.3/8.14.3/Submit) id o6FK1mGq088944;
	Thu, 15 Jul 2010 22:01:48 +0200 (CEST) (envelope-from olli)
From: Oliver Fromme <olli@lurza.secnetix.de>
Message-Id: <201007152001.o6FK1mGq088944@lurza.secnetix.de>
To: jkim@FreeBSD.org (Jung-uk Kim)
Date: Thu, 15 Jul 2010 22:01:48 +0200 (CEST)
In-Reply-To: <201007151507.33998.jkim@FreeBSD.org>
X-Mailer: ELM [version 2.5 PL8]
MIME-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 8bit
X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.3.4
	(lurza.secnetix.de [127.0.0.1]);
	Thu, 15 Jul 2010 22:02:03 +0200 (CEST)
Cc: freebsd-stable@FreeBSD.org, Andriy Gapon <avg@icyb.net.ua>
Subject: Re: 8.1-PRERELEASE: CPU packages not detected correctly
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: Thu, 15 Jul 2010 20:02:05 -0000


Jung-uk Kim wrote:
 > On Thursday 15 July 2010 01:56 pm, Andriy Gapon wrote:
 > > on 15/07/2010 19:57 Oliver Fromme said the following:
 > > > I patched topo_probe() so it calls topo_probe_0x4() after
 > > > topo_probe_0xb() if cpu_cores is still 0.  I think this
 > > > is a better fallback procedure.  With this patch, cpu_cores
 > > > gets the value 4 which is the correct one, finally:
 > > [...]
 > > I think that your addition achieves this effect, perhaps just not
 > > as explicitly as I would preferred.
 > > 
 > > Jung-uk, what do you think?
 > 
 > Yes, you're right.  Please try new patch:
 > 
 > http://people.freebsd.org/~jkim/mp_machdep2.diff

Thank you!

I will have access to that particular machine on Monday again,
so testing the new patch will have to wait until Monday.
But from looking at your patch it should have the same result
as my simpler patch, so it should work fine.

Best regards
   Oliver

-- 
Oliver Fromme, secnetix GmbH & Co. KG, Marktplatz 29, 85567 Grafing b. M.
Handelsregister: Registergericht Muenchen, HRA 74606,  Geschäftsfuehrung:
secnetix Verwaltungsgesellsch. mbH, Handelsregister: Registergericht Mün-
chen, HRB 125758,  Geschäftsführer: Maik Bachmann, Olaf Erb, Ralf Gebhart

FreeBSD-Dienstleistungen, -Produkte und mehr:  http://www.secnetix.de/bsd

"C++ is the only current language making COBOL look good."
        -- Bertrand Meyer