Re: [Qemu-devel] [PATCH 10/10] aio-win32: add support for sockets

2014-09-15 Thread Paolo Bonzini
Il 15/09/2014 03:18, TeLeMan ha scritto: > On Sat, Sep 13, 2014 at 6:33 PM, Paolo Bonzini wrote: >> Il 13/09/2014 04:22, TeLeMan ha scritto: >>> On Fri, Sep 12, 2014 at 6:05 PM, Paolo Bonzini wrote: Il 12/09/2014 03:39, TeLeMan ha scritto: > On Wed, Jul 9, 2014 at 5:53 PM, Paolo Bonzini

Re: [Qemu-devel] [PATCH 10/10] aio-win32: add support for sockets

2014-09-14 Thread TeLeMan
On Sat, Sep 13, 2014 at 6:33 PM, Paolo Bonzini wrote: > Il 13/09/2014 04:22, TeLeMan ha scritto: >> On Fri, Sep 12, 2014 at 6:05 PM, Paolo Bonzini wrote: >>> Il 12/09/2014 03:39, TeLeMan ha scritto: On Wed, Jul 9, 2014 at 5:53 PM, Paolo Bonzini wrote: > diff --git a/aio-win32.c b/aio-wi

Re: [Qemu-devel] [PATCH 10/10] aio-win32: add support for sockets

2014-09-13 Thread Paolo Bonzini
Il 13/09/2014 04:22, TeLeMan ha scritto: > On Fri, Sep 12, 2014 at 6:05 PM, Paolo Bonzini wrote: >> Il 12/09/2014 03:39, TeLeMan ha scritto: >>> On Wed, Jul 9, 2014 at 5:53 PM, Paolo Bonzini wrote: diff --git a/aio-win32.c b/aio-win32.c index 4542270..61e3d2d 100644 --- a/aio-win32

Re: [Qemu-devel] [PATCH 10/10] aio-win32: add support for sockets

2014-09-12 Thread TeLeMan
On Fri, Sep 12, 2014 at 6:05 PM, Paolo Bonzini wrote: > Il 12/09/2014 03:39, TeLeMan ha scritto: >> On Wed, Jul 9, 2014 at 5:53 PM, Paolo Bonzini wrote: >>> diff --git a/aio-win32.c b/aio-win32.c >>> index 4542270..61e3d2d 100644 >>> --- a/aio-win32.c >>> +++ b/aio-win32.c >>> +bool was_dispa

Re: [Qemu-devel] [PATCH 10/10] aio-win32: add support for sockets

2014-09-12 Thread Paolo Bonzini
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Il 12/09/2014 14:51, Stefan Hajnoczi ha scritto: >>> >>> when (ret - WAIT_OBJECT_0) >= count and have_select_revents is >>> true, the following events[ret - WAIT_OBJECT_0] will be >>> overflowed. > Thanks for your review. Paolo has hardware problems

Re: [Qemu-devel] [PATCH 10/10] aio-win32: add support for sockets

2014-09-12 Thread Stefan Hajnoczi
On Fri, Sep 12, 2014 at 09:39:16AM +0800, TeLeMan wrote: > On Wed, Jul 9, 2014 at 5:53 PM, Paolo Bonzini wrote: > > Uses the same select/WSAEventSelect scheme as main-loop.c. > > WSAEventSelect() is edge-triggered, so it cannot be used > > directly, but it is still used as a way to exit from a > >

Re: [Qemu-devel] [PATCH 10/10] aio-win32: add support for sockets

2014-09-12 Thread Paolo Bonzini
Il 12/09/2014 03:39, TeLeMan ha scritto: > On Wed, Jul 9, 2014 at 5:53 PM, Paolo Bonzini wrote: >> diff --git a/aio-win32.c b/aio-win32.c >> index 4542270..61e3d2d 100644 >> --- a/aio-win32.c >> +++ b/aio-win32.c >> +bool was_dispatching, progress, have_select_revents, first; > have_select_rev

Re: [Qemu-devel] [PATCH 10/10] aio-win32: add support for sockets

2014-09-11 Thread TeLeMan
On Wed, Jul 9, 2014 at 5:53 PM, Paolo Bonzini wrote: > Uses the same select/WSAEventSelect scheme as main-loop.c. > WSAEventSelect() is edge-triggered, so it cannot be used > directly, but it is still used as a way to exit from a > blocking g_poll(). > > Before g_poll() is called, we poll sockets

Re: [Qemu-devel] [PATCH 10/10] aio-win32: add support for sockets

2014-09-11 Thread TeLeMan
On Wed, Jul 9, 2014 at 5:53 PM, Paolo Bonzini wrote: > Uses the same select/WSAEventSelect scheme as main-loop.c. > WSAEventSelect() is edge-triggered, so it cannot be used > directly, but it is still used as a way to exit from a > blocking g_poll(). > > Before g_poll() is called, we poll sockets

[Qemu-devel] [PATCH 10/10] aio-win32: add support for sockets

2014-07-09 Thread Paolo Bonzini
Uses the same select/WSAEventSelect scheme as main-loop.c. WSAEventSelect() is edge-triggered, so it cannot be used directly, but it is still used as a way to exit from a blocking g_poll(). Before g_poll() is called, we poll sockets with a non-blocking select() to achieve the level-triggered seman