[PATCH] PPC64 iSeries: don't share request queues in viocd

Nick Piggin nickpiggin at yahoo.com.au
Mon Nov 15 22:44:33 EST 2004


Christoph Hellwig wrote:
> On Mon, Nov 15, 2004 at 04:53:57PM +1100, Stephen Rothwell wrote:
> 
>>Hi Andrew,
>>
>>This patch fixes the virtual cdrom driver to not share a single request
>>queue.  Sharing the queue causes an oops if you remove the module and more
>>than one cdrom exists.
> 
> 
> Maybe you should fix that underlying bug?  Queues are supposed to be
> shareable.
> 

I think shared queues are actually quite fundamentally broken at the
moment (as pointed out to me by Al). It stems from the refcounting /
conceptual relationship between a gendisk and a queue (I think - been
a while since I looked at the code).

I had something which just about fixed it up except that I couldn't
work out an appropriate place and name for the "queue" in the sysfs
hierarcy (IIRC I just had it as a sequentially increasing number, in
/sys/block/).

It is a relationship that I don't think sysfs can capture very well:
queues are shared between multiple other objects, but they have no
meaning outside the context of one of these objects.



More information about the Linuxppc64-dev mailing list