On Tue, May 17, 2022 at 10:34:41AM +0800, Wan Jiabing wrote: > Fix following coccicheck warning: > ./virt/kvm/vfio.c:258:1-7: preceding lock on line 236 > > If kvm_vfio_file_iommu_group() failed, code would goto err_fdput with > mutex_lock acquired and then return ret. It might cause potential > deadlock. Move mutex_unlock bellow err_fdput tag to fix it. > > Fixes: d55d9e7a45721 ("kvm/vfio: Store the struct file in the kvm_vfio_group") > Signed-off-by: Wan Jiabing <wanjiabing@xxxxxxxx> > --- > virt/kvm/vfio.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Reviewed-by: Jason Gunthorpe <jgg@xxxxxxxxxx> Thanks, Jason