Page MenuHomeFreeBSD

e1000: Fix lem/em UDP rx csum offload
ClosedPublic

Authored by kbowling on Aug 7 2021, 4:36 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Oct 11, 3:22 PM
Unknown Object (File)
Thu, Oct 9, 6:37 AM
Unknown Object (File)
Mon, Oct 6, 4:28 PM
Unknown Object (File)
Sep 14 2025, 10:54 AM
Unknown Object (File)
Sep 13 2025, 5:28 AM
Unknown Object (File)
Sep 11 2025, 11:28 AM
Unknown Object (File)
Sep 11 2025, 3:16 AM
Unknown Object (File)
Sep 10 2025, 11:09 PM
Subscribers

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kbowling created this revision.
kbowling retitled this revision from e1000: Fix UDP rx checksum offload to e1000: Fix lem/em UDP rx csum offload.Aug 7 2021, 4:38 AM
karels added inline comments.
sys/dev/e1000/em_txrx.c
740

Isn't this test redundant?

kbowling marked an inline comment as done.
kbowling added inline comments.
sys/dev/e1000/em_txrx.c
740

Yes, thanks!

sys/dev/e1000/em_txrx.c
740

fwiw, the comment might have been worth keeping.

sys/dev/e1000/em_txrx.c
740

Yeah, seems like it

kbowling added a subscriber: grehan.
kbowling added inline comments.
sys/dev/e1000/em_txrx.c
740

I'm indifferent to it, I'd want to go add it back to igb and igc if it stays here. @grehan thoughts?

This revision is now accepted and ready to land.Aug 9 2021, 9:22 PM