if_ bnxt: Fix interrupt storm on Thor2
AcceptedPublic
Actions

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

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Sep 6, 3:11 AM

	Unknown Object (File)
	Sat, Sep 5, 7:21 PM

	Unknown Object (File)
	Sat, Sep 5, 7:20 PM

	Unknown Object (File)
	Fri, Sep 4, 8:52 PM

	Unknown Object (File)
	Fri, Sep 4, 2:09 AM

	Unknown Object (File)
	Thu, Sep 3, 4:47 PM

	Unknown Object (File)
	Wed, Sep 2, 9:43 AM

	Unknown Object (File)
	Tue, Sep 1, 10:35 PM

View All 35 Files

Subscribers

None

Details

Reviewers

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

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.Fri, Aug 21, 8:28 AM

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

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

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 Thor2AcceptedPublicActions