Re: [Qemu-devel] qemu_rdma_cleanup seg - related to 5a91337?

2014-02-17 Thread Michael R. Hines
On 02/17/2014 05:06 PM, Dr. David Alan Gilbert wrote: * Michael R. Hines (mrhi...@linux.vnet.ibm.com) wrote: On 02/06/2014 08:26 PM, Dr. David Alan Gilbert wrote: Hi Isaku, I hit a seg in qemu_rdma_cleanup in the code changed by your '[PATCH] rdma: clean up of qemu_rdma_cleanup()' migratio

Re: [Qemu-devel] qemu_rdma_cleanup seg - related to 5a91337?

2014-02-17 Thread Dr. David Alan Gilbert
* Michael R. Hines (mrhi...@linux.vnet.ibm.com) wrote: > On 02/06/2014 08:26 PM, Dr. David Alan Gilbert wrote: > >Hi Isaku, > >I hit a seg in qemu_rdma_cleanup in the code changed by your > >'[PATCH] rdma: clean up of qemu_rdma_cleanup()' > > > >migration-rdma.c ~ 2241 > > > > if (rdma->qp)

Re: [Qemu-devel] qemu_rdma_cleanup seg - related to 5a91337?

2014-02-16 Thread Michael R. Hines
On 02/06/2014 08:26 PM, Dr. David Alan Gilbert wrote: Hi Isaku, I hit a seg in qemu_rdma_cleanup in the code changed by your '[PATCH] rdma: clean up of qemu_rdma_cleanup()' migration-rdma.c ~ 2241 if (rdma->qp) { rdma_destroy_qp(rdma->cm_id); rdma->qp = NULL; }

[Qemu-devel] qemu_rdma_cleanup seg - related to 5a91337?

2014-02-06 Thread Dr. David Alan Gilbert
Hi Isaku, I hit a seg in qemu_rdma_cleanup in the code changed by your '[PATCH] rdma: clean up of qemu_rdma_cleanup()' migration-rdma.c ~ 2241 if (rdma->qp) { rdma_destroy_qp(rdma->cm_id); rdma->qp = NULL; } Your patch changed that to free cm_id at that point rather th