if_ bnxt: Fix interrupt storm on Thor2
ClosedPublic
Actions

Authored by sumit.saxena_broadcom.com on Aug 3 2026, 12:11 PM.

Details

Reviewers

chandrakanth.patil_broadcom.com
sreekanth.reddy_broadcom.com
gallatin
shurd

Commits

rG8c69a5dc60b3: if_bnxt: Fix interrupt storm on Thor2

Summary

An interrupt storm was observed on Thor2 because the Rx CQ consumer
index was always one less than it should be. Correct the Rx CQ
consumer index to fix the storm.

Reported by: Drew Gallatin

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

sumit.saxena_broadcom.com created this revision.Aug 3 2026, 12:11 PM

Herald added a reviewer: shurd. · View Herald TranscriptAug 3 2026, 12:11 PM

sumit.saxena_broadcom.com requested review of this revision.Aug 3 2026, 12:11 PM

sumit.saxena_broadcom.com edited the summary of this revision. (Show Details)Aug 3 2026, 6:27 PM

Why is the if needed? UINT32_MAX +1 == 0

In D58602#1345383, @gallatin wrote:

Why is the if needed? UINT32_MAX +1 == 0

Yes, the 'if' is not needed.

sumit.saxena_broadcom.com updated this revision to Diff 184630.Aug 21 2026, 8:28 AM

gallatin accepted this revision.Aug 21 2026, 2:08 PM

This revision is now accepted and ready to land.Aug 21 2026, 2:08 PM

Closed by commit rG8c69a5dc60b3: if_bnxt: Fix interrupt storm on Thor2 (authored by sreekanth.reddy_broadcom.com, committed by sumit.saxena_broadcom.com). · Explain WhyFri, Oct 9, 10:13 AM

This revision was automatically updated to reflect the committed changes.

sumit.saxena_broadcom.com added a commit: rG8c69a5dc60b3: if_bnxt: Fix interrupt storm on Thor2.

Herald added a subscriber: imp. · View Herald TranscriptFri, Oct 9, 10:13 AM

Revision Contents
Changeset List

Path

Size

sys/

dev/

bnxt/

bnxt_en/

if_bnxt.c

3 lines

Diff 184630

View Options

sys/dev/bnxt/bnxt_en/if_bnxt.c

if_ bnxt: Fix interrupt storm on Thor2ClosedPublicActions