Re: [PATCH] PART1: Proposed init & module changes for 2.5

Rusty Russell (rusty@rustcorp.com.au)
Mon, 24 Sep 2001 14:35:54 +1000


In message <4103.1001293289@kao2.melbourne.sgi.com> you write:
> On Mon, 24 Sep 2001 08:42:25 +1000,
> Rusty Russell <rusty@rustcorp.com.au> wrote:
> >In message <20010923124336.D30515@nightmaster.csn.tu-chemnitz.de> you write:
> >> Can the startcall or the initcall still be called after stopcall?
> >
> >No: at this stage the module will have to be reloaded, exactly because
> >of assumptions like zero-initialization.
>
> When we discussed this at linux.conf.au in Sydney, we agreed that we
> could call startfn after stopfn to handle the quiesce unload algorithm.
> That handles the rmmod race without exporting mod use count to
> everything, i.e.
>
> rmmod
> if use count == 0, call stopfn()
> synchronize_kernel()
> if use count == 0, exitfn()
> if use count != 0, startfn()
>
> That catches the race where a second cpu has entered the module but not
> done MOD_INC_USECOUNT yet. The module is now in use but stopfn has
> been called, the best thing to do is accept that rmmod lost the race
> and reinstate the module.

Hi Keith!

Yes, this was my original intention, but the more I thought
about it, the more I wondered if it was worth it. In fact, my patch
adds CONFIG_MODULE_UNLOAD, because most people don't actually need it,
and I don't really care if most modules are not unloadable. Loading
modules makes sense. Unloading them is pretty much an optimization.

The reason I eventually decided to split initialization in
this version anyway, was that so many drivers get it wrong! Things
like registering interrupt handlers before they have set up their
internal state, etc. Splitting it into two functions is merely a
mechanism to get them to think about things a little harder 8)

And honestly, I got distracted by the PARAM stuff 8)

Cheers!
Rusty.

--
Premature optmztion is rt of all evl. --DK
-
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/