[Intel-gfx] [PATCH v4] dma-buf: Rename struct fence to dma_fence

2016-10-25 Thread Daniel Vetter
On Tue, Oct 25, 2016 at 01:00:45PM +0100, Chris Wilson wrote:
> I plan to usurp the short name of struct fence for a core kernel struct,
> and so I need to rename the specialised fence/timeline for DMA
> operations to make room.
> 
> A consensus was reached in
> https://lists.freedesktop.org/archives/dri-devel/2016-July/113083.html
> that making clear this fence applies to DMA operations was a good thing.
> Since then the patch has grown a bit as usage increases, so hopefully it
> remains a good thing!
> 
> (v2...: rebase, rerun spatch)
> v3: Compile on msm, spotted a manual fixup that I broke.
> v4: Try again for msm, sorry Daniel

Fourth times's the charm it seems. Applied to drm-misc, thanks.
-Daniel

> 
> coccinelle script:
> @@
> 
> @@
> - struct fence
> + struct dma_fence
> @@
> 
> @@
> - struct fence_ops
> + struct dma_fence_ops
> @@
> 
> @@
> - struct fence_cb
> + struct dma_fence_cb
> @@
> 
> @@
> - struct fence_array
> + struct dma_fence_array
> @@
> 
> @@
> - enum fence_flag_bits
> + enum dma_fence_flag_bits
> @@
> 
> @@
> (
> - fence_init
> + dma_fence_init
> |
> - fence_release
> + dma_fence_release
> |
> - fence_free
> + dma_fence_free
> |
> - fence_get
> + dma_fence_get
> |
> - fence_get_rcu
> + dma_fence_get_rcu
> |
> - fence_put
> + dma_fence_put
> |
> - fence_signal
> + dma_fence_signal
> |
> - fence_signal_locked
> + dma_fence_signal_locked
> |
> - fence_default_wait
> + dma_fence_default_wait
> |
> - fence_add_callback
> + dma_fence_add_callback
> |
> - fence_remove_callback
> + dma_fence_remove_callback
> |
> - fence_enable_sw_signaling
> + dma_fence_enable_sw_signaling
> |
> - fence_is_signaled_locked
> + dma_fence_is_signaled_locked
> |
> - fence_is_signaled
> + dma_fence_is_signaled
> |
> - fence_is_later
> + dma_fence_is_later
> |
> - fence_later
> + dma_fence_later
> |
> - fence_wait_timeout
> + dma_fence_wait_timeout
> |
> - fence_wait_any_timeout
> + dma_fence_wait_any_timeout
> |
> - fence_wait
> + dma_fence_wait
> |
> - fence_context_alloc
> + dma_fence_context_alloc
> |
> - fence_array_create
> + dma_fence_array_create
> |
> - to_fence_array
> + to_dma_fence_array
> |
> - fence_is_array
> + dma_fence_is_array
> |
> - trace_fence_emit
> + trace_dma_fence_emit
> |
> - FENCE_TRACE
> + DMA_FENCE_TRACE
> |
> - FENCE_WARN
> + DMA_FENCE_WARN
> |
> - FENCE_ERR
> + DMA_FENCE_ERR
> )
>  (
>  ...
>  )
> 
> Signed-off-by: Chris Wilson 
> Reviewed-by: Gustavo Padovan 
> Acked-by: Sumit Semwal 
> Acked-by: Christian König 
> ---
>  Documentation/sync_file.txt|  14 +-
>  drivers/base/Kconfig   |   6 +-
>  drivers/dma-buf/Kconfig|   2 +-
>  drivers/dma-buf/Makefile   |   2 +-
>  drivers/dma-buf/dma-buf.c  |  28 +--
>  .../dma-buf/{fence-array.c => dma-fence-array.c}   |  91 
>  drivers/dma-buf/{fence.c => dma-fence.c}   | 199 -
>  drivers/dma-buf/reservation.c  |  94 
>  drivers/dma-buf/seqno-fence.c  |  18 +-
>  drivers/dma-buf/sw_sync.c  |  48 ++---
>  drivers/dma-buf/sync_debug.c   |  13 +-
>  drivers/dma-buf/sync_debug.h   |   9 +-
>  drivers/dma-buf/sync_file.c|  63 +++---
>  drivers/gpu/drm/amd/amdgpu/amdgpu.h|  54 ++---
>  drivers/gpu/drm/amd/amdgpu/amdgpu_benchmark.c  |   8 +-
>  drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c |  16 +-
>  drivers/gpu/drm/amd/amdgpu/amdgpu_ctx.c|  22 +-
>  drivers/gpu/drm/amd/amdgpu/amdgpu_device.c |  14 +-
>  drivers/gpu/drm/amd/amdgpu/amdgpu_display.c|  16 +-
>  drivers/gpu/drm/amd/amdgpu/amdgpu_fence.c  |  58 ++---
>  drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c |   6 +-
>  drivers/gpu/drm/amd/amdgpu/amdgpu_job.c|  22 +-
>  drivers/gpu/drm/amd/amdgpu/amdgpu_object.c |  14 +-
>  drivers/gpu/drm/amd/amdgpu/amdgpu_object.h |   8 +-
>  drivers/gpu/drm/amd/amdgpu/amdgpu_sa.c |  24 +--
>  drivers/gpu/drm/amd/amdgpu/amdgpu_sync.c   |  48 +++--
>  drivers/gpu/drm/amd/amdgpu/amdgpu_test.c   |  12 +-
>  drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h  |   4 +-
>  drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c|  10 +-
>  drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.h|   4 +-
>  drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c|  26 +--
>  drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.h|   4 +-
>  drivers/gpu/drm/amd/amdgpu/amdgpu_vce.c|  26 +--
>  drivers/gpu/drm/amd/amdgpu/amdgpu_vce.h|   4 +-
>  drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c |  79 +++
>  drivers/gpu/drm/amd/amdgpu/cik_sdma.c  |   6 +-
>  drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c  |   6 +-
>  drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c  |   6 +-
>  drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c  |  12 +-

