[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 09/17] hw/rdma: Fix missing conversion to rdma_e
From: |
Yuval Shaia |
Subject: |
Re: [Qemu-devel] [PATCH 09/17] hw/rdma: Fix missing conversion to rdma_error_report() |
Date: |
Tue, 17 Sep 2019 17:51:57 +0300 |
User-agent: |
Mutt/1.10.1 (2018-07-13) |
On Tue, Sep 17, 2019 at 12:21:27PM +0200, Greg Kurz wrote:
> Commit 4d71b38ae8fa converted many error_setg() call sites to
> rdma_error_report(), but it forgot to convert a companion
> error_append_hint(). Since no guy doesn't set errp anymore in
> pvrdma_realize(), errp remains NULL and error_append_hint() does
> nothing.
>
> Also error_append_hint() was a poor choice since its "intended use
> is adding helpful hints on the human user interface" and "not for
> clarifying a confusing error message".
>
> Call rdma_error_report() instead.
Thanks,
So are you suggesting to replace all other error_setg calls with
rdma_error_report instead?
>
> Fixes: 4d71b38ae8fa "hw/rdma: Switch to generic error reporting way"
> Signed-off-by: Greg Kurz <address@hidden>
> ---
> hw/rdma/vmw/pvrdma_main.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/hw/rdma/vmw/pvrdma_main.c b/hw/rdma/vmw/pvrdma_main.c
> index 3e36e130139c..d370ae07ca6a 100644
> --- a/hw/rdma/vmw/pvrdma_main.c
> +++ b/hw/rdma/vmw/pvrdma_main.c
> @@ -667,7 +667,7 @@ static void pvrdma_realize(PCIDevice *pdev, Error **errp)
> out:
> if (rc) {
> pvrdma_fini(pdev);
> - error_append_hint(errp, "Device failed to load\n");
> + rdma_error_report("Device failed to load");
Reviewed-by: Yuval Shaia <address@hidden>
> }
> }
>
>
>
- Re: [Qemu-devel] [PATCH 02/17] block: Pass local error object pointer to error_append_hint(), (continued)
- [Qemu-devel] [PATCH 03/17] char/spice: Pass local error object pointer to error_append_hint(), Greg Kurz, 2019/09/17
- [Qemu-devel] [PATCH 04/17] ppc: Pass local error object pointer to error_append_hint(), Greg Kurz, 2019/09/17
- [Qemu-devel] [PATCH 05/17] arm: Pass local error object pointer to error_append_hint(), Greg Kurz, 2019/09/17
- [Qemu-devel] [PATCH 06/17] vfio: Pass local error object pointer to error_append_hint(), Greg Kurz, 2019/09/17
- [Qemu-devel] [PATCH 07/17] virtio-pci: Pass local error object pointer to error_append_hint(), Greg Kurz, 2019/09/17
- [Qemu-devel] [PATCH 08/17] pcie_root_port: Pass local error object pointer to error_append_hint(), Greg Kurz, 2019/09/17
- [Qemu-devel] [PATCH 09/17] hw/rdma: Fix missing conversion to rdma_error_report(), Greg Kurz, 2019/09/17
- Re: [Qemu-devel] [PATCH 09/17] hw/rdma: Fix missing conversion to rdma_error_report(),
Yuval Shaia <=
- [Qemu-devel] [PATCH 10/17] s390x/css: Pass local error object pointer to error_append_hint(), Greg Kurz, 2019/09/17
- Re: [Qemu-devel] [PATCH 10/17] s390x/css: Pass local error object pointer to error_append_hint(), Cornelia Huck, 2019/09/17
- Re: [Qemu-devel] [PATCH 10/17] s390x/css: Pass local error object pointer to error_append_hint(), David Hildenbrand, 2019/09/17
- Re: [Qemu-devel] [PATCH 10/17] s390x/css: Pass local error object pointer to error_append_hint(), Greg Kurz, 2019/09/17
- Re: [Qemu-devel] [PATCH 10/17] s390x/css: Pass local error object pointer to error_append_hint(), Cornelia Huck, 2019/09/18
- Re: [Qemu-devel] [PATCH 10/17] s390x/css: Pass local error object pointer to error_append_hint(), Eric Blake, 2019/09/18
- Re: [Qemu-devel] [PATCH 10/17] s390x/css: Pass local error object pointer to error_append_hint(), Cornelia Huck, 2019/09/19
[Qemu-devel] [PATCH 11/17] scsi: Pass local error object pointer to error_append_hint(), Greg Kurz, 2019/09/17
[Qemu-devel] [PATCH 12/17] migration: Pass local error object pointer to error_append_hint(), Greg Kurz, 2019/09/17