Message ID | 1398193438-23805-1-git-send-email-mcgrof@do-not-panic.com |
---|---|
State | Accepted, archived |
Delegated to: | David Miller |
Headers | show |
From: "Luis R. Rodriguez" <mcgrof@do-not-panic.com> Date: Tue, 22 Apr 2014 12:03:58 -0700 > From: "Luis R. Rodriguez" <mcgrof@suse.com> > > Johannes noted this is not needed, all of the fragment > accessors don't need CONFIG_NET_NS. This goes test compiled with > CONFIG_BT_6LOWPAN=y and a disabled CONFIG_NET_NS. > > CC: Alexander Smirnov <alex.bluesman.smirnov@gmail.com> > Cc: Dmitry Eremin-Solenikov <dbaryshkov@gmail.com> > Cc: linux-zigbee-devel@lists.sourceforge.net > Cc: David S. Miller" <davem@davemloft.net> > Cc: netdev@vger.kernel.org > Cc: linux-kernel@vger.kernel.org > Cc: Johannes Berg <johannes@sipsolutions.net> > Signed-off-by: Luis R. Rodriguez <mcgrof@suse.com> Applied, thanks Luis. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi, On Tue, Apr 22, 2014 at 12:03:58PM -0700, Luis R. Rodriguez wrote: > From: "Luis R. Rodriguez" <mcgrof@suse.com> > > Johannes noted this is not needed, all of the fragment > accessors don't need CONFIG_NET_NS. This goes test compiled with > CONFIG_BT_6LOWPAN=y and a disabled CONFIG_NET_NS. > a little note about this here. There exists two 6LoWPAN standard. One for bluetooth low energy and one for IEEE 802.15.4. The actual namespace is only for IEEE 802.15.4, because we need fragmentation there. 6LoWPAN fragmentation for bluetooth low energy is already handled by the MAC-Layer. So this has nothing to do with CONFIG_BT_6LOWPAN. - Alex -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Thu, Apr 24, 2014 at 9:44 AM, Alexander Aring <alex.aring@gmail.com> wrote: > Hi, > > On Tue, Apr 22, 2014 at 12:03:58PM -0700, Luis R. Rodriguez wrote: >> From: "Luis R. Rodriguez" <mcgrof@suse.com> >> >> Johannes noted this is not needed, all of the fragment >> accessors don't need CONFIG_NET_NS. This goes test compiled with >> CONFIG_BT_6LOWPAN=y and a disabled CONFIG_NET_NS. >> > > a little note about this here. There exists two 6LoWPAN standard. One > for bluetooth low energy and one for IEEE 802.15.4. > > The actual namespace is only for IEEE 802.15.4, because we need > fragmentation there. 6LoWPAN fragmentation for bluetooth low energy is > already handled by the MAC-Layer. So this has nothing to do with > CONFIG_BT_6LOWPAN. Thanks for the clarification, I actually did mean CONFIG_IEEE802154_6LOWPAN however, I goofed that on the commit log. Luis -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi Luis, On Thu, Apr 24, 2014 at 10:25:58AM -0700, Luis R. Rodriguez wrote: > On Thu, Apr 24, 2014 at 9:44 AM, Alexander Aring <alex.aring@gmail.com> wrote: > > Hi, > > > > On Tue, Apr 22, 2014 at 12:03:58PM -0700, Luis R. Rodriguez wrote: > >> From: "Luis R. Rodriguez" <mcgrof@suse.com> > >> > >> Johannes noted this is not needed, all of the fragment > >> accessors don't need CONFIG_NET_NS. This goes test compiled with > >> CONFIG_BT_6LOWPAN=y and a disabled CONFIG_NET_NS. > >> > > > > a little note about this here. There exists two 6LoWPAN standard. One > > for bluetooth low energy and one for IEEE 802.15.4. > > > > The actual namespace is only for IEEE 802.15.4, because we need > > fragmentation there. 6LoWPAN fragmentation for bluetooth low energy is > > already handled by the MAC-Layer. So this has nothing to do with > > CONFIG_BT_6LOWPAN. > > Thanks for the clarification, I actually did mean > CONFIG_IEEE802154_6LOWPAN however, I goofed that on the commit log. > ok. But I need to say thanks for sending this patch! :-) It's nice to see that the community helps to improving the code which I produced and it's really not perfect at the moment. (I was a little bit shocked that somebody makes the effort to making a backport about that). - Alex -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Thu, Apr 24, 2014 at 10:33 AM, Alexander Aring <alex.aring@gmail.com> wrote: > Hi Luis, > > On Thu, Apr 24, 2014 at 10:25:58AM -0700, Luis R. Rodriguez wrote: >> On Thu, Apr 24, 2014 at 9:44 AM, Alexander Aring <alex.aring@gmail.com> wrote: >> > Hi, >> > >> > On Tue, Apr 22, 2014 at 12:03:58PM -0700, Luis R. Rodriguez wrote: >> >> From: "Luis R. Rodriguez" <mcgrof@suse.com> >> >> >> >> Johannes noted this is not needed, all of the fragment >> >> accessors don't need CONFIG_NET_NS. This goes test compiled with >> >> CONFIG_BT_6LOWPAN=y and a disabled CONFIG_NET_NS. >> >> >> > >> > a little note about this here. There exists two 6LoWPAN standard. One >> > for bluetooth low energy and one for IEEE 802.15.4. >> > >> > The actual namespace is only for IEEE 802.15.4, because we need >> > fragmentation there. 6LoWPAN fragmentation for bluetooth low energy is >> > already handled by the MAC-Layer. So this has nothing to do with >> > CONFIG_BT_6LOWPAN. >> >> Thanks for the clarification, I actually did mean >> CONFIG_IEEE802154_6LOWPAN however, I goofed that on the commit log. >> > > ok. But I need to say thanks for sending this patch! :-) > > It's nice to see that the community helps to improving the code which I > produced and it's really not perfect at the moment. (I was a little bit > shocked that somebody makes the effort to making a backport about that). To be clear -- we strive for automatic backport for the entire Linux kernel, so the way this should be seen is that if something gets added to Linux it will eventually get backported automatically. We're not there yet to scale rapid integration of most used drivers but that is a lofty objective. In this case what triggered the backport was that Hauke orginally had added backport support for CONFIG_IEEE802154, when 6 Lowpan was merged upstream we took that in as well, that required a few changes to help with the automatic backport, and I'm glad these have gone in now. I should also note that I haven't personally tested the 6lowpan backport but reports from users on the backport with IEEE802154 or 6lowpan would be greatly appreciated, typically for backports the way it works is if things compile it should work as the backports code only consists of about 1% of the code used and bugs have been infrequent on that codebase. Since 6lowpan is ever changing, as noted in earlier threads, folks can use the latest linux-next based backports release, this is listed on the temporary backports release page [0]. [0] http://drvbp1.linux-foundation.org/~mcgrof/rel-html/backports/ Luis -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/include/net/net_namespace.h b/include/net/net_namespace.h index 066cf60..ea3cede 100644 --- a/include/net/net_namespace.h +++ b/include/net/net_namespace.h @@ -386,15 +386,8 @@ net_ieee802154_lowpan(struct net *net) { return &net->ieee802154_lowpan; } -#else -static inline struct netns_ieee802154_lowpan * -net_ieee802154_lowpan(struct net *net) -{ - return NULL; -} #endif - /* For callers who don't really care about whether it's IPv4 or IPv6 */ static inline void rt_genid_bump_all(struct net *net) {