public inbox for b.a.t.m.a.n@lists.open-mesh.org
 help / color / mirror / Atom feed
From: Antonio Quartulli <ordex@autistici.org>
To: 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.] [PATCHv2] batman-adv: forward late OGMs from best next hop
Date: Sun, 26 May 2013 22:23:30 +0200	[thread overview]
Message-ID: <20130526202330.GP1679@ritirata.org> (raw)
In-Reply-To: <20130526151443.GA27074@pandem0nium>

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

Hey guys,

On Sun, May 26, 2013 at 05:14:43PM +0200, Simon Wunderlich wrote:
[cut...]

> 
> Regarding your suggestion to forward multiple packets with the same seqno, but
> better TQ: We currently would ignore them in A for beeing a duplicate. If we were
> to change that, we would have to change the TQ ring buffer as well. However this ring
> buffer currently has no specific order, so it's not so easy to update a specific value
> in this ring buffer because we don't know which TQ value belongs to which seqno.
> If packets arrive out of order, they will be saved in the ring buffer in the
> order they come in, regardless of the seqno. Have a look at batadv_iv_ogm_orig_update(),
> maybe you can find a decent solution for that. :)
> 

Honestly, I think that we should avoid adding features to batman iv any longer
(this is my point of view) and rather concentrate the effort in pushing them
into batman v (which is going to come soonish[tm]).

Therefore, if you think there is something which we can do better, I'd prefer to
push such improvement into the new version of the protocol, which is going to be
"simpler" (to some extends) and since it is still under development it would be
easier to change/improve/modify.

batman iv already has many "patches and corner cases" which I think is better to
do not change too much to avoid introducing some new nice bugs :)


these were just my 2 cents


Cheers!


-- 
Antonio Quartulli

..each of us alone is worth nothing..
Ernesto "Che" Guevara

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

  reply	other threads:[~2013-05-26 20:23 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-05-23 11:07 [B.A.T.M.A.N.] [PATCHv2] batman-adv: forward late OGMs from best next hop Simon Wunderlich
2013-05-23 11:17 ` Antonio Quartulli
2013-05-25 21:07 ` Linus Lüssing
2013-05-25 21:42   ` Antonio Quartulli
2013-05-26 15:14     ` Simon Wunderlich
2013-05-26 20:23       ` Antonio Quartulli [this message]
2013-06-09  4:30 ` Marek Lindner

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=20130526202330.GP1679@ritirata.org \
    --to=ordex@autistici.org \
    --cc=b.a.t.m.a.n@lists.open-mesh.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox