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 > > Log: > gcov support > > 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
signature.asc
Description: PGP signature