[PATCH 10/10] nvme_fc: correct nvme status set on abort

Hannes Reinecke hare at suse.de
Sun May 14 23:45:40 PDT 2017


On 05/13/2017 09:03 PM, James Smart wrote:
> correct nvme status set on abort. Patch that changed status to being actual
> nvme status crossed in the night with the patch that added abort values.
> 
> Signed-off-by: James Smart <james.smart at broadcom.com>
> ---
>  drivers/nvme/host/fc.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/nvme/host/fc.c b/drivers/nvme/host/fc.c
> index 78973407430a..8dd06e97736d 100644
> --- a/drivers/nvme/host/fc.c
> +++ b/drivers/nvme/host/fc.c
> @@ -1375,9 +1375,9 @@ nvme_fc_fcpio_done(struct nvmefc_fcp_req *req)
>  	complete_rq = __nvme_fc_fcpop_chk_teardowns(ctrl, op);
>  	if (!complete_rq) {
>  		if (unlikely(op->flags & FCOP_FLAGS_TERMIO)) {
> -			status = cpu_to_le16(NVME_SC_ABORT_REQ);
> +			status = cpu_to_le16(NVME_SC_ABORT_REQ << 1);
>  			if (blk_queue_dying(rq->q))
> -				status |= cpu_to_le16(NVME_SC_DNR);
> +				status |= cpu_to_le16(NVME_SC_DNR << 1);
>  		}
>  		nvme_end_request(rq, status, result);
>  	} else
> 
Reviewed-by: Hannes Reinecke <hare at suse.com>

Cheers,

Hannes
-- 
Dr. Hannes Reinecke		   Teamlead Storage & Networking
hare at suse.de			               +49 911 74053 688
SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: F. Imendörffer, J. Smithard, J. Guild, D. Upmanyu, G. Norton
HRB 21284 (AG Nürnberg)



More information about the Linux-nvme mailing list