[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 00/31] acpi, vhost, misc: fixes, features
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-devel] [PULL 00/31] acpi, vhost, misc: fixes, features |
Date: |
Fri, 8 Jun 2018 14:52:23 -0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.8.0 |
On 06/04/2018 07:28 AM, Peter Maydell wrote:
> On 1 June 2018 at 17:26, Michael S. Tsirkin <address@hidden> wrote:
>> The following changes since commit 63b88968f139b6a77f2f81e6f1eedf70c0170a85:
>>
>> intel-iommu: rework the page walk logic (2018-05-23 17:34:05 +0300)
>>
>> are available in the Git repository at:
>>
>> git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git tags/for_upstream
>>
>> for you to fetch changes up to 25b1d45a1975fd8624c37b5bf42e8502ccf53460:
>>
>> vhost-blk: turn on pre-defined RO feature bit (2018-06-01 19:20:38 +0300)
>>
>> ----------------------------------------------------------------
>> acpi, vhost, misc: fixes, features
>>
>> vDPA support, fix to vhost blk RO bit handling, some include path
>> cleanups, NFIT ACPI table.
>>
>> Signed-off-by: Michael S. Tsirkin <address@hidden>
>>
>> ----------------------------------------------------------------
>
> Applied, thanks.
Not sure this is a problem, but something experienced after rebasing:
#1
/source/qemu/qemu-nbd.c: In function ‘version’:
/source/qemu/qemu-nbd.c:133:6: error: expected ‘)’ before
‘QEMU_FULL_VERSION’
"%s " QEMU_FULL_VERSION "\n"
^~~~~~~~~~~~~~~~~~
)
/source/qemu/qemu-nbd.c:133:3: error: format ‘%s’ expects a matching
‘char *’ argument [-Werror=format=]
"%s " QEMU_FULL_VERSION "\n"
~^
cc1: all warnings being treated as errors
It took me a while to figure this out, probably because today is Friday.
$ git status --ignored show me I had a "qemu-version.h" generated long
time ago in the source directory.
- I only do out-of-tree builds.
- sometimes I run "make help/docker/vm-test" and friends in the source
directory, and my 'make' is aliased as 'make -jN'.
I doubt those make calls generate this file, as ./configure generates
it, so I probably messed with a ./configure in source dir.
Now, building out-of-tree dirs was finding the old /qemu-version.h
before the generated one in current dir.
I was not noticing it due to the /qemu-version.h entry in /.gitignore .
#2
CC vl.o
/source/qemu/vl.c: In function ‘main’:
/source/qemu/vl.c:3052:18: error: ‘QEMU_OPTION_blockdev’ undeclared
(first use in this function); did you mean ‘QEMU_OPTION_clock’?
case QEMU_OPTION_blockdev:
^~~~~~~~~~~~~~~~~~~~
QEMU_OPTION_clock
/source/qemu/vl.c:3052:18: note: each undeclared identifier is reported
only once for each function it appears in
/source/qemu/vl.c:3171:18: error: ‘QEMU_OPTION_nic’ undeclared (first
use in this function); did you mean ‘QEMU_OPTION_rtc’?
case QEMU_OPTION_nic:
^~~~~~~~~~~~~~~
QEMU_OPTION_rtc
/source/qemu/vl.c:3574:18: error: ‘QEMU_OPTION_preconfig’ undeclared
(first use in this function); did you mean ‘QEMU_OPTION_writeconfig’?
case QEMU_OPTION_preconfig:
^~~~~~~~~~~~~~~~~~~~~
QEMU_OPTION_writeconfig
This is the same story, but with /qemu-options.def (also in .gitignore).
So 'rm ${srcdir}/qemu-version.h ${srcdir}/qemu-options.def' fixed my
problems.
Should we add a check for this kind of mistakes when running out-of-tree
builds? I'll try something.
Regards,
Phil.
- [Qemu-devel] [PULL 24/31] colo: use local path for local headers, (continued)
- [Qemu-devel] [PULL 24/31] colo: use local path for local headers, Michael S. Tsirkin, 2018/06/01
- [Qemu-devel] [PULL 25/31] qga: use local path for local headers, Michael S. Tsirkin, 2018/06/01
- [Qemu-devel] [PULL 27/31] arch_init: sort architectures, Michael S. Tsirkin, 2018/06/01
- [Qemu-devel] [PULL 28/31] tests/.gitignore: add entry for generated file, Michael S. Tsirkin, 2018/06/01
- [Qemu-devel] [PULL 29/31] nvdimm, acpi: support NFIT platform capabilities, Michael S. Tsirkin, 2018/06/01
- [Qemu-devel] [PULL 26/31] ui: use local path for local headers, Michael S. Tsirkin, 2018/06/01
- [Qemu-devel] [PULL 23/31] migration: use local path for local headers, Michael S. Tsirkin, 2018/06/01
- [Qemu-devel] [PULL 30/31] ACPI testing: test NFIT platform capabilities, Michael S. Tsirkin, 2018/06/01
- [Qemu-devel] [PULL 31/31] vhost-blk: turn on pre-defined RO feature bit, Michael S. Tsirkin, 2018/06/01
- Re: [Qemu-devel] [PULL 00/31] acpi, vhost, misc: fixes, features, Peter Maydell, 2018/06/04
- Re: [Qemu-devel] [PULL 00/31] acpi, vhost, misc: fixes, features,
Philippe Mathieu-Daudé <=
- Re: [Qemu-devel] [PULL 00/31] acpi, vhost, misc: fixes, features, Philippe Mathieu-Daudé, 2018/06/08
- Re: [Qemu-devel] [PULL 00/31] acpi, vhost, misc: fixes, features, Peter Maydell, 2018/06/09
- Re: [Qemu-devel] [PULL 00/31] acpi, vhost, misc: fixes, features, Thomas Huth, 2018/06/06