From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([209.51.188.92]:41841) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1hD3tW-0007t9-BJ for qemu-devel@nongnu.org; Sun, 07 Apr 2019 05:15:19 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1hD3tV-0007Mc-Ba for qemu-devel@nongnu.org; Sun, 07 Apr 2019 05:15:18 -0400 Received: from userp2130.oracle.com ([156.151.31.86]:52564) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1hD3tV-0007MC-0i for qemu-devel@nongnu.org; Sun, 07 Apr 2019 05:15:17 -0400 Date: Sun, 7 Apr 2019 12:15:04 +0300 From: Yuval Shaia Message-ID: <20190407091503.GB6737@lap1> References: <20190403113343.26384-1-kamalheib1@gmail.com> <20190403181937.GE3446@lap1> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20190403181937.GE3446@lap1> Subject: Re: [Qemu-devel] [PATCH v3 0/4] pvrdma: Add support for SRQ List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Kamal Heib Cc: qemu-devel@nongnu.org On Wed, Apr 03, 2019 at 09:19:38PM +0300, Yuval Shaia wrote: > On Wed, Apr 03, 2019 at 02:33:39PM +0300, Kamal Heib wrote: > > This series implements the SRQ (Shared Receive Queue) for the pvrdma > > device, It also includes all the needed functions and definitions for > > support SRQ in the backend and resource management layers. > > > > Changes from v2->3: > > - Patch #1: > > -- Fix commit message. > > -- Remove initialization of backend_qp from rdma_backend_post_srq_recv(). > > -- Add rx_srq counter. > > - Patch #2: > > -- Add checks for srq attrs. > > - Patch #3: > > -- Move initialization of recv_cq_handle to be under is_srq. > > -- Rearrange destroy_qp() to avoid use after free. > > - Patch #4: > > -- Avoid use after free. > > -- Fix indentation. > > > > Changes from v1->v2: > > - Handle checkpatch.pl warnings. > > Appreciate if you can post v4 with a new patch to display the new counter > (function rdma_dump_device_counters). Please ignore this comment, counter is already added - i missed that. With that - patchset lgtm, from my perspective ready to merge. > > > > > Kamal Heib (4): > > hw/rdma: Add SRQ support to backend layer > > hw/rdma: Add support for managing SRQ resource > > hw/rdma: Modify create/destroy QP to support SRQ > > hw/pvrdma: Add support for SRQ > > > > hw/rdma/rdma_backend.c | 125 +++++++++++++++++++++- > > hw/rdma/rdma_backend.h | 18 +++- > > hw/rdma/rdma_backend_defs.h | 5 + > > hw/rdma/rdma_rm.c | 117 +++++++++++++++++++- > > hw/rdma/rdma_rm.h | 13 ++- > > hw/rdma/rdma_rm_defs.h | 10 ++ > > hw/rdma/vmw/pvrdma_cmd.c | 206 ++++++++++++++++++++++++++++++++---- > > hw/rdma/vmw/pvrdma_main.c | 16 +++ > > hw/rdma/vmw/pvrdma_qp_ops.c | 46 +++++++- > > hw/rdma/vmw/pvrdma_qp_ops.h | 1 + > > 10 files changed, 521 insertions(+), 36 deletions(-) > > > > -- > > 2.20.1 > > > > >