Page MenuHomeFreeBSD

Remove extra kern_kldload from VBoxNetFlt-freebsd.c

Authored by madpilot on Jun 11 2021, 1:35 PM.
Referenced Files
Tue, May 30, 10:00 AM
Unknown Object (File)
Wed, May 24, 2:10 AM
Unknown Object (File)
Mon, May 15, 5:19 AM
Unknown Object (File)
Thu, May 4, 4:43 PM
Unknown Object (File)
Apr 24 2023, 3:39 PM
Unknown Object (File)
Apr 20 2023, 2:51 AM
Unknown Object (File)
Apr 14 2023, 5:26 AM
Unknown Object (File)
Mar 16 2023, 1:13 AM



As reported in bug 256505 since base commit e266a0f7f001 performing kern_kldload while loading a kernel module is not allowed.

The VBoxNetFlt module does this to load ng_ether. Historically ng_ether was missing MODULE_VERSION information and could not be properly depended upon.

This has been fixed in base r238844 more than ten years ago though, so we can unconditionally use a proper MODULE_DEPEND.

Test Plan

Compiles fine in poudriere with all supported releases, and works as expected on head.

Diff Detail

R11 FreeBSD ports repository
Lint Not Applicable
Tests Not Applicable

Event Timeline

Forgot to mention, I renamed the patch file to the new naming convention.

madpilot edited the summary of this revision. (Show Details)
madpilot edited the summary of this revision. (Show Details)
madpilot edited the test plan for this revision. (Show Details)

I also plan to merge this to quarterly once committed, I'm waiting for some feedback on the bug report to make sure it works fine.

This revision is now accepted and ready to land.Jun 14 2021, 6:54 PM