Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S269184AbUINSnJ (ORCPT ); Tue, 14 Sep 2004 14:43:09 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S269695AbUINSiT (ORCPT ); Tue, 14 Sep 2004 14:38:19 -0400 Received: from [69.28.190.101] ([69.28.190.101]:48863 "EHLO havoc.gtf.org") by vger.kernel.org with ESMTP id S269218AbUINSfD (ORCPT ); Tue, 14 Sep 2004 14:35:03 -0400 Date: Tue, 14 Sep 2004 14:35:02 -0400 From: Jeff Garzik To: James Bottomley Cc: Mark Lord , Linux Kernel , SCSI Mailing List , "linux-ide@vger.kernel.org" , Alan Cox Subject: Re: [PATCH] New QStor SATA/RAID Driver for 2.6.9-rc2 Message-ID: <20040914183502.GA23277@havoc.gtf.org> References: <41471163.10709@rtr.ca> <414723B0.1090600@pobox.com> <1095186343.2008.29.camel@mulgrave> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1095186343.2008.29.camel@mulgrave> User-Agent: Mutt/1.4.1i Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 818 Lines: 22 On Tue, Sep 14, 2004 at 02:25:35PM -0400, James Bottomley wrote: > Sleeping in the eh paths is fine (as long as you drop the locks that the > EH thread has uselessly taken for you). Indeed it's often required > since the return is supposed to tell the eh thread whether the action > was successful or not. I'm not sure this true for all arches? The lock is taken in the SCSI layer with spin_lock_irqsave(), but the low-level driver cannot perform the exact opposite, spin_unlock_irqrestore(). The best they can do is spin_lock_irq(), which isnt 100% the same. Jeff - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/