Re: [PATCH] scsi-misc-2.5 remove scsi_device list_lock

[Jens Axboe <axboe@suse.de>]

On Fri, Apr 25 2003, Christoph Hellwig wrote:
> > -		spin_lock_irqsave(&d->list_lock, flags);
> > +		spin_lock_irqsave(&d->sdev_lock, flags);
> >  		list_for_each_entry(cmd, &d->cmd_list, list) {
> >  			if(cmd->serial_number == 0){
> 
> <snip>
> 
> > -	spin_lock_irqsave(&sdev->list_lock, flags);
> > +	spin_lock_irqsave(sdev->request_queue->queue_lock, flags);
> >  	list_for_each_entry(scmd, &sdev->cmd_list, list) {
> 
> 
> Oh, please, please use the lock consistantly.  Yes, I know that the
> two are the same currently, but it might not be obvious to every reader
> and queue_lock might change to something else in the future.

Again... Besides, this alias _must_ only be used by the block layer.
Every other driver should use the real one.

-- 
Jens Axboe