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

2020-04-13 Thread Yuval Shaia
For the hw/rdma stuff: Reviewed-by: Yuval Shaia Tested-by: Yuval Shaia Thanks, Yuval On Wed, 1 Apr 2020 at 19:20, Simran Singhal wrote: > Replace manual lock()/unlock() calls with lock guard macros > (QEMU_LOCK_GUARD/WITH_QEMU_LOCK_GUARD). > > Signed-off-by: Simran Singhal > --- >

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

2020-04-03 Thread Stefan Hajnoczi
On Wed, Apr 01, 2020 at 09:50:23PM +0530, Simran Singhal wrote: > Replace manual lock()/unlock() calls with lock guard macros > (QEMU_LOCK_GUARD/WITH_QEMU_LOCK_GUARD). > > Signed-off-by: Simran Singhal > --- > hw/hyperv/hyperv.c | 15 ++--- > hw/rdma/rdma_backend.c | 50

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

2020-04-03 Thread Simran Singhal
Hello Stefan On Fri, Apr 3, 2020 at 6:30 PM Stefan Hajnoczi wrote: > On Wed, Apr 01, 2020 at 11:57:34AM -0700, no-re...@patchew.org wrote: > > Patchew URL: > https://patchew.org/QEMU/20200401162023.GA15912@simran-Inspiron-5558/ > > > > > > > > Hi, > > > > This series failed the asan build test.

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

2020-04-03 Thread Stefan Hajnoczi
On Wed, Apr 01, 2020 at 11:57:34AM -0700, no-re...@patchew.org 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

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

2020-04-01 Thread no-reply
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

[PATCH] lockable: Replace locks with lock guard macros

2020-04-01 Thread Simran Singhal
Replace manual lock()/unlock() calls with lock guard macros (QEMU_LOCK_GUARD/WITH_QEMU_LOCK_GUARD). Signed-off-by: Simran Singhal --- hw/hyperv/hyperv.c | 15 ++--- hw/rdma/rdma_backend.c | 50 +- hw/rdma/rdma_rm.c | 3 +--