Page MenuHomeFreeBSD

linux(4): Get rid of Linuxulator's kernel build options.
ClosedPublic

Authored by dchagin on Jun 13 2021, 2:28 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Dec 4, 3:24 PM
Unknown Object (File)
Wed, Dec 4, 3:24 PM
Unknown Object (File)
Wed, Dec 4, 3:24 PM
Unknown Object (File)
Wed, Dec 4, 3:24 PM
Unknown Object (File)
Wed, Dec 4, 2:58 PM
Unknown Object (File)
Nov 20 2024, 12:20 PM
Unknown Object (File)
Nov 15 2024, 2:06 PM
Unknown Object (File)
Nov 12 2024, 5:10 AM
Subscribers

Details

Summary

Stop confusing people and retire COMPAT_LINUX and COMPAT_LINUX32 kernel
build options. Since we have 64-bit Linux emulation we can't build both
emulators into the kernel. And I don't think it matters, Linux emulation
depends on loadable modules (via rc).

Cut LINPROCFS and LINSYSFS for consistency.

I haven't removed opt_compat.h, maybe we should as there no any COMPAT
option left.

PR: 215061
MFC after: 2 weeks

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

To make sure I understand this: this change is to no longer allow Linuxulator to be built into the kernel, and only allow it to be built as a module?

share/man/man4/tdfx.4
52–56
By loading the
56
modules, you can enable the linux ioctl code for this driver, where the only supported
sys/amd64/conf/NOTES
263–264

"both" what? I think you mean

# To enable Linuxulator support, one must also load linux.ko and tdfx_linux.ko

Is that correct?

sys/i386/conf/NOTES
432
# To enable Linuxulator support, load tdfx and tdfx_linux as modules.

The i386 and amd64 versions of this comment should match; I think this version is clearer.

fixed Ravi's comments. thanks!

To make sure I understand this: this change is to no longer allow Linuxulator to be built into the kernel, and only allow it to be built as a module?

yes, you are right

sys/amd64/conf/NOTES
263–264

sentence correct, but I removed it as amd64 does not support tdfx, and fixed this sentence for i386 NOTES. thanks)

bcr added a subscriber: bcr.

OK for the manpage part of the change. You need to bump the .Dd for this content change when you commit.

This revision is now accepted and ready to land.Jun 15 2021, 8:51 AM
This revision was automatically updated to reflect the committed changes.
dchagin marked an inline comment as done.