Re: [PATCH] ncr5380: Don't re-enter NCR5380_select() when aborting a command

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Please ignore this patch. It isn't sufficient to fix the problem. I'll 
send another patch that does fix it.

On Tue, 26 Jan 2016, Finn Thain wrote:

> Fixes: 707d62b37fbb ("ncr5380: Fix EH during arbitration and selection")
> Signed-off-by: Finn Thain <fthain@xxxxxxxxxxxxxxxxxxx>
> 
> ---
>  drivers/scsi/NCR5380.c       |    2 +-
>  drivers/scsi/atari_NCR5380.c |    2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> Index: linux/drivers/scsi/NCR5380.c
> ===================================================================
> --- linux.orig/drivers/scsi/NCR5380.c	2016-01-26 13:31:10.000000000 +1100
> +++ linux/drivers/scsi/NCR5380.c	2016-01-26 13:31:10.000000000 +1100
> @@ -2337,7 +2337,7 @@ static int NCR5380_abort(struct scsi_cmn
>  		dsprintk(NDEBUG_ABORT, instance,
>  		         "abort: removed %p from disconnected list\n", cmd);
>  		cmd->result = DID_ERROR << 16;
> -		if (!hostdata->connected)
> +		if (!hostdata->connected && !hostdata->selecting)
>  			NCR5380_select(instance, cmd);
>  		if (hostdata->connected != cmd) {
>  			complete_cmd(instance, cmd);
> Index: linux/drivers/scsi/atari_NCR5380.c
> ===================================================================
> --- linux.orig/drivers/scsi/atari_NCR5380.c	2016-01-26 13:31:10.000000000 +1100
> +++ linux/drivers/scsi/atari_NCR5380.c	2016-01-26 13:31:10.000000000 +1100
> @@ -2532,7 +2532,7 @@ static int NCR5380_abort(struct scsi_cmn
>  		dsprintk(NDEBUG_ABORT, instance,
>  		         "abort: removed %p from disconnected list\n", cmd);
>  		cmd->result = DID_ERROR << 16;
> -		if (!hostdata->connected)
> +		if (!hostdata->connected && !hostdata->selecting)
>  			NCR5380_select(instance, cmd);
>  		if (hostdata->connected != cmd) {
>  			complete_cmd(instance, cmd);
> 
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux