Commit a408c0ca authored by Christian König's avatar Christian König
Browse files

dma-buf: use inline lock for the dma-fence-chain 



Using the inline lock is now the recommended way for dma_fence
implementations.

So use this approach for the framework's internal fences as well.

Also saves about 4 bytes for the external spinlock.

Signed-off-by: default avatarChristian König <christian.koenig@amd.com>
Reviewed-by: default avatarTvrtko Ursulin <tvrtko.ursulin@igalia.com>
Reviewed-by: default avatarPhilipp Stanner <phasta@kernel.org>
Reviewed-by: default avatarBoris Brezillon <boris.brezillon@collabora.com>
Link: https://lore.kernel.org/r/20260219160822.1529-9-christian.koenig@amd.com
parent 59432439

drivers/dma-buf/dma-fence-chain.c

+1 −2
Original line number Diff line number Diff line
@@ -245,7 +245,6 @@ void dma_fence_chain_init(struct dma_fence_chain *chain, 
	struct dma_fence_chain *prev_chain = to_dma_fence_chain(prev);

	uint64_t context;



	spin_lock_init(&chain->lock);

	rcu_assign_pointer(chain->prev, prev);

	chain->fence = fence;

	chain->prev_seqno = 0;
@@ -261,7 +260,7 @@ void dma_fence_chain_init(struct dma_fence_chain *chain, 
			seqno = max(prev->seqno, seqno);

	}



	dma_fence_init64(&chain->base, &dma_fence_chain_ops, &chain->lock,

	dma_fence_init64(&chain->base, &dma_fence_chain_ops, NULL,

			 context, seqno);



	/*

include/linux/dma-fence-chain.h

+0 −1
Original line number Diff line number Diff line
@@ -46,7 +46,6 @@ struct dma_fence_chain { 
		 */

		struct irq_work work;

	};

	spinlock_t lock;

};