On Tue, 06/16 14:57, Stefan Hajnoczi wrote:
> On Wed, Jun 10, 2015 at 04:59:44PM +0800, Fam Zheng wrote:
> > diff --git a/iohandler.c b/iohandler.c
> > index 826f713..f5d7704 100644
> > --- a/iohandler.c
> > +++ b/iohandler.c
> > @@ -32,111 +32,39 @@
> >  #include <sys/wait.h>
> >  #endif
> >  
> > -typedef struct IOHandlerRecord {
> > -    IOHandler *fd_read;
> > -    IOHandler *fd_write;
> > -    void *opaque;
> > -    QLIST_ENTRY(IOHandlerRecord) next;
> > -    int fd;
> > -    int pollfds_idx;
> > -    bool deleted;
> > -} IOHandlerRecord;
> > +static AioContext *iohandler_ctx;
> 
> Please include a comment explaining why this AioContext is needed in
> addition to the global qemu_aio_context.

OK!

It's needed because iohandlers shouldn't be polled by aio_poll().

Fam

Reply via email to