[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v5 17/24] hw/pvrdma: Fill error code in command'
From: |
Yuval Shaia |
Subject: |
Re: [Qemu-devel] [PATCH v5 17/24] hw/pvrdma: Fill error code in command's response |
Date: |
Sun, 25 Nov 2018 13:53:57 +0200 |
User-agent: |
Mutt/1.10.1 (2018-07-13) |
> > err = cmd_handlers[dsr_info->req->hdr.cmd].exec(dev, dsr_info->req,
> > - dsr_info->rsp);
> > + dsr_info->rsp);
> > out:
> > set_reg_val(dev, PVRDMA_REG_ERR, err);
> > post_interrupt(dev, INTR_VEC_CMD_RING);
>
>
> As I responded in V4 thread :) one might forget to set hdr.error to 0
> resulting in errors hard to debug.
>
> Please consider clearing the field on init.
Will fix.
>
> Reviewed-by: Marcel Apfelbaum <address@hidden>
Please review v6.
>
> Thanks,
> Marcel
- Re: [Qemu-devel] [PATCH v5 13/24] hw/pvrdma: Make sure PCI function 0 is vmxnet3, (continued)
- [Qemu-devel] [PATCH v5 16/24] hw/pvrdma: Fill all CQE fields, Yuval Shaia, 2018/11/22
- [Qemu-devel] [PATCH v5 23/24] hw/pvrdma: Do not clean resources on shutdown, Yuval Shaia, 2018/11/22
- [Qemu-devel] [PATCH v5 15/24] hw/pvrdma: Make device state depend on Ethernet function state, Yuval Shaia, 2018/11/22
- [Qemu-devel] [PATCH v5 24/24] docs: Update pvrdma device documentation, Yuval Shaia, 2018/11/22
- [Qemu-devel] [PATCH v5 18/24] hw/rdma: Remove unneeded code that handles more that one port, Yuval Shaia, 2018/11/22
- [Qemu-devel] [PATCH v5 17/24] hw/pvrdma: Fill error code in command's response, Yuval Shaia, 2018/11/22
- [Qemu-devel] [PATCH v5 06/24] hw/pvrdma: Make function reset_device return void, Yuval Shaia, 2018/11/22
- [Qemu-devel] [PATCH v5 12/24] vmxnet3: Move some definitions to header file, Yuval Shaia, 2018/11/22
- [Qemu-devel] [PATCH v5 07/24] hw/pvrdma: Make default pkey 0xFFFF, Yuval Shaia, 2018/11/22
- [Qemu-devel] [PATCH v5 05/24] hw/rdma: Add support for MAD packets, Yuval Shaia, 2018/11/22
- [Qemu-devel] [PATCH v5 09/24] hw/pvrdma: Set the correct opcode for send completion, Yuval Shaia, 2018/11/22
- [Qemu-devel] [PATCH v5 03/24] hw/rdma: Return qpn 1 if ibqp is NULL, Yuval Shaia, 2018/11/22
- [Qemu-devel] [PATCH v5 10/24] qapi: Define new QMP message for pvrdma, Yuval Shaia, 2018/11/22