f54d186700
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 <chris@chris-wilson.co.uk> Reviewed-by: Gustavo Padovan <gustavo.padovan@collabora.co.uk> Acked-by: Sumit Semwal <sumit.semwal@linaro.org> Acked-by: Christian König <christian.koenig@amd.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch> Link: http://patchwork.freedesktop.org/patch/msgid/20161025120045.28839-1-chris@chris-wilson.co.uk
147 lines
4.4 KiB
C
147 lines
4.4 KiB
C
/*
|
|
* dma-fence-array: aggregate fences to be waited together
|
|
*
|
|
* Copyright (C) 2016 Collabora Ltd
|
|
* Copyright (C) 2016 Advanced Micro Devices, Inc.
|
|
* Authors:
|
|
* Gustavo Padovan <gustavo@padovan.org>
|
|
* Christian König <christian.koenig@amd.com>
|
|
*
|
|
* This program is free software; you can redistribute it and/or modify it
|
|
* under the terms of the GNU General Public License version 2 as published by
|
|
* the Free Software Foundation.
|
|
*
|
|
* This program is distributed in the hope that it will be useful, but WITHOUT
|
|
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
|
|
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for
|
|
* more details.
|
|
*/
|
|
|
|
#include <linux/export.h>
|
|
#include <linux/slab.h>
|
|
#include <linux/dma-fence-array.h>
|
|
|
|
static const char *dma_fence_array_get_driver_name(struct dma_fence *fence)
|
|
{
|
|
return "dma_fence_array";
|
|
}
|
|
|
|
static const char *dma_fence_array_get_timeline_name(struct dma_fence *fence)
|
|
{
|
|
return "unbound";
|
|
}
|
|
|
|
static void dma_fence_array_cb_func(struct dma_fence *f,
|
|
struct dma_fence_cb *cb)
|
|
{
|
|
struct dma_fence_array_cb *array_cb =
|
|
container_of(cb, struct dma_fence_array_cb, cb);
|
|
struct dma_fence_array *array = array_cb->array;
|
|
|
|
if (atomic_dec_and_test(&array->num_pending))
|
|
dma_fence_signal(&array->base);
|
|
dma_fence_put(&array->base);
|
|
}
|
|
|
|
static bool dma_fence_array_enable_signaling(struct dma_fence *fence)
|
|
{
|
|
struct dma_fence_array *array = to_dma_fence_array(fence);
|
|
struct dma_fence_array_cb *cb = (void *)(&array[1]);
|
|
unsigned i;
|
|
|
|
for (i = 0; i < array->num_fences; ++i) {
|
|
cb[i].array = array;
|
|
/*
|
|
* As we may report that the fence is signaled before all
|
|
* callbacks are complete, we need to take an additional
|
|
* reference count on the array so that we do not free it too
|
|
* early. The core fence handling will only hold the reference
|
|
* until we signal the array as complete (but that is now
|
|
* insufficient).
|
|
*/
|
|
dma_fence_get(&array->base);
|
|
if (dma_fence_add_callback(array->fences[i], &cb[i].cb,
|
|
dma_fence_array_cb_func)) {
|
|
dma_fence_put(&array->base);
|
|
if (atomic_dec_and_test(&array->num_pending))
|
|
return false;
|
|
}
|
|
}
|
|
|
|
return true;
|
|
}
|
|
|
|
static bool dma_fence_array_signaled(struct dma_fence *fence)
|
|
{
|
|
struct dma_fence_array *array = to_dma_fence_array(fence);
|
|
|
|
return atomic_read(&array->num_pending) <= 0;
|
|
}
|
|
|
|
static void dma_fence_array_release(struct dma_fence *fence)
|
|
{
|
|
struct dma_fence_array *array = to_dma_fence_array(fence);
|
|
unsigned i;
|
|
|
|
for (i = 0; i < array->num_fences; ++i)
|
|
dma_fence_put(array->fences[i]);
|
|
|
|
kfree(array->fences);
|
|
dma_fence_free(fence);
|
|
}
|
|
|
|
const struct dma_fence_ops dma_fence_array_ops = {
|
|
.get_driver_name = dma_fence_array_get_driver_name,
|
|
.get_timeline_name = dma_fence_array_get_timeline_name,
|
|
.enable_signaling = dma_fence_array_enable_signaling,
|
|
.signaled = dma_fence_array_signaled,
|
|
.wait = dma_fence_default_wait,
|
|
.release = dma_fence_array_release,
|
|
};
|
|
EXPORT_SYMBOL(dma_fence_array_ops);
|
|
|
|
/**
|
|
* dma_fence_array_create - Create a custom fence array
|
|
* @num_fences: [in] number of fences to add in the array
|
|
* @fences: [in] array containing the fences
|
|
* @context: [in] fence context to use
|
|
* @seqno: [in] sequence number to use
|
|
* @signal_on_any: [in] signal on any fence in the array
|
|
*
|
|
* Allocate a dma_fence_array object and initialize the base fence with
|
|
* dma_fence_init().
|
|
* In case of error it returns NULL.
|
|
*
|
|
* The caller should allocate the fences array with num_fences size
|
|
* and fill it with the fences it wants to add to the object. Ownership of this
|
|
* array is taken and dma_fence_put() is used on each fence on release.
|
|
*
|
|
* If @signal_on_any is true the fence array signals if any fence in the array
|
|
* signals, otherwise it signals when all fences in the array signal.
|
|
*/
|
|
struct dma_fence_array *dma_fence_array_create(int num_fences,
|
|
struct dma_fence **fences,
|
|
u64 context, unsigned seqno,
|
|
bool signal_on_any)
|
|
{
|
|
struct dma_fence_array *array;
|
|
size_t size = sizeof(*array);
|
|
|
|
/* Allocate the callback structures behind the array. */
|
|
size += num_fences * sizeof(struct dma_fence_array_cb);
|
|
array = kzalloc(size, GFP_KERNEL);
|
|
if (!array)
|
|
return NULL;
|
|
|
|
spin_lock_init(&array->lock);
|
|
dma_fence_init(&array->base, &dma_fence_array_ops, &array->lock,
|
|
context, seqno);
|
|
|
|
array->num_fences = num_fences;
|
|
atomic_set(&array->num_pending, signal_on_any ? 1 : num_fences);
|
|
array->fences = fences;
|
|
|
|
return array;
|
|
}
|
|
EXPORT_SYMBOL(dma_fence_array_create);
|