Differential D30074

ixgbe: Clean up ix_txrx
ClosedPublic
Actions

Authored by kbowling on May 1 2021, 11:43 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Apr 9, 3:00 AM

	Unknown Object (File)
	Mar 17 2024, 11:22 AM

	Unknown Object (File)
	Mar 17 2024, 11:22 AM

	Unknown Object (File)
	Mar 17 2024, 11:22 AM

	Unknown Object (File)
	Mar 17 2024, 11:22 AM

	Unknown Object (File)
	Mar 17 2024, 11:09 AM

	Unknown Object (File)
	Feb 12 2024, 4:27 AM

	Unknown Object (File)
	Dec 23 2023, 12:54 AM

View All 32 Files

Subscribers

Details

Reviewers

erj
krzysztof.galazka_intel.com
gallatin
grehan

Group Reviewers

Intel Networking

Commits

rGfd350dd7ff94: ixgbe: Clean up ix_txrx
rGfe6803c18e81: ixgbe: Clean up ix_txrx
rG51e46835e10a: ixgbe: Clean up ix_txrx

Summary

The intention here is to reduce differences with D30072. The main functional change is logical simplification in ixgbe_rx_checksum.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

kbowling created this revision.May 1 2021, 11:43 PM

Herald added a subscriber: ae. · View Herald TranscriptMay 1 2021, 11:43 PM

kbowling requested review of this revision.May 1 2021, 11:43 PM

Same comments regarding __predict() as in the e1000 review.

sys/dev/ixgbe/ix_txrx.c
438	I thought that intel shipped this as part of a backport to 11.x? I assume removing this will cause them unneeded pain,

kbowling marked an inline comment as done.May 3 2021, 9:53 PM

kbowling added inline comments.

sys/dev/ixgbe/ix_txrx.c
438	It's possible, however this would have been some point in 11-CURRENT. Removing the ifdef should be safe for 11-RELEASE et al https://cgit.freebsd.org/src/commit/sys/sys/param.h?h=stable/11&id=10204535afded7f13a48badf29bcc1a33dc3c132

Thanks for catching those mistakes. Can you take a look, I'm happy with this at the moment.

gallatin accepted this revision.May 10 2021, 3:03 PM

This revision is now accepted and ready to land.May 10 2021, 3:03 PM

Closed by commit rG51e46835e10a: ixgbe: Clean up ix_txrx (authored by Kevin Bowling <kbowling@FreeBSD.org>). · Explain WhyJul 18 2021, 6:26 AM

This revision was automatically updated to reflect the committed changes.

Kevin Bowling <kbowling@FreeBSD.org> added a commit: rG51e46835e10a: ixgbe: Clean up ix_txrx.

Kevin Bowling <kbowling@FreeBSD.org> mentioned this in rG9fd0cda92d33: e1000: Add missing branch prediction.Jul 20 2021, 7:23 AM

Kevin Bowling <kbowling@FreeBSD.org> added a commit: rGfe6803c18e81: ixgbe: Clean up ix_txrx.Jul 30 2021, 12:31 AM

Kevin Bowling <kbowling@FreeBSD.org> added a commit: rGfd350dd7ff94: ixgbe: Clean up ix_txrx.Jul 30 2021, 12:42 AM

Kevin Bowling <kbowling@FreeBSD.org> mentioned this in rGb91bb9b07f75: e1000: Add missing branch prediction.Aug 3 2021, 2:05 AM

Kevin Bowling <kbowling@FreeBSD.org> mentioned this in rGa1b9408f97d1: e1000: Add missing branch prediction.Aug 3 2021, 2:12 AM

Revision Contents
Changeset List

Path

Size

sys/

dev/

ixgbe/

87 lines

Diff 88929

sys/dev/ixgbe/ix_txrx.c

Loading...