netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
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.

      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).