aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorEd Maste <emaste@FreeBSD.org>2025-11-25 17:12:36 +0000
committerEd Maste <emaste@FreeBSD.org>2025-11-25 17:14:10 +0000
commitfad4c92b78a123f87195173ac118655fa8e325cd (patch)
tree5fe6941c2f715f22f2df6fb0911651d3590c5b06
parent301b8a806f79292ee3324ca938a4d938df1911e8 (diff)
Revert "kernel linker: Disable local sym resolution by default"
I've received a report of a failure from resolving kern_kmq_open. Revert for now as we are currently in stabweek. This reverts commit 9562994a7aacee2baae6ddee1a7b558b48ae39ef. Reported by: mav
-rw-r--r--UPDATING9
-rw-r--r--sys/kern/link_elf.c2
-rw-r--r--sys/kern/link_elf_obj.c2
3 files changed, 2 insertions, 11 deletions
diff --git a/UPDATING b/UPDATING
index 7f0cb44181d3..aaef0e5b4cd3 100644
--- a/UPDATING
+++ b/UPDATING
@@ -27,15 +27,6 @@ NOTE TO PEOPLE WHO THINK THAT FreeBSD 16.x IS SLOW:
world, or to merely disable the most expensive debugging functionality
at runtime, run "ln -s 'abort:false,junk:false' /etc/malloc.conf".)
-20251125:
- The `debug.link_elf_leak_locals` and `debug.link_elf_obj_leak_locals`
- sysctls now default to 0, so the kernel module linker no longer
- performs symbol resolution against local symbols from other modules.
-
- If a kernel module now fails to load because of an unresolved symbol,
- set these sysctls to 1 and file a bug report including details about
- the affected module.
-
20251115:
The FreeBSD-base repository is now defined in /etc/pkg/FreeBSD.conf,
disabled by default. In -CURRENT and -STABLE this points at nightly
diff --git a/sys/kern/link_elf.c b/sys/kern/link_elf.c
index f910fc3d5df5..ebd203858b66 100644
--- a/sys/kern/link_elf.c
+++ b/sys/kern/link_elf.c
@@ -203,7 +203,7 @@ static struct linker_class link_elf_class = {
link_elf_methods, sizeof(struct elf_file)
};
-static bool link_elf_leak_locals = false;
+static bool link_elf_leak_locals = true;
SYSCTL_BOOL(_debug, OID_AUTO, link_elf_leak_locals,
CTLFLAG_RWTUN, &link_elf_leak_locals, 0,
"Allow local symbols to participate in global module symbol resolution");
diff --git a/sys/kern/link_elf_obj.c b/sys/kern/link_elf_obj.c
index 759cd75cba31..a3a53a39bfd6 100644
--- a/sys/kern/link_elf_obj.c
+++ b/sys/kern/link_elf_obj.c
@@ -192,7 +192,7 @@ static struct linker_class link_elf_class = {
link_elf_methods, sizeof(struct elf_file)
};
-static bool link_elf_obj_leak_locals = false;
+static bool link_elf_obj_leak_locals = true;
SYSCTL_BOOL(_debug, OID_AUTO, link_elf_obj_leak_locals,
CTLFLAG_RWTUN, &link_elf_obj_leak_locals, 0,
"Allow local symbols to participate in global module symbol resolution");