RE: [PATCH v7 15/22] vfio-iommufd: Add detach_ioas support for physical VFIO devices

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> From: Liu, Yi L <yi.l.liu@xxxxxxxxx>
> Sent: Friday, March 17, 2023 11:07 AM
> 
> > From: Tian, Kevin <kevin.tian@xxxxxxxxx>
> > Sent: Friday, March 17, 2023 10:14 AM
> >
> > > From: Liu, Yi L <yi.l.liu@xxxxxxxxx>
> > > 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.:)

Huh, bad eyes. 😊




[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux