Differential D56743
net/bone-kmods: Add port Authored by dave_freedave.net on Apr 30 2026, 11:44 AM. Tags None Referenced Files
Details
These are netgraph kernel modules I maintain outside src. I saw 292993 go by recently. Not the first issue with vnet(9) + netgraph(4) with if_vmove. And this port has the wormhole which removes the need for if_vmove entirely would be another solution for users. I run these nodes and test them on stable/14, stable/15, and CURRENT. However they require patches for stable/14 and the port requires at least stable/15. Rather than repeat myself you can look at the old src review: D50244
Diff Detail
Event TimelineComment Actions Dave, I'm very sorry for not looking at your earlier submissions. Trust me, this is not a reluctance to add them to the src, but just a lack of time. Would you like me to review your nodes on github or would you prefer to refresh the phabricator revisions where you suggested the nodes to add to src? I will try to find some time. Comment Actions I tagged so you and zlei for awareness so the next time a vnet + netgraph + if_vmove comes along you have an alternate to point people at. I would love for you to take a look as I value your opinion! But bong-kmods has 2 nodes I never put up for review: ng_pcap(4) and ng_ula4tag(4). And I have 3 more more to make ... I feel like ports are a good place for these to at least start.
Comment Actions "Bag of Netgraph"... That's really clever! I like that a lot. I just want to make sure that you've thought through and are okay with the implications. I'd suggest at least putting its name in the descr so people know it's not just named after drug paraphernalia. Comment Actions
Comment Actions poudriere testport -j FreeBSD15 -p local net/bong-kmods output: @adamw You will notice this in my make.conf from log above: .if ${OSVERSION:U0} > 1600000
net_bong-kmods_SET+=» INVARIANTS
.endifThat is so my kernel builds don't get confused but so that poudriere(8) builds for CURRENT automatically set INVARIANTS because that is the default way CURRENT builds. But any kernel can have INVARIANTS enabled. I didn't see any better option from 286717 to handle this. Comment Actions Closing the loop here: I sent you an email a minute ago letting you know that I've been pretty sick and haven't been paying enough attention to my checklist. I like the descr change you made. I personally wouldn't name stuff designed for corporate environments "bong" (any more than I'd name them "methpipe") BUT the good news is that you're not me and you already did the important thing, which is hear my concern and reflect on it. If you are confident about the name, then let me know and I'll get these committed! If you are on the fence, here are some things that come to mind:
Comment Actions Ok wow, I like "bone" I mean I don't really want to change it on github but I now that seems like a great idea...
Comment Actions I don't know that its worth doing flavors. It would be nice if official FreeBSD16 ports enabled INVARIANTS but not essential. Comment Actions Switch to new mono repo for bone-kmods/bone-utils I like using invariants to make the two flavors of bone-kmods. Not much really changed here. Comment Actions Anything more needed for this and D56744 ? I am running bone-kmods-invariants on my CURRENT, bone-kmods on stable/15, and both have bone-utils. Comment Actions poudriere testport -j FreeBSD15 -p local net/bone-kmods poudriere testport -j FreeBSD15 -p local net/bone-kmods@invariants attaching recent poudriere test builds.
| ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||