Page MenuHomeFreeBSD
Differential D37817

Mechanically convert sfxge(4) to DrvAPI
ClosedPublic
Actions

Authored by jhibbits on Dec 22 2022, 3:30 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, May 11, 6:35 AM
Unknown Object (File)
Thu, May 7, 4:50 AM
Unknown Object (File)
Sat, May 2, 11:51 PM
Unknown Object (File)
Sat, May 2, 1:22 PM
Unknown Object (File)
Sun, Apr 19, 5:35 PM
Unknown Object (File)
Sat, Apr 18, 10:22 AM
Unknown Object (File)
Fri, Apr 17, 9:37 PM
Unknown Object (File)
Apr 11 2026, 10:30 PM
View All Files
Subscribers
imp
zlei

Details

Reviewers
arybchik
zlei
Group Reviewers
network
Commits
rG04abf87b2a56: Mechanically convert sfxge(4) to IfAPI

Diff Detail

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

Event Timeline

jhibbits created this revision.Dec 22 2022, 3:30 PM
Herald added a subscriber: imp. · View Herald TranscriptDec 22 2022, 3:30 PM
jhibbits requested review of this revision.Dec 22 2022, 3:30 PM
Harbormaster completed remote builds in B48829: Diff 114443.Dec 22 2022, 3:30 PM
jhibbits added a parent revision: D37816: Mechanically convert sge(4) to DrvAPI.Dec 22 2022, 3:30 PM
jhibbits added a child revision: D37818: Mechanically convert sk(4) to DrvAPI.
jhibbits added reviewers: glebius, melifaro.Jan 10 2023, 9:20 PM
jhibbits edited reviewers, added: network, arybchik; removed: glebius, melifaro.Feb 6 2023, 7:59 PM
zlei accepted this revision.Feb 7 2023, 2:23 AM
zlei added a subscriber: zlei.

Looks good to me.

sys/dev/sfxge/sfxge.h
273

Small nits, align member ifnet

sys/dev/sfxge/sfxge_port.c
519–520

Use if_getlladdr()

This revision is now accepted and ready to land.Feb 7 2023, 2:23 AM
arybchik accepted this revision.Feb 7 2023, 7:38 AM

Thanks, LGTM

Closed by commit rG04abf87b2a56: Mechanically convert sfxge(4) to IfAPI (authored by jhibbits). · Explain WhyFeb 7 2023, 7:19 PM
This revision was automatically updated to reflect the committed changes.
jhibbits added a commit: rG04abf87b2a56: Mechanically convert sfxge(4) to IfAPI.

Revision Contents
Changeset List

PathSize
sys/
dev/
sfxge/
6 lines
95 lines
29 lines
8 lines
4 lines
10 lines

Diff 116727

View Options

sys/dev/sfxge/sfxge.h

Loading...
View Options

sys/dev/sfxge/sfxge.c

Loading...
View Options

sys/dev/sfxge/sfxge_port.c

Loading...
View Options

sys/dev/sfxge/sfxge_rx.c

Loading...
View Options

sys/dev/sfxge/sfxge_tx.h

Loading...
View Options

sys/dev/sfxge/sfxge_tx.c

Loading...