[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 2/9] block/nvme: Have nvme_create_queue_pair() report errors
From: |
Klaus Jensen |
Subject: |
Re: [PATCH 2/9] block/nvme: Have nvme_create_queue_pair() report errors consistently |
Date: |
Wed, 25 Aug 2021 13:24:44 +0200 |
On Aug 24 16:11, Philippe Mathieu-Daudé wrote:
> nvme_create_queue_pair() does not return a boolean value (indicating
> eventual error) but a pointer, and is inconsistent in how it fills the
> error handler. To fulfill callers expectations, always set an error
> message on failure.
>
> Reported-by: Auger Eric <eric.auger@redhat.com>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> ---
> block/nvme.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/block/nvme.c b/block/nvme.c
> index e8dbbc23177..6642c104aa4 100644
> --- a/block/nvme.c
> +++ b/block/nvme.c
> @@ -220,6 +220,7 @@ static NVMeQueuePair
> *nvme_create_queue_pair(BDRVNVMeState *s,
>
> q = g_try_new0(NVMeQueuePair, 1);
> if (!q) {
> + error_setg(errp, "Cannot allocate queue pair");
> return NULL;
> }
> trace_nvme_create_queue_pair(idx, q, size, aio_context,
> @@ -228,6 +229,7 @@ static NVMeQueuePair
> *nvme_create_queue_pair(BDRVNVMeState *s,
> qemu_real_host_page_size);
> q->prp_list_pages = qemu_try_memalign(qemu_real_host_page_size, bytes);
> if (!q->prp_list_pages) {
> + error_setg(errp, "Cannot allocate PRP page list");
> goto fail;
> }
> memset(q->prp_list_pages, 0, bytes);
> --
> 2.31.1
>
Looks like
r = qemu_vfio_dma_map(s->vfio, q->prp_list_pages, bytes,
false, &prp_list_iova);
if (r) {
goto fail;
}
should set an error message as well.
signature.asc
Description: PGP signature
- [PATCH 0/9] block/nvme: Rework error reporting, Philippe Mathieu-Daudé, 2021/08/24
- [PATCH 1/9] block/nvme: Use safer trace format string, Philippe Mathieu-Daudé, 2021/08/24
- [PATCH 2/9] block/nvme: Have nvme_create_queue_pair() report errors consistently, Philippe Mathieu-Daudé, 2021/08/24
- Re: [PATCH 2/9] block/nvme: Have nvme_create_queue_pair() report errors consistently,
Klaus Jensen <=
- [PATCH 3/9] util/vfio-helpers: Let qemu_vfio_verify_mappings() use error_report(), Philippe Mathieu-Daudé, 2021/08/24
- [PATCH 4/9] util/vfio-helpers: Replace qemu_mutex_lock() calls with QEMU_LOCK_GUARD, Philippe Mathieu-Daudé, 2021/08/24
- [PATCH 5/9] util/vfio-helpers: Remove unreachable code in qemu_vfio_dma_map(), Philippe Mathieu-Daudé, 2021/08/24
- [PATCH 6/9] util/vfio-helpers: Pass Error handle to qemu_vfio_dma_map(), Philippe Mathieu-Daudé, 2021/08/24
- [PATCH 7/9] util/vfio-helpers: Have qemu_vfio_dma_map() propagate Error, Philippe Mathieu-Daudé, 2021/08/24