From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:54078) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WGjzf-0005ht-AV for qemu-devel@nongnu.org; Fri, 21 Feb 2014 01:54:04 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WGjzU-0000YD-UC for qemu-devel@nongnu.org; Fri, 21 Feb 2014 01:53:55 -0500 Received: from e9.ny.us.ibm.com ([32.97.182.139]:34452) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WGjzU-0000Y6-Pv for qemu-devel@nongnu.org; Fri, 21 Feb 2014 01:53:44 -0500 Received: from /spool/local by e9.ny.us.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Fri, 21 Feb 2014 01:53:44 -0500 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable From: Michael Roth In-Reply-To: <1392690846-30926-1-git-send-email-mrhines@linux.vnet.ibm.com> References: <1392690846-30926-1-git-send-email-mrhines@linux.vnet.ibm.com> Message-ID: <20140221065338.20750.48718@loki> Date: Fri, 21 Feb 2014 00:53:38 -0600 Subject: Re: [Qemu-devel] [PATCH] rdma: bug fixes List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: mrhines@linux.vnet.ibm.com, qemu-devel@nongnu.org Cc: "Michael R. Hines" , hinesmr@cn.ibm.com, qemu-stable@nongnu.org, quintela@redhat.com Quoting mrhines@linux.vnet.ibm.com (2014-02-17 20:34:06) > From: "Michael R. Hines" > = > 1. Fix small memory leak in parsing inet address from command line in dat= a_init() > 2. Fix ibv_post_send() return value check and pass error code back up cor= rectly. > 3. Fix rdma_destroy_qp() segfault after failure to connect to destination. > = > Reported-by: frank.yangjie@gmail.com > Reported-by: dgilbert@redhat.com > Signed-off-by: Michael R. Hines Reviewed-by: Michael Roth > --- > migration-rdma.c | 20 ++++++++++---------- > 1 file changed, 10 insertions(+), 10 deletions(-) > = > diff --git a/migration-rdma.c b/migration-rdma.c > index f94f3b4..29351a6 100644 > --- a/migration-rdma.c > +++ b/migration-rdma.c > @@ -1589,13 +1589,11 @@ static int qemu_rdma_post_send_control(RDMAContex= t *rdma, uint8_t *buf, > } > = > = > - if (ibv_post_send(rdma->qp, &send_wr, &bad_wr)) { > - return -1; > - } > + ret =3D ibv_post_send(rdma->qp, &send_wr, &bad_wr); > = > - if (ret < 0) { > + if (ret > 0) { > fprintf(stderr, "Failed to use post IB SEND for control!\n"); > - return ret; > + return -ret; > } > = > ret =3D qemu_rdma_block_for_wrid(rdma, RDMA_WRID_SEND_CONTROL, NULL); > @@ -2237,10 +2235,6 @@ static void qemu_rdma_cleanup(RDMAContext *rdma) > } > } > = > - if (rdma->qp) { > - rdma_destroy_qp(rdma->cm_id); > - rdma->qp =3D NULL; > - } > if (rdma->cq) { > ibv_destroy_cq(rdma->cq); > rdma->cq =3D NULL; > @@ -2258,6 +2252,10 @@ static void qemu_rdma_cleanup(RDMAContext *rdma) > rdma->listen_id =3D NULL; > } > if (rdma->cm_id) { > + if (rdma->qp) { > + rdma_destroy_qp(rdma->cm_id); > + rdma->qp =3D NULL; > + } > rdma_destroy_id(rdma->cm_id); > rdma->cm_id =3D NULL; > } > @@ -2512,8 +2510,10 @@ static void *qemu_rdma_data_init(const char *host_= port, Error **errp) > } else { > ERROR(errp, "bad RDMA migration address '%s'", host_port); > g_free(rdma); > - return NULL; > + rdma =3D NULL; > } > + > + qapi_free_InetSocketAddress(addr); > } > = > return rdma; > -- = > 1.8.1.2