linux-spdx.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Greg KH <gregkh@linuxfoundation.org>
To: Dave Hansen <dave.hansen@intel.com>
Cc: linux-kernel@vger.kernel.org, dan.j.williams@intel.com,
	h.peter.anvin@intel.com, tglx@linutronix.de, corbet@lwn.net,
	linux-spdx@vger.kernel.org, linux-doc@vger.kernel.org
Subject: Re: [PATCH] Documentation: clarify driver licensing rules
Date: Wed, 12 Aug 2020 16:58:54 +0200	[thread overview]
Message-ID: <20200812145854.GA2616348@kroah.com> (raw)
In-Reply-To: <dd23860c-7e45-2d43-0405-c8037f4a7d8f@intel.com>

On Wed, Aug 12, 2020 at 07:45:00AM -0700, Dave Hansen wrote:
> On 8/12/20 1:23 AM, Greg KH wrote:
> > On Tue, Aug 11, 2020 at 10:17:48AM -0700, Dave Hansen wrote:
> >> But, this left submitters (and the folks who help them pick licenses)
> >> a bit confused. They have read things like
> >> Documentation/process/license-rules.rst which says:
> >>
> >> 	individual source files can have a different license
> >> 	which is required to be compatible with the GPL-2.0
> >>
> >> and Documentation/process/submitting-drivers.rst:
> >>
> >> 	We don't insist on any kind of exclusive GPL licensing,
> >> 	and if you wish ... you may well wish to release under
> >> 	multiple licenses.
> > 
> > Both of these are fine, but maybe you need to put:
> > 	"don't try to do stupid things just because you can!"
> > somewhere in here instead?
> 
> Folks never think what _they_ are doing is stupid, so I fear that would
> fall on deaf ears.

True, but one can dream...

> ...
> >>  Licensing:
> >> -		The code must be released to us under the
> >> -		GNU General Public License. We don't insist on any kind
> >> -		of exclusive GPL licensing, and if you wish the driver
> >> -		to be useful to other communities such as BSD you may well
> >> -		wish to release under multiple licenses.
> >> +		The code must be released to us under the GNU General Public
> >> +		License. While there are no kernel-wide rules, some maintainers
> >> +		may insist on exclusive GPL licensing by default.
> > 
> > Maintainers should not do that, it is not their place to do so.  They
> > _can_ push back on, again, stupid things, but in the end, they should
> > accept anything that is a compatible license with the kernel as it is
> > really up to the copyright owner as to what license they wish to use.
> 
> I wonder if we're not quite on the same page.  I thought the pitfall
> recently was from overly-aggressive dual-licensing on code that wasn't
> likely to be able to be used in another project.  Was that the misstep?
>  Or was it that the code shouldn't have been dual-licensed in the first
> place because it was too intertwined with GPLv2 code to be anything but
> purely GPLv2?

Both of those have been the cases recently, it's not just one recent
submission that has had this problem :(

> > So while I like the intent here, I don't think this wording change is
> > good as-is.
> > 
> > As it stands, the text makes sense, but as always, if you have legal
> > questions, you should be talking to a lawyer, not a kernel developer :)
> 
> I'd like to do two things with this Documentation/.  First, to _get_
> folks to go talk to their lawyers.  Second, the lawyers and the
> non-lawyers who do licensing _will_ read this documentation.  If they
> understand what the kernel expects, they are in the best position to
> pass that understanding on to developers since they're the gatekeepers.
>   That will hopefully make your job easier because it will filter out
> some of the stupid things before you see them.

I agree with your goal here, I just don't agree that we should be saying
"some maintainers may insist on exclusive GPL licensing" as no
maintainer should be insisting on this.  They should be pushing back on
the "that is just dumb as it doesn't even do what you think it does!"
patches, like I have been seeing recently.

I do not want to codify the behavior I have seen at times of some
maintainers who refuse to accept anything but GPL-only licensed code.
So try rewording your patch a bit as I think we are in violent agreement
here :)

thanks,

greg k-h

      reply	other threads:[~2020-08-12 14:58 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-08-11 17:17 [PATCH] Documentation: clarify driver licensing rules Dave Hansen
2020-08-12  8:23 ` Greg KH
2020-08-12 14:45   ` Dave Hansen
2020-08-12 14:58     ` Greg KH [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20200812145854.GA2616348@kroah.com \
    --to=gregkh@linuxfoundation.org \
    --cc=corbet@lwn.net \
    --cc=dan.j.williams@intel.com \
    --cc=dave.hansen@intel.com \
    --cc=h.peter.anvin@intel.com \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-spdx@vger.kernel.org \
    --cc=tglx@linutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).