Page MenuHomeFreeBSD
Differential D59427

ipfw: cleanup !FreeBSD and !_KERNEL code
ClosedPublic
Actions

Authored by glebius on Sep 4 2026, 9:57 PM.
Tags
None
Referenced Files
F175144657: D59427.diff
Thu, Oct 8, 3:22 PM
Unknown Object (File)
Tue, Oct 6, 12:59 PM
Unknown Object (File)
Sun, Oct 4, 7:26 PM
Unknown Object (File)
Thu, Oct 1, 1:37 PM
Unknown Object (File)
Thu, Oct 1, 4:24 AM
Unknown Object (File)
Tue, Sep 29, 8:14 PM
Unknown Object (File)
Tue, Sep 29, 7:04 AM
Unknown Object (File)
Sun, Sep 27, 11:40 PM
View All 51 Files
Subscribers
donner
imp
melifaro
vegeta_tuxpowered.net

Details

Reviewers
ae
lytboris_gmail.com
Group Reviewers
network
Commits
rG58031cbd54b8: ipfw: cleanup !FreeBSD and !_KERNEL code
Summary

These are leftovers from last import from Luigi Rizzo in 2013, that were
never compiled checked since and lots of changes have had happened since
and more changes to come. Let's not pretend ipfw is buildable on
something else than FreeBSD nor it can be compiled for userland.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 76593
Build 73476: arc lint + arc unit

Event Timeline

glebius created this revision.Sep 4 2026, 9:57 PM
Herald added subscribers: vegeta_tuxpowered.net, donner, melifaro, imp. · View Herald TranscriptSep 4 2026, 9:57 PM
glebius requested review of this revision.Sep 4 2026, 9:57 PM
Harbormaster completed remote builds in B76539: Diff 185873.Sep 4 2026, 9:57 PM
lytboris_gmail.com added a comment.Sep 6 2026, 2:11 PM

There are two USERSPACE ifdefs in dummynet code as well:

ip_dn_io.c:#ifdef USERSPACE
ip_dn_private.h:#ifdef USERSPACE

Should we remove them as well?

glebius updated this revision to Diff 186008.Sep 6 2026, 4:37 PM
  • Cover dummynet, too.
Harbormaster completed remote builds in B76593: Diff 186008.Sep 6 2026, 4:37 PM
lytboris_gmail.com accepted this revision.Sep 6 2026, 5:10 PM

LGTM

This revision is now accepted and ready to land.Sep 6 2026, 5:10 PM
Closed by commit rG58031cbd54b8: ipfw: cleanup !FreeBSD and !_KERNEL code (authored by glebius). · Explain WhyFri, Sep 18, 10:51 PM
This revision was automatically updated to reflect the committed changes.
glebius added a commit: rG58031cbd54b8: ipfw: cleanup !FreeBSD and !_KERNEL code.

Revision Contents
Changeset List

PathSize
sys/
netpfil/
ipfw/
20 lines
28 lines
28 lines
39 lines
5 lines

Diff 186008

View Options

sys/netpfil/ipfw/dn_sched_qfq.c

Loading...
View Options

sys/netpfil/ipfw/ip_dn_io.c

Loading...
View Options

sys/netpfil/ipfw/ip_dn_private.h

Loading...
View Options

sys/netpfil/ipfw/ip_fw2.c

Loading...
View Options

sys/netpfil/ipfw/ip_fw_log.c

Loading...