diff options
author | Jakub Kicinski <kuba@kernel.org> | 2023-07-19 18:04:40 -0700 |
---|---|---|
committer | Jakub Kicinski <kuba@kernel.org> | 2023-07-20 20:09:13 -0700 |
commit | 2b56b3d9924160d84c37d085628525666e3b1cea (patch) | |
tree | 97eaeaaedfd419060ec1785752a2dc9e5a004c17 /arch/x86 | |
parent | 9b1a00fda61ee9874e11c24c6be7ddc36e8a4598 (diff) |
eth: bnxt: handle invalid Tx completions more gracefully
Invalid Tx completions should never happen (tm) but when they do
they crash the host, because driver blindly trusts that there is
a valid skb pointer on the ring.
The completions I've seen appear to be some form of FW / HW
miscalculation or staleness, they have typical (small) values
(<100), but they are most often higher than number of queued
descriptors. They usually happen after boot.
Instead of crashing, print a warning and schedule a reset.
Link: https://lore.kernel.org/r/20230720010440.1967136-4-kuba@kernel.org
Reviewed-by: Michael Chan <michael.chan@broadcom.com>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Diffstat (limited to 'arch/x86')
0 files changed, 0 insertions, 0 deletions