[PATCH v4] dma-buf: Rename struct fence to dma_fence

2016-10-25 Thread Chris Wilson
I plan to usurp the short name of struct fence for a core kernel struct,
and so I need to rename the specialised fence/timeline for DMA
operations to make room.

A consensus was reached in
https://lists.freedesktop.org/archives/dri-devel/2016-July/113083.html
that making clear this fence applies to DMA operations was a good thing.
Since then the patch has grown a bit as usage increases, so hopefully it
remains a good thing!

(v2...: rebase, rerun spatch)
v3: Compile on msm, spotted a manual fixup that I broke.
v4: Try again for msm, sorry Daniel

coccinelle script:
@@

@@
- struct fence
+ struct dma_fence
@@

@@
- struct fence_ops
+ struct dma_fence_ops
@@

@@
- struct fence_cb
+ struct dma_fence_cb
@@

@@
- struct fence_array
+ struct dma_fence_array
@@

@@
- enum fence_flag_bits
+ enum dma_fence_flag_bits
@@

@@
(
- fence_init
+ dma_fence_init
|
- fence_release
+ dma_fence_release
|
- fence_free
+ dma_fence_free
|
- fence_get
+ dma_fence_get
|
- fence_get_rcu
+ dma_fence_get_rcu
|
- fence_put
+ dma_fence_put
|
- fence_signal
+ dma_fence_signal
|
- fence_signal_locked
+ dma_fence_signal_locked
|
- fence_default_wait
+ dma_fence_default_wait
|
- fence_add_callback
+ dma_fence_add_callback
|
- fence_remove_callback
+ dma_fence_remove_callback
|
- fence_enable_sw_signaling
+ dma_fence_enable_sw_signaling
|
- fence_is_signaled_locked
+ dma_fence_is_signaled_locked
|
- fence_is_signaled
+ dma_fence_is_signaled
|
- fence_is_later
+ dma_fence_is_later
|
- fence_later
+ dma_fence_later
|
- fence_wait_timeout
+ dma_fence_wait_timeout
|
- fence_wait_any_timeout
+ dma_fence_wait_any_timeout
|
- fence_wait
+ dma_fence_wait
|
- fence_context_alloc
+ dma_fence_context_alloc
|
- fence_array_create
+ dma_fence_array_create
|
- to_fence_array
+ to_dma_fence_array
|
- fence_is_array
+ dma_fence_is_array
|
- trace_fence_emit
+ trace_dma_fence_emit
|
- FENCE_TRACE
+ DMA_FENCE_TRACE
|
- FENCE_WARN
+ DMA_FENCE_WARN
|
- FENCE_ERR
+ DMA_FENCE_ERR
)
 (
 ...
 )

Signed-off-by: Chris Wilson 
Reviewed-by: Gustavo Padovan 
Acked-by: Sumit Semwal 
Acked-by: Christian König 
---
 Documentation/sync_file.txt|  14 +-
 drivers/base/Kconfig   |   6 +-
 drivers/dma-buf/Kconfig|   2 +-
 drivers/dma-buf/Makefile   |   2 +-
 drivers/dma-buf/dma-buf.c  |  28 +--
 .../dma-buf/{fence-array.c => dma-fence-array.c}   |  91 
 drivers/dma-buf/{fence.c => dma-fence.c}   | 199 -
 drivers/dma-buf/reservation.c  |  94 
 drivers/dma-buf/seqno-fence.c  |  18 +-
 drivers/dma-buf/sw_sync.c  |  48 ++---
 drivers/dma-buf/sync_debug.c   |  13 +-
 drivers/dma-buf/sync_debug.h   |   9 +-
 drivers/dma-buf/sync_file.c|  63 +++---
 drivers/gpu/drm/amd/amdgpu/amdgpu.h|  54 ++---
 drivers/gpu/drm/amd/amdgpu/amdgpu_benchmark.c  |   8 +-
 drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c |  16 +-
 drivers/gpu/drm/amd/amdgpu/amdgpu_ctx.c|  22 +-
 drivers/gpu/drm/amd/amdgpu/amdgpu_device.c |  14 +-
 drivers/gpu/drm/amd/amdgpu/amdgpu_display.c|  16 +-
 drivers/gpu/drm/amd/amdgpu/amdgpu_fence.c  |  58 ++---
 drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c |   6 +-
 drivers/gpu/drm/amd/amdgpu/amdgpu_job.c|  22 +-
 drivers/gpu/drm/amd/amdgpu/amdgpu_object.c |  14 +-
 drivers/gpu/drm/amd/amdgpu/amdgpu_object.h |   8 +-
 drivers/gpu/drm/amd/amdgpu/amdgpu_sa.c |  24 +--
 drivers/gpu/drm/amd/amdgpu/amdgpu_sync.c   |  48 +++--
 drivers/gpu/drm/amd/amdgpu/amdgpu_test.c   |  12 +-
 drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h  |   4 +-
 drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c|  10 +-
 drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.h|   4 +-
 drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c|  26 +--
 drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.h|   4 +-
 drivers/gpu/drm/amd/amdgpu/amdgpu_vce.c|  26 +--
 drivers/gpu/drm/amd/amdgpu/amdgpu_vce.h|   4 +-
 drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c |  79 +++
 drivers/gpu/drm/amd/amdgpu/cik_sdma.c  |   6 +-
 drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c  |   6 +-
 drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c  |   6 +-
 drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c  |  12 +-
 drivers/gpu/drm/amd/amdgpu/sdma_v2_4.c |   6 +-
 drivers/gpu/drm/amd/amdgpu/sdma_v3_0.c |   6 +-
 drivers/gpu/drm/amd/amdgpu/si_dma.c|   6 +-
 drivers/gpu/drm/amd/scheduler/gpu_sched_trace.h|   4 +-
 drivers/gpu/drm/amd/scheduler/gpu_scheduler.c  |  67 +++---
 drivers/gpu/drm/amd/scheduler/gpu_scheduler.h  |  26 +--
 drivers/gpu/drm/amd/scheduler/sched_fence.c|  48 +++--
 drivers/gpu/drm/drm_atomic.c