[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 6/6] RFH: We lost "connect" events
From: |
Juan Quintela |
Subject: |
Re: [Qemu-devel] [PATCH 6/6] RFH: We lost "connect" events |
Date: |
Mon, 19 Aug 2019 12:33:45 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.2 (gnu/linux) |
Daniel P. Berrangé <address@hidden> wrote:
> On Wed, Aug 14, 2019 at 04:02:18AM +0200, Juan Quintela wrote:
>> When we have lots of channels, sometimes multifd migration fails
>> with the following error:
>> after some time, sending side decides to send another packet through
>> that channel, and it is now when we get the above error.
>>
>> Any good ideas?
>
> In inet_listen_saddr() we call
>
> if (!listen(slisten, 1)) {
>
> note the second parameter sets the socket backlog, which is the max
> number of pending socket connections we allow. My guess is that the
> target QEMU is not accepting incoming connections quickly enough and
> thus you hit the limit & the kernel starts dropping the incoming
> connections.
>
> As a quick test, just hack this code to pass a value of 100 and see
> if it makes your test reliable. If it does, then we'll need to figure
> out a nice way to handle backlog instead of hardcoding it at 1.
I will test.
But notice that the qemu_connect() on source side says that things went
right. It is the destination what is *not* calling the callback. Or
at least that is what I think it is happening.
Later, Juan.
- [Qemu-devel] [PATCH 3/6] migration: Make sure that all multifd channels have been created, (continued)
- [Qemu-devel] [PATCH 3/6] migration: Make sure that all multifd channels have been created, Juan Quintela, 2019/08/13
- [Qemu-devel] [PATCH 4/6] migration: Make multifd threads wait until all have been created, Juan Quintela, 2019/08/13
- [Qemu-devel] [PATCH 5/6] migration: add some multifd traces, Juan Quintela, 2019/08/13
- [Qemu-devel] [PATCH 6/6] RFH: We lost "connect" events, Juan Quintela, 2019/08/13
- Re: [Qemu-devel] [PATCH 6/6] RFH: We lost "connect" events, Daniel P . Berrangé, 2019/08/19
- Re: [Qemu-devel] [PATCH 6/6] RFH: We lost "connect" events, Juan Quintela, 2019/08/19
- Re: [Qemu-devel] [PATCH 6/6] RFH: We lost "connect" events, Daniel P . Berrangé, 2019/08/19
- Re: [Qemu-devel] [PATCH 6/6] RFH: We lost "connect" events, Juan Quintela, 2019/08/19
- Re: [Qemu-devel] [PATCH 6/6] RFH: We lost "connect" events, Peter Maydell, 2019/08/19
- Re: [Qemu-devel] [PATCH 6/6] RFH: We lost "connect" events, Daniel P . Berrangé, 2019/08/19
- Re: [Qemu-devel] [PATCH 6/6] RFH: We lost "connect" events, Juan Quintela, 2019/08/19
Re: [Qemu-devel] [PATCH 0/6] Fix multifd with big number of channels, no-reply, 2019/08/14
Re: [Qemu-devel] [PATCH 0/6] Fix multifd with big number of channels, Dr. David Alan Gilbert, 2019/08/14
Re: [Qemu-devel] [PATCH 0/6] Fix multifd with big number of channels, no-reply, 2019/08/14