Re: [PATCH 0/4] KVM: VFIO: use mutex guards to simplify control flow

[Alex Williamson <alex@shazbot.org>]

On Fri, 13 Mar 2026 13:20:38 +0100
Carlos López <clopez@suse.de> wrote:

> Replace several uses of mutex_lock() / mutex_unlock() pairs with mutex
> guards, allowing removal of all gotos in virt/kvm/vfio.c.
> 
> In this series I also include a small refactor that allows greatly
> simplifying kvm_vfio_file_del(), and skipping an unnecessary call to
> kvm_vfio_update_coherency() if the list of files managed by the KVM
> VFIO device does not change.
> 
> Carlos López (4):
>   KVM: VFIO: clean up control flow in kvm_vfio_file_add()
>   KVM: VFIO: use mutex guard in kvm_vfio_file_set_spapr_tce()
>   KVM: VFIO: deduplicate file release logic
>   KVM: VFIO: update coherency only if file was deleted
> 
>  virt/kvm/vfio.c | 97 ++++++++++++++++++-------------------------------
>  1 file changed, 35 insertions(+), 62 deletions(-)
> 
> 
> base-commit: d2ea4ff1ce50787a98a3900b3fb1636f3620b7cf

Series looks good to me.  This is nicely isolated, so I assume Paolo
will take it through the KVM tree, but I can also take it through vfio
if preferred.

Reviewed-by: Alex Williamson <alex@shazbot.org>

Thanks,
Alex