bnxt: set hardware checksum only if required
ClosedPublic
Actions

Authored by anaelle.cazuc_stormshield.eu on Nov 27 2025, 10:32 AM.

Details

Reviewers

shurd
sreekanth.reddy_broadcom.com
tuexen

Group Reviewers

network

Commits

rG2e94e1631ef5: bnxt: set hardware checksum only if required

Summary

The test condition in the bnxt driver for TCP/UDP transmit hardware checksum offload is invalid: only the TCP / UDP csum bits should be tested
Only the relevant ipi_csum_flags bits are now tested as it's done in other iflib drivers

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

anaelle.cazuc_stormshield.eu created this revision.Nov 27 2025, 10:32 AM

Herald added a reviewer: shurd. · View Herald TranscriptNov 27 2025, 10:32 AM

Herald added a subscriber: imp. · View Herald Transcript

A branch on github is available here to ease merge: https://github.com/stormshield-acazuc/freebsd-src/tree/bnxt_csum

markj added a reviewer: sreekanth.reddy_broadcom.com.Nov 27 2025, 2:25 PM

tuexen accepted this revision.Nov 27 2025, 2:31 PM

This revision is now accepted and ready to land.Nov 27 2025, 2:31 PM

Closed by commit rG2e94e1631ef5: bnxt: set hardware checksum only if required (authored by anaelle.cazuc_stormshield.eu, committed by emaste). · Explain WhyFeb 5 2026, 2:14 PM

This revision was automatically updated to reflect the committed changes.

emaste added a commit: rG2e94e1631ef5: bnxt: set hardware checksum only if required.

Revision Contents
Changeset List

Path

Size

sys/

dev/

bnxt/

bnxt_en/

bnxt_txrx.c

22 lines

Diff 167172

View Options

sys/dev/bnxt/bnxt_en/bnxt_txrx.c

bnxt: set hardware checksum only if requiredClosedPublicActions