Re: IDE locking problem

[Manfred Spraul <manfred@colorfullife.com>]

>
>
>Then, IDE could do something like:
>
> - set dead flag
> - wait for all pending requests to drain (easy: insert a barrier
>   in the queue and wait on it, with a hack for the barrier insertion
>   to bypass the dead flag... ugh... maybe a blk_terminate_queue()
>   doing all that would be helpful ?)
> - unregister blkdev
> - then tear down the queue (leaving the "empty" queue with the dead
>   flag set, not just memset(...,0,...), so that any bozo keeping a
>   reference to it will be rejected trying to insert request instead
>   of trying to tap an uninitalized queue object
>
>What do you think ?
>  
>
The last step is bad - sooner or later the queue will be kfreed, and if 
there are bozos around that still have references, they would access 
random memory. It must be guaranteed that all references expired before 
the tear down begins. Just leaving a dead flag set is not sufficient.

--
    Manfred