LWN.net Logo

Re: [PATCH] Remove module reference counting.

From:  Rusty Russell <rusty@rustcorp.com.au>
To:  Linus Torvalds <torvalds@osdl.org>
Subject:  Re: [PATCH] Remove module reference counting.
Date:  Mon, 28 Jul 2003 05:34:36 +1000
Cc:  Stephen Hemminger <shemminger@osdl.org>, "David S. Miller" <davem@redhat.com>, arjanv@redhat.com, Greg KH <greg@kroah.com>, Kernel Mailing List <linux-kernel@vger.kernel.org>

In message <Pine.LNX.4.44.0307261230110.1841-100000@home.osdl.org> you write:
> 
> First off - we're not changing fundamental module stuff any more.

OK.  Who are you and what have you done with the real Linus?

I guess it's back to fixing up reference counting in the rest of the
kernel.  It's not hard, it's just not done. 8(

> > No, it would just leak memory.  Not really a concern for developers.
> > It's fairly trivial to hack up a backdoor "remove all freed modules
> > and be damned" thing for developers if there's real demand.
> 
> It's not just a developer thing. At least installers etc used to do some 
> device probing by loading modules and depending on the result.

A similar case.  At this point you don't have random users trying to
access things, so freeing is actually fairly safe (modulo other
bugs).

The kudzu one and Alan's USB firmware example bother me more: they
load then unload modules currently?  Since modern device drivers are
not supposed to fail to load just because there isn't currently
hardware, I guess they'd have to.

Oh well,
Rusty.
--
  Anyone who quotes me in their sig is an idiot. -- Rusty Russell.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


(Log in to post comments)

Copyright © 2003, Eklektix, Inc.
Comments and public postings are copyrighted by their creators.
Linux is a registered trademark of Linus Torvalds
Powered by Rackspace Managed Hosting.