[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/3] proper locking on bitmap add/remove paths
From: |
John Snow |
Subject: |
Re: [Qemu-devel] [PATCH 0/3] proper locking on bitmap add/remove paths |
Date: |
Wed, 11 Sep 2019 17:46:48 -0400 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.8.0 |
On 9/11/19 10:09 AM, Vladimir Sementsov-Ogievskiy wrote:
> 10.09.2019 23:37, address@hidden wrote:
>> Patchew URL: https://patchew.org/QEMU/address@hidden/
>>
>>
>>
>> Hi,
>>
>> This series failed the docker-quick@centos7 build test. Please find the
>> testing commands and
>> their output below. If you have Docker installed, you can probably reproduce
>> it
>> locally.
>>
>> === TEST SCRIPT BEGIN ===
>> #!/bin/bash
>> make docker-image-centos7 V=1 NETWORK=1
>> time make docker-test-quick@centos7 SHOW_ENV=1 J=14 NETWORK=1
(Was patchew even using clang?)
>> === TEST SCRIPT END ===
>>
>> libudev no
>> default devices yes
>>
>> warning: Python 2 support is deprecated
>> warning: Python 3 will be required for building future versions of QEMU
>>
>> NOTE: cross-compilers enabled: 'cc'
>> GEN x86_64-softmmu/config-devices.mak.tmp
>> ---
>> CC block/qed-cluster.o
>> CC block/qed-check.o
>> /tmp/qemu-test/src/block/qcow2-bitmap.c: In function
>> 'qcow2_co_remove_persistent_dirty_bitmap':
>> /tmp/qemu-test/src/block/qcow2-bitmap.c:502:8: error: 'bm' may be used
>> uninitialized in this function [-Werror=maybe-uninitialized]
>> if (bm == NULL) {
>> ^
>> /tmp/qemu-test/src/block/qcow2-bitmap.c:1413:18: note: 'bm' was declared here
>>
>>
>> The full log is available at
>> http://patchew.org/logs/address@hidden/testing.docker-quick@centos7/?type=message.
>> ---
>> Email generated automatically by Patchew [https://patchew.org/].
>> Please send your feedback to address@hidden
>>
>
> Who knows, how to clang Qemu?
>
> I try with
> ./configure --target-list=x86_64-softmmu --enable-debug --disable-virtfs
> --enable-werror --audio-drv-list=oss --extra-cflags=-Wall --enable-sanitizers
> --cc=clang --cxx=clang++
> make -j9
>
../../configure --target-list="x86_64-softmmu" --cc=clang --cxx=clang++
--host-cc=clang
works OK for me in fedora 30.
--target-list="x86_64-softmmu" --cc=clang --cxx=clang++ --host-cc=clang
--enable-werror --extra-cflags=-Wall
Seems OK too, and finally adding
--enable-sanitizers
also appears to work alright.
clang version 8.0.0 (Fedora 8.0.0-1.fc30)
Target: x86_64-unknown-linux-gnu
Thread model: posix
InstalledDir: /usr/bin
Maybe something bad in your ccache or some intermediate state in your
build dir. I use separate build directories for gcc and clang just in case.
--js