Date: Sat, 1 Sep 2001 03:36:28 +0200 (CEST) From: Christian Carstensen <cc@devcon.net> To: <current@freebsd.org> Subject: dynamic sysctl problem and hot fix Message-ID: <Pine.LNX.4.33.0109010335010.4823-100000@lilly.subway.devcon.net>
next in thread | raw e-mail | index | archive | help
hi,
i just came across a problem with dynamic sysctls:
when unloading a driver module that used dyn sysctls, my system paniced
with "oid too high". that problem is caused by sysctl_ctx_free() in
kern/kern_sysctl.c, that first deregisters all oids in the list to see if
a error occurs. then, all oids are being reregistered and, if there was no
error, they're finally removed.
during the second phase, sysctl_register_oid(e1->entry) is called with
n := e1->entry->oid_number being the old oid number with n >
CTL_AUTO_START.
that leads to panic("static sysctl too high") in sysctl_register_oid.
one approach might be to initialize the oid_number field to contain the
value OID_AUTO before calling sysctl_regiser_oid, but i'm unsure about the
side effects of doing that in sysctl_ctx_free().
alternatively, the "old" oid number could be reused, the following patch
should do, but it's just a workaround.
best,
christian
--
"Sorry, no defects found. Please try a different search"
[http://www.cisco.com/support/bugtools/bugtool.shtml]
Index: kern_sysctl.c
===================================================================
RCS file: /usr/cvs/src/sys/kern/kern_sysctl.c,v
retrieving revision 1.113
diff -r1.113 kern_sysctl.c
83a84,96
> static struct sysctl_oid *
> sysctl_find_oidnumber(const int number, struct sysctl_oid_list *list)
> {
> struct sysctl_oid *oidp;
>
> SLIST_FOREACH(oidp, list, oid_link) {
> if (oidp->oid_number == number) {
> return (oidp);
> }
> }
> return (NULL);
> }
>
125c138,139
< panic("static sysctl oid too high: %d", oidp->oid_number);
---
> if (sysctl_find_oidnumber(oidp->oid_number, parent))
> panic("static sysctl oid too high: %d", oidp->oid_number);
177c191
< if (error)
---
> if (error)
To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-current" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?Pine.LNX.4.33.0109010335010.4823-100000>