[PATCH] Dynamic kernel environment support (kgetenv() and friends), ACPI quirks

[Thomas E. Spanjaard]

Matthew Dillon wrote:
    * sysctl_kenv() (which you have #ifdef notyet'd so the code is not
      compiled in yet)... you are missing a spin_unlock.  heh.  yah yah,
      and probably ten other things too.
Yeah, I didn't really put any effort in making that one actually work 
yet. I could just leave that out entirely, to implement at a later time. 
Stale dead code is worse than nonexisting code :).

    * kenv_uninit() is holding a spin lock across a kfree(), which is
      not legal.  Of course, insofar as I can tell, kenv_uninit() will
      never run since it is a SYSUNINIT in a mandatry kernel module,
      so you could probably remove the whole routine and remove the
      SYSUNINIT :-).
Yeah, I should just set kenv_isdynamic=0 right after locking, then 
unlocking again, which would be sufficient. The theory for the SYSUNINIT 
is that, in theory, one could make the dynamic stuff a kmod and keep 
only the static stuff in the kernel; the kfooenv functions would need 
significant changes though and the usefulness is approximately zero :). 
I'll just remove that stuff.

Cheers,
--
        Thomas E. Spanjaard
        tgen at netphreax.net
Attachment:
signature.asc
-------------- next part --------------
A non-text attachment was scrubbed...
Name: pgp00008.pgp
Type: application/octet-stream
Size: 186 bytes
Desc: "Description: OpenPGP digital signature"
URL: <http://lists.dragonflybsd.org/pipermail/kernel/attachments/20070113/8fa969c2/attachment-0016.obj>