diff mbox

dma/fence: Export enable-signaling tracepoint for emission by drivers

Message ID 20170124115758.31353-1-chris@chris-wilson.co.uk (mailing list archive)
State New, archived
Headers show

Commit Message

Chris Wilson Jan. 24, 2017, 11:57 a.m. UTC
Currently this tracepoint is solely used by dma_fence_enable_sw_signaling,
however I have a need to manually perform the hw enabling of the
signaling and would like to emit this tracepoint for completeness.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Sumit Semwal <sumit.semwal@linaro.org>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
---
 drivers/dma-buf/dma-fence.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Chris Wilson Jan. 27, 2017, 2:48 p.m. UTC | #1
On Tue, Jan 24, 2017 at 11:57:58AM +0000, Chris Wilson wrote:
> Currently this tracepoint is solely used by dma_fence_enable_sw_signaling,
> however I have a need to manually perform the hw enabling of the
> signaling and would like to emit this tracepoint for completeness.
> 
> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
> Cc: Sumit Semwal <sumit.semwal@linaro.org>
> Cc: Daniel Vetter <daniel.vetter@ffwll.ch>

Anybody for a quick ack and backmerge? :)

> ---
>  drivers/dma-buf/dma-fence.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/dma-buf/dma-fence.c b/drivers/dma-buf/dma-fence.c
> index a1bfc098ea10..d1f1f456f5c4 100644
> --- a/drivers/dma-buf/dma-fence.c
> +++ b/drivers/dma-buf/dma-fence.c
> @@ -28,6 +28,7 @@
>  
>  EXPORT_TRACEPOINT_SYMBOL(dma_fence_annotate_wait_on);
>  EXPORT_TRACEPOINT_SYMBOL(dma_fence_emit);
> +EXPORT_TRACEPOINT_SYMBOL(dma_fence_enable_signal);
>  
>  /*
>   * fence context counter: each execution context should have its own
> -- 
> 2.11.0
>
Gustavo Padovan Jan. 27, 2017, 2:55 p.m. UTC | #2
Hi Chris,

2017-01-24 Chris Wilson <chris@chris-wilson.co.uk>:

> Currently this tracepoint is solely used by dma_fence_enable_sw_signaling,
> however I have a need to manually perform the hw enabling of the
> signaling and would like to emit this tracepoint for completeness.
> 
> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
> Cc: Sumit Semwal <sumit.semwal@linaro.org>
> Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
> ---
>  drivers/dma-buf/dma-fence.c | 1 +
>  1 file changed, 1 insertion(+)

Reviewed-by: Gustavo Padovan <gustavo.padovan@collabora.com>

Gustavo
Sumit Semwal Jan. 27, 2017, 4:53 p.m. UTC | #3
fwiw,


On 27 January 2017 at 20:25, Gustavo Padovan <gustavo@padovan.org> wrote:
> Hi Chris,
>
> 2017-01-24 Chris Wilson <chris@chris-wilson.co.uk>:
>
>> Currently this tracepoint is solely used by dma_fence_enable_sw_signaling,
>> however I have a need to manually perform the hw enabling of the
>> signaling and would like to emit this tracepoint for completeness.
>>
>> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
>> Cc: Sumit Semwal <sumit.semwal@linaro.org>
>> Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
>> ---
>>  drivers/dma-buf/dma-fence.c | 1 +
>>  1 file changed, 1 insertion(+)
>
> Reviewed-by: Gustavo Padovan <gustavo.padovan@collabora.com>
>
Feel free to add my a-b :)
> Gustavo
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
Gustavo Padovan Jan. 27, 2017, 5:37 p.m. UTC | #4
Hi Chris,

2017-01-24 Chris Wilson <chris@chris-wilson.co.uk>:

> Currently this tracepoint is solely used by dma_fence_enable_sw_signaling,
> however I have a need to manually perform the hw enabling of the
> signaling and would like to emit this tracepoint for completeness.
> 
> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
> Cc: Sumit Semwal <sumit.semwal@linaro.org>
> Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
> ---
>  drivers/dma-buf/dma-fence.c | 1 +
>  1 file changed, 1 insertion(+)

Applied to drm-misc-next Thanks! 

Gustavo
diff mbox

Patch

diff --git a/drivers/dma-buf/dma-fence.c b/drivers/dma-buf/dma-fence.c
index a1bfc098ea10..d1f1f456f5c4 100644
--- a/drivers/dma-buf/dma-fence.c
+++ b/drivers/dma-buf/dma-fence.c
@@ -28,6 +28,7 @@ 
 
 EXPORT_TRACEPOINT_SYMBOL(dma_fence_annotate_wait_on);
 EXPORT_TRACEPOINT_SYMBOL(dma_fence_emit);
+EXPORT_TRACEPOINT_SYMBOL(dma_fence_enable_signal);
 
 /*
  * fence context counter: each execution context should have its own