"Michael R. Hines" <[email protected]> wrote: > 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?
Pointer, please? I was waiting for Michael Reviewed-by from Michael. Later, Juan. > > - Michael > > - Michael
