> -----Original Message----- > From: Michael R. Hines [mailto:[email protected]] > Sent: Thursday, May 15, 2014 8:44 AM > To: Gonglei (Arei); [email protected] > Cc: Huangweidong (C); [email protected]; [email protected]; > [email protected]; [email protected]; [email protected]; > Moyuxiang > Subject: Re: [Qemu-devel] [PATCH] rdma: Fix block during rdma migration > > On 05/09/2014 12:25 PM, Gonglei (Arei) wrote: > > Hi, > > > >> -----Original Message----- > >> From: Michael R. Hines [mailto:[email protected]] > >> Sent: Tuesday, April 01, 2014 8:42 AM > >> To: Gonglei (Arei); [email protected] > >> Cc: Huangweidong (C); [email protected]; [email protected]; > >> [email protected]; [email protected]; Moyuxiang; > >> [email protected] > >> Subject: Re: [Qemu-devel] [PATCH] rdma: Fix block during rdma migration > >> > >> On 03/29/2014 03:39 PM, [email protected] wrote: > >>> From: Mo Yuxiang <[email protected]> > >>> > >>> If the networking break or there's something wrong with rdma > >>> device(ib0 with no IP) during rdma migration, the main_loop of > >>> qemu will be blocked in rdma_destroy_id. I add rdma_ack_cm_event > >>> to fix this bug. > >>> > >>> Signed-off-by: Mo Yuxiang <[email protected]> > >>> Signed-off-by: Gonglei <[email protected]> > >>> --- > >>> migration-rdma.c | 1 + > >>> 1 file changed, 1 insertion(+) > >>> > >>> diff --git a/migration-rdma.c b/migration-rdma.c > >>> index eeb4302..f60749b 100644 > >>> --- a/migration-rdma.c > >>> +++ b/migration-rdma.c > >>> @@ -949,6 +949,7 @@ route: > >>> ERROR(errp, "result not equal to event_addr_resolved %s", > >>> rdma_event_str(cm_event->event)); > >>> perror("rdma_resolve_addr"); > >>> + rdma_ack_cm_event(cm_event); > >>> ret = -EINVAL; > >>> goto err_resolve_get_addr; > >>> } > >> Reviewed-by: Michael R. Hines <[email protected]> > >> > >> Good catch. =) That's an obvious bug. It looks like I need > >> to do a much better job of "kill -9" inside the regression > >> testing scripts - probably i should try killing the migration > >> prematurely at different periods just to be sure there are > >> no more places where the connection state is not getting > >> cleaned up...... > >> > >> - Michael > >> > > Michael, do you have a plan to pull this patch to master? Thanks. > > > > Best regards, > > -Gonglei > > > > Sorry for the late reply, but I'm not the maintainer for migration, > that's Juan > (I can only signoff on patches like everyone else =). > > I also have outstanding RDMA patches myself that have not yet been pulled. > > Would you mind pinging Juan for both of us? > Thanks. The patch is Cc'ing Juan, maybe he is very busy. I have post v2 even, but I have not gotten any reply. I have no idea how to do next.
Best regards, -Gonglei
