From mboxrd@z Thu Jan 1 00:00:00 1970 From: James Bottomley Subject: Re: [PATCH v2 1/2] scsi: fix race between simultaneous decrements of ->host_failed Date: Wed, 01 Jun 2016 11:38:54 -0400 Message-ID: <1464795534.23285.4.camel@linux.vnet.ibm.com> References: <1464683898-9877-1-git-send-email-fangwei1@huawei.com> <38efc40d-fec9-3278-abb8-2a78c4146a8c@sandisk.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <38efc40d-fec9-3278-abb8-2a78c4146a8c@sandisk.com> Sender: linux-scsi-owner@vger.kernel.org To: Bart Van Assche , Kevin Groeneveld , Wei Fang , "tj@kernel.org" , "martin.petersen@oracle.com" , "corbet@lwn.net" Cc: "hch@infradead.org" , "dan.j.williams@intel.com" , "linux-ide@vger.kernel.org" , "linux-scsi@vger.kernel.org" , "linux-doc@vger.kernel.org" List-Id: linux-ide@vger.kernel.org On Wed, 2016-06-01 at 08:29 -0700, Bart Van Assche wrote: > On 06/01/2016 07:36 AM, Kevin Groeneveld wrote: > > > Subject: [PATCH v2 1/2] scsi: fix race between simultaneous > > > decrements of ->host_failed > > > > I wonder if this could be related to > > http://www.spinics.net/lists/linux-scsi/msg86808.html? > > > > I never did get to the bottom of that. If I have time I hope to > > retest my scsi hang issue with this patch. > > I've never seen that hang with the ib_srp driver (which is a SCSI > LLD). That probably means that the SATA code uses the SCSI error > handler in another way than other SCSI LLDs. It's the ata_sas error handler, which is split between libsas and libata. It does one thread of execution per ata port when the strategy handler is invoked. This behaviour is pretty much unique at the moment. James