From owner-svn-src-all@freebsd.org Tue May 3 06:12:38 2016 Return-Path: Delivered-To: svn-src-all@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 45886B2BABA; Tue, 3 May 2016 06:12:38 +0000 (UTC) (envelope-from brde@optusnet.com.au) Received: from mail108.syd.optusnet.com.au (mail108.syd.optusnet.com.au [211.29.132.59]) by mx1.freebsd.org (Postfix) with ESMTP id 0D8A516FA; Tue, 3 May 2016 06:12:37 +0000 (UTC) (envelope-from brde@optusnet.com.au) Received: from c122-106-149-109.carlnfd1.nsw.optusnet.com.au (c122-106-149-109.carlnfd1.nsw.optusnet.com.au [122.106.149.109]) by mail108.syd.optusnet.com.au (Postfix) with ESMTPS id DB4391A37B6; Tue, 3 May 2016 15:52:56 +1000 (AEST) Date: Tue, 3 May 2016 15:52:56 +1000 (EST) From: Bruce Evans X-X-Sender: bde@besplex.bde.org To: Pedro Giffuni cc: John Baldwin , "Ngie Cooper (yaneurabeya)" , src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r298933 - in head: share/man/man9 sys/amd64/include sys/dev/acpica sys/dev/drm2 sys/dev/drm2/i915 sys/kern sys/sys sys/x86/acpica sys/x86/x86 In-Reply-To: Message-ID: <20160503152502.A939@besplex.bde.org> References: <201605021800.u42I0cjK084243@repo.freebsd.org> <4F040E00-AB92-4D32-99F5-9BCB02578DC0@gmail.com> <2097917.RNSsKXUJ7U@ralph.baldwin.cx> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed X-Optus-CM-Score: 0 X-Optus-CM-Analysis: v=2.1 cv=EfU1O6SC c=1 sm=1 tr=0 a=R/f3m204ZbWUO/0rwPSMPw==:117 a=L9H7d07YOLsA:10 a=9cW_t1CCXrUA:10 a=s5jvgZ67dGcA:10 a=kj9zAlcOel0A:10 a=6I5d2MoRAAAA:8 a=gkHm1csUnXyMWDEhC90A:9 a=CjuIK1q_8ugA:10 X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 03 May 2016 06:12:38 -0000 On Mon, 2 May 2016, Pedro Giffuni wrote: > > On 05/02/16 15:52, John Baldwin wrote: >> On Monday, May 02, 2016 11:45:41 AM Ngie Cooper wrote: >>> >>>> On May 2, 2016, at 11:00, John Baldwin wrote: >>>> >>>> Author: jhb >>>> [... excessive quoting trimmed] >>>> New Revision: 298933 >>>> URL: https://svnweb.freebsd.org/changeset/base/298933 >>>> >>>> Log: >>>> Add a new bus method to fetch device-specific CPU sets. >>>> [... excessive quoting trimmed] >>>> kernel >>> >>> This broke the build with gcc: >>> https://jenkins.freebsd.org/job/FreeBSD_HEAD_amd64_gcc/1211/ >> >> I saw. What is odd though is that my tinderbox builds all passed. This >> might >> be due to the recent howmany() changes since _bitset.h only needed >> >> before but now needs (which is borderline to being pointless >> for >> a _foo.h header). > > TBH, I thought so too, but I avoided applying such changes to headers, > and I haven't touched _bitset.h, _foo.h headers cannot use howmany() due to namespace pollution. _bitset.h was already broken, unless it is supposed to be kernel-only -- it uses howmany(). It is kernel-only according to its documention -- bitset is only documented in kernel manpages (in a single unreadable one than is linked ad nauseum). It is otherwise fairly clean. It defines the symbols BITSET_DEFINE, BITSET_T_INITIALIZER, and BITSET_SET in the application namespace This is not completely clean for a _foo.h header. All other BITSET* macros are already in bitset.h I think only BITSET_DEFINE should be in _bitset.h (for use declarations in other headers). select.h avoids this problem by defining its own howmany() macro. This seems to be correct except for the bogus ifdef around the private macro. This ifdef is a little more than a style bug (verboseness) -- it breaks detection of other definitions that might be different. Lexical differences wouldn't matter, but it is easier to never have them. Old versions of select polluted . The select macros just used howmany(). howmany() was in too. Bruce