From fb1021a26db14b199c50e3eed77a1daa371f282e Mon Sep 17 00:00:00 2001 From: Susant Sahani Date: Mon, 21 Jul 2014 18:52:14 +0530 Subject: [PATCH] networkd: bond add support for lacp rate LacpduTransmitRate option specifies the rate in which link partner to transmit LACPDU packets in 802.3ad mode. Possible values slow : Request partner to transmit LACPDUs every 30 seconds fast : Request partner to transmit LACPDUs every 1 second The default is slow. chages: 1. Added enum bond_lacp_rate_table 2. gperf LacpduTransmitRate Test: conf file: [NetDev] Name=bond1 Kind=bond [Bond] Mode=802.3ad LacpduTransmitRate=fast test: cat /proc/net/bonding/bond1 Ethernet Channel Bonding Driver: v3.7.1 (April 27, 2011) Bonding Mode: IEEE 802.3ad Dynamic link aggregation Transmit Hash Policy: layer2+3 (2) MII Status: up MII Polling Interval (ms): 0 Up Delay (ms): 0 Down Delay (ms): 0 802.3ad info LACP rate: fast Min links: 0 Aggregator selection policy (ad_select): stable bond bond1 has no active aggregator [tomegun: renamed from LacpduTransmitRate to LACPTransmitRate, manpage fixes and dropped bond_ prefix from variables] --- man/systemd.netdev.xml | 42 ++++++++++++++++--------- src/network/networkd-netdev-bond.c | 20 ++++++++++++ src/network/networkd-netdev-bond.h | 13 ++++++++ src/network/networkd-netdev-gperf.gperf | 1 + 4 files changed, 61 insertions(+), 15 deletions(-) diff --git a/man/systemd.netdev.xml b/man/systemd.netdev.xml index ba212ec06..cea416f94 100644 --- a/man/systemd.netdev.xml +++ b/man/systemd.netdev.xml @@ -408,14 +408,15 @@ Mode= - Specifies one of the bonding policies. The default is balance-rr (round robin). - balance-rr - active-backup - balance-xor - broadcast - 802.3ad - balance-tlb - balance-alb + Specifies one of the bonding policies. The default is + balance-rr (round robin). Possible values are + balance-rr, + active-backup, + balance-xor, + broadcast, + 802.3ad, + balance-tlb, and + balance-alb. @@ -424,17 +425,28 @@ TransmitHashPolicy= Selects the transmit hash policy to use for slave selection in - balance-xor, 802.3ad, and tlb modes. - layer2 - layer3+4 - layer2+3 - encap2+3 - 802.3ad - encap3+4 + balance-xor, 802.3ad, and tlb modes. Possible values are + layer2, + layer3+4, + layer2+3, + encap2+3, + 802.3ad, and + encap3+4. + + LACPTransmitRate= + + Specifies the rate with which link partner + transmits Link Aggregation Control Protocol Data Unit packets + in 802.3ad mode. Possible values are + slow, which requests partner to transmit LACPDUs every 30 seconds, and + fast, which requests partner to transmit LACPDUs every second. + The default value is slow. + + diff --git a/src/network/networkd-netdev-bond.c b/src/network/networkd-netdev-bond.c index c4d1fcd17..e3f333356 100644 --- a/src/network/networkd-netdev-bond.c +++ b/src/network/networkd-netdev-bond.c @@ -57,6 +57,14 @@ DEFINE_CONFIG_PARSE_ENUM(config_parse_bond_xmit_hash_policy, BondXmitHashPolicy, "Failed to parse bond transmit hash policy") +static const char* const bond_lacp_rate_table[_NETDEV_BOND_LACP_RATE_MAX] = { + [NETDEV_BOND_LACP_RATE_SLOW] = "slow", + [NETDEV_BOND_LACP_RATE_FAST] = "fast", +}; + +DEFINE_STRING_TABLE_LOOKUP(bond_lacp_rate, BondLacpRate); +DEFINE_CONFIG_PARSE_ENUM(config_parse_bond_lacp_rate, bond_lacp_rate, BondLacpRate, "Failed to parse bond lacp rate") + static uint8_t bond_mode_to_kernel(BondMode mode) { switch (mode) { case NETDEV_BOND_MODE_BALANCE_RR: @@ -126,6 +134,17 @@ static int netdev_bond_fill_message_create(NetDev *netdev, Link *link, sd_rtnl_m } } + if (b->lacp_rate != _NETDEV_BOND_LACP_RATE_INVALID && + b->mode == NETDEV_BOND_MODE_802_3AD) { + r = sd_rtnl_message_append_u8(m, IFLA_BOND_AD_LACP_RATE, b->lacp_rate ); + if (r < 0) { + log_error_netdev(netdev, + "Could not append IFLA_BOND_AD_LACP_RATE attribute: %s", + strerror(-r)); + return r; + } + } + return 0; } @@ -137,6 +156,7 @@ static void bond_init(NetDev *netdev) { b->mode = _NETDEV_BOND_MODE_INVALID; b->xmit_hash_policy = _NETDEV_BOND_XMIT_HASH_POLICY_INVALID; + b->lacp_rate = _NETDEV_BOND_LACP_RATE_INVALID; } const NetDevVTable bond_vtable = { diff --git a/src/network/networkd-netdev-bond.h b/src/network/networkd-netdev-bond.h index 0e9955ee3..fea076817 100644 --- a/src/network/networkd-netdev-bond.h +++ b/src/network/networkd-netdev-bond.h @@ -47,11 +47,20 @@ typedef enum BondXmitHashPolicy { _NETDEV_BOND_XMIT_HASH_POLICY_INVALID = -1 } BondXmitHashPolicy; + +typedef enum BondLacpRate { + NETDEV_BOND_LACP_RATE_SLOW, + NETDEV_BOND_LACP_RATE_FAST, + _NETDEV_BOND_LACP_RATE_MAX, + _NETDEV_BOND_LACP_RATE_INVALID = -1, +} BondLacpRate; + struct Bond { NetDev meta; BondMode mode; BondXmitHashPolicy xmit_hash_policy; + BondLacpRate lacp_rate; }; extern const NetDevVTable bond_vtable; @@ -62,5 +71,9 @@ BondMode bond_mode_from_string(const char *d) _pure_; const char *bond_xmit_hash_policy_to_string(BondXmitHashPolicy d) _const_; BondXmitHashPolicy bond_xmit_hash_policy_from_string(const char *d) _pure_; +const char *bond_lacp_rate_to_string(BondLacpRate d) _const_; +BondLacpRate bond_lacp_rate_from_string(const char *d) _pure_; + int config_parse_bond_mode(const char *unit, const char *filename, unsigned line, const char *section, unsigned section_line, const char *lvalue, int ltype, const char *rvalue, void *data, void *userdata); int config_parse_bond_xmit_hash_policy(const char *unit, const char *filename, unsigned line, const char *section, unsigned section_line, const char *lvalue, int ltype, const char *rvalue, void *data, void *userdata); +int config_parse_bond_lacp_rate(const char *unit, const char *filename, unsigned line, const char *section, unsigned section_line, const char *lvalue, int ltype, const char *rvalue, void *data, void *userdata); diff --git a/src/network/networkd-netdev-gperf.gperf b/src/network/networkd-netdev-gperf.gperf index bafdccfc1..40fae75c4 100644 --- a/src/network/networkd-netdev-gperf.gperf +++ b/src/network/networkd-netdev-gperf.gperf @@ -53,3 +53,4 @@ Tap.User, config_parse_string, 0, Tap.Group, config_parse_string, 0, offsetof(TunTap, group_name) Bond.Mode, config_parse_bond_mode, 0, offsetof(Bond, mode) Bond.TransmitHashPolicy, config_parse_bond_xmit_hash_policy, 0, offsetof(Bond, xmit_hash_policy) +Bond.LACPTransmitRate, config_parse_bond_lacp_rate, 0, offsetof(Bond, lacp_rate) -- 2.30.2