Page MenuHomeFreeBSD
Differential D32131

ixgbe: Rename 'struct adapter' to 'struct ixgbe_softc'
ClosedPublic
Actions

Authored by kbowling on Sep 25 2021, 1:44 AM.
Tags
None
Referenced Files
F72492320: D32131.id95737.diff
Fri, Dec 8, 10:34 PM
F72465012: D32131.id95737.diff
Fri, Dec 8, 10:21 AM
F72461472: D32131.id95737.diff
Fri, Dec 8, 9:09 AM
Unknown Object (File)
Fri, Dec 8, 3:30 AM
Unknown Object (File)
Fri, Dec 8, 3:30 AM
Unknown Object (File)
Fri, Dec 8, 3:30 AM
Unknown Object (File)
Fri, Dec 8, 3:06 AM
Unknown Object (File)
Mon, Dec 4, 4:26 AM
View All 35 Files
Subscribers
ae

Details

Reviewers
jhb
markj
imp
Group Reviewers
Intel Networking
Commits
rG2b81559f4fd6: ixgbe: whitespace cleanup pass
rG90e279aecd64: ixgbe: Rename 'struct adapter' to 'struct ixgbe_softc'
rG456b1d96e8c4: ixgbe: whitespace cleanup pass
rG4418b03c6759: ixgbe: Rename 'struct adapter' to 'struct ixgbe_softc'
rG15d077995bd2: ixgbe: whitespace cleanup pass
rGb1d5caf3c750: ixgbe: Rename 'struct adapter' to 'struct ixgbe_softc'
Summary

Rename the 'struct adapter' to 'struct ixgbe_softc' to avoid type ambiguity in things like kgdb.

Test Plan

Build and run
Visual inspection

Diff Detail

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

Event Timeline

kbowling created this revision.Sep 25 2021, 1:44 AM
Herald added a subscriber: ae. · View Herald TranscriptSep 25 2021, 1:44 AM
kbowling requested review of this revision.Sep 25 2021, 1:44 AM
kbowling added a reviewer: Intel Networking.
kbowling planned changes to this revision.Sep 25 2021, 1:48 AM

Need to fix ixv

kbowling updated this revision to Diff 95696.Sep 25 2021, 2:02 AM
kbowling updated this revision to Diff 95697.Sep 25 2021, 2:06 AM
markj added inline comments.Sep 25 2021, 2:30 PM
sys/dev/ixgbe/ixgbe.h
332

All the indentation here looks strange now. Are you editing with tabstop=4?

kbowling updated this revision to Diff 95737.Sep 25 2021, 6:33 PM

ts=8

kbowling marked an inline comment as done.Sep 25 2021, 6:34 PM
kbowling added inline comments.
sys/dev/ixgbe/ixgbe.h
332

Yes good call!

markj accepted this revision.Sep 26 2021, 3:53 PM

The whitespace changes should be committed separately.

This revision is now accepted and ready to land.Sep 26 2021, 3:53 PM
Closed by commit rGb1d5caf3c750: ixgbe: Rename 'struct adapter' to 'struct ixgbe_softc' (authored by Kevin Bowling <kbowling@FreeBSD.org>). · Explain WhySep 26 2021, 6:19 PM
This revision was automatically updated to reflect the committed changes.
Kevin Bowling <kbowling@FreeBSD.org> added a commit: rGb1d5caf3c750: ixgbe: Rename 'struct adapter' to 'struct ixgbe_softc'.
Kevin Bowling <kbowling@FreeBSD.org> added a commit: rG15d077995bd2: ixgbe: whitespace cleanup pass.Sep 26 2021, 6:32 PM
Kevin Bowling <kbowling@FreeBSD.org> added a commit: rG4418b03c6759: ixgbe: Rename 'struct adapter' to 'struct ixgbe_softc'.Oct 3 2021, 1:33 AM
Kevin Bowling <kbowling@FreeBSD.org> added a commit: rG456b1d96e8c4: ixgbe: whitespace cleanup pass.
Kevin Bowling <kbowling@FreeBSD.org> added a commit: rG90e279aecd64: ixgbe: Rename 'struct adapter' to 'struct ixgbe_softc'.Oct 3 2021, 5:04 AM
Kevin Bowling <kbowling@FreeBSD.org> added a commit: rG2b81559f4fd6: ixgbe: whitespace cleanup pass.

Revision Contents
Changeset List

PathSize
sys/
dev/
ixgbe/
186 lines
26 lines
1263 lines
602 lines
296 lines
47 lines
22 lines
2 lines
2 lines
20 lines
8 lines

Diff 95754

View Options

sys/dev/ixgbe/if_bypass.c

Loading...
View Options

sys/dev/ixgbe/if_fdir.c

Loading...
View Options

sys/dev/ixgbe/if_ix.c

Loading...
View Options

sys/dev/ixgbe/if_ixv.c

Loading...
View Options

sys/dev/ixgbe/if_sriov.c

Loading...
View Options

sys/dev/ixgbe/ix_txrx.c

Loading...
View Options

sys/dev/ixgbe/ixgbe.h

Loading...
View Options

sys/dev/ixgbe/ixgbe_bypass.h

Loading...
View Options

sys/dev/ixgbe/ixgbe_fdir.h

Loading...
View Options

sys/dev/ixgbe/ixgbe_osdep.c

Loading...
View Options

sys/dev/ixgbe/ixgbe_sriov.h

Loading...