[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 00/11] qapi: add support for lists of native
From: |
mdroth |
Subject: |
Re: [Qemu-devel] [PATCH v3 00/11] qapi: add support for lists of native types |
Date: |
Wed, 15 May 2013 14:13:09 -0500 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Wed, May 15, 2013 at 02:05:58PM -0400, Luiz Capitulino wrote:
> On Wed, 15 May 2013 12:42:24 -0500
> mdroth <address@hidden> wrote:
>
> > On Wed, May 15, 2013 at 11:04:27AM -0400, Luiz Capitulino wrote:
> > > On Wed, 15 May 2013 09:32:37 -0500
> > > mdroth <address@hidden> wrote:
> > >
> > > > On Wed, May 15, 2013 at 09:17:46AM -0400, Luiz Capitulino wrote:
> > > > > On Fri, 10 May 2013 17:45:59 -0500
> > > > > Michael Roth <address@hidden> wrote:
> > > > >
> > > > > > These patches apply on top of qemu.git master, and can also be
> > > > > > obtained from:
> > > > > > git://github.com/mdroth/qemu.git qapi-native-lists
> > > > > >
> > > > > > Sending this now since a number of series have popped up in the
> > > > > > past that
> > > > > > wanted this, and Amos has some pending patches (query-mac-tables)
> > > > > > that rely
> > > > > > on this as well.
> > > > > >
> > > > > > These patches add support for specifying lists of native qapi types
> > > > > > (int/bool/str/number/int8/uint8/etc) like so:
> > > > > >
> > > > > > { 'type': 'foo',
> > > > > > 'data': { 'bar': ['int'] }}
> > > > > >
> > > > > > for a 'bar' field that is a list of type 'int',
> > > > > >
> > > > > > { 'type': 'foo2',
> > > > > > 'data': { 'bar2': ['str'] }}
> > > > > >
> > > > > > for a 'bar2' field that is a list of type 'str', and so on.
> > > > > >
> > > > > > This uses linked list types for the native C representations, just
> > > > > > as we do
> > > > > > for complex schema-defined types. In the future we may add schema
> > > > > > annotations
> > > > > > of some sort to specify a more natural/efficient array type for the
> > > > > > C
> > > > > > representations, but this should serve the majority of uses-cases
> > > > > > for now.
> > > > >
> > > > > I'm getting a build breakage when building all targets:
> > > >
> > > > Hmm, tested all target builds and didn't/don't see any issues. Is this a
> > > > clean build? Can you confirm whether or not int8List/etc are declared in
> > > > qapi-types.h in your current build dir?
> > >
> > > Yes, it's a clean build and yes, the int?List declarations are being
> > > generated.
> > > Tested a little bit more and this also happens with make -j1 and when
> > > building only one target (x86_64 in my case).
> >
> > The only way I've managed to reproduce this is by having a stale
> > qapi-types.h hanging around in $SRC_DIR while i'm building in a
> > different $BUILD_DIR. Can you confirm that's not what's happening here?
>
> Yes, it was :( I had an old qapi-types.h in $SRC_DIR, but was building
> in a different $BUILD_DIR. I am really sorry for having wasted your time
> on this.
>
> I've applied this series to qmp-next branch, but I have no idea how I ended
> up with a qapi-types.h file in $SRC_DIR. I have an alias to build qemu that
> I use for several months now...
>
No problem, only thought to check that scenario because it happens to me
all the time :)
- [Qemu-devel] [PATCH 09/11] qapi: add native list coverage for visitor serialization tests, (continued)
- [Qemu-devel] [PATCH 09/11] qapi: add native list coverage for visitor serialization tests, Michael Roth, 2013/05/10
- [Qemu-devel] [PATCH 10/11] qapi: add native list coverage for QMP output visitor tests, Michael Roth, 2013/05/10
- [Qemu-devel] [PATCH 11/11] qapi: add native list coverage for QMP input visitor tests, Michael Roth, 2013/05/10
- Re: [Qemu-devel] [PATCH v3 00/11] qapi: add support for lists of native types, Laszlo Ersek, 2013/05/13
- Re: [Qemu-devel] [PATCH v3 00/11] qapi: add support for lists of native types, Amos Kong, 2013/05/13
- Re: [Qemu-devel] [PATCH v3 00/11] qapi: add support for lists of native types, Luiz Capitulino, 2013/05/15
- Re: [Qemu-devel] [PATCH v3 00/11] qapi: add support for lists of native types, mdroth, 2013/05/15
- Re: [Qemu-devel] [PATCH v3 00/11] qapi: add support for lists of native types, Luiz Capitulino, 2013/05/15
- Re: [Qemu-devel] [PATCH v3 00/11] qapi: add support for lists of native types, mdroth, 2013/05/15
- Re: [Qemu-devel] [PATCH v3 00/11] qapi: add support for lists of native types, Luiz Capitulino, 2013/05/15
- Re: [Qemu-devel] [PATCH v3 00/11] qapi: add support for lists of native types,
mdroth <=
- Re: [Qemu-devel] [PATCH v3 00/11] qapi: add support for lists of native types, Laszlo Ersek, 2013/05/16
- Re: [Qemu-devel] [PATCH v3 00/11] qapi: add support for lists of native types, Peter Maydell, 2013/05/16
- Re: [Qemu-devel] [PATCH v3 00/11] qapi: add support for lists of native types, mdroth, 2013/05/17