summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorKeith Busch <keith.busch@intel.com>2019-03-13 18:54:55 +0100
committerJens Axboe <axboe@kernel.dk>2019-03-13 12:05:39 -0600
commit415df90b437f2b026ed37af2f812e41fc06c7f90 (patch)
tree77168476baed67924d6edd43749f7ade5b0b291a
parentd9d53ed3f77ff4057ce714c0d169c28a653504e7 (diff)
downloadlinux-415df90b437f2b026ed37af2f812e41fc06c7f90.tar.gz
linux-415df90b437f2b026ed37af2f812e41fc06c7f90.tar.bz2
linux-415df90b437f2b026ed37af2f812e41fc06c7f90.zip
nvme: don't warn on block content change effects
A write or flush IO passthrough command is expected to change the logical block content, so don't warn on these as no additional handling is necessary. Signed-off-by: Keith Busch <keith.busch@intel.com> Reviewed-by: Sagi Grimberg <sagi@grimberg.me> Signed-off-by: Christoph Hellwig <hch@lst.de> Signed-off-by: Jens Axboe <axboe@kernel.dk>
-rw-r--r--drivers/nvme/host/core.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/nvme/host/core.c b/drivers/nvme/host/core.c
index 07bf2bff3a76..dc1641247b17 100644
--- a/drivers/nvme/host/core.c
+++ b/drivers/nvme/host/core.c
@@ -1250,7 +1250,7 @@ static u32 nvme_passthru_start(struct nvme_ctrl *ctrl, struct nvme_ns *ns,
if (ns) {
if (ctrl->effects)
effects = le32_to_cpu(ctrl->effects->iocs[opcode]);
- if (effects & ~NVME_CMD_EFFECTS_CSUPP)
+ if (effects & ~(NVME_CMD_EFFECTS_CSUPP | NVME_CMD_EFFECTS_LBCC))
dev_warn(ctrl->device,
"IO command:%02x has unhandled effects:%08x\n",
opcode, effects);