From: Marek Lindner <mareklindner@neomailbox.ch>
To: b.a.t.m.a.n@lists.open-mesh.org
Cc: Sven Eckelmann <sven@narfation.org>
Subject: Re: [B.A.T.M.A.N.] [PATCH 2/4] batctl: Reduce IPv4 focus of mac resolve functions
Date: Tue, 24 Sep 2013 20:09:13 +0800 [thread overview]
Message-ID: <1750733.3MepXfCQX9@diderot> (raw)
In-Reply-To: <1379860946-32669-2-git-send-email-sven@narfation.org>
On Sunday 22 September 2013 16:42:24 Sven Eckelmann wrote:
> The first implementation of the IP translation functions in
> cb75e019e74fc11f9547dadd51abbf3c9df2c5e7 ("batctl: Allow to use IPv4
> addresses for ping/traceroute") had is focus on implementing it only for
> IPv4. This is a limitation when other layer 3 protocols are used.
>
> For example, a port to IPv6 would either have to copy the functionality or
> do a major refactoring when implementing the it resolv functionality. This
> is not necessary when the functions already expect different layer 3
> protocols in the first place and make it easy to find the right place to
> modify them.
>
> Signed-off-by: Sven Eckelmann <sven@narfation.org>
> ---
> functions.c | 86
> ++++++++++++++++++++++++++++++++++++++++++++----------------- 1 file
> changed, 63 insertions(+), 23 deletions(-)
Applied in revision daf6aa6.
Thanks,
Marek
next prev parent reply other threads:[~2013-09-24 12:09 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-09-22 14:42 [B.A.T.M.A.N.] [PATCH 1/4] batctl: Add missing includes and remove unused includes Sven Eckelmann
2013-09-22 14:42 ` [B.A.T.M.A.N.] [PATCH 2/4] batctl: Reduce IPv4 focus of mac resolve functions Sven Eckelmann
2013-09-24 12:09 ` Marek Lindner [this message]
2013-09-22 14:42 ` [B.A.T.M.A.N.] [PATCH 3/4] batctl: Replace /proc/net/arp based resolver with rtnl Sven Eckelmann
2013-09-22 17:09 ` [B.A.T.M.A.N.] [PATCHv2 " Sven Eckelmann
2013-09-24 12:11 ` Marek Lindner
2013-09-22 14:43 ` [B.A.T.M.A.N.] [PATCH 4/4] batctl: Add support for IPv6 to address resolver Sven Eckelmann
2013-09-24 12:15 ` Marek Lindner
2013-09-24 15:20 ` Antonio Quartulli
2013-09-24 18:28 ` Sven Eckelmann
2013-09-23 7:03 ` [B.A.T.M.A.N.] [PATCH 1/4] batctl: Add missing includes and remove unused includes Antonio Quartulli
2013-09-23 8:18 ` Sven Eckelmann
2013-09-23 8:20 ` Antonio Quartulli
2013-09-24 12:06 ` 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=1750733.3MepXfCQX9@diderot \
--to=mareklindner@neomailbox.ch \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox