Re: [PATCH 0/2] backends/iommufd: Remove mutex

2024-01-04 Thread Cédric Le Goater 

On 1/2/24 13:32, Cédric Le Goater wrote:

Hello !

Coverity has some reports regarding the IOMMUFDBackend mutex. Since
the IOMMUFDBackend routines are called from the QEMU main thread, this
series simply suggests removing the mutex and rely on the BQL to
handle concurrent access.

Thanks,

C.

Cédric Le Goater (2):
   backends/iommufd: Remove check on number of backend users
   backends/iommufd: Remove mutex

  include/sysemu/iommufd.h |  2 --
  backends/iommufd.c   | 12 
  2 files changed, 14 deletions(-)




Applied to vfio-next.

Thanks,

C.

RE: [PATCH 0/2] backends/iommufd: Remove mutex

2024-01-03 Thread Duan, Zhenzhong 


>-Original Message-
>From: Cédric Le Goater 
>Sent: Tuesday, January 2, 2024 8:32 PM
>To: qemu-devel@nongnu.org
>Cc: Liu, Yi L ; Eric Auger ; Duan,
>Zhenzhong ; Cédric Le Goater
>
>Subject: [PATCH 0/2] backends/iommufd: Remove mutex
>
>Hello !
>
>Coverity has some reports regarding the IOMMUFDBackend mutex. Since
>the IOMMUFDBackend routines are called from the QEMU main thread, this
>series simply suggests removing the mutex and rely on the BQL to
>handle concurrent access.

For the whole series,
Reviewed-by: Zhenzhong Duan 

Thanks
Zhenzhong

>
>Thanks,
>
>C.
>
>Cédric Le Goater (2):
>  backends/iommufd: Remove check on number of backend users
>  backends/iommufd: Remove mutex
>
> include/sysemu/iommufd.h |  2 --
> backends/iommufd.c   | 12 
> 2 files changed, 14 deletions(-)
>
>--
>2.43.0

Re: [PATCH 0/2] backends/iommufd: Remove mutex

2024-01-02 Thread Eric Auger 
Hi Cédric,

On 1/2/24 13:32, Cédric Le Goater wrote:
> Hello !
>
> Coverity has some reports regarding the IOMMUFDBackend mutex. Since
> the IOMMUFDBackend routines are called from the QEMU main thread, this
> series simply suggests removing the mutex and rely on the BQL to
> handle concurrent access.
>
> Thanks,
>
> C.
>
> Cédric Le Goater (2):
>   backends/iommufd: Remove check on number of backend users
>   backends/iommufd: Remove mutex

for the series:
Reviewed-by: Eric Auger 

Thank you for the fix!

Eric
>
>  include/sysemu/iommufd.h |  2 --
>  backends/iommufd.c   | 12 
>  2 files changed, 14 deletions(-)
>