From owner-svn-src-all@freebsd.org Tue Feb 26 02:18:48 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 1E11B15004A4; Tue, 26 Feb 2019 02:18:48 +0000 (UTC) (envelope-from freebsd@pdx.rh.CN85.dnsmgr.net) Received: from pdx.rh.CN85.dnsmgr.net (br1.CN84in.dnsmgr.net [69.59.192.140]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 9D0EA900B4; Tue, 26 Feb 2019 02:18:47 +0000 (UTC) (envelope-from freebsd@pdx.rh.CN85.dnsmgr.net) Received: from pdx.rh.CN85.dnsmgr.net (localhost [127.0.0.1]) by pdx.rh.CN85.dnsmgr.net (8.13.3/8.13.3) with ESMTP id x1Q2IgkV042693; Mon, 25 Feb 2019 18:18:42 -0800 (PST) (envelope-from freebsd@pdx.rh.CN85.dnsmgr.net) Received: (from freebsd@localhost) by pdx.rh.CN85.dnsmgr.net (8.13.3/8.13.3/Submit) id x1Q2Ig4r042692; Mon, 25 Feb 2019 18:18:42 -0800 (PST) (envelope-from freebsd) From: "Rodney W. Grimes" Message-Id: <201902260218.x1Q2Ig4r042692@pdx.rh.CN85.dnsmgr.net> Subject: Re: svn commit: r344487 - in head/sys: conf gnu/gcov In-Reply-To: To: "K. Macy" Date: Mon, 25 Feb 2019 18:18:42 -0800 (PST) CC: Brooks Davis , Matt Macy , svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers Reply-To: rgrimes@freebsd.org X-Mailer: ELM [version 2.4ME+ PL121h (25)] MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset=US-ASCII X-Rspamd-Queue-Id: 9D0EA900B4 X-Spamd-Bar: ------ Authentication-Results: mx1.freebsd.org X-Spamd-Result: default: False [-6.93 / 15.00]; NEURAL_HAM_MEDIUM(-1.00)[-0.999,0]; NEURAL_HAM_SHORT(-0.93)[-0.928,0]; REPLY(-4.00)[]; NEURAL_HAM_LONG(-1.00)[-1.000,0] 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: Tue, 26 Feb 2019 02:18:48 -0000 > > 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. > > I've copied the full copyright attribution that is in the > corresponding files on Linux. Is there some reason why FreeBSD > requires the files to be inflated with the full license text where the > original lacks it? I think for a few reasons, I doubt you copied the whole distribution that this file came from, as I am sure that distribution included a LICENSE file. Second if you actually read the GPL v2 documentation and follow what it says it says you must do this, just because some one else does not follow the rules of what the GPL v2 says does not give us to knowingling not do it. Third this is a particular dangerious area for BSD to be mixing a GPL code with its kernel, to my knowlege we have never had any gpl code in the kernel, no have we ever allowed it, but thats a seperate argument, that should be made. > > 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. > > Yup. Thanks > > > > This commit needed more through review. > > How would this be achieved:? I had several people on the review and no > one had substantive feedback. I have very seriuos concerns how you can even make that comment now given that ngie@ pointed out during the review that this GPL code and you dismissed it as a non issue. This shows lack of knowlege as to the projects GPL goals, and lack of concern that you might of wanted to ask first, rather than now have to deal with it post commit. When ngie@ pointed out an issue you could of posted to a list with your review asking for more people. Everyone should strive to find reviewers, if the patch doesn't trigger enough herard rules (which we should also work on, but seems the phabricator admins are deaf or non existance as multiple requests go unanswered), it is very easy to go drop a link to your review in an appropriate mailling list -current being choice of last resort. If you do that I am pretty sure you'll get plenty of feedback. > > > > > 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 Given the high push in the last few years to be GPL free, and that being well publically aired at conferences and in commit mail as we try to achive that goal I can not see why here do we need to add policy for what really should be common knowledge, yet in areas that we clearly do not follow or have policy there seems to be no reason what so ever to fix what we dont follow or add what we do want. IMHO, this commit should be a huge red flag, we seem to have a problem Houston, in communications! > -M -- Rod Grimes rgrimes@freebsd.org