From owner-freebsd-bugs@FreeBSD.ORG  Mon Aug  9 00:30:11 2010
Return-Path: <owner-freebsd-bugs@FreeBSD.ORG>
Delivered-To: freebsd-bugs@hub.freebsd.org
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id 27512106567C
	for <freebsd-bugs@hub.freebsd.org>;
	Mon,  9 Aug 2010 00:30:11 +0000 (UTC)
	(envelope-from gnats@FreeBSD.org)
Received: from freefall.freebsd.org (freefall.freebsd.org
	[IPv6:2001:4f8:fff6::28])
	by mx1.freebsd.org (Postfix) with ESMTP id B8F208FC18
	for <freebsd-bugs@hub.freebsd.org>;
	Mon,  9 Aug 2010 00:30:10 +0000 (UTC)
Received: from freefall.freebsd.org (localhost [127.0.0.1])
	by freefall.freebsd.org (8.14.4/8.14.4) with ESMTP id o790UAm1085182
	for <freebsd-bugs@freefall.freebsd.org>; Mon, 9 Aug 2010 00:30:10 GMT
	(envelope-from gnats@freefall.freebsd.org)
Received: (from gnats@localhost)
	by freefall.freebsd.org (8.14.4/8.14.4/Submit) id o790UAPn085176;
	Mon, 9 Aug 2010 00:30:10 GMT (envelope-from gnats)
Date: Mon, 9 Aug 2010 00:30:10 GMT
Message-Id: <201008090030.o790UAPn085176@freefall.freebsd.org>
To: freebsd-bugs@FreeBSD.org
From: dfilter@FreeBSD.ORG (dfilter service)
Cc: 
Subject: Re: kern/148698: commit references a PR
X-BeenThere: freebsd-bugs@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
Reply-To: dfilter service <dfilter@FreeBSD.ORG>
List-Id: Bug reports <freebsd-bugs.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-bugs>,
	<mailto:freebsd-bugs-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-bugs>
List-Post: <mailto:freebsd-bugs@freebsd.org>
List-Help: <mailto:freebsd-bugs-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-bugs>,
	<mailto:freebsd-bugs-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Mon, 09 Aug 2010 00:30:11 -0000

The following reply was made to PR kern/148698; it has been noted by GNATS.

From: dfilter@FreeBSD.ORG (dfilter service)
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: kern/148698: commit references a PR
Date: Mon,  9 Aug 2010 00:24:11 +0000 (UTC)

 Author: attilio
 Date: Mon Aug  9 00:23:57 2010
 New Revision: 211087
 URL: http://svn.freebsd.org/changeset/base/211087
 
 Log:
   The r208165 fixed a bug related to unsigned integer overflowing for the
   number of CPUs detection.
   However, that was not mention at all, the problem was not reported, the
   patch has not been MFCed and the fix is mostly improper.
   
   Fix the original overflow (caused when 32 CPUs must be detected) by
   just using a different mathematical computation (it also makes more
   explicit the size of operands involved, which is good in the moment
   waiting for a more complete support for a large number of CPUs).
   
   PR:		kern/148698
   Submitted by:	Joe Landers <jlanders at vmware dot com>
   Tested by:	gianni
   MFC after:	10 days
 
 Modified:
   head/sys/kern/subr_smp.c
 
 Modified: head/sys/kern/subr_smp.c
 ==============================================================================
 --- head/sys/kern/subr_smp.c	Sun Aug  8 23:24:23 2010	(r211086)
 +++ head/sys/kern/subr_smp.c	Mon Aug  9 00:23:57 2010	(r211087)
 @@ -504,10 +504,7 @@ smp_topo_none(void)
  	top = &group[0];
  	top->cg_parent = NULL;
  	top->cg_child = NULL;
 -	if (mp_ncpus == sizeof(top->cg_mask) * 8)
 -		top->cg_mask = -1;
 -	else
 -		top->cg_mask = (1 << mp_ncpus) - 1;
 +	top->cg_mask = ~0U >> (32 - mp_ncpus);
  	top->cg_count = mp_ncpus;
  	top->cg_children = 0;
  	top->cg_level = CG_SHARE_NONE;
 _______________________________________________
 svn-src-all@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/svn-src-all
 To unsubscribe, send any mail to "svn-src-all-unsubscribe@freebsd.org"