Message ID | 20201202115249.37690-32-hare@suse.de (mailing list archive) |
---|---|
State | Superseded |
Headers | show |
Series | SCSI result handling cleanup, part 1 | expand |
On Wed, Dec 02, 2020 at 12:52:46PM +0100, Hannes Reinecke wrote: > ILLEGAL_COMMAND is a sense code, not a driver byte. Looks good, Reviewed-by: Christoph Hellwig <hch@lst.de>
diff --git a/drivers/scsi/storvsc_drv.c b/drivers/scsi/storvsc_drv.c index 0c65fbd41035..c6ba1878b1e6 100644 --- a/drivers/scsi/storvsc_drv.c +++ b/drivers/scsi/storvsc_drv.c @@ -1641,7 +1641,7 @@ static bool storvsc_scsi_cmd_ok(struct scsi_cmnd *scmnd) * this. So, don't send it. */ case SET_WINDOW: - scmnd->result = ILLEGAL_REQUEST << 16; + scmnd->result = DID_ERROR << 16; allowed = false; break; default:
ILLEGAL_COMMAND is a sense code, not a driver byte. Signed-off-by: Hannes Reinecke <hare@suse.de> --- drivers/scsi/storvsc_drv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)