Jason Gunthorpe <[email protected]> writes: > On Wed, Jul 01, 2026 at 11:19:24AM +0530, Aneesh Kumar K.V (Arm) wrote: > >> @@ -840,7 +843,7 @@ static void swiotlb_del_pool(struct device *dev, struct >> io_tlb_pool *pool) >> list_del_rcu(&pool->node); >> spin_unlock_irqrestore(&dev->dma_io_tlb_lock, flags); >> >> - call_rcu(&pool->rcu, swiotlb_dyn_free); >> + swiotlb_schedule_dyn_free(pool); > > This knows if the pool has cc_shared memory in it since there is now a > flag tracking that? So should the work queue version be conditional on > that flag? >
We could definitely do that. I was not sure whether it was worth it, considering that it would add two paths for SWIOTLB dynamic free. -aneesh
