Re: [Qemu-devel] [PULL v2 00/52] Misc patches for 2017-01-12

[Paolo Bonzini <pbonzini@redhat.com>]

On 12/01/2018 14:31, Peter Maydell wrote:
> On 12 January 2018 at 12:24, Paolo Bonzini <pbonzini@redhat.com> wrote:
>> The following changes since commit 997eba28a3ed5400a80f754bf3a1c8044b75b9ff:
>>
>>   Merge remote-tracking branch 'remotes/pmaydell/tags/pull-target-arm-20180111' into staging (2018-01-11 14:34:41 +0000)
>>
>> are available in the git repository at:
>>
>>
>>   git://github.com/bonzini/qemu.git tags/for-upstream
>>
>> for you to fetch changes up to a88ae3be7bf0550adf877f5422c837dbf4bf4a39:
>>
>>   scripts/analyse-locks-simpletrace.py: script to analyse lock times (2018-01-12 13:22:05 +0100)
>>
>> ----------------------------------------------------------------
>> * QemuMutex tracing improvements (Alex)
>> * ram_addr_t optimization (David)
>> * SCSI fixes (Fam, Stefan, me)
>> * do {} while (0) fixes (Eric)
>> * KVM fix for PMU (Jan)
>> * memory leak fixes from ASAN (Marc-André)
>> * migration fix for HPET, icount, loadvm (Maria, Pavel)
>> * hflags fixes (me, Tao)
>> * block/iscsi uninitialized variable (Peter L.)
>> * full support for GMainContexts in character devices (Peter Xu)
>> * more boot-serial-test (Thomas)
>> * Memory leak fix (Zhecheng)
>>
>> ----------------------------------------------------------------
> 
> This fails to build on x86-64 Linux, with an --enable-debug build, using
> gcc (Ubuntu 5.4.0-6ubuntu1~16.04.5) 5.4.0 20160609
> 
>   CC      util/coroutine-ucontext.o
> /home/petmay01/linaro/qemu-for-merges/util/coroutine-ucontext.c: In
> function ‘finish_switch_fiber’:
> /home/petmay01/linaro/qemu-for-merges/util/coroutine-ucontext.c:75:5:
> error: implicit declaration of function
> ‘__sanitizer_finish_switch_fiber’
> [-Werror=implicit-function-declaration]
>      __sanitizer_finish_switch_fiber(fake_stack_save, &bottom_old, &size_old);
>      ^
> /home/petmay01/linaro/qemu-for-merges/util/coroutine-ucontext.c:75:5:
> error: nested extern declaration of ‘__sanitizer_finish_switch_fiber’
> [-Werror=nested-externs]
> /home/petmay01/linaro/qemu-for-merges/util/coroutine-ucontext.c: In
> function ‘start_switch_fiber’:
> /home/petmay01/linaro/qemu-for-merges/util/coroutine-ucontext.c:88:5:
> error: implicit declaration of function
> ‘__sanitizer_start_switch_fiber’
> [-Werror=implicit-function-declaration]
>      __sanitizer_start_switch_fiber(fake_stack_save, bottom, size);
>      ^
> /home/petmay01/linaro/qemu-for-merges/util/coroutine-ucontext.c:88:5:
> error: nested extern declaration of ‘__sanitizer_start_switch_fiber’
> [-Werror=nested-externs]
> 
> 
> It looks like older versions of gcc and the sanitizer don't
> provide those switch_fiber functions.

I'll drop that patch.  Marc-André, can you refine the configure test?

Paolo