From: Jeff Garzik <jgarzik@pobox.com>
To: Jay Vosburgh <fubar@us.ibm.com>
Cc: netdev@vger.kernel.org
Subject: Re: [PATCH 8/8] bonding: Add "follow" option to fail_over_mac
Date: Thu, 22 May 2008 07:09:19 -0400 [thread overview]
Message-ID: <4835545F.1080206@pobox.com> (raw)
In-Reply-To: <12110838193155-git-send-email-fubar@us.ibm.com>
Jay Vosburgh wrote:
> Add a "follow" selection for fail_over_mac. This option
> causes the MAC address to move from slave to slave as the active
> slave changes. This is in addition to the existing fail_over_mac option
> that causes the bond's MAC address to change during failover.
>
> This new option is useful for devices that cannot tolerate
> multiple ports using the same MAC address simultaneously, either
> because it confuses them or incurs a performance penalty (as is the
> case with some LPAR-aware multiport devices). Because the MAC of the
> bond itself does not change, the "follow" option is slightly more
> reliable during failover and doesn't change the MAC of the bond during
> operation.
>
> This patch requires a previous ARP monitor change to properly
> handle RTNL during failovers.
>
> Signed-off-by: Jay Vosburgh <fubar@us.ibm.com>
> ---
> Documentation/networking/bonding.txt | 96 ++++++++++++------
> drivers/net/bonding/bond_main.c | 185 ++++++++++++++++++++++++++-------
> drivers/net/bonding/bond_sysfs.c | 36 +++----
> drivers/net/bonding/bonding.h | 4 +
> 4 files changed, 232 insertions(+), 89 deletions(-)
applied 1-8
next prev parent reply other threads:[~2008-05-22 11:09 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-05-18 4:10 [PATCH net-next-2.6 0/8] bonding: Fixes and updates Jay Vosburgh
[not found] ` <12110838151824-git-send-email-fubar@us.ibm.com>
2008-05-18 4:10 ` [PATCH 2/8] bonding: remove test for IP in ARP monitor Jay Vosburgh
2008-05-18 4:10 ` [PATCH 3/8] bonding: Remove redundant argument from bond_create Jay Vosburgh
2008-05-18 4:10 ` [PATCH 4/8] bonding: Relax unneeded _safe lists iterations Jay Vosburgh
2008-05-18 4:10 ` [PATCH 5/8] bonding: Remove unneeded list_empty checks Jay Vosburgh
2008-05-18 4:10 ` [PATCH 6/8] bonding: Send more than one gratuitous ARP when slave takes over Jay Vosburgh
2008-05-18 4:10 ` [PATCH 7/8] bonding: refactor ARP active-backup monitor Jay Vosburgh
2008-05-18 4:10 ` [PATCH 8/8] bonding: Add "follow" option to fail_over_mac Jay Vosburgh
2008-05-22 11:09 ` Jeff Garzik [this message]
2008-05-20 21:54 ` [PATCH 6/8] bonding: Send more than one gratuitous ARP when slave takes over Andrew Morton
2008-05-20 21:46 ` [PATCH 3/8] bonding: Remove redundant argument from bond_create Andrew Morton
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=4835545F.1080206@pobox.com \
--to=jgarzik@pobox.com \
--cc=fubar@us.ibm.com \
--cc=netdev@vger.kernel.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.