Page MenuHomeFreeBSD

dummynet: Does not depend on ipfw
ClosedPublic

Authored by kp on Sep 10 2021, 4:24 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mar 7 2024, 8:01 PM
Unknown Object (File)
Jan 15 2024, 10:51 PM
Unknown Object (File)
Dec 28 2023, 7:09 AM
Unknown Object (File)
Dec 20 2023, 3:11 AM
Unknown Object (File)
Dec 14 2023, 9:43 PM
Unknown Object (File)
Nov 22 2023, 6:08 AM
Unknown Object (File)
Nov 13 2023, 7:24 AM
Unknown Object (File)
Nov 11 2023, 12:32 PM

Details

Summary

Allow the dummynet module to be loaded without ipfw, as a first step
towards making pf use it for packet scheduling.

MFC after: 2 weeks
Sponsored by: Rubicon Communications, LLC ("Netgate")

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 41461
Build 38350: arc lint + arc unit

Event Timeline

kp requested review of this revision.Sep 10 2021, 4:24 PM

May this cause an unwanted effect on automatic module loading?
So may somebody depend on this dependency in the local setup?

This revision is now accepted and ready to land.Sep 10 2021, 6:33 PM

May this cause an unwanted effect on automatic module loading?
So may somebody depend on this dependency in the local setup?

Yes, that's a concern. Especially for merging it back to stable/13 and stable/12.
I'm planning to MFC pf/dummynet , but perhaps users in 12 and 13 are just going to have to live with the dummynet module autoloading ipfw. It's something to think about.

In D31903#720341, @kp wrote:

May this cause an unwanted effect on automatic module loading?
So may somebody depend on this dependency in the local setup?

Yes, that's a concern. Especially for merging it back to stable/13 and stable/12.
I'm planning to MFC pf/dummynet , but perhaps users in 12 and 13 are just going to have to live with the dummynet module autoloading ipfw. It's something to think about.

With default to deny it might be not exactly the desired outcome. Do you think that a separate module w/ dummynet-for-pf (for 12/13) is too much of a hassle? Especially given ae@‘s plan to change dummynet for ipfw?

Do you think that a separate module w/ dummynet-for-pf (for 12/13) is too much of a hassle?

That's an interesting idea. I'll give that a try. That may turn out to be the easiest fix.

Especially given ae@‘s plan to change dummynet for ipfw?

I'm hoping that we can move pf along to the new dummynet code at the same time as ipfw. Happily the interface between firewall and dummynet is pretty simple, so I don't really expect that to be a lot of work.

In D31903#720423, @kp wrote:

Do you think that a separate module w/ dummynet-for-pf (for 12/13) is too much of a hassle?

That's an interesting idea. I'll give that a try. That may turn out to be the easiest fix.

https://reviews.freebsd.org/D32079

It introduced 'pfdummynet.ko', which depends on pf like dummynet.ko depends on ipfw. That's intended for stable/13 and stable/12. I'd do this for main, but not MFC it.
pfdummynet and dummynet can't be loaded at the same time, but that seems like a minor issue at most. Users will have to pick between doing dummynet with ipfw or doing it with pf.

This revision was automatically updated to reflect the committed changes.