diff options
| author |  Jakub Kicinski <kuba@kernel.org>
| 2024-04-23 17:21:48 -0700 |
|---|---|---|
| committer | Jakub Kicinski <kuba@kernel.org>
| 2024-04-24 20:16:43 -0700 |
| commit | 730117730709992c9f6535dd7b47638ee561ec45 (patch) | |
| tree | 7c5762fd7cdd3f0f77ec13eb29279cfcd77f9aec /net | |
| parent | igc: Fix LED-related deadlock on driver unbind (diff) | |
| download | linux-730117730709992c9f6535dd7b47638ee561ec45.tar.gz linux-730117730709992c9f6535dd7b47638ee561ec45.tar.bz2 linux-730117730709992c9f6535dd7b47638ee561ec45.zip | |
eth: bnxt: fix counting packets discarded due to OOM and netpoll
I added OOM and netpoll discard counters, naively assuming that
the cpr pointer is pointing to a common completion ring.
Turns out that is usually *a* completion ring but not *the*
completion ring which bnapi->cp_ring points to. bnapi->cp_ring
is where the stats are read from, so we end up reporting 0
thru ethtool -S and qstat even though the drop events have happened.
Make 100% sure we're recording statistics in the correct structure.
Fixes: 907fd4a294db ("bnxt: count discards due to memory allocation errors")
Reviewed-by: Michael Chan <michael.chan@broadcom.com>
Link: https://lore.kernel.org/r/20240424002148.3937059-1-kuba@kernel.org
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Diffstat (limited to 'net')
0 files changed, 0 insertions, 0 deletions