|
 |
9ae3a8 |
From 41048b716132906fe54b6dbca48421a24f78affb Mon Sep 17 00:00:00 2001
|
|
|
9ae3a8 |
From: Dr. David Alan Gilbert (git) <dgilbert@redhat.com>
|
|
|
9ae3a8 |
Date: Fri, 11 Jul 2014 09:05:28 +0200
|
|
|
9ae3a8 |
Subject: [PATCH 01/11] rdma: bug fixes
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
Message-id: <1405069528-20466-2-git-send-email-dgilbert@redhat.com>
|
|
|
9ae3a8 |
Patchwork-id: 59778
|
|
|
9ae3a8 |
O-Subject: [RHEL-7.1 qemu-kvm PATCH 1/1] rdma: bug fixes
|
|
|
9ae3a8 |
Bugzilla: 1107821
|
|
|
9ae3a8 |
RH-Acked-by: Luiz Capitulino <lcapitulino@redhat.com>
|
|
|
9ae3a8 |
RH-Acked-by: Amit Shah <amit.shah@redhat.com>
|
|
|
9ae3a8 |
RH-Acked-by: Laszlo Ersek <lersek@redhat.com>
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
From: "Michael R. Hines" <mrhines@us.ibm.com>
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
1. Fix small memory leak in parsing inet address from command line in data_init()
|
|
|
9ae3a8 |
2. Fix ibv_post_send() return value check and pass error code back up correctly.
|
|
|
9ae3a8 |
3. Fix rdma_destroy_qp() segfault after failure to connect to destination.
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
Reported-by: frank.yangjie@gmail.com
|
|
|
9ae3a8 |
Reported-by: dgilbert@redhat.com
|
|
|
9ae3a8 |
Signed-off-by: Michael R. Hines <mrhines@us.ibm.com>
|
|
|
9ae3a8 |
Signed-off-by: Juan Quintela <quintela@redhat.com>
|
|
|
9ae3a8 |
(cherry picked from commit e325b49a320b493cc5d69e263751ff716dc458fe)
|
|
|
9ae3a8 |
---
|
|
|
9ae3a8 |
migration-rdma.c | 20 ++++++++++----------
|
|
|
9ae3a8 |
1 file changed, 10 insertions(+), 10 deletions(-)
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
9ae3a8 |
---
|
|
|
9ae3a8 |
migration-rdma.c | 20 ++++++++++----------
|
|
|
9ae3a8 |
1 files changed, 10 insertions(+), 10 deletions(-)
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
diff --git a/migration-rdma.c b/migration-rdma.c
|
|
|
9ae3a8 |
index f79be20..ba4ad76 100644
|
|
|
9ae3a8 |
--- a/migration-rdma.c
|
|
|
9ae3a8 |
+++ b/migration-rdma.c
|
|
|
9ae3a8 |
@@ -1589,13 +1589,11 @@ static int qemu_rdma_post_send_control(RDMAContext *rdma, uint8_t *buf,
|
|
|
9ae3a8 |
}
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
- if (ibv_post_send(rdma->qp, &send_wr, &bad_wr)) {
|
|
|
9ae3a8 |
- return -1;
|
|
|
9ae3a8 |
- }
|
|
|
9ae3a8 |
+ ret = ibv_post_send(rdma->qp, &send_wr, &bad_wr);
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
- if (ret < 0) {
|
|
|
9ae3a8 |
+ if (ret > 0) {
|
|
|
9ae3a8 |
fprintf(stderr, "Failed to use post IB SEND for control!\n");
|
|
|
9ae3a8 |
- return ret;
|
|
|
9ae3a8 |
+ return -ret;
|
|
|
9ae3a8 |
}
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
ret = qemu_rdma_block_for_wrid(rdma, RDMA_WRID_SEND_CONTROL, NULL);
|
|
|
9ae3a8 |
@@ -2237,10 +2235,6 @@ static void qemu_rdma_cleanup(RDMAContext *rdma)
|
|
|
9ae3a8 |
}
|
|
|
9ae3a8 |
}
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
- if (rdma->qp) {
|
|
|
9ae3a8 |
- rdma_destroy_qp(rdma->cm_id);
|
|
|
9ae3a8 |
- rdma->qp = NULL;
|
|
|
9ae3a8 |
- }
|
|
|
9ae3a8 |
if (rdma->cq) {
|
|
|
9ae3a8 |
ibv_destroy_cq(rdma->cq);
|
|
|
9ae3a8 |
rdma->cq = NULL;
|
|
|
9ae3a8 |
@@ -2258,6 +2252,10 @@ static void qemu_rdma_cleanup(RDMAContext *rdma)
|
|
|
9ae3a8 |
rdma->listen_id = NULL;
|
|
|
9ae3a8 |
}
|
|
|
9ae3a8 |
if (rdma->cm_id) {
|
|
|
9ae3a8 |
+ if (rdma->qp) {
|
|
|
9ae3a8 |
+ rdma_destroy_qp(rdma->cm_id);
|
|
|
9ae3a8 |
+ rdma->qp = NULL;
|
|
|
9ae3a8 |
+ }
|
|
|
9ae3a8 |
rdma_destroy_id(rdma->cm_id);
|
|
|
9ae3a8 |
rdma->cm_id = NULL;
|
|
|
9ae3a8 |
}
|
|
|
9ae3a8 |
@@ -2512,8 +2510,10 @@ static void *qemu_rdma_data_init(const char *host_port, Error **errp)
|
|
|
9ae3a8 |
} else {
|
|
|
9ae3a8 |
ERROR(errp, "bad RDMA migration address '%s'", host_port);
|
|
|
9ae3a8 |
g_free(rdma);
|
|
|
9ae3a8 |
- return NULL;
|
|
|
9ae3a8 |
+ rdma = NULL;
|
|
|
9ae3a8 |
}
|
|
|
9ae3a8 |
+
|
|
|
9ae3a8 |
+ qapi_free_InetSocketAddress(addr);
|
|
|
9ae3a8 |
}
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
return rdma;
|
|
|
9ae3a8 |
--
|
|
|
9ae3a8 |
1.7.1
|
|
|
9ae3a8 |
|