All of lore.kernel.org
 help / color / mirror / Atom feed
From: Antonio Quartulli <antonio@meshcoding.com>
To: Sven Eckelmann <sven@narfation.org>
Cc: The list for a Better Approach To Mobile Ad-hoc Networking
	<b.a.t.m.a.n@lists.open-mesh.org>
Subject: Re: [B.A.T.M.A.N.] [PATCH] batman-adv: Use safer default config for optional features
Date: Sat, 07 Feb 2015 09:58:49 +0100	[thread overview]
Message-ID: <54D5D3C9.1090502@meshcoding.com> (raw)
In-Reply-To: <54743686.1YRlhzGWHK@sven-edge>

[-- Attachment #1: Type: text/plain, Size: 2274 bytes --]



On 07/02/15 08:27, Sven Eckelmann wrote:
> On Friday 06 February 2015 14:20:28 Antonio Quartulli wrote:
>> To me this looks like the caching effect of DAT: bat0 is likely to
>> change MAC address everytime you recreate the interface (unless you
>> statically assign the MAC) therefore I presume that the other node (the
>> one which did not reboot) was still caching the old bat0 MAC address
>> (each entry requires some minutes before being invalidated/refreshed).
> 
> Thanks for the explanation.
> 
> So you are basically saying that DAT cannot detect when some nodes found a 
> conflict in their ARP table (when receiving IP packets with a different MAC 
> for an IP or similar things)? 

Right.
Well, the scenario "incoming packets altering the local ARP table and
generating a conflict" was never really considered.

> And there is also the problem when no local 
> information is stored and only a conflict with some data data on another 
> unrelated node is happened.

I haven't really understood this issue.

> 
> The first conflict (the conflict in the local ARP table) is not detected 
> because David wanted that batman-adv isn't accessing the ARP table?

Even without reading the the ARP table we could "detect the conflict" by
checking incoming packets and by matching their IP/MAC couple with what
we have in DAT...At the moment we only assume that the IP/MAC couple is
stable enough and that in the worst case the user will wait for the
cache to get invalidated.


Still, I think this is an optimization that we may want to implement,
but not a real issue that pushes us to disable DAT by default.

> 
>> This means that any communication willing to contact the rebooted node
>> was targetting the old address and therefore the two were not be able
>> talk until the cache was refreshed.
>>
>> Can this be the case?
> 
> Yes, unfortunately I hadn't the time to analyze it further and have no logs of 
> any similar problem. Thats why I cannot check if this is contradicted by 
> anything I've done (or not done). So it is a very plausible scenario which 
> you've described.

No problem, I just wanted to get your feeling/feedback about that :)


Thanks a lot.

Cheers,

-- 
Antonio Quartulli


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

  reply	other threads:[~2015-02-07  8:58 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-02-05 16:22 [B.A.T.M.A.N.] [PATCH] batman-adv: Use safer default config for optional features Sven Eckelmann
2015-02-06  7:26 ` Antonio Quartulli
2015-02-06 12:38   ` Sven Eckelmann
2015-02-06 13:20     ` Antonio Quartulli
2015-02-07  7:27       ` Sven Eckelmann
2015-02-07  8:58         ` Antonio Quartulli [this message]
2015-02-18  7:33 ` Marek Lindner
2015-02-18 17:18   ` Martin Hundebøll

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=54D5D3C9.1090502@meshcoding.com \
    --to=antonio@meshcoding.com \
    --cc=b.a.t.m.a.n@lists.open-mesh.org \
    --cc=sven@narfation.org \
    /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.