Return-Path: Message-ID: <1437031525.2896.69.camel@linux.intel.com> Subject: Re: [RFC v2 0/4] Adding stateful compression to IPHC From: Jukka Rissanen To: "Duda, Lukasz" Cc: "alex.aring@gmail.com" , "linux-wpan@vger.kernel.org" , "linux-bluetooth@vger.kernel.org" Date: Thu, 16 Jul 2015 10:25:25 +0300 In-Reply-To: References: <1436788233-13960-1-git-send-email-lukasz.duda@nordicsemi.no> <1436792982.2896.32.camel@linux.intel.com> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Sender: linux-wpan-owner@vger.kernel.org List-ID: Hi Lukasz, On ke, 2015-07-15 at 13:42 +0000, Duda, Lukasz wrote: > Hi Jukka, > > > Hi Lukasz, > > > > On ma, 2015-07-13 at 13:50 +0200, Lukasz Duda wrote: > > > Hi all, > > > > > > Resending the RFC patch set after some clean up. > > > > > > The following patches introduce support for stateful compression in IPHC. > > > > > > Patch #1: Introduce debugfs entry for 6lowpan module > > > Patch #2: Add stateful compression component of 6lowpan module > > > Patch #3: Add stateful compression support for iphc.c > > > Patch #4: Enable stateful compression in bluetooth_6lowpan > > > > > > Usage of stateful compression is described in Patch #2. > > > > > > Notes > > > ===== > > > > > > RFC v2: > > > - Splitting patch into smaller incremental patches > > > - Moving debugfs logic from iphc.c into context.c > > > - Updating patch set to align with style and coding guideline > > > > > > RFC v1: > > > - Initial patch set: http://www.spinics.net/lists/linux- > > bluetooth/msg62930.html > > > > > > Limitations > > > =========== > > > > > > - Temporarly use of debugfs to be able to add context table entries. > > > - Current module does not support context time-outs. > > > - No support for multicast addresses stateful compression. > > > > > > Discussion: Idea on how to make the 6lowpan context tables out of debug > > mode > > > > > ============================================================ > > ================ > > > > > > The patch provided here is just a temporary solution where the contexts are > > > manually added. The proper way of adding/removing contexts would be to > > make > > > ndisc.c parse 6CO options and act upon it. Generated Router Advertisement > > > packets with 6CO option (for example from RADVD) will be handled in the > > > ndisc_router_discovery function. > > > > > > Also, other flags like ARO, ABRO etc. which are specified in RFC6775 need > > > to be handled in the ipv6 module in order to do it in a neat way. > > > Potentially the RFC6775 extensions could also be used by other protocols > > > than BLE and 802.15.4. > > > > > > There is a need for parsing the 6CO option in ndisc.c which parses Router > > > Advertisement messages. Today the function calls to IPHC (6lowpan module) > > are > > > quite hard to implement in ndisc.c, as there is no guarantee that the > > 6lowpan > > > module will be loaded or not. It would make sense to build 6lowpan module > > > as in-build part of IPv6. The features could be compiled in by using #defines. > > > > > > What do you think about moving 6lowpan as a component of IPv6 and > > modify ndisc.c > > > to handled 6LoWPAN specific options? Do you see any other solution to > > make sure > > > that 6CO is parsed and acted upon and still keep 6lowpan as a stand-alone > > module? > > > > Have you considered that 6lowpan module could register a callback in > > ipv6 module. When needed, ipv6 could then call the 6lowpan callback if > > needed. This way there would be no need to embed 6lowpan functionality > > to ipv6 module. > > > > > > Cheers, > > Jukka > > > > I did consider that solution but I assume you think about the way of > registration used for NHC callbacks (for example nhc_udp.c) in the 6lowpan > module. I was thinking more something like the notifier API. Perhaps it can be utilized to send information about received ndisc events. So in 6lowpan module you would just do static struct notifier_block ndisc_notifier = { .notifier_call = ndisc_event, }; and register the notifier handler, and then in ndisc.c:ndisc_rcv() could then just send information about RAs etc. > > This could imply a new API for ndisc.c for example > ndisc_register_lowpan_callback(). This callback can be issued from the > 6lowpan module in order to register a callback. > When an RA message comes, the registered callback will be issued. > Should this be called only when a specific option is detected, or for all RA > messages in general? Perhaps more general solution would be preferable instead of being specific to 6lowpan needs. > > How do you see a way of handling all RFC6775 options like ARO,DAR,DAC etc. > Should this be extended in ndisc.c itself, or should it be handled within > the 6lowpan module (e.g. by using this registered lowpan callback). > Because when receiving NS with ARO it is expected to answer with NA that > contains ARO option with reply status. If the options are specific to 6lowpan, then I would say the 6lowpan module needs to deal with them. > > I believe that using callbacks like this could be used for parsing 6CO > very well which does not need any interaction. However, would there be > any chance of getting such dynamic registration of callbacks into ipv6 > module? Is there any example that implements such solution to see how > it works. I do not know and we are just pondering different ideas here. Could you ask the gurus in netdev mailing list their opinion/guidance? Cheers, Jukka