Hi Werner,

On Thu, Oct 24, 2013 at 11:16:19AM -0300, Werner Almesberger wrote:
> Alexander Aring wrote:
> > Subject: [Linux-zigbee-devel] [PATCH net-next 11/12] 6lowpan: set 6lowpan
> >  network and transport header
> 
> Acked-by: Werner Almesberger <wer...@almesberger.net>
> 
> (not "reviewed" since I didn't try to hunt down the uses.)
> 
> It seems to me that 08/12, 10/12, 11/12, and 12/12 would make a
> nice patch set on their own. They're no longer "trivial" in the
> sense that one can see with just one look that they're correct,
> and they together clean up and extend the use of skb_*_header
>
okay I remove these patches from this patchstack, the most benefit is
that we don't have the static calculation from the mac header. The
mac_header size depends on the flow control which isn't evaluate.

But with this patch we set the mac header size and evaluate the fc for
that. For example, the current size matches only for intra pan
communication and I remember boardcasting/muticasting is a problem too,
because this size calculation is only for EUI64 addresses so parsing of
header will failed. (A broadcast address is short-address 0xffff)

Another point is that we calculate every offset with the mac_len but we
can get the pointers for network_header, mac_header if we set this once.

- Alex

------------------------------------------------------------------------------
October Webinars: Code for Performance
Free Intel webinars can help you accelerate application performance.
Explore tips for MPI, OpenMP, advanced profiling, and more. Get the most from 
the latest Intel processors and coprocessors. See abstracts and register >
http://pubads.g.doubleclick.net/gampad/clk?id=60135991&iu=/4140/ostg.clktrk
_______________________________________________
Linux-zigbee-devel mailing list
Linux-zigbee-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-zigbee-devel

Reply via email to