[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [Bug 1179731] [NEW] is networking broken on windows hos
From: |
TeLeMan |
Subject: |
Re: [Qemu-devel] [Bug 1179731] [NEW] is networking broken on windows hosts? |
Date: |
Thu, 16 May 2013 13:52:49 +0800 |
The patch is working on 134a03e0b3d34b01b68107104c525c3bff1211d4 and
is not working from cbff4b342b000a7642125dbdabf61113e05eee44.
--
SUN OF A BEACH
On Wed, May 15, 2013 at 4:37 PM, Paolo Bonzini <address@hidden> wrote:
> Il 15/05/2013 03:38, TeLeMan ha scritto:
>> On Tue, May 14, 2013 at 7:55 PM, Paolo Bonzini <address@hidden> wrote:
>>> Il 14/05/2013 13:48, Paolo Bonzini ha scritto:
>>>> Il 14/05/2013 13:39, TeLeMan ha scritto:
>>>>> On Tue, May 14, 2013 at 6:46 PM, Paolo Bonzini <address@hidden> wrote:
>>>>>> Il 14/05/2013 12:24, TeLeMan ha scritto:
>>>>>>> On Tue, May 14, 2013 at 3:51 PM, Stefan Hajnoczi <address@hidden> wrote:
>>>>>>>> On Tue, May 14, 2013 at 12:02:24AM -0000, therock247uk wrote:
>>>>>>>>> just wondering as i just compiled the latest git and qemu goes into
>>>>>>>>> none
>>>>>>>>> responding mode when i try to do any networking stuff on guests (both
>>>>>>>>> linux and windows)
>>>>>>>>
>>>>>>>> Works for me on qemu.git/master on Linux:
>>>>>>>>
>>>>>>>> $ git rev-parse HEAD
>>>>>>>> b087143b4d010451208264b7c841436aafe1cbb1
>>>>>>>> $ x86_64-softmmu/qemu-system-x86_64 -m 1024 -enable-kvm -cpu host \
>>>>>>>> -drive if=virtio,cache=none,file=test.img
>>>>>>>>
>>>>>>>> Please include more information, like the QEMU command-line and commit
>>>>>>>> ID.
>>>>>>>>
>>>>>>>> Stefan
>>>>>>>>
>>>>>>>
>>>>>>> This regression occurs on the Windows host. SLIRP hangs in sorecvfrom().
>>>>>>
>>>>>> Can you bisect it?
>>>>>>
>>>>>> Paolo
>>>>>>
>>>>> The first break is the commit
>>>>> 5e3bc735d93dd23f074b5116fd11e1ad8cd4962f. But it need more packets
>>>>> than HEAD to reproduce this regression.
>>>>
>>>> Please check if this partial revert of that commit fixes it:
>>>
>>> Yeah, this should work... WSAEventSelect is edge-triggered and the
>>> event will not be signaled if the socket handler does not consume all
>>> the data in the socket buffer.
>>
>> Unfortunately, it does not work.
>
> Ok... as you can see the patch is just moving a block of code just
> before g_main_context_prepare(context, &max_priority).
>
> Can you please try doing the same on top of these six commits:
>
> 134a03e0b3d34b01b68107104c525c3bff1211d4
> cbff4b342b000a7642125dbdabf61113e05eee44
> 48ce11ff972c733afaed3e2a2613a2e56081ec92
> 8917c3bdba37d6fe4393db0fad3fabbde9530d6b
> a3e4b4a8091cc4fcf7cb619570c72c54c2d6a6e9
> 9cbaacf999b01b27dc3a22502705178057af66de
>
> Paolo
- [Qemu-devel] [PATCH v4 0/8] block: drive-backup live backup command, Stefan Hajnoczi, 2013/05/16
- [Qemu-devel] [PATCH v4 3/8] block: add drive-backup QMP command, Stefan Hajnoczi, 2013/05/16
- [Qemu-devel] [PATCH v4 7/8] blockdev: add Abort transaction, Stefan Hajnoczi, 2013/05/16
- [Qemu-devel] [PATCH v4 2/8] block: add basic backup support to block driver, Stefan Hajnoczi, 2013/05/16
- [Qemu-devel] [PATCH v4 5/8] blockdev: rename BlkTransactionStates to singular, Stefan Hajnoczi, 2013/05/16
- [Qemu-devel] [PATCH v4 4/8] qemu-iotests: add 055 drive-backup test case, Stefan Hajnoczi, 2013/05/16
- [Qemu-devel] [PATCH v4 1/8] block: add bdrv_add_before_write_notifier(), Stefan Hajnoczi, 2013/05/16
- [Qemu-devel] [PATCH v4 6/8] blockdev: add DriveBackup transaction, Stefan Hajnoczi, 2013/05/16
- [Qemu-devel] [PATCH v4 8/8] qemu-iotests: test 'drive-backup' transaction in 055, Stefan Hajnoczi, 2013/05/16