Re: [PATCH] lockable: Replace locks with lock guard macros

[Stefan Hajnoczi]

On Wed, Apr 01, 2020 at 11:57:34AM -0700, address@hidden wrote:
> Patchew URL: 
> https://patchew.org/QEMU/20200401162023.GA15912@simran-Inspiron-5558/
> 
> 
> 
> Hi,
> 
> This series failed the asan 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
> export ARCH=x86_64
> make docker-image-fedora V=1 NETWORK=1
> time make docker-test-debug@fedora TARGET_LIST=x86_64-softmmu J=14 NETWORK=1
> === TEST SCRIPT END ===
> 
>   CC      x86_64-softmmu/hw/i386/x86.o
>   CC      x86_64-softmmu/hw/i386/pc.o
>   CC      x86_64-softmmu/hw/i386/pc_sysfw.o
> /tmp/qemu-test/src/hw/rdma/rdma_utils.c:74:5: error: unused variable 
> 'qemu_lockable_auto__COUNTER__' [-Werror,-Wunused-variable]
>     QEMU_LOCK_GUARD(&list->lock);
>     ^
> /tmp/qemu-test/src/include/qemu/lockable.h:173:29: note: expanded from macro 
> 'QEMU_LOCK_GUARD'

This patch series depends on Daniel Brodsky's "[PATCH v4 0/2] Replaced
locks with lock guard macros" patch series.  I think Daniel intends to
fix the unused variable warnings.

Stefan

signature.asc
Description: PGP signature