Page MenuHomeFreeBSD

bnxt_en: Fix Rx CQ consumer index on Thor2 to prevent interrupt storm
Needs RevisionPublic

Authored by chandrakanth.patil_broadcom.com on Aug 21 2025, 4:08 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Nov 9, 12:41 PM
Unknown Object (File)
Sun, Nov 9, 12:41 PM
Unknown Object (File)
Sun, Nov 9, 8:34 AM
Unknown Object (File)
Sat, Nov 8, 8:02 AM
Unknown Object (File)
Thu, Nov 6, 8:21 PM
Unknown Object (File)
Thu, Nov 6, 7:22 PM
Unknown Object (File)
Oct 2 2025, 4:20 PM
Unknown Object (File)
Sep 20 2025, 11:30 PM
Subscribers

Details

Summary

On Thor2, the Rx Completion Queue consumer index was incorrect, causing
continuous interrupts. Updated the index handling to stop the interrupt storm.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

its been long the patches not reviewed so please review the patch and provide your comments.

gallatin added a subscriber: gallatin.
gallatin added inline comments.
sys/dev/bnxt/bnxt_en/if_bnxt.c
1492

This if-then is just another increment. Eg, you're saying
if cons == 0xffffffff, then saying cons = 0. is exactly the same as saying cons = cons++ because cons++ will wrap back to zero on that increment.

So it would be simpler to skip the if/then and just always increment.

This revision now requires changes to proceed.Sep 18 2025, 2:17 PM