> From: Tian, Kevin <kevin.t...@intel.com>
> Sent: Friday, March 17, 2023 10:14 AM
>
> > From: Liu, Yi L <yi.l....@intel.com>
> > Sent: Thursday, March 16, 2023 8:55 PM
> >
> > +void vfio_iommufd_physical_detach_ioas(struct vfio_device *vdev)
> > +{
> > + lockdep_assert_held(&vdev->dev_set->lock);
> > +
> > + if (WARN_ON(!vdev->iommufd_device) || !vdev->iommufd_attached)
> > + return;
> > +
>
> the 2nd check shouldn't be a WARN_ON. It's user-triggerable.
Yes. it has already been removed in this version.:)
