diff options
| author | Artur Rojek <ar@semihalf.com> | 2021-08-12 08:34:17 +0000 |
|---|---|---|
| committer | Marcin Wojtas <mw@FreeBSD.org> | 2021-09-01 23:05:38 +0000 |
| commit | c81f8c26115a64b9a97ecdb2a64e824dd839ee73 (patch) | |
| tree | de77c65d22b91a487124fa80db4849c5d42b2e0e | |
| parent | 6bc90e8acf884f659415a919bc9cd9cfea701e0a (diff) | |
| download | src-c81f8c26115a64b9a97ecdb2a64e824dd839ee73.tar.gz src-c81f8c26115a64b9a97ecdb2a64e824dd839ee73.zip | |
ena: Avoid unnecessary mbuf collapses for LLQ condition
In case of Low-latency Queue, one small enough descriptor can be pushed
directly to the ENA hw, thus saving one fragment. Check for this
condition before performing collapse.
Obtained from: Semihalf
MFC after: 2 weeks
Sponsored by: Amazon, Inc.
| -rw-r--r-- | sys/dev/ena/ena_datapath.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/sys/dev/ena/ena_datapath.c b/sys/dev/ena/ena_datapath.c index 15bd09c489cf..c0066b1209cc 100644 --- a/sys/dev/ena/ena_datapath.c +++ b/sys/dev/ena/ena_datapath.c @@ -801,6 +801,11 @@ ena_check_and_collapse_mbuf(struct ena_ring *tx_ring, struct mbuf **mbuf) /* One segment must be reserved for configuration descriptor. */ if (num_frags < adapter->max_tx_sgl_size) return (0); + + if ((num_frags == adapter->max_tx_sgl_size) && + ((*mbuf)->m_pkthdr.len < tx_ring->tx_max_header_size)) + return (0); + counter_u64_add(tx_ring->tx_stats.collapse, 1); collapsed_mbuf = m_collapse(*mbuf, M_NOWAIT, |