Page MenuHomeFreeBSD

linux(4): Fix loadable modules after r367395
ClosedPublic

Authored by cem on Nov 6 2020, 5:35 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, May 11, 5:59 PM
Unknown Object (File)
Mon, May 11, 11:19 AM
Unknown Object (File)
Sun, May 10, 11:12 PM
Unknown Object (File)
Sun, May 10, 6:48 AM
Unknown Object (File)
Fri, Apr 24, 9:15 PM
Unknown Object (File)
Sun, Apr 19, 3:41 PM
Unknown Object (File)
Sun, Apr 19, 3:40 PM
Unknown Object (File)
Apr 9 2026, 3:40 PM
Subscribers

Details

Summary

Move dtrace SDT definitions into linux_common module code. Also, build
linux_dummy.c into the linux_common kld -- we don't need separate
versions of these stubs for 32- and 64-bit emulation.

PR: 250897
Reported by: several
X-MFC-With: r367395

Test Plan

Tinderbox i386/amd64/arm64 passes.

Loaded GENERIC in my test VM and loaded all three of linux.ko, linux_common.ko,
and linux64.ko without issue. Ran chrome.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

cem requested review of this revision.Nov 6 2020, 5:35 PM
cem created this revision.

I don't quite understand the problem, nor the fix, but if it works - it works :-)

Move MI dummy stubs and SDT definition into linux_common.

This tinderboxes and both modules load without issue on amd64. I was able to run a 64-bit binary fine.

sys/compat/linux/linux_dummy.c
39 ↗(On Diff #79281)

This comment might be confusing without the context of the removed lines - I would rephrase it to say that we use the same stubs for 32- and 64-bit

sys/compat/linux/linux_dummy.c
39 ↗(On Diff #79281)

I've rewritten this comment somewhat better in what I'll commit, but it's still not especially satisfying. Oh well.

This revision was not accepted when it landed; it landed in state Needs Review.Nov 6 2020, 10:05 PM
This revision was automatically updated to reflect the committed changes.