[v5] intel_iommu: Fix locking issues

[PATCH v5 0/2] intel_iommu: Fix locking issues

CLEMENT MATHIEU--DRIF posted 2 patches 6 months, 2 weeks ago

Diff against v4
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20250430124750.240412-1-clement.mathieu--drif@eviden.com

Maintainers: "Michael S. Tsirkin" <mst@redhat.com>, Jason Wang <jasowang@redhat.com>, Yi Liu <yi.l.liu@intel.com>, "Clément Mathieu--Drif" <clement.mathieu--drif@eviden.com>, Marcel Apfelbaum <marcel.apfelbaum@gmail.com>, Paolo Bonzini <pbonzini@redhat.com>, Richard Henderson <richard.henderson@linaro.org>, Eduardo Habkost <eduardo@habkost.net>

hw/i386/intel_iommu.c | 35 +++++++++++++++++++++++++----------
1 file changed, 25 insertions(+), 10 deletions(-)

Expand all Fold all

[PATCH v5 0/2] intel_iommu: Fix locking issues

Posted by CLEMENT MATHIEU--DRIF 6 months, 2 weeks ago

This series introduces 2 fixes and improves locking style
consistency in the VT-d device.

Changes since v4:
	- Re-check if the address space is present once both bql and
	  iommu lock are held.


Clement Mathieu--Drif (2):
  intel_iommu: Use BQL_LOCK_GUARD to manage cleanup automatically
  intel_iommu: Take locks when looking for and creating address spaces

 hw/i386/intel_iommu.c | 35 +++++++++++++++++++++++++----------
 1 file changed, 25 insertions(+), 10 deletions(-)

-- 
2.49.0

RE: [PATCH v5 0/2] intel_iommu: Fix locking issues

Posted by Duan, Zhenzhong 6 months, 1 week ago

Hi Clement,

>-----Original Message-----
>From: CLEMENT MATHIEU--DRIF <clement.mathieu--drif@eviden.com>
>Subject: [PATCH v5 0/2] intel_iommu: Fix locking issues
>
>This series introduces 2 fixes and improves locking style
>consistency in the VT-d device.
>
>Changes since v4:
>	- Re-check if the address space is present once both bql and
>	  iommu lock are held.

Yes, this is cleaner. For the whole series:

Reviewed-by: Zhenzhong Duan <zhenzhong.duan@intel.com>

Thanks
Zhenzhong

Re: [PATCH v5 0/2] intel_iommu: Fix locking issues

Posted by CLEMENT MATHIEU--DRIF 6 months, 1 week ago

Cc'ing Paolo

On 30/04/2025 2:48 pm, CLEMENT MATHIEU--DRIF wrote:
> This series introduces 2 fixes and improves locking style
> consistency in the VT-d device.
> 
> Changes since v4:
> 	- Re-check if the address space is present once both bql and
> 	  iommu lock are held.
> 
> 
> Clement Mathieu--Drif (2):
>    intel_iommu: Use BQL_LOCK_GUARD to manage cleanup automatically
>    intel_iommu: Take locks when looking for and creating address spaces
> 
>   hw/i386/intel_iommu.c | 35 +++++++++++++++++++++++++----------
>   1 file changed, 25 insertions(+), 10 deletions(-)
>