Re: [Qemu-devel] [PATCH] rdma: Fix cleanup in error paths

2015-03-25 Thread Juan Quintela
Padmanabh Ratnakar wrote: > As part of commit e325b49a320b493cc5d69e263751ff716dc458fe, > order in which resources are destroyed was changed for fixing > a seg fault. Due to this change, CQ will never get destroyed as > CQ should be destroyed after QP destruction. Seg fault is caused > improper cl

Re: [Qemu-devel] [PATCH] rdma: Fix cleanup in error paths

2015-03-25 Thread Padmanabh Ratnakar
> > This other three are nice, remove code, and make it correct with the case > that qp has to be removed first. > > So, should we drop the listen_id part, or there is a reason for it? > > Later, Juan. > listen_id is used only in server side. Saw that listen_id gets created before cm_id. So f

Re: [Qemu-devel] [PATCH] rdma: Fix cleanup in error paths

2015-03-25 Thread Padmanabh Ratnakar
> -Original Message- > From: Dr. David Alan Gilbert [mailto:dgilb...@redhat.com] > Sent: Wednesday, March 25, 2015 4:10 PM > To: Padmanabh Ratnakar > Cc: quint...@redhat.com; amit.s...@redhat.com; mrhi...@us.ibm.com; > Padmanabh Ratnakar; Meghana Cheripady; qemu-devel@nongnu.org > Subject:

Re: [Qemu-devel] [PATCH] rdma: Fix cleanup in error paths

2015-03-25 Thread Juan Quintela
Padmanabh Ratnakar wrote: > As part of commit e325b49a320b493cc5d69e263751ff716dc458fe, > order in which resources are destroyed was changed for fixing > a seg fault. Due to this change, CQ will never get destroyed as > CQ should be destroyed after QP destruction. Seg fault is caused > improper cl

Re: [Qemu-devel] [PATCH] rdma: Fix cleanup in error paths

2015-03-25 Thread Dr. David Alan Gilbert
> As part of commit e325b49a320b493cc5d69e263751ff716dc458fe, > order in which resources are destroyed was changed for fixing > a seg fault. Due to this change, CQ will never get destroyed as > CQ should be destroyed after QP destruction. Seg fault is caused > improper cleanup when connection fails

[Qemu-devel] [PATCH] rdma: Fix cleanup in error paths

2015-03-25 Thread Padmanabh Ratnakar
As part of commit e325b49a320b493cc5d69e263751ff716dc458fe, order in which resources are destroyed was changed for fixing a seg fault. Due to this change, CQ will never get destroyed as CQ should be destroyed after QP destruction. Seg fault is caused improper cleanup when connection fails. Fixing c