[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 03/25] block/nvme: Report warning with warn_report()
From: |
Auger Eric |
Subject: |
Re: [PATCH 03/25] block/nvme: Report warning with warn_report() |
Date: |
Wed, 28 Oct 2020 11:22:00 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.5.0 |
Hi Philippe,
On 10/27/20 2:55 PM, Philippe Mathieu-Daudé wrote:
> Instead of displaying warning on stderr, use warn_report()
> which also displays it on the monitor.
>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Reviewed-by: Eric Auger <eric.auger@redhat.com>
Thanks
Eric
> ---
> block/nvme.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/block/nvme.c b/block/nvme.c
> index 739a0a700cb..6f1d7f9b2a1 100644
> --- a/block/nvme.c
> +++ b/block/nvme.c
> @@ -399,8 +399,8 @@ static bool nvme_process_completion(NVMeQueuePair *q)
> }
> cid = le16_to_cpu(c->cid);
> if (cid == 0 || cid > NVME_QUEUE_SIZE) {
> - fprintf(stderr, "Unexpected CID in completion queue: %" PRIu32
> "\n",
> - cid);
> + warn_report("NVMe: Unexpected CID in completion queue:
> %"PRIu32", "
> + "queue size: %u", cid, NVME_QUEUE_SIZE);
> continue;
> }
> trace_nvme_complete_command(s, q->index, cid);
>
[PATCH 05/25] block/nvme: Trace nvme_poll_queue() per queue, Philippe Mathieu-Daudé, 2020/10/27
[PATCH 06/25] block/nvme: Improve nvme_free_req_queue_wait() trace information, Philippe Mathieu-Daudé, 2020/10/27
[PATCH 09/25] block/nvme: Move definitions before structure declarations, Philippe Mathieu-Daudé, 2020/10/27