Page MenuHomeFreeBSD
Differential D37214

LinuxKPI: ethtool.h add more definitions
ClosedPublic
Actions

Authored by bz on Oct 30 2022, 5:29 PM.
Tags
None
Referenced Files
F86804990: D37214.diff
Tue, Jun 25, 8:22 PM
F86782102: D37214.diff
Tue, Jun 25, 12:29 PM
Unknown Object (File)
Sun, Jun 9, 5:48 AM
Unknown Object (File)
May 14 2024, 8:09 AM
Unknown Object (File)
Mar 19 2024, 2:26 PM
Unknown Object (File)
Dec 23 2023, 1:27 AM
Unknown Object (File)
Dec 12 2023, 7:44 AM
Unknown Object (File)
Nov 7 2023, 1:18 AM
View All 31 Files
Subscribers
emaste
imp

Details

Reviewers
None
Group Reviewers
linuxkpi
Commits
rG2078d0ea1e9f: LinuxKPI: ethtool.h add more definitions
rG1b91eba37b29: LinuxKPI: ethtool.h add more definitions
Summary

While we do not currently use this add the definitions to avoid other
longer-term maintenance problems with drivers.

MFC after: 3 days

Diff Detail

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

Event Timeline

bz created this revision.Oct 30 2022, 5:29 PM
Herald added subscribers: emaste, imp. · View Herald TranscriptOct 30 2022, 5:29 PM
bz requested review of this revision.Oct 30 2022, 5:29 PM
Harbormaster completed remote builds in B48095: Diff 112412.Oct 30 2022, 5:29 PM
bz added a comment.Nov 7 2022, 1:15 PM

Anyone?

bz added a comment.Nov 15 2022, 11:03 PM

If no one reviews I'll commit this the next days.

emaste added inline comments.Nov 16 2022, 5:18 PM
sys/compat/linuxkpi/common/include/linux/ethtool.h
48

these are char[] arrays in Linux so I worry this may introduce some misbehaviour in the future (even if they're not being used in a way that would cause that, today)

bz marked an inline comment as done.Nov 18 2022, 9:03 PM
bz added inline comments.
sys/compat/linuxkpi/common/include/linux/ethtool.h
48

Based on a simple user space program, they all seem to be of size [32]; I'll adjust the file and post an update in a minute

bz updated this revision to Diff 113334.Nov 18 2022, 9:05 PM
bz marked an inline comment as done.

Make struct members fixed length char[] based on imput from @emaste and
a simple user space program using the original header file and LinuxKPI.

Also migrate ETH_GSTRING_LEN into here from if_ether.h.

Harbormaster completed remote builds in B48429: Diff 113334.Nov 18 2022, 9:05 PM
This revision was not accepted when it landed; it landed in state Needs Review.Nov 28 2022, 5:30 PM
Closed by commit rG1b91eba37b29: LinuxKPI: ethtool.h add more definitions (authored by bz). · Explain Why
This revision was automatically updated to reflect the committed changes.
bz added a commit: rG1b91eba37b29: LinuxKPI: ethtool.h add more definitions.
bz added a commit: rG2078d0ea1e9f: LinuxKPI: ethtool.h add more definitions.Jan 18 2023, 4:25 PM

Revision Contents
Changeset List

PathSize
sys/
compat/
linuxkpi/
common/
include/
linux/
20 lines
2 lines

Diff 113570

View Options

sys/compat/linuxkpi/common/include/linux/ethtool.h

Loading...
View Options

sys/compat/linuxkpi/common/include/linux/if_ether.h

Loading...