From: Andy Gospodarek <andy@greyhouse.net>
To: Anton Nayshtut <Anton.Nayshtut@wilocity.com>,
Jay Vosburgh <fubar@us.ibm.com>,
Veaceslav Falico <vfalico@redhat.com>,
"David S. Miller" <davem@davemloft.net>,
Cong Wang <xiyou.wangcong@gmail.com>,
Nicolas Schichan <nschichan@freebox.fr>,
Eric Dumazet <edumazet@google.com>
Cc: linux-kernel@vger.kernel.org, netdev@vger.kernel.org
Subject: Re: [PATCH 0/4] bonding: L2DA mode
Date: Thu, 21 Nov 2013 10:09:14 -0500 [thread overview]
Message-ID: <528E221A.6090307@greyhouse.net> (raw)
In-Reply-To: <1385045738-29726-1-git-send-email-Anton.Nayshtut@wilocity.com>
On 11/21/2013 09:55 AM, Anton Nayshtut wrote:
> L2 Destination Address based (L2DA) mode allows bonding to send packets using
> different slaves according to packets L2 Destination Address.
>
> In L2DA mode, the bonding maintains a default slave and DA/slave map.
>
> Upon a packet transmission, the bonding examines DA of the packet and tries to
> find a corresponding slave within the map. If found, the slave is used for the
> packet transmission. Otherwise, the default slave is used. If the default slave
> is unable to transmit at this moment, the bonding tries to fall back to an
> arbitrary slave that can transmit.
>
> Both the default slave and the map can be controlled via sysfs or by ioctls.
>
> Anton Nayshtut (4):
> bonding: L2DA mode added
> bonding: L2DA mode intergated
> bonding: L2DA command IOCTL
> bonding: L2DA query IOCTL
>
> drivers/net/bonding/Makefile | 2 +-
> drivers/net/bonding/bond_l2da.c | 425 +++++++++++++++++++++++++++++++++++++
> drivers/net/bonding/bond_l2da.h | 56 +++++
> drivers/net/bonding/bond_main.c | 172 ++++++++++++++-
> drivers/net/bonding/bond_options.c | 17 +-
> drivers/net/bonding/bond_sysfs.c | 223 ++++++++++++++++++-
> drivers/net/bonding/bonding.h | 7 +
> include/uapi/linux/if_bonding.h | 32 +++
> include/uapi/linux/sockios.h | 4 +-
> net/core/dev_ioctl.c | 4 +
> net/socket.c | 4 +
> 11 files changed, 937 insertions(+), 9 deletions(-)
> create mode 100644 drivers/net/bonding/bond_l2da.c
> create mode 100644 drivers/net/bonding/bond_l2da.h
>
I have not done a full review of this, but I notice right away that
there are no patches to Documentation/networking/bonding.txt. You can
wait until there are more comments before submitting a v2 of this
series, but please make sure it includes a documentation update when
submitted.
prev parent reply other threads:[~2013-11-21 15:09 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-11-21 14:55 [PATCH 0/4] bonding: L2DA mode Anton Nayshtut
2013-11-21 14:55 ` [PATCH 1/4] bonding: L2DA mode added Anton Nayshtut
2013-11-21 15:32 ` Eric Dumazet
2013-11-21 15:37 ` Nikolay Aleksandrov
2013-11-25 9:10 ` Anton Nayshtut
2013-11-21 14:55 ` [PATCH 2/4] bonding: L2DA mode intergated Anton Nayshtut
2013-11-21 14:55 ` [PATCH 3/4] bonding: L2DA command IOCTL Anton Nayshtut
2013-11-21 14:55 ` [PATCH 4/4] bonding: L2DA query IOCTL Anton Nayshtut
2013-11-21 15:09 ` Andy Gospodarek [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=528E221A.6090307@greyhouse.net \
--to=andy@greyhouse.net \
--cc=Anton.Nayshtut@wilocity.com \
--cc=davem@davemloft.net \
--cc=edumazet@google.com \
--cc=fubar@us.ibm.com \
--cc=linux-kernel@vger.kernel.org \
--cc=netdev@vger.kernel.org \
--cc=nschichan@freebox.fr \
--cc=vfalico@redhat.com \
--cc=xiyou.wangcong@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).