diff options
author | Keith Busch <kbusch@kernel.org> | 2021-03-17 21:33:41 +0100 |
---|---|---|
committer | Christoph Hellwig <hch@lst.de> | 2021-04-02 18:48:28 +0200 |
commit | ed4a854b062b841ebc1aa576f27daf72d07150a5 (patch) | |
tree | d102ebb2412b818e72e9f524fee81b6ba53b82de /drivers/nvme | |
parent | nvme: use driver pdu command for passthrough (diff) | |
download | linux-ed4a854b062b841ebc1aa576f27daf72d07150a5.tar.xz linux-ed4a854b062b841ebc1aa576f27daf72d07150a5.zip |
nvme: warn of unhandled effects only once
We don't need to repeatedly spam the kernel logs with the same warning
about unhandled passthrough IO effects. Just one warning is sufficient
to observe this condition occurs.
Signed-off-by: Keith Busch <kbusch@kernel.org>
Reviewed-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>
Signed-off-by: Christoph Hellwig <hch@lst.de>
Diffstat (limited to 'drivers/nvme')
-rw-r--r-- | drivers/nvme/host/core.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/nvme/host/core.c b/drivers/nvme/host/core.c index c3f94eb90669..40215a0246e4 100644 --- a/drivers/nvme/host/core.c +++ b/drivers/nvme/host/core.c @@ -1072,9 +1072,9 @@ u32 nvme_command_effects(struct nvme_ctrl *ctrl, struct nvme_ns *ns, u8 opcode) if (ns->head->effects) effects = le32_to_cpu(ns->head->effects->iocs[opcode]); if (effects & ~(NVME_CMD_EFFECTS_CSUPP | NVME_CMD_EFFECTS_LBCC)) - dev_warn(ctrl->device, - "IO command:%02x has unhandled effects:%08x\n", - opcode, effects); + dev_warn_once(ctrl->device, + "IO command:%02x has unhandled effects:%08x\n", + opcode, effects); return 0; } |