From: Jason Ekstrand <jason@jlekstrand.net>
To: dri-devel@lists.freedesktop.org
Cc: Jason Ekstrand <jason.ekstrand@intel.com>,
Jason Ekstrand <jason@jlekstrand.net>
Subject: [PATCH 6/9] dma-buf/dma-fence: Allow wait_any_timeout without default_wait
Date: Tue, 8 Aug 2017 15:46:06 -0700 [thread overview]
Message-ID: <1502232369-19753-7-git-send-email-jason.ekstrand@intel.com> (raw)
In-Reply-To: <1502232369-19753-1-git-send-email-jason.ekstrand@intel.com>
dma_fence_wait_any_timeout only relies on two things to work correctly:
1) The SIGNALED flag to be set upon fence signal
2) The callback list to get called upon fence signal
Both of these are part of the core dma-fence API so it should work fine
even with a non-default wait function.
Signed-off-by: Jason Ekstrand <jason@jlekstrand.net>
---
drivers/dma-buf/dma-fence.c | 5 -----
1 file changed, 5 deletions(-)
diff --git a/drivers/dma-buf/dma-fence.c b/drivers/dma-buf/dma-fence.c
index 57da14c..1218692 100644
--- a/drivers/dma-buf/dma-fence.c
+++ b/drivers/dma-buf/dma-fence.c
@@ -501,11 +501,6 @@ dma_fence_wait_any_timeout(struct dma_fence **fences, uint32_t count,
for (i = 0; i < count; ++i) {
struct dma_fence *fence = fences[i];
- if (fence->ops->wait != dma_fence_default_wait) {
- ret = -EINVAL;
- goto fence_rm_cb;
- }
-
cb[i].task = current;
if (dma_fence_add_callback(fence, &cb[i].base,
dma_fence_default_wait_cb)) {
--
2.5.0.400.gff86faf
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2017-08-08 22:46 UTC|newest]
Thread overview: 55+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-08-08 22:46 [PATCH 0/9] drm/syncobj: Add full-featured wait support Jason Ekstrand
2017-08-08 22:46 ` [PATCH 1/9] drm/syncobj: Rename fence_get to find_fence Jason Ekstrand
2017-08-08 22:46 ` [PATCH 2/9] drm/syncobj: Lock around drm_syncobj::fence Jason Ekstrand
2017-08-09 21:21 ` Chris Wilson
2017-08-10 0:31 ` Jason Ekstrand
2017-08-10 10:55 ` Chris Wilson
2017-08-10 19:10 ` Chris Wilson
2017-08-08 22:46 ` [PATCH 3/9] drm/syncobj: Remove file_private from replace_fence Jason Ekstrand
2017-08-08 22:46 ` [PATCH 4/9] i915: Add support for drm syncobjs Jason Ekstrand
2017-08-08 22:46 ` [PATCH 5/9] drm/syncobj: add sync obj wait interface. (v8) Jason Ekstrand
2017-08-08 22:46 ` Jason Ekstrand [this message]
2017-08-08 22:46 ` [PATCH 7/9] drm/syncobj: Add a reset ioctl Jason Ekstrand
2017-08-08 22:46 ` [PATCH 8/9] drm/syncobj: Add a callback mechanism for replace_fence Jason Ekstrand
2017-08-08 22:46 ` [PATCH 9/9] drm/syncobj: Allow wait for submit and signal behavior Jason Ekstrand
2017-08-09 17:00 ` [PATCH] drm/syncobj: Allow wait for submit and signal behavior (v2) Jason Ekstrand
2017-08-09 17:57 ` Chris Wilson
2017-08-09 18:25 ` Christian König
2017-08-09 21:09 ` Jason Ekstrand
2017-08-09 22:41 ` Chris Wilson
2017-08-09 23:53 ` Jason Ekstrand
2017-08-10 11:00 ` Chris Wilson
2017-08-10 14:42 ` Jason Ekstrand
2017-08-10 12:26 ` Christian König
2017-08-10 14:32 ` Jason Ekstrand
2017-08-10 14:41 ` Christian König
2017-08-09 21:31 ` Chris Wilson
2017-08-09 21:54 ` Jason Ekstrand
2017-08-10 12:26 ` Christian König
2017-08-10 1:35 ` [PATCH v3 9/9] drm/syncobj: Allow wait for submit and signal behavior (v3) Jason Ekstrand
2017-08-11 22:39 ` [PATCH 0/9] drm/syncobj: Add full-featured wait support (v2) Jason Ekstrand
2017-08-11 22:39 ` [PATCH 1/9] drm/syncobj: Rename fence_get to find_fence Jason Ekstrand
2017-08-11 22:39 ` [PATCH 2/9] drm/syncobj: Add a race-free drm_syncobj_fence_get helper Jason Ekstrand
2017-08-14 2:03 ` kbuild test robot
2017-08-11 22:39 ` [PATCH 3/9] i915: Add support for drm syncobjs Jason Ekstrand
2017-08-14 2:58 ` Jason Ekstrand
2017-08-11 22:39 ` [PATCH 4/9] drm/syncobj: add sync obj wait interface. (v8) Jason Ekstrand
2017-08-11 22:39 ` [PATCH 5/9] dma-buf/dma-fence: Allow wait_any_timeout without default_wait (v2) Jason Ekstrand
2017-08-11 22:39 ` [PATCH 6/9] drm/syncobj: Add a reset ioctl Jason Ekstrand
2017-08-11 22:39 ` [PATCH 7/9] dma-buf/dma-fence: Signal all callbacks from dma_fence_release() Jason Ekstrand
2018-01-31 12:32 ` Gustavo Padovan
2018-01-31 15:53 ` Chris Wilson
2017-08-11 22:39 ` [PATCH 8/9] dma-buf/dma-fence: Add a mechanism for proxy fences Jason Ekstrand
2017-08-11 22:39 ` [PATCH 9/9] drm/syncobj: Allow wait for submit and signal behavior (v4) Jason Ekstrand
2017-08-13 13:19 ` [PATCH 0/9] drm/syncobj: Add full-featured wait support (v2) Christian König
2017-08-13 15:26 ` Jason Ekstrand
2017-08-13 15:52 ` Christian König
2017-08-13 23:14 ` Jason Ekstrand
2017-08-14 5:49 ` Jason Ekstrand
2017-08-14 7:36 ` Christian König
2017-08-14 15:08 ` Jason Ekstrand
2017-08-16 15:52 ` Jason Ekstrand
2017-08-16 16:53 ` Christian König
2017-08-16 20:10 ` Jason Ekstrand
2017-08-21 21:42 ` Jason Ekstrand
2017-08-22 8:30 ` Christian König
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1502232369-19753-7-git-send-email-jason.ekstrand@intel.com \
--to=jason@jlekstrand.net \
--cc=dri-devel@lists.freedesktop.org \
--cc=jason.ekstrand@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox