[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 08/22] hw/pvrdma: Set the correct opcode for
From: |
Yuval Shaia |
Subject: |
Re: [Qemu-devel] [PATCH v2 08/22] hw/pvrdma: Set the correct opcode for recv completion |
Date: |
Sun, 11 Nov 2018 10:43:07 +0200 |
User-agent: |
Mutt/1.10.1 (2018-07-13) |
On Sat, Nov 10, 2018 at 08:18:58PM +0200, Marcel Apfelbaum wrote:
>
>
> On 11/8/18 6:08 PM, Yuval Shaia wrote:
> > The function pvrdma_post_cqe populates CQE entry with opcode from the
> > given completion element. For receive operation value was not set. Fix
> > it by setting it to IBV_WC_RECV.
> >
> > Signed-off-by: Yuval Shaia <address@hidden>
> > ---
> > hw/rdma/vmw/pvrdma_qp_ops.c | 3 ++-
> > 1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/hw/rdma/vmw/pvrdma_qp_ops.c b/hw/rdma/vmw/pvrdma_qp_ops.c
> > index 762700a205..7b0f440fda 100644
> > --- a/hw/rdma/vmw/pvrdma_qp_ops.c
> > +++ b/hw/rdma/vmw/pvrdma_qp_ops.c
> > @@ -196,8 +196,9 @@ int pvrdma_qp_recv(PVRDMADev *dev, uint32_t qp_handle)
> > comp_ctx = g_malloc(sizeof(CompHandlerCtx));
> > comp_ctx->dev = dev;
> > comp_ctx->cq_handle = qp->recv_cq_handle;
> > - comp_ctx->cqe.qp = qp_handle;
> > comp_ctx->cqe.wr_id = wqe->hdr.wr_id;
> > + comp_ctx->cqe.qp = qp_handle;
>
> Not sure the above chunk is needed.
Right, it is not related to the change but did it "while there" to be
consisted with settings order in pvrdma_qp_send :)
>
> > + comp_ctx->cqe.opcode = IBV_WC_RECV;
>
> Anyway
>
> Reviewed-by: Marcel Apfelbaum<address@hidden>
Thanks.
>
> Thanks,
> Marcel
>
> > rdma_backend_post_recv(&dev->backend_dev, &dev->rdma_dev_res,
> > &qp->backend_qp, qp->qp_type,
>
- [Qemu-devel] [PATCH v2 05/22] hw/rdma: Add support for MAD packets, (continued)
- [Qemu-devel] [PATCH v2 05/22] hw/rdma: Add support for MAD packets, Yuval Shaia, 2018/11/08
- [Qemu-devel] [PATCH v2 02/22] hw/rdma: Add ability to force notification without re-arm, Yuval Shaia, 2018/11/08
- [Qemu-devel] [PATCH v2 04/22] hw/rdma: Abort send-op if fail to create addr handler, Yuval Shaia, 2018/11/08
- [Qemu-devel] [PATCH v2 06/22] hw/pvrdma: Make function reset_device return void, Yuval Shaia, 2018/11/08
- [Qemu-devel] [PATCH v2 08/22] hw/pvrdma: Set the correct opcode for recv completion, Yuval Shaia, 2018/11/08
- [Qemu-devel] [PATCH v2 07/22] hw/pvrdma: Make default pkey 0xFFFF, Yuval Shaia, 2018/11/08
- [Qemu-devel] [PATCH v2 10/22] json: Define new QMP message for pvrdma, Yuval Shaia, 2018/11/08
- [Qemu-devel] [PATCH v2 11/22] hw/pvrdma: Add support to allow guest to configure GID table, Yuval Shaia, 2018/11/08
- [Qemu-devel] [PATCH v2 09/22] hw/pvrdma: Set the correct opcode for send completion, Yuval Shaia, 2018/11/08
- [Qemu-devel] [PATCH v2 13/22] hw/pvrdma: Make sure PCI function 0 is vmxnet3, Yuval Shaia, 2018/11/08