From owner-svn-src-head@freebsd.org Fri Dec 4 18:20:26 2015 Return-Path: Delivered-To: svn-src-head@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 74099A41A80; Fri, 4 Dec 2015 18:20:26 +0000 (UTC) (envelope-from bdrewery@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:1900:2254:206c::16:87]) by mx1.freebsd.org (Postfix) with ESMTP id 56E731508; Fri, 4 Dec 2015 18:20:26 +0000 (UTC) (envelope-from bdrewery@FreeBSD.org) Received: from mail.xzibition.com (localhost [IPv6:::1]) by freefall.freebsd.org (Postfix) with ESMTP id 5023E17EE; Fri, 4 Dec 2015 18:20:26 +0000 (UTC) (envelope-from bdrewery@FreeBSD.org) Received: from mail.xzibition.com (localhost [172.31.3.2]) by mail.xzibition.com (Postfix) with ESMTP id F2FE118924; Fri, 4 Dec 2015 18:20:25 +0000 (UTC) X-Virus-Scanned: amavisd-new at mail.xzibition.com Received: from mail.xzibition.com ([172.31.3.2]) by mail.xzibition.com (mail.xzibition.com [172.31.3.2]) (amavisd-new, port 10026) with LMTP id lpUQxnUq27UM; Fri, 4 Dec 2015 18:20:22 +0000 (UTC) Subject: Re: svn commit: r291744 - in head/sys: conf modules DKIM-Filter: OpenDKIM Filter v2.9.2 mail.xzibition.com 6E8EC1891B To: John Baldwin References: <201512040427.tB44RL8S067654@repo.freebsd.org> <2092924.RITcBJdNZd@ralph.baldwin.cx> Cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org From: Bryan Drewery Openpgp: id=F9173CB2C3AAEA7A5C8A1F0935D771BB6E4697CF; url=http://www.shatow.net/bryan/bryan2.asc Organization: FreeBSD Message-ID: <5661D967.2030108@FreeBSD.org> Date: Fri, 4 Dec 2015 10:20:23 -0800 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:38.0) Gecko/20100101 Thunderbird/38.4.0 MIME-Version: 1.0 In-Reply-To: <2092924.RITcBJdNZd@ralph.baldwin.cx> Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="C4OQN3JHRU0f2UxXhPgF8fHbG3N5NPqn5" X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 04 Dec 2015 18:20:26 -0000 This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --C4OQN3JHRU0f2UxXhPgF8fHbG3N5NPqn5 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: quoted-printable On 12/4/2015 9:50 AM, John Baldwin wrote: > On Friday, December 04, 2015 04:27:21 AM Bryan Drewery wrote: >> Author: bdrewery >> Date: Fri Dec 4 04:27:21 2015 >> New Revision: 291744 >> URL: https://svnweb.freebsd.org/changeset/base/291744 >> >> Log: >> Calculate MPATH for sys/modules to save 92% time in a basic 'obj' tr= ee-walk. >> =20 >> Sponsored by: EMC / Isilon Storage Division >> >> Modified: >> head/sys/conf/kmod.mk >> head/sys/modules/Makefile >> >> Modified: head/sys/conf/kmod.mk >> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D >> --- head/sys/conf/kmod.mk Fri Dec 4 03:54:18 2015 (r291743) >> +++ head/sys/conf/kmod.mk Fri Dec 4 04:27:21 2015 (r291744) >> @@ -370,8 +370,10 @@ vnode_if_typedef.h: >> .endif >> =20 >> # Build _if.[ch] from _if.m, and clean them when we're done. >> +# This is duplicated in sys/modules/Makefile. >> .if !defined(__MPATH) >> __MPATH!=3Dfind ${SYSDIR:tA}/ -name \*_if.m >> +.export __MPATH >> .endif >> _MFILES=3D${__MPATH:T:O} >> _MPATH=3D${__MPATH:H:O:u} >=20 > It's also duplicated in sys/conf/kern.pre.mk which uses this to export = it: >=20 > # Calculate path for .m files early, if needed. > .if !defined(__MPATH) > __MPATH!=3Dfind ${S:tA}/ -name \*_if.m > .endif > .... > MKMODULESENV+=3D __MPATH=3D"${__MPATH}" >=20 Ah. I do want to consolidate this logic. Further I want to only generate these files *ONCE* and not for every sub-directory that needs them. It's on my list to tackle soon. I'll put that one out for review. > So, the comment should probably reference the two other places that wil= l pass > an __MPATH down (kern.pre.mk and sys/modules/Makefile). >=20 > Note though that you should only be seeing this if you are building mod= ules > outside of a kernel build (e.g. MODULES_WITH_WORLD). modules built as = part of > a kernel build will inherit __MPATH from kern.pre.mk. >=20 > When I ran a buildkernel with truss -af and grepp'd for all the execve'= s of > /usr/bin/find I only found about 6 for a GENERIC kernel, so I believe t= hat > the kern.pre.mk bits were working already. >=20 I've been running a lot of manual builds against sys/modules to verify various changes. I actually have not been running buildworld much lately.= --=20 Regards, Bryan Drewery --C4OQN3JHRU0f2UxXhPgF8fHbG3N5NPqn5 Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQEcBAEBAgAGBQJWYdlnAAoJEDXXcbtuRpfP7okIAKtWVMYJgGCMViWih+YM9ZpV 0nRn7jT4qsog+Eli7A+86XgmAYMlsZvrJvxexPMGwL3mIjeqL7yed7VI44450NjQ RlPbBgxOjUKGxnEiP8F+oO4RbuNRr70XIDW3vPkOPVp+bz7phnXkQYPVFhR0hYBi PPHH8KKBOOXGZvoQNHXuc/xqSrlHCIjGwPomnKeXolrAIjMnSCe7juMKspizui5c OLB/qxuAVY+Wt31llKkhQ+j/OFq+00K/c9OfN3bZt4giYI93kTd9AcnMP+7tnYd1 +9nWEng+2QgtYpKg8Zj139kBCP3yw016y2EpgWIxCn4g35qgaKGmw2VLtjY3bgI= =5w1l -----END PGP SIGNATURE----- --C4OQN3JHRU0f2UxXhPgF8fHbG3N5NPqn5--