[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 0/7] Block patches
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PULL 0/7] Block patches |
Date: |
Mon, 9 Nov 2015 10:01:18 +0000 |
User-agent: |
Mutt/1.5.23 (2015-06-09) |
On Mon, Nov 09, 2015 at 08:35:56AM +0100, Markus Armbruster wrote:
> Fam Zheng <address@hidden> writes:
>
> > On Fri, 11/06 18:07, Peter Maydell wrote:
> >> On 6 November 2015 at 17:52, Stefan Hajnoczi <address@hidden> wrote:
> >> > The following changes since commit
> >> > 4b59f39bc9a03afcc74b2fa28da7c3189fca507c:
> >> >
> >> > Merge remote-tracking branch
> >> > 'remotes/mjt/tags/pull-trivial-patches-2015-11-06' into staging
> >> > (2015-11-06 12:50:24 +0000)
> >> >
> >> > are available in the git repository at:
> >> >
> >> > git://github.com/stefanha/qemu.git tags/block-pull-request
> >> >
> >> > for you to fetch changes up to 6f707181b1bd6ccf2d2fd9397039c7ef6fa4a9fd:
> >> >
> >> > blockdev: acquire AioContext in hmp_commit() (2015-11-06 15:41:00
> >> > +0000)
> >> >
> >> > ----------------------------------------------------------------
> >>
> >> Build failure on OSX :-(
> >>
> >> /Users/pm215/src/qemu-for-merges/aio-posix.c CC block/qcow.o
> >> :442:37: error: no member named 'epollfd' in 'struct AioContext'
> >> epoll_handler.pfd.fd = ctx->epollfd;
> >> ~~~ ^
> >>
> >
> > :(
> >
> > I think it is harmless to always include this member in AioContext. Stefan,
> > could you squash this into patch 5? Thanks!
> >
> > Fam
> >
> > ---
> >
> > diff --git a/include/block/aio.h b/include/block/aio.h
> > index 91737d5..735f1f8 100644
> > --- a/include/block/aio.h
> > +++ b/include/block/aio.h
> > @@ -124,11 +124,9 @@ struct AioContext {
> > QEMUTimerListGroup tlg;
> >
> > int external_disable_cnt;
> > -#ifdef CONFIG_EPOLL
> > int epollfd;
> > bool epoll_enabled;
> > bool epoll_available;
> > -#endif
> > };
>
> Replace by the ifdeffery by a comment pointing to CONFIG_EPOLL, perhaps?
Done. Will send a v2 pull request.
Stefan
signature.asc
Description: PGP signature
- [Qemu-devel] [PULL 1/7] dataplane: simplify indirect descriptor read, (continued)
- [Qemu-devel] [PULL 1/7] dataplane: simplify indirect descriptor read, Stefan Hajnoczi, 2015/11/06
- [Qemu-devel] [PULL 2/7] dataplane: support non-contigious s/g, Stefan Hajnoczi, 2015/11/06
- [Qemu-devel] [PULL 3/7] aio: Introduce aio_external_disabled, Stefan Hajnoczi, 2015/11/06
- [Qemu-devel] [PULL 4/7] aio: Introduce aio_context_setup, Stefan Hajnoczi, 2015/11/06
- [Qemu-devel] [PULL 5/7] aio: Introduce aio-epoll.c, Stefan Hajnoczi, 2015/11/06
- [Qemu-devel] [PULL 7/7] blockdev: acquire AioContext in hmp_commit(), Stefan Hajnoczi, 2015/11/06
- [Qemu-devel] [PULL 6/7] monitor: add missed aio_context_acquire into vm_completion call, Stefan Hajnoczi, 2015/11/06
- Re: [Qemu-devel] [PULL 0/7] Block patches, Peter Maydell, 2015/11/06