Re: [PATCH 0/3] dma-buf: dma-fence: Remove dma_fence_init64()
From: Timur Kristóf
Date: Thu Nov 05 2026 - 11:51:49 EST
On Thu, 2026-11-05 at 12:35 +0100, Philipp Stanner wrote:
> We recently had a discussion about reworking dma_fence init to close
> a
> race-window [1].
>
> The community agreement seems to have been that we can get rid of
> dma_fence_init64(), so let's do that already. In the meantime we can
> continue our discussions regarding the race in [1].
>
> This also touches the Rust dma_fence, which actually should use 64-
> bit
> seqnos (was forgotten so far). Rust fence has no users currently, so
> needs no backporting, so I would like to address that in a distinct
> patch.
>
> Tested on Nouveau with and without the SEQNO64 flag. Nouveau also
> uses
> the USER flag, so I hope we're fine.
>
>
> P.
>
> [1]
> https://lore.kernel.org/dri-devel/20261029095542.1526750-3-phasta@xxxxxxxxxx/
Makes good sense to me.
For the entire series:
Reviewed-by: Timur Kristóf <timur.kristof@xxxxxxxxx>
(ps. I had to remove some recipients from this email because the
original had more than 100 recipients and gmail wouldn't let me reply
to all of them. Apologies for that.)
>
>
> Philipp Stanner (3):
> dma-buf: dma-fence: Add flags parameter to dma_fence_init()
> dma-buf: Port users of dma_fence_init64()
> dma-buf: dma-fence: Remove dma_fence_init64()
>
> drivers/accel/amdxdna/amdxdna_ctx.c | 2 +-
> drivers/accel/ethosu/ethosu_job.c | 2 +-
> drivers/accel/ivpu/ivpu_job.c | 2 +-
> drivers/accel/rocket/rocket_job.c | 2 +-
> drivers/dma-buf/dma-fence-array.c | 2 +-
> drivers/dma-buf/dma-fence-chain.c | 4 +-
> drivers/dma-buf/dma-fence.c | 58 +++++++++--------
> --
> drivers/dma-buf/st-dma-fence-chain.c | 2 +-
> drivers/dma-buf/st-dma-fence-unwrap.c | 2 +-
> drivers/dma-buf/st-dma-fence.c | 2 +-
> drivers/dma-buf/st-dma-resv.c | 2 +-
> drivers/dma-buf/sw_sync.c | 2 +-
> .../gpu/drm/amd/amdgpu/amdgpu_amdkfd_fence.c | 2 +-
> .../drm/amd/amdgpu/amdgpu_eviction_fence.c | 7 ++-
> drivers/gpu/drm/amd/amdgpu/amdgpu_fence.c | 2 +-
> .../gpu/drm/amd/amdgpu/amdgpu_userq_fence.c | 5 +-
> .../gpu/drm/amd/amdgpu/amdgpu_vm_tlb_fence.c | 5 +-
> drivers/gpu/drm/drm_crtc.c | 2 +-
> drivers/gpu/drm/drm_writeback.c | 2 +-
> drivers/gpu/drm/etnaviv/etnaviv_gpu.c | 2 +-
> drivers/gpu/drm/i915/gem/i915_gem_ttm_move.c | 2 +-
> drivers/gpu/drm/i915/i915_request.c | 2 +-
> drivers/gpu/drm/i915/i915_sw_fence_work.c | 2 +-
> drivers/gpu/drm/i915/i915_vma_resource.c | 2 +-
> .../gpu/drm/i915/selftests/i915_sw_fence.c | 2 +-
> drivers/gpu/drm/imagination/pvr_ccb.c | 2 +-
> drivers/gpu/drm/imagination/pvr_queue.c | 2 +-
> drivers/gpu/drm/lima/lima_sched.c | 2 +-
> drivers/gpu/drm/msm/msm_fence.c | 2 +-
> drivers/gpu/drm/nouveau/nouveau_fence.c | 4 +-
> drivers/gpu/drm/panfrost/panfrost_job.c | 2 +-
> drivers/gpu/drm/panthor/panthor_sched.c | 2 +-
> drivers/gpu/drm/qxl/qxl_release.c | 2 +-
> drivers/gpu/drm/radeon/radeon_fence.c | 2 +-
> drivers/gpu/drm/scheduler/sched_fence.c | 4 +-
> .../gpu/drm/scheduler/tests/mock_scheduler.c | 2 +-
> drivers/gpu/drm/ttm/tests/ttm_bo_test.c | 2 +-
> .../gpu/drm/ttm/tests/ttm_bo_validate_test.c | 2 +-
> drivers/gpu/drm/v3d/v3d_fence.c | 2 +-
> drivers/gpu/drm/vc4/vc4_gem.c | 2 +-
> drivers/gpu/drm/vgem/vgem_fence.c | 2 +-
> drivers/gpu/drm/virtio/virtgpu_fence.c | 2 +-
> drivers/gpu/drm/vmwgfx/vmwgfx_fence.c | 2 +-
> drivers/gpu/drm/xe/xe_hw_fence.c | 2 +-
> drivers/gpu/drm/xe/xe_oa.c | 2 +-
> drivers/gpu/drm/xe/xe_preempt_fence.c | 2 +-
> drivers/gpu/drm/xe/xe_tlb_inval.c | 2 +-
> drivers/gpu/host1x/fence.c | 2 +-
> drivers/iio/industrialio-buffer.c | 2 +-
> drivers/usb/gadget/function/f_fs.c | 2 +-
> include/linux/dma-fence.h | 14 +++--
> rust/kernel/dma_buf/dma_fence.rs | 2 +-
> 52 files changed, 95 insertions(+), 94 deletions(-)
>
>
> base-commit: 8258a37702b9b9cf27f5c0ea07625e3e6bd74f5a