On Mon, 2002-03-04 at 17:12, Craig Christophel wrote:
> Also if there is only infrequent useage of the list to change the list, 
> perhaps you should use a rwlock_t instead.   It's just as light as a spinlock 
> but has the advantage of allowing multiple readers.

How about rwsem? See attached.

I think I did this properly, but there may be a few boo-boos. 

particularly:
o pci_proc_attach_device()
o pdev_enable_device()
o pdev_fixup_irq()

look like reader safe to me, but I could be wrong. 

and 
o pci_fixup_device()
o pci_announce_device

are writers, as I'm not sure what drv->probe or fixup->hook are doing.


thanks,
-john