diff options
| author | Christophe Fontaine <cfontain@redhat.com> | 2022-07-07 14:31:31 +0200 |
|---|---|---|
| committer | Ilya Maximets <i.maximets@ovn.org> | 2022-07-15 23:08:38 +0200 |
| commit | 1b53826d6c740c78857b66b1769650ffd3de7c63 (patch) | |
| tree | c5aa31152ad4bd34fc8deb4234ec9e79bcdbb2f7 /vswitchd | |
| parent | 73ba04fd77a1c4ea98dd289268b9db9067ce3519 (diff) | |
| download | openvswitch-1b53826d6c740c78857b66b1769650ffd3de7c63.tar.gz | |
ofproto/bond: Add knob 'all-members-active'.
This config param allows the delivery of broadcast and multicast
packets to the secondary interface of non-lacp bonds, equivalent
to the option 'all_slaves_active' for Linux kernel bonds.
Reported-at: https://bugzilla.redhat.com/1720935
Signed-off-by: Christophe Fontaine <cfontain@redhat.com>
Signed-off-by: Ilya Maximets <i.maximets@ovn.org>
Diffstat (limited to 'vswitchd')
| -rw-r--r-- | vswitchd/bridge.c | 3 | ||||
| -rw-r--r-- | vswitchd/vswitch.xml | 14 |
2 files changed, 17 insertions, 0 deletions
diff --git a/vswitchd/bridge.c b/vswitchd/bridge.c index b704410d3..1db74de9d 100644 --- a/vswitchd/bridge.c +++ b/vswitchd/bridge.c @@ -4618,6 +4618,9 @@ port_configure_bond(struct port *port, struct bond_settings *s) s->use_lb_output_action = (s->balance == BM_TCP) && smap_get_bool(&port->cfg->other_config, "lb-output-action", false); + /* all_members_active is disabled by default. */ + s->all_members_active = smap_get_bool(&port->cfg->other_config, + "all-members-active", false); } /* Returns true if 'port' is synthetic, that is, if we constructed it locally diff --git a/vswitchd/vswitch.xml b/vswitchd/vswitch.xml index 3eb3d4af2..4a9284f6b 100644 --- a/vswitchd/vswitch.xml +++ b/vswitchd/vswitch.xml @@ -2122,6 +2122,20 @@ <code>true</code>). </column> + <column name="other_config" key="all-members-active" + type='{"type": "boolean"}'> + <p> + Enable/Disable delivery of broadcast/multicast packets on secondary + interface of a balance-slb bond. Relevant only when + <ref column="lacp"/> is <code>off</code>. + </p> + <p> + This parameter is identical to <code>all_slaves_active</code> for + Linux kernel bonds. Disabled by default as it is not a desirable + configuration for most users. + </p> + </column> + <group title="Link Failure Detection"> <p> An important part of link bonding is detecting that links are down so |