Re: [PATCH] module: Clarify GPL-Compatible is OK

From: Arend van Spriel
Date: Sun Apr 08 2012 - 08:43:06 EST


On 04/07/2012 03:02 AM, Luis R. Rodriguez wrote:
On Fri, Apr 6, 2012 at 5:51 PM, Luis R. Rodriguez<mcgrof@xxxxxxxxxxxxx> wrote:

4) Use the patch and leave it to the person who wants to extract code
to figure out the exact module license.

After some though I think this is the only approach possible. Consider
ath5k which has files under difference licenses:

* Dual BSD/GPL
* GPL
* ISC

The person taking code must check the header of each file.

Luis

I would say that regardless your patch the "your preferred adjective" license disclaimer in the source files trumps any MODULE_LICENSE value so checking the files is always a necessary step. It is the only place to look for the specific license information and it is on a per-file basis as stated above. MODULE_LICENSE clearly groups these so a generalization seems justified just for that.

So MODULE_LICENSE is really just about allowing EXPORT_SYMBOL_GPL or not. I admit that statement does not cover everything as the MODULE_LICENSE is part of the module information, but as said it may or may not cover a number of different licenses used within the module.

So this specific macro does not, or even should not, help people figure out the license. We could consider some other tagging of each and every source file and add some scripting support to do the task, but that will be one hell of a patch so I will forever deny suggesting it ;-)

Gr. AvS

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/