From nobody Tue Oct 05 16:49:20 2021 X-Original-To: dev-commits-ports-main@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 0AE7212D0ABB; Tue, 5 Oct 2021 16:49:26 +0000 (UTC) (envelope-from pkubaj@anongoth.pl) Received: from mail.anongoth.pl (mail.anongoth.pl [46.248.190.61]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X448 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "anongoth.pl", Issuer "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4HP3TK5y1fz3NgS; Tue, 5 Oct 2021 16:49:25 +0000 (UTC) (envelope-from pkubaj@anongoth.pl) Received: from anongoth.pl (unknown [192.168.1.15]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (Client did not present a certificate) (Authenticated sender: pkubaj@anongoth.pl) by mail.anongoth.pl (Postfix) with ESMTPSA id C2164D6655; Tue, 5 Oct 2021 18:49:22 +0200 (CEST) Date: Tue, 5 Oct 2021 18:49:20 +0200 From: Piotr Kubaj To: Mathieu Arnold Cc: ports-committers@freebsd.org, dev-commits-ports-all@freebsd.org, dev-commits-ports-main@freebsd.org Subject: Re: git: fb5f03a87cf4 - main - Mk/bsd.lto.mk: add global LTO support for ports Message-ID: References: <202109301834.18UIYKrL013410@gitrepo.freebsd.org> <20211004133056.z7e2iyrhlwprvvvp@aching.in.mat.cc> <20211005163152.gv2vwvg4nuqwga7q@aching.in.mat.cc> List-Id: Commits to the main branch of the FreeBSD ports repository List-Archive: https://lists.freebsd.org/archives/dev-commits-ports-main List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-ports-main@freebsd.org X-BeenThere: dev-commits-ports-main@freebsd.org MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="nx4PpliHdBPes8uY" Content-Disposition: inline In-Reply-To: <20211005163152.gv2vwvg4nuqwga7q@aching.in.mat.cc> X-Rspamd-Queue-Id: 4HP3TK5y1fz3NgS X-Spamd-Bar: ---- Authentication-Results: mx1.freebsd.org; none X-Spamd-Result: default: False [-4.00 / 15.00]; REPLY(-4.00)[] X-Spam: Yes X-ThisMailContainsUnwantedMimeParts: N --nx4PpliHdBPes8uY Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On 21-10-05 18:31:52, Mathieu Arnold wrote: > On Tue, Oct 05, 2021 at 06:27:15PM +0200, Piotr Kubaj wrote: > > On 21-10-04 15:30:56, Mathieu Arnold wrote: > > > On Thu, Sep 30, 2021 at 06:34:20PM +0000, Piotr Kubaj wrote: > > > > The branch main has been updated by pkubaj: > > > >=20 > > > > URL: https://cgit.FreeBSD.org/ports/commit/?id=3Dfb5f03a87cf432751f= ae1f0ae7f29c9d4fc65917 > > > >=20 > > > > commit fb5f03a87cf432751fae1f0ae7f29c9d4fc65917 > > > > Author: Piotr Kubaj > > > > AuthorDate: 2021-09-30 18:27:50 +0000 > > > > Commit: Piotr Kubaj > > > > CommitDate: 2021-09-30 18:27:50 +0000 > > > >=20 > > > > Mk/bsd.lto.mk: add global LTO support for ports > > > > =20 > > > > It's well known that LTO provides both performance and size ben= efits for > > > > binaries. > > > > =20 > > > > Add preliminary, opt-in support for global LTO enforcement to p= orts. Ports that > > > > provide LTO option on their own and the ones that don't work wi= th LTO will need > > > > to set LTO_UNSAFE in the future. > > > > =20 > > > > PR: 258536 > > >=20 > > > Not to be picky about approval and all, but this was added to the > > > framework, and the framework is maintained by portmgr. When you want= to > > > add something to it, you must consult with portmgr before anything ge= ts > > > committed. > > >=20 > > > In that case, we would have told you not to do it this way, but to ma= ke > > > this a Mk/Uses/lto.mk. > > >=20 > > > So please, turn this into a USES=3Dlto. > >=20 > > I did consult, but no one replied. >=20 > There is absolutely no maintainer timeout for the framework, you cannot > just add code there without explicit approval. And this is a port of a bigger issue, where portmgr ignores emails until nu= merously asked for (if one is lucky). As one of users wrote in https://bugs.freebsd.org/bugzilla/show_bug.cgi?id= =3D251117, for which portmgr is assigned, "portmgr@ more and more feels to = me like a black hole, or /dev/null: Anything sent there seems to disappear = without effect." Since it was a change that doesn't change anything out-of-the-box, I decide= d to commit it. >=20 > > IMO adding it to USES is not a good idea, since USES are supposed to be= used per port and my idea was to force LTO for all ports, same way that SS= P already does. >=20 > All I see in the patch is a USE_LTO knob, and a LTO_UNSAFE one, without > any documentation of what it is for, what it does, what it might do, > what it is about, or anything else. Neither has SSP, I don't see any documentation for it (including commiter h= andbook which has just one line regarding USES=3Dkmod at https://docs.freeb= sd.org/en/books/porters-handbook/book/#uses-kmod). >=20 > --=20 > Mathieu Arnold --nx4PpliHdBPes8uY Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCgAdFiEEycyIeNkkgohzsoorelmbhSCDnJ0FAmFcghAACgkQelmbhSCD nJ0wpQ//fYKMm0WBk1bpSrPhLY0IruNs1oHFpUQrhJj2rzak/ko0oFBVxRTTJc7E Y3kbLYdDlFzR7uH3Aiz8O2TuqtcoSc8A4eOfdVmo37D9QJLcifu7jtg9RzlslhMT AZrAzCwavk78IK8nG/BeqOtAgcC5dzGPEOx79RSKRIA0G2qieyqDIOiVgv6nS8b9 g+t7gSzfQSEeADtIpjneIhdG+/uRLr7JPir4JqVXQekBi3V5bBWcjhRof3DmaqHT EG1ie8nVbZMKIdj8JzgpLtCPFud2AWpW6hAdubCWFlJL6+dkstmGrUM+fyKh5tCn b5o4uyCRYNtgRiys+cSvJFILdiUAVSlnNm4tiiP1aRihkBAsk8GSYM7KkyfPx63b NvUr6UGFqFE7OSbxFPx9RnhzT45Yxye7vB01k0Q29Qav9Y2ga/PvnoWn1AKIIV/2 YsbBFKJ5G4Q9Xkt7QlfUoqVdrtNDcUtHj/DbaDYssULFj6a7hyEDqqLRtamVMewe BJk+Yap+gaxcizb+zHWnBm0nZoqNfi95/NNkhRARi8NhIPgIxUQJNzn2C/KDoQDk yqf2mgUTgQjAK5JKOtxg7XAg0X06/4Kj6zmkeFoXntbdQpSH32ACItp6vof7GPJc cDXXAIpz+6CSs+BQZXyIvQqu7PFuC3DTTOc596TH5HycmXFBR4Q= =7irM -----END PGP SIGNATURE----- --nx4PpliHdBPes8uY--