From owner-svn-src-head@freebsd.org Wed Jan 3 18:43:58 2018 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 1801FEA943A; Wed, 3 Jan 2018 18:43:58 +0000 (UTC) (envelope-from bdrewery@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2610:1c1:1:6074::16:84]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "freefall.freebsd.org", Issuer "Let's Encrypt Authority X3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id DB03C80802; Wed, 3 Jan 2018 18:43:57 +0000 (UTC) (envelope-from bdrewery@FreeBSD.org) Received: from mail.xzibition.com (unknown [127.0.1.132]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by freefall.freebsd.org (Postfix) with ESMTPS id E51A19C03; Wed, 3 Jan 2018 18:43:56 +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 E29A653C2; Wed, 3 Jan 2018 18:43:55 +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 d2q54CAsuldO; Wed, 3 Jan 2018 18:43:52 +0000 (UTC) Subject: Re: svn commit: r327523 - in head/stand/mips/beri: boot2 loader DKIM-Filter: OpenDKIM Filter v2.9.2 mail.xzibition.com 9556053BD To: John Baldwin , src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org References: <201801031735.w03HZcFL082840@repo.freebsd.org> From: Bryan Drewery Openpgp: id=F9173CB2C3AAEA7A5C8A1F0935D771BB6E4697CF; url=http://www.shatow.net/bryan/bryan2.asc Organization: FreeBSD Message-ID: Date: Wed, 3 Jan 2018 10:43:54 -0800 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.5.2 MIME-Version: 1.0 In-Reply-To: <201801031735.w03HZcFL082840@repo.freebsd.org> Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="9QFbdP79QaMRBzkLnEpFPK0UZxWj9iFRD" X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.25 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: Wed, 03 Jan 2018 18:43:58 -0000 This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --9QFbdP79QaMRBzkLnEpFPK0UZxWj9iFRD Content-Type: multipart/mixed; boundary="kDsDxgNUvCdjCitpP16g67rCdkFoE9hCl"; protected-headers="v1" From: Bryan Drewery To: John Baldwin , src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Message-ID: Subject: Re: svn commit: r327523 - in head/stand/mips/beri: boot2 loader References: <201801031735.w03HZcFL082840@repo.freebsd.org> In-Reply-To: <201801031735.w03HZcFL082840@repo.freebsd.org> --kDsDxgNUvCdjCitpP16g67rCdkFoE9hCl Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: quoted-printable On 1/3/2018 9:35 AM, John Baldwin wrote: > Author: jhb > Date: Wed Jan 3 17:35:38 2018 > New Revision: 327523 > URL: https://svnweb.freebsd.org/changeset/base/327523 >=20 > Log: > Don't clobber system LDFLAGS for beri boot loaders. > =20 > Prior to r325114, bsd.init.mk was included after assignments to CFLAG= S and > LDFLAGS in these Makefiles. After r325114, bare assignments (=3D rat= her than > +=3D) lost system-assigned default values that are needed when compil= ing with > an external toolchain. CFLAGS in both Makefiles already uses +=3D. = This > commit changes LDFLAGS to use +=3D. While here, depend on the LDFLAG= S update > in the parent Makefile.inc to set -nostdlib. > =20 Hmm, I'm not sure it is even right for these files to be including bsd.init.mk. It says at the top: # The include file includes , # ../Makefile.inc and ; this is used at the # top of all files that actually "build something". There's a bunch of other files doing this as well. The biggest problem here is that including bsd.init.mk is like including bsd.prog.mk first and then putting the rest of the Makefile after it. It changes a lot of assumptions in share/mk files. And this particular file still includes bsd.prog.mk, so things really could go wrong. I speak of things like dependency handling and beforebuild handling. I think it's more likely these files just need src.opts.mk. > Sponsored by: DARPA / AFRL >=20 > Modified: > head/stand/mips/beri/boot2/Makefile > head/stand/mips/beri/loader/Makefile >=20 > Modified: head/stand/mips/beri/boot2/Makefile > =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/stand/mips/beri/boot2/Makefile Wed Jan 3 17:28:01 2018 (r3275= 22) > +++ head/stand/mips/beri/boot2/Makefile Wed Jan 3 17:35:38 2018 (r3275= 23) > @@ -53,8 +53,7 @@ CFLAGS+=3D -I${LDRSRC} \ > -fno-pic -mno-abicalls \ > -g > =20 > -LDFLAGS=3D -nostdlib \ > - -static \ > +LDFLAGS+=3D -static \ > -Wl,-N \ > -G0 \ > -L${.CURDIR} >=20 > Modified: head/stand/mips/beri/loader/Makefile > =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/stand/mips/beri/loader/Makefile Wed Jan 3 17:28:01 2018 (r327= 522) > +++ head/stand/mips/beri/loader/Makefile Wed Jan 3 17:35:38 2018 (r327= 523) > @@ -85,8 +85,7 @@ CFLAGS+=3D -G0 \ > -mno-abicalls \ > -g > =20 > -LDFLAGS=3D -nostdlib \ > - -static \ > +LDFLAGS+=3D -static \ > -T ${.CURDIR}/loader.ldscript \ > -L${.CURDIR} \ > -e __start >=20 --=20 Regards, Bryan Drewery --kDsDxgNUvCdjCitpP16g67rCdkFoE9hCl-- --9QFbdP79QaMRBzkLnEpFPK0UZxWj9iFRD 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 iQEcBAEBAgAGBQJaTSRqAAoJEDXXcbtuRpfPvLYH/iARgeKrcFRyUWjnWOvnqT1c ZnpnUxKatCtDVDMC+khhtHhUjvb+kFC07JjMMCaebd33quxF/ohKIQKv8varCVw9 Cy7FEERbiqwBhCeEDokbooRgIW26vbYzCKCt3d1gpdP76CamJqA8Sm6x6K9+qQ9S udZ39AAAAv5IrIbVAUjro7h4y5JK+DuL2CCxv6O2HpLBHs7JTxchPdI24iaEHMy3 L1mZAtq29s4JlgQXFM1qnOhtLyEcTP8KrtSdJUxUdr7edtd1AMBkFNamFf30Hc/2 ka5M+mqErI/V9RwMIYEoRVaizCGJJAVnFHmHpUw1yZ7GN4ynIY19JLpEhNITyXs= =iEAA -----END PGP SIGNATURE----- --9QFbdP79QaMRBzkLnEpFPK0UZxWj9iFRD--