diff options
| author | Mark Johnston <markj@FreeBSD.org> | 2021-02-24 15:08:53 +0000 |
|---|---|---|
| committer | Mark Johnston <markj@FreeBSD.org> | 2021-02-24 15:08:53 +0000 |
| commit | b6999635b195e8c0dd5c08ac9ff268c92107cc66 (patch) | |
| tree | f5013c01acd4556668a2e78a0441b608fde733fc | |
| parent | e70eb40271512dfbca7cecf823e4b445e3989c2e (diff) | |
| download | src-b6999635b195e8c0dd5c08ac9ff268c92107cc66.tar.gz src-b6999635b195e8c0dd5c08ac9ff268c92107cc66.zip | |
iflib: Avoid double counting in rxeof
iflib_rxeof() was counting everything twice. This was introduced when
pfil hooks were added to the iflib receive path. We want to count rx
packets/bytes before the pfil hooks are executed, so remove the counter
adjustments that are executed after.
PR: 253583
Reviewed by: gallatin, erj
MFC after: 3 days
Sponsored by: The FreeBSD Foundation
Differential Revision: https://reviews.freebsd.org/D28900
| -rw-r--r-- | sys/net/iflib.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/sys/net/iflib.c b/sys/net/iflib.c index 6312499feac8..5366b652373d 100644 --- a/sys/net/iflib.c +++ b/sys/net/iflib.c @@ -2973,8 +2973,6 @@ iflib_rxeof(iflib_rxq_t rxq, qidx_t budget) if (!IP_ALIGNED(m) && (m = iflib_fixup_rx(m)) == NULL) continue; #endif - rx_bytes += m->m_pkthdr.len; - rx_pkts++; #if defined(INET6) || defined(INET) if (lro_enabled) { if (!lro_possible) { |