[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/6] qapi: introduce the SysEmuTarget enumeratio
From: |
Laszlo Ersek |
Subject: |
Re: [Qemu-devel] [PATCH 0/6] qapi: introduce the SysEmuTarget enumeration |
Date: |
Wed, 25 Apr 2018 14:26:44 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0 |
On 04/25/18 00:03, address@hidden wrote:
> Hi,
>
> This series failed address@hidden build test. Please find the testing
> commands and
> their output below. If you have Docker installed, you can probably reproduce
> it
> locally.
[snip]
> qapi/qapi-types-misc.h:654:20: error: expected identifier or '(' before
> numeric constant
> CpuInfoX86 i386;
> ^
> qapi/qapi-types-misc.h:749:22: error: expected identifier or '(' before
> numeric constant
> CpuInfoOther i386;
> ^
> make: *** [/tmp/qemu-test/src/rules.mak:66: qapi/qapi-types-misc.o] Error 1
> make: *** Waiting for unfinished jobs....
> qapi/qapi-types-misc.h:654:20: error: expected identifier or '(' before
> numeric constant
> CpuInfoX86 i386;
> ^
> qapi/qapi-types-misc.h:749:22: error: expected identifier or '(' before
> numeric constant
> CpuInfoOther i386;
> ^
> make: *** [/tmp/qemu-test/src/rules.mak:66: qapi/qapi-types.o] Error 1
> qapi/qapi-types-misc.h:654:20: error: expected identifier or '(' before
> numeric constant
> CpuInfoX86 i386;
> ^
> qapi/qapi-types-misc.h:749:22: error: expected identifier or '(' before
> numeric constant
> CpuInfoOther i386;
> ^
Wow, is "i386" a macro on mingw, one that expands to a number? That is
incredibly st...range.
I do need this enum constant to be "i386" in the QAPI schema, because
that's what TARGET_NAME will map to.
Can I let the generator use the "q_" prefix here somehow?
Hmm, I think that trick is already happening for "sparc":
641 struct CpuInfo {
642 /* Members inherited from CpuInfoBase: */
643 char *qom_path;
644 int64_t thread_id;
645 bool has_props;
646 CpuInstanceProperties *props;
647 CpuInfoArch arch;
648 int64_t CPU;
649 bool current;
650 bool halted;
651 SysEmuTarget target;
652 /* Own members: */
653 union { /* union tag is @target */
654 CpuInfoX86 i386;
655 CpuInfoX86 x86_64;
656 CpuInfoSPARC q_sparc; <----------------- here
Alright, I'll add "i386" to "polluted_words" in "scripts/qapi/common.py".
Sigh, what else is there already... I see Eric's commit 86ae191163d4 :(
Laszlo
- Re: [Qemu-devel] [PATCH 4/6] qapi: change the type of TargetInfo.arch from string to enum SysEmuTarget, (continued)
- [Qemu-devel] [PATCH 6/6] qapi: discriminate CpuInfo[Fast] on SysEmuTarget, not CpuInfoArch, Laszlo Ersek, 2018/04/24
- Re: [Qemu-devel] [PATCH 6/6] qapi: discriminate CpuInfo[Fast] on SysEmuTarget, not CpuInfoArch, Markus Armbruster, 2018/04/25
- Re: [Qemu-devel] [PATCH 6/6] qapi: discriminate CpuInfo[Fast] on SysEmuTarget, not CpuInfoArch, Laszlo Ersek, 2018/04/25
- Re: [Qemu-devel] [PATCH 6/6] qapi: discriminate CpuInfo[Fast] on SysEmuTarget, not CpuInfoArch, Markus Armbruster, 2018/04/26
- Re: [Qemu-devel] [PATCH 6/6] qapi: discriminate CpuInfo[Fast] on SysEmuTarget, not CpuInfoArch, Laszlo Ersek, 2018/04/26
- Re: [Qemu-devel] [PATCH 6/6] qapi: discriminate CpuInfo[Fast] on SysEmuTarget, not CpuInfoArch, Markus Armbruster, 2018/04/26
- Re: [Qemu-devel] [PATCH 6/6] qapi: discriminate CpuInfo[Fast] on SysEmuTarget, not CpuInfoArch, Laszlo Ersek, 2018/04/26
Re: [Qemu-devel] [PATCH 0/6] qapi: introduce the SysEmuTarget enumeration, no-reply, 2018/04/26
- Re: [Qemu-devel] [PATCH 0/6] qapi: introduce the SysEmuTarget enumeration,
Laszlo Ersek <=