qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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


From: Simran Singhal
Subject: Re: [PATCH] lockable: Replace locks with lock guard macros
Date: Fri, 3 Apr 2020 18:35:24 +0530

Hello Stefan

On Fri, Apr 3, 2020 at 6:30 PM Stefan Hajnoczi <address@hidden> wrote:
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.

Yes, I saw that patch series.
I'll send the next version after Daniel Brodsky fixes the unused variable warnings.

Thanks & Regards
Simran Singhal

Stefan

reply via email to

[Prev in Thread] Current Thread [Next in Thread]