namespace support requires network modules to say "GPL"
- From: Mark Lord <lkml@xxxxxx>
- Date: Sat, 01 Dec 2007 08:10:17 -0500
Now that we have network namespace support merged it is time to...
revisit the sysfs support so we can remove the dependency on !SYSFS.
Now that the namespace updates are part of 2.6.24,
there is a major inconsistency in network EXPORT_SYMBOLs.
It used to be that an external network module could get away without
having to add a MODULE_LICENSE("GPL*") line to the source.
In support of that, common networking functions (still) use EXPORT_SYMBOL()
rather than the more restrictive EXPORT_SYMBOL_GPL().
Eg. register_netdev(), sk_alloc(), __dev_get_by_name().
But now, none of those three are actually usable by default,
because they all require "init_net", which is EXPORT_SYMBOL_GPL().
So.. It appears that one of three things should really happen next:
1) Change the other exports to also be EXPORT_SYMBOL_GPL.
2) Have register_netdev, sk_alloc, and __dev_get_by_name default
to using init_net when NULL is specified in the namespace field.
or
3) Change init_net to be EXPORT_SYMBOL_GPL.
Right now, things are just a bit inconsistent, and it's not clear
whether the namespace changes intended this consequence or not.
Cheers
(as for me, I think all kernel modules are GPL, whether they have
the MODULE_LICENSE line or not, so flames to /dev/null on that).
--
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/
- Follow-Ups:
- Re: namespace support requires network modules to say "GPL"
- From: Stephen Hemminger
- Re: namespace support requires network modules to say "GPL"
- From: Mark Lord
- Re: namespace support requires network modules to say "GPL"
- References:
- [PATCH 0/10] sysfs network namespace support
- From: Eric W. Biederman
- [PATCH 0/10] sysfs network namespace support
- Prev by Date: Re: [PATCH] xfs: revert to double-buffering readdir
- Next by Date: Re: possible BUG: unable to handle kernel NULL pointer dereference at virtual address 00000000
- Previous by thread: [PATCH 10/10] net: Enable tagging for net_class directories in sysfs
- Next by thread: Re: namespace support requires network modules to say "GPL"
- Index(es):
Relevant Pages
|