Re: 2.5.47bk2 + current modutils == broken hotplug

[David Brownell <david-b@pacbell.net>]

This is about another of the limitations/flaws of the current
module framework, which I'm hoping the new one could fix...


>>Is it true that the infrastructure newly in place can easily be
>>made to provide (from user-space) the policy of "driver remains
>>loaded until the devices it's bound to are all unplugged"?
> 
> 
> That should always be true, unless I'm missing something.  What kind
> of devices?

Any device whose driver gets modprobed through hotplug ... it'd
be good to have the module unload automatically when the hardware
is gone.  The pcmcia tools do this already, with a more limited
problem domain, by tracking device-to-driver associations in the
kernel.  But other hotpluggable frameworks can't do it today, and
it seems to be the module framework which gets in the way.


Consider two instances of such a device.  Hotplug one, the driver
is loaded, refcount zero since it's not opened.  Then rmmod on
unplug would be appropriate:  the hardware is gone.

But instead of unplugging it, plug in a second.  Now rmmod is no
longer an appropriate default policy on unplug, even though the
module "refcount" is still zero, since the user could still try
access the other device.  (Maybe they were plugging in devices
until they found the one with the data they were after, and just
hadn't looked at that the other one yet.)


It's a case where the current modprobe/rmmod scheme is counting
the wrong kinds of things.  Changing module refcounts to track the
device bindings would be a "hard" policy, interfering with driver
developer and sysadmins (gotta reboot or unplug to change drivers).
Tracking such device-to-driver bindings in userland is fragile.
Today's workaround is not to rmmod ... an undesirable answer.

So I keep thinking the right answer is to have a separate count
to track hardware bindings, or at any rate some "soft" policy
hook is needed to prevent rmmod in such cases.  Sysadmins and
driver developers would be able to override the policy, but
Joe Tux-user would just stick to the default.

- Dave


>>That'd be a user-friendly policy, but we'd still need to handle
>>today's developer-oriented "sysadmin can always remove module"
>>policy.  (Me, I'd run with the "user friendly" policy except
>>when hacking a driver.  Then I'd debug/rmmod/update/modprobe.)
> 
> 
> rmmod -f is about as unfriendly as you can get, really 8)
> 
> Cheers,
> Rusty.
> --
>   Anyone who quotes me in their sig is an idiot. -- Rusty Russell.
>