diff options
author | nabijaczleweli <nabijaczleweli@gmail.com> | 2020-04-29 02:00:25 +0200 |
---|---|---|
committer | nabijaczleweli <nabijaczleweli@gmail.com> | 2020-04-29 18:57:13 +0200 |
commit | e81f5fc4e86cef5a99e37eaaa2e5fb869bc76039 (patch) | |
tree | be1d79f32dcec30948f3a92e9da659df116a1edd | |
parent | 80af9bdabe2aa6421568d5d20f4c513cf49a3e43 (diff) | |
download | systemd-e81f5fc4e86cef5a99e37eaaa2e5fb869bc76039.tar.gz |
link: Allow configuring RX mini and jumbo ring sizes, too
This now covers all ethtool_ringparam configurables (as of v5.6;
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/include/uapi/linux/ethtool.h?h=v5.6#n488)
-rw-r--r-- | man/systemd.link.xml | 14 | ||||
-rw-r--r-- | src/shared/ethtool-util.c | 16 | ||||
-rw-r--r-- | src/shared/ethtool-util.h | 4 | ||||
-rw-r--r-- | src/udev/net/link-config-gperf.gperf | 2 | ||||
-rw-r--r-- | src/udev/net/link-config.c | 2 | ||||
-rw-r--r-- | test/fuzz/fuzz-link-parser/directives.link | 2 |
6 files changed, 39 insertions, 1 deletions
diff --git a/man/systemd.link.xml b/man/systemd.link.xml index 34d4834cca..7e17bf095f 100644 --- a/man/systemd.link.xml +++ b/man/systemd.link.xml @@ -696,6 +696,20 @@ </listitem> </varlistentry> <varlistentry> + <term><varname>RxMiniBufferSize=</varname></term> + <listitem> + <para>Takes an integer. Specifies the maximum number of pending packets in the NIC mini receive buffer. + When unset, the kernel's default will be used.</para> + </listitem> + </varlistentry> + <varlistentry> + <term><varname>RxJumboBufferSize=</varname></term> + <listitem> + <para>Takes an integer. Specifies the maximum number of pending packets in the NIC jumbo receive buffer. + When unset, the kernel's default will be used.</para> + </listitem> + </varlistentry> + <varlistentry> <term><varname>TxBufferSize=</varname></term> <listitem> <para>Takes an integer. Specifies the maximum number of pending packets in the NIC transmit buffer. diff --git a/src/shared/ethtool-util.c b/src/shared/ethtool-util.c index 703e48025e..0cde87f5ac 100644 --- a/src/shared/ethtool-util.c +++ b/src/shared/ethtool-util.c @@ -436,6 +436,16 @@ int ethtool_set_nic_buffer_size(int *ethtool_fd, const char *ifname, netdev_ring need_update = true; } + if (ring->rx_mini_pending_set && ecmd.rx_mini_pending != ring->rx_mini_pending) { + ecmd.rx_mini_pending = ring->rx_mini_pending; + need_update = true; + } + + if (ring->rx_jumbo_pending_set && ecmd.rx_jumbo_pending != ring->rx_jumbo_pending) { + ecmd.rx_jumbo_pending = ring->rx_jumbo_pending; + need_update = true; + } + if (ring->tx_pending_set && ecmd.tx_pending != ring->tx_pending) { ecmd.tx_pending = ring->tx_pending; need_update = true; @@ -1032,6 +1042,12 @@ int config_parse_nic_buffer_size(const char *unit, if (streq(lvalue, "RxBufferSize")) { ring->rx_pending = k; ring->rx_pending_set = true; + } else if (streq(lvalue, "RxMiniBufferSize")) { + ring->rx_mini_pending = k; + ring->rx_mini_pending_set = true; + } else if (streq(lvalue, "RxJumboBufferSize")) { + ring->rx_jumbo_pending = k; + ring->rx_jumbo_pending_set = true; } else if (streq(lvalue, "TxBufferSize")) { ring->tx_pending = k; ring->tx_pending_set = true; diff --git a/src/shared/ethtool-util.h b/src/shared/ethtool-util.h index 55c41f5bc7..4730241708 100644 --- a/src/shared/ethtool-util.h +++ b/src/shared/ethtool-util.h @@ -84,9 +84,13 @@ typedef struct netdev_channels { typedef struct netdev_ring_param { uint32_t rx_pending; + uint32_t rx_mini_pending; + uint32_t rx_jumbo_pending; uint32_t tx_pending; bool rx_pending_set; + bool rx_mini_pending_set; + bool rx_jumbo_pending_set; bool tx_pending_set; } netdev_ring_param; diff --git a/src/udev/net/link-config-gperf.gperf b/src/udev/net/link-config-gperf.gperf index 2784246dd7..60a9d21c1d 100644 --- a/src/udev/net/link-config-gperf.gperf +++ b/src/udev/net/link-config-gperf.gperf @@ -59,6 +59,8 @@ Link.OtherChannels, config_parse_channel, 0, Link.CombinedChannels, config_parse_channel, 0, offsetof(link_config, channels) Link.Advertise, config_parse_advertise, 0, offsetof(link_config, advertise) Link.RxBufferSize, config_parse_nic_buffer_size, 0, offsetof(link_config, ring) +Link.RxMiniBufferSize, config_parse_nic_buffer_size, 0, offsetof(link_config, ring) +Link.RxJumboBufferSize, config_parse_nic_buffer_size, 0, offsetof(link_config, ring) Link.TxBufferSize, config_parse_nic_buffer_size, 0, offsetof(link_config, ring) Link.RxFlowControl, config_parse_tristate, 0, offsetof(link_config, rx_flow_control) Link.TxFlowControl, config_parse_tristate, 0, offsetof(link_config, tx_flow_control) diff --git a/src/udev/net/link-config.c b/src/udev/net/link-config.c index 48039511fa..098aa59913 100644 --- a/src/udev/net/link-config.c +++ b/src/udev/net/link-config.c @@ -407,7 +407,7 @@ int link_config_apply(link_config_ctx *ctx, link_config *config, log_warning_errno(r, "Could not set channels of %s: %m", old_name); } - if (config->ring.rx_pending_set || config->ring.tx_pending_set) { + if (config->ring.rx_pending_set || config->ring.rx_mini_pending_set || config->ring.rx_jumbo_pending_set || config->ring.tx_pending_set) { r = ethtool_set_nic_buffer_size(&ctx->ethtool_fd, old_name, &config->ring); if (r < 0) log_warning_errno(r, "Could not set ring buffer of %s: %m", old_name); diff --git a/test/fuzz/fuzz-link-parser/directives.link b/test/fuzz/fuzz-link-parser/directives.link index fe71d26d89..7d23a9ecf8 100644 --- a/test/fuzz/fuzz-link-parser/directives.link +++ b/test/fuzz/fuzz-link-parser/directives.link @@ -40,6 +40,8 @@ OtherChannels= CombinedChannels= Advertise= RxBufferSize= +RxMiniBufferSize= +RxJumboBufferSize= TxBufferSize= RxFlowControl= TxFlowControl= |