On 1/30/26 14:56, Jason Gunthorpe wrote: > On Fri, Jan 30, 2026 at 02:21:08PM +0100, Christian König wrote: > >> That would work for me. >> >> Question is if you really want to do it this way? See usually >> exporters try to avoid blocking such functions. > > Yes, it has to be this way, revoke is a synchronous user space > triggered operation around things like FLR or device close. We can't > defer it into some background operation like pm.
Yeah, but you only need that in a couple of use cases and not all. Especially when the device is idle exporters usually don't want runtime PM to kick in and try to suspend the device while importers are still using it. > >>>>> } >>>>> fput(priv->dmabuf->file); >>>> >>>> This is also extremely questionable. Why doesn't the dmabuf have >>>> a reference while on the linked list? > > If we hold a refcount while on the list then the FD can never be > closed. > > There is locking protecting the list so that it is safe and close > continues to work right. Ah! You use get_file_active(), yeah that should work. Thanks, Christian. > > Jason
