Re: smp dead lock of io_request_lock/queue_lock patch

From: Christoph Hellwig
Date: Sat Jan 17 2004 - 14:18:09 EST


On Sat, Jan 17, 2004 at 02:07:33PM -0500, Doug Ledford wrote:
> #ifdef SCSI_HAS_HOST_LOCK
> #if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,0)
> adapter->lock_ptr = &adapter->lock;
> host->lock = &adapter->lock;
> #else
> adapter->lock_ptr = &adapter->lock;
> host->host_lock = &adapter->lock;
> #endif
> #else
> adapter->lock_ptr = &io_request_lock;
> #endif

Still looks wrong for the 2.6 case which should just be;

adapter->lock_ptr = shost->host_lock;

as I just stated in the review for the megaraid update.

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/