From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Ahern Subject: Re: [PATCH iproute2 net-next] bridge: add support for backup port Date: Sat, 13 Oct 2018 20:30:59 -0600 Message-ID: <883b40ac-0d13-daf8-c30e-24c2b25d0b9b@gmail.com> References: <20181012114255.17217-1-nikolay@cumulusnetworks.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: roopa@cumulusnetworks.com, dsahern@kernel.org To: Nikolay Aleksandrov , netdev@vger.kernel.org Return-path: Received: from mail-pl1-f195.google.com ([209.85.214.195]:34621 "EHLO mail-pl1-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725734AbeJNKKW (ORCPT ); Sun, 14 Oct 2018 06:10:22 -0400 Received: by mail-pl1-f195.google.com with SMTP id f18-v6so7646687plr.1 for ; Sat, 13 Oct 2018 19:31:02 -0700 (PDT) In-Reply-To: <20181012114255.17217-1-nikolay@cumulusnetworks.com> Content-Language: en-US Sender: netdev-owner@vger.kernel.org List-ID: On 10/12/18 5:42 AM, Nikolay Aleksandrov wrote: > This patch adds support for the new backup port option that can be set > on a bridge port. If the port's carrier goes down all of the traffic > gets redirected to the configured backup port. We add the following new > arguments: > $ ip link set dev brport type bridge_slave backup_port brport2 > $ ip link set dev brport type bridge_slave nobackup_port > > $ bridge link set dev brport backup_port brport2 > $ bridge link set dev brport nobackup_port > > The man pages are updated respectively. > Also 2 minor style adjustments: > - add missing space to bridge man page's state argument > - use lower starting case for vlan_tunnel in ip-link man page (to be > consistent with the rest) > > Signed-off-by: Nikolay Aleksandrov > --- > bridge/link.c | 26 ++++++++++++++++++++++++++ > ip/iplink_bridge_slave.c | 18 ++++++++++++++++++ > man/man8/bridge.8 | 13 ++++++++++++- > man/man8/ip-link.8.in | 14 ++++++++++++-- > 4 files changed, 68 insertions(+), 3 deletions(-) > applied to iproute2-next. Thanks