Hi Alex, On ti, 2014-09-30 at 12:03 +0200, Alexander Aring wrote: > On Tue, Sep 30, 2014 at 03:23:09PM +0530, Varka Bhadram wrote: > > On 09/30/2014 03:13 PM, Alexander Aring wrote: > > > > > >>May be IEEE-802.11 packets send to IPv6 Layer. In that case Its not dead code. > > >> > > > > > >802.11 data frames will be converted to ethernet frames. > > > > > >Why we now talking about 802.11? 6LoWPAN set always PACKET_HOST and then > > >we never have PACKET_BROADCAST set in IPv6 Layer. Then matching pkt_type > > >with PACKET_BROADCAST is dead code. > > > > We don't know in which scenario the IPv6 Layer checking that statement. > > > > The control is reaching there when they got ICMPv6 message of typeICMPV6_MGM_REPORT <http://lxr.free-electrons.com/ident?i=ICMPV6_MGM_REPORT> [1] . > > > > We don't know the significance of it..?? > > > > Yep, we don't know it and it doesn't matter, but override the pkt_type > to PACKET_HOST when we have a broadcast frame is wrong. > > Broadcast means it's IPv6 BROADCAST or MULTICAST, the detection if > LOGICAL MULTICAST over BROADCAST xor BROADCAST over BROADCAST is IPv6 Layer. I probably missed something from earlier mails but why are we talking about broadcast with IPv6? After all IPv6 supports only multicast and broadcasting is not supported. Also what is this "logical multicast over broadcast"? > > These ICMPv6 messages check on this and it's currently broken. > > - Alex Cheers, Jukka -- To unsubscribe from this list: send the line "unsubscribe linux-wpan" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
