[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/6] fix w32 sockets
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 0/6] fix w32 sockets |
Date: |
Tue, 03 Apr 2012 13:06:31 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:11.0) Gecko/20120329 Thunderbird/11.0.1 |
Il 20/03/2012 10:49, Paolo Bonzini ha scritto:
> The w32 main loop has been mostly broken by the introduction of the
> glib main loop. glib's g_poll does not use sockets on w32, so we
> need a separate approach.
>
> Patch 1 is a simple cleanup that is needed later in the series.
>
> Patch 2 and patch 3 completely separate the way the main loop waits
> on POSIX and w32 systems, and drop glib source handling from the w32
> main loop.
>
> Patch 4 fixes a longstanding bug in how sockets are handled, also
> simplifying the code in the process. On top of this simplification,
> patch 5 starts using g_poll in the w32 main loop and patch 6 adds
> back glib source handling.
>
> I didn't test this in the conditions explained in bug 916720, but I
> tested both a TCP monitor and an stdio monitor and both work (under
> Wine that is).
>
> Stefan, can you please take care of shepherding the patches in
> (pinging etc.)?
>
> Paolo Bonzini (6):
> slirp: use socket_set_nonblock
> main loop: use msec-based timeout in glib_select_fill
> main-loop: disable fd_set-based glib integration under w32
> main-loop: interrupt wait when data arrives on a socket
> main-loop: replace WaitForMultipleObjects with g_poll
> main-loop: integrate glib sources for w32
Patch 1 is now in separately through the slirp tree, so the other 5 can
now be applied. Blue, can you commit them?
Paolo
- Re: [Qemu-devel] [PATCH 0/6] fix w32 sockets,
Paolo Bonzini <=