On 2016/6/2 10:37, Wei Fang wrote:

Hi, Kevin,

On 2016/6/1 22:36, Kevin Groeneveld wrote:

quoted

quoted

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.

The concurrently decrements of host_failed only lead to abnormal
of host_failed, host_busy will be zero after error handler, and
the result may be host_failed > host_busy forever. But in your
case, host_busy > host_failed, so I think it's not the same
case. I'm afraid that this patch can't fix your scsi hang issue.

Something wrong in my words. host_busy may not be zero after error
handler, but the result is true that missing decrement of host_failed
may lead to host_failed > host_busy.

Thanks,
Wei