RE: [RFC v2 0/4] Adding stateful compression to IPHC

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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.

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?

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.

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.

Best regards,
Łukasz Duda
��.n��������+%������w��{.n�����{����^n�r������&��z�ޗ�zf���h���~����������_��+v���)ߣ�

[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux