Re: [PATCH RFC v2 03/13] iommufd: File descriptor, context, kconfig and makefiles

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

 



On Sun, Sep 04, 2022 at 04:19:04PM +0800, Baolu Lu wrote:
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 589517372408ca..abd041f5e00f4c 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -10609,6 +10609,16 @@ L:	linux-mips@xxxxxxxxxxxxxxx
> >   S:	Maintained
> >   F:	drivers/net/ethernet/sgi/ioc3-eth.c
> > +IOMMU FD
> > +M:	Jason Gunthorpe <jgg@xxxxxxxxxx>
> > +M:	Kevin Tian <kevin.tian@xxxxxxxxx>
> > +L:	iommu@xxxxxxxxxxxxxxxxxxxxxxxxxx
> 
> This mailing list has already been replaced with iommu@xxxxxxxxxxxxxxx.

It is also not sorted.. I fixed both

> > +/**
> > + * iommufd_put_object_keep_user() - Release part of the refcount on obj
> > + * @obj - Object to release
> > + *
> > + * Objects have two protections to ensure that userspace has a consistent
> > + * experience with destruction. Normally objects are locked so that destroy will
> > + * block while there are concurrent users, and wait for the object to be
> > + * unlocked.
> > + *
> > + * However, destroy can also be blocked by holding users reference counts on the
> > + * objects, in that case destroy will immediately return EBUSY and will not wait
> > + * for reference counts to go to zero.
> > + *
> > + * This function releases the destroy lock and destroy will return EBUSY.
> 
> This reads odd. Does it release or acquire a destroy lock.

I changed this line to
  This function switches from blocking userspace to returning EBUSY.

And the rest too, thanks

Jason



[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