[PATCH] nvmet-fc: Remove a set-but-not-used variable
Sagi Grimberg
sagi at grimberg.me
Thu Jun 15 01:58:06 PDT 2017
already applied, but FWIW,
Reviewed-by: Sagi Grimberg <sagi at grimberg.me>
On 08/06/17 19:43, Bart Van Assche wrote:
> From: Bart Van Assche <bart.vanassche at sandisk.com>
>
> This was detected by building the nvmet-fc driver with W=1.
>
> Signed-off-by: Bart Van Assche <bart.vanassche at sandisk.com>
> Cc: James Smart <james.smart at broadcom.com>
> Cc: Christoph Hellwig <hch at lst.de>
> Cc: Johannes Thumshirn <jthumshirn at suse.de>
> ---
> drivers/nvme/target/fcloop.c | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/drivers/nvme/target/fcloop.c b/drivers/nvme/target/fcloop.c
> index 294a6611fb24..1bb9d5b311b1 100644
> --- a/drivers/nvme/target/fcloop.c
> +++ b/drivers/nvme/target/fcloop.c
> @@ -569,7 +569,6 @@ fcloop_tgt_fcp_abort(struct nvmet_fc_target_port *tgtport,
> struct nvmefc_tgt_fcp_req *tgt_fcpreq)
> {
> struct fcloop_fcpreq *tfcp_req = tgt_fcp_req_to_fcpreq(tgt_fcpreq);
> - int active;
>
> /*
> * mark aborted only in case there were 2 threads in transport
> @@ -577,7 +576,6 @@ fcloop_tgt_fcp_abort(struct nvmet_fc_target_port *tgtport,
> * after the abort request
> */
> spin_lock(&tfcp_req->reqlock);
> - active = tfcp_req->active;
> tfcp_req->aborted = true;
> spin_unlock(&tfcp_req->reqlock);
>
>
More information about the Linux-nvme
mailing list