From owner-svn-src-all@freebsd.org Mon Feb 25 17:58:20 2019 Return-Path: Delivered-To: svn-src-all@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id C0C191510A84; Mon, 25 Feb 2019 17:58:20 +0000 (UTC) (envelope-from brooks@spindle.one-eyed-alien.net) Received: from spindle.one-eyed-alien.net (spindle.one-eyed-alien.net [199.48.129.229]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 249B87085A; Mon, 25 Feb 2019 17:58:15 +0000 (UTC) (envelope-from brooks@spindle.one-eyed-alien.net) Received: by spindle.one-eyed-alien.net (Postfix, from userid 3001) id 156F63C475F; Mon, 25 Feb 2019 17:58:09 +0000 (UTC) Date: Mon, 25 Feb 2019 17:58:09 +0000 From: Brooks Davis To: Matt Macy Cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r344487 - in head/sys: conf gnu/gcov Message-ID: <20190225175809.GB47081@spindle.one-eyed-alien.net> References: <201902232114.x1NLE0cH085345@repo.freebsd.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="9jxsPFA5p3P2qPhR" Content-Disposition: inline In-Reply-To: <201902232114.x1NLE0cH085345@repo.freebsd.org> User-Agent: Mutt/1.9.4 (2018-02-28) X-Rspamd-Queue-Id: 249B87085A X-Spamd-Bar: --- Authentication-Results: mx1.freebsd.org X-Spamd-Result: default: False [-3.64 / 15.00]; ARC_NA(0.00)[]; NEURAL_HAM_MEDIUM(-1.00)[-0.996,0]; FROM_HAS_DN(0.00)[]; RCPT_COUNT_THREE(0.00)[4]; TO_DN_SOME(0.00)[]; TO_MATCH_ENVRCPT_ALL(0.00)[]; MIME_GOOD(-0.20)[multipart/signed,text/plain]; RCVD_TLS_LAST(0.00)[]; DMARC_NA(0.00)[freebsd.org]; AUTH_NA(1.00)[]; NEURAL_HAM_LONG(-1.00)[-0.999,0]; IP_SCORE(-3.60)[ip: (-9.34), ipnet: 199.48.128.0/22(-4.66), asn: 36236(-3.90), country: US(-0.07)]; MX_GOOD(-0.01)[spindle.one-eyed-alien.net]; MX_MISSING(3.50)[requested record is not found]; NEURAL_HAM_SHORT(-0.64)[-0.641,0]; R_SPF_NA(0.00)[]; SIGNED_PGP(-2.00)[]; FORGED_SENDER(0.30)[brooks@freebsd.org,brooks@spindle.one-eyed-alien.net]; R_DKIM_NA(0.00)[]; MIME_TRACE(0.00)[0:+,1:+]; ASN(0.00)[asn:36236, ipnet:199.48.128.0/22, country:US]; FROM_NEQ_ENVFROM(0.00)[brooks@freebsd.org,brooks@spindle.one-eyed-alien.net]; RCVD_COUNT_TWO(0.00)[2] X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.29 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: Mon, 25 Feb 2019 17:58:21 -0000 --9jxsPFA5p3P2qPhR Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Sat, Feb 23, 2019 at 09:14:00PM +0000, Matt Macy wrote: > Author: mmacy > Date: Sat Feb 23 21:14:00 2019 > New Revision: 344487 > URL: https://svnweb.freebsd.org/changeset/base/344487 >=20 > Log: > gcov support > =20 > add gcov support and export results as files in debugfs We had a brief discussion of this commit within a subset of core. This addition of GPLv2 code is fine as the code is easily removal to a module (per kmoore@) should the day come that we're read to evict all GPL code. The modest increase in activation energy for that task seems worth it for the short-term gains of reduced integration cost (this code will greatly improve our ZFS-on-Linux test coverage.) Rod rightly points out that we haven't accepted SPDX tags alone as license statements. The standard GPL v2.0 boiler plate should be added to this file along side the tag. An additional issue is that the a warning tag was not added to sys/conf/files. A warning along the lines of: warning "kernel contains GPLv2 licensed GCOV" needs to be added. This commit needed more through review. We intend to update our license policy to require core sign off for new GPL code to ensure we're not adding new, tightly integrated dependencies, to document that we're doing so knowingly, and to make sure steps aren't missed. The current document is at: https://www.freebsd.org/internal/software-license.html -- Brooks --9jxsPFA5p3P2qPhR Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEcBAEBAgAGBQJcdCywAAoJEKzQXbSebgfAbL8H/A9EBVqx4ZQWU2cJDmUFQE8A NIA5ft21MhjOrh9UD/vFrYiBb75muK1tnIxlnk6wGVs2fYNCWR5SgJ0ng0559xdi s7UsouFVfS7fOdm111CAwrmUFxt6NoSHnVPvKO0FxLxoz7zyyC9L5rKxEy4vq4ua 6EpMqh+S8UGWCOUrEDFmFHJwVWnv0uFiMjn9L7uytIA4yzvpT1bCDZToGek+FKGN c+oo9HMKlWQUilPMJPkP5dd1dzADmpad1y8zhX6sn9p0cNGHtaUCYc8Mcy+BXQ3q 8G1YB5G26q6oWBW0kEvXm/ZLFsRuKQ7gnN3d5zfjH0LNNCjQTSuGxLn2xKQecEc= =tVeO -----END PGP SIGNATURE----- --9jxsPFA5p3P2qPhR--