lkml.org 
[lkml]   [2003]   [Mar]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: PCI driver module unload race?

> Somehow you have to protect dev->driver, you cannot resolve the driver
> pointer without holding the bus lock or holding a reference. If you don't
> get a reference, any access via this pointer must be done under the bus
> lock.

struct device_driver has a separate reference count than the module
reference count. In the driver core, this reference count is used. It is
acquired by taking the bus's lock.

The module refcount isn't used, because it resides in the driver, so we
have to guarantee we can dereference the pointer before we deref the
pointer to increment the refcount. It's a bloody mess, and we work around
it.

The driver has an unload_sem that is locked until the driver's refcount
goes to 0. When it does, it's unlocked. A driver_unregister() call will
try and take this semaphore while unregistering, meaning it will block
until outstanding references go away.

I don't particularly love it, but it's simple enough and it works.
Ideally, we'd have one reference count and this wouldn't be an issue.
However, with the evolution of the driver core and the module core in 2.5,
these details haven't had a chance to be worked. I hope in 2.7, we can
achieve more unification between the two.

> > Yeah, I still think there are some nasty issues with regards to being in
> > a sysfs directory, with a open file handle, and the module is removed.
> > But I haven't checked stuff like that in a while.
> >
> > CONFIG_MODULE_UNLOAD, just say no.
>
> That's certainly an option, but I'm afraid not too many people will do
> this.

Greg, and Rusty, are right. Dealing with this is a PITA, and I think will
always be. I'm willing to take the Nancy Reagan platform, too.


-pat

-
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/

\
 
 \ /
  Last update: 2005-03-22 13:33    [W:0.077 / U:0.976 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site