Re: [PATCH] scsi: storvsc: fix SRB_STATUS_ABORTED handling

2016-03-08 Thread Martin K. Petersen
> "Vitaly" == Vitaly Kuznetsov writes: Vitaly> Commit 3209f9d780d1 ("scsi: storvsc: Fix a bug in the handling Vitaly> of SRB status flags") filtered SRB_STATUS_AUTOSENSE_VALID out Vitaly> effectively making the (SRB_STATUS_ABORTED | Vitaly> SRB_STATUS_AUTOSENSE_VALID)

Re: [PATCH] scsi: storvsc: fix SRB_STATUS_ABORTED handling

2016-03-08 Thread Martin K. Petersen
> "Vitaly" == Vitaly Kuznetsov writes: Vitaly> Commit 3209f9d780d1 ("scsi: storvsc: Fix a bug in the handling Vitaly> of SRB status flags") filtered SRB_STATUS_AUTOSENSE_VALID out Vitaly> effectively making the (SRB_STATUS_ABORTED | Vitaly> SRB_STATUS_AUTOSENSE_VALID) case a dead code. The

RE: [PATCH] scsi: storvsc: fix SRB_STATUS_ABORTED handling

2016-03-07 Thread KY Srinivasan
; Haiyang Zhang > <haiya...@microsoft.com>; Cathy Avery <cav...@redhat.com>; James E.J. > Bottomley <jbottom...@odin.com>; Martin K. Petersen" > <martin.peter...@oracle.com> > Subject: [PATCH] scsi: storvsc: fix SRB_STATUS_ABORTED handling > > Commit 3209f9d780d1 (&qu

RE: [PATCH] scsi: storvsc: fix SRB_STATUS_ABORTED handling

2016-03-07 Thread KY Srinivasan
ames E.J. > Bottomley ; Martin K. Petersen" > > Subject: [PATCH] scsi: storvsc: fix SRB_STATUS_ABORTED handling > > Commit 3209f9d780d1 ("scsi: storvsc: Fix a bug in the handling of SRB > status flags") filtered SRB_STATUS_AUTOSENSE_VALID out effectively > maki

[PATCH] scsi: storvsc: fix SRB_STATUS_ABORTED handling

2016-03-07 Thread Vitaly Kuznetsov
Commit 3209f9d780d1 ("scsi: storvsc: Fix a bug in the handling of SRB status flags") filtered SRB_STATUS_AUTOSENSE_VALID out effectively making the (SRB_STATUS_ABORTED | SRB_STATUS_AUTOSENSE_VALID) case a dead code. The logic from this branch (e.g. storvsc_device_scan() call) is still required,

[PATCH] scsi: storvsc: fix SRB_STATUS_ABORTED handling

2016-03-07 Thread Vitaly Kuznetsov
Commit 3209f9d780d1 ("scsi: storvsc: Fix a bug in the handling of SRB status flags") filtered SRB_STATUS_AUTOSENSE_VALID out effectively making the (SRB_STATUS_ABORTED | SRB_STATUS_AUTOSENSE_VALID) case a dead code. The logic from this branch (e.g. storvsc_device_scan() call) is still required,