All of lore.kernel.org
 help / color / mirror / Atom feed
From: Alexander Aring <alex.aring@gmail.com>
To: Varka Bhadram <varkabhadram@gmail.com>
Cc: linux-wpan - ML <linux-wpan@vger.kernel.org>
Subject: Re: Regarding dev->addr_len
Date: Fri, 8 Aug 2014 12:47:15 +0200	[thread overview]
Message-ID: <20140808104712.GA16641@omega> (raw)
In-Reply-To: <53E49E62.3010908@gmail.com>

Hi,

On Fri, Aug 08, 2014 at 03:24:42PM +0530, Varka Bhadram wrote:
> 
> Entire Network layer may be working with 48-bit or 64-bit MAC address only.
> 

That's not correct. The network layer can handle xx bit mac addresses
depends on addr_len. Our mac layer have something special which isn't in
any other mac layers. We have two kinds of mac addresses which can be
used mixed in destination and source address in the mac header.

The problem is 6LoWPAN is an adaption layer and the whole
net/ipv6/ndisc.c implementation can handle only one kind of mac address
with one addr_len.

According we have two kinds with different length we need also other
ICMPv6 nd messages. The neighbor discovery cache need to hold a
information if it's a short or extended and send the right nd messages.

> May be we need to change this behavior to work with 16-bit also ...?
> 

Okay, create patches to change the core netdev api to handle two kinds
of dev_addr's. I am prerry sure this will not accept at mainline. We
need another idea and if you read my other mail on netdev you will get a
imagine about that what I try to do. Sorry, this is hard but we can't
change the whole network API only for 802.15.4.

One reply talked about a private data isnide a neighbor entry which is
set by L2, that's another option and I like it.

> Also for a interface we are going to have one netdevice structure ...?
> 

I don't know what you mean here. But another idea of mine was to have a
second interface for short addresses only, but this can't work because
short and extended addresses can be used at the same time. If you mean
that.

> If yes, dynamically updating dev_addr and addr_len creates problems..?
> 
Look at net/ipv6/ndisc.c in about one minute you will see that this will
create problems of course.

- Alex

      reply	other threads:[~2014-08-08 10:47 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-08-08  8:40 Regarding dev->addr_len Varka Bhadram
2014-08-08  9:33 ` Alexander Aring
2014-08-08  9:54   ` Varka Bhadram
2014-08-08 10:47     ` Alexander Aring [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20140808104712.GA16641@omega \
    --to=alex.aring@gmail.com \
    --cc=linux-wpan@vger.kernel.org \
    --cc=varkabhadram@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.