diff options
| author | Max Gurtovoy <maxg@mellanox.com> | 2017-01-23 10:01:12 +0100 |
|---|---|---|
| committer | Jens Axboe <axboe@fb.com> | 2017-02-22 21:34:00 +0100 |
| commit | 2dbf5816bb52ee3198882c1d198440e4bac62253 (patch) | |
| tree | f694821390b376f3adc0b0ef1e37594f5b616f78 /drivers/nvme/target/fabrics-cmd.c | |
| parent | nvme: Make controller state visible via sysfs (diff) | |
| download | linux-2dbf5816bb52ee3198882c1d198440e4bac62253.tar.xz linux-2dbf5816bb52ee3198882c1d198440e4bac62253.zip | |
nvmet: avoid dereferencing nvmet_req
No need to dereference req twice to get the cmd when we already
have it stored in a local variable.
Signed-off-by: Max Gurtovoy <maxg@mellanox.com>
Reviewed-by: Parav Pandit <parav@mellanox.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Sagi Grimberg <sagi@grimberg.me>
Signed-off-by: Jens Axboe <axboe@fb.com>
Diffstat (limited to 'drivers/nvme/target/fabrics-cmd.c')
| -rw-r--r-- | drivers/nvme/target/fabrics-cmd.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/nvme/target/fabrics-cmd.c b/drivers/nvme/target/fabrics-cmd.c index 18dc314601c3..8bd022af3df6 100644 --- a/drivers/nvme/target/fabrics-cmd.c +++ b/drivers/nvme/target/fabrics-cmd.c @@ -220,7 +220,7 @@ int nvmet_parse_connect_cmd(struct nvmet_req *req) req->ns = NULL; - if (req->cmd->common.opcode != nvme_fabrics_command) { + if (cmd->common.opcode != nvme_fabrics_command) { pr_err("invalid command 0x%x on unconnected queue.\n", cmd->fabrics.opcode); return NVME_SC_INVALID_OPCODE | NVME_SC_DNR; |