diff options
| author | Eric Anholt <eric@anholt.net> | 2018-04-04 09:58:23 -0700 |
|---|---|---|
| committer | Eric Anholt <eric@anholt.net> | 2018-04-12 11:20:50 -0700 |
| commit | d9c525ed2240ff450f36a5d83c9c2c66087cd2bb (patch) | |
| tree | 938358f0a3e7fc844f99766e068340a875eee166 | |
| parent | aedfd8ede4c2e0ecad0a3796796b2f0610482ced (diff) | |
broadcom/vc5: Drop the finished_seqno optimization.
With the DRM scheduler changes, I'm about to remove all seqnos from the
UABI.
| -rw-r--r-- | src/gallium/drivers/vc5/vc5_bufmgr.c | 4 | ||||
| -rw-r--r-- | src/gallium/drivers/vc5/vc5_screen.h | 7 |
2 files changed, 0 insertions, 11 deletions
diff --git a/src/gallium/drivers/vc5/vc5_bufmgr.c b/src/gallium/drivers/vc5/vc5_bufmgr.c index 5e068400fd8..bced512ae0d 100644 --- a/src/gallium/drivers/vc5/vc5_bufmgr.c +++ b/src/gallium/drivers/vc5/vc5_bufmgr.c @@ -469,9 +469,6 @@ bool vc5_wait_seqno(struct vc5_screen *screen, uint64_t seqno, uint64_t timeout_ns, const char *reason) { - if (screen->finished_seqno >= seqno) - return true; - if (unlikely(V3D_DEBUG & V3D_DEBUG_PERF) && timeout_ns && reason) { if (vc5_wait_seqno_ioctl(screen->fd, seqno, 0) == -ETIME) { fprintf(stderr, "Blocking on seqno %lld for %s\n", @@ -489,7 +486,6 @@ vc5_wait_seqno(struct vc5_screen *screen, uint64_t seqno, uint64_t timeout_ns, return false; } - screen->finished_seqno = seqno; return true; } diff --git a/src/gallium/drivers/vc5/vc5_screen.h b/src/gallium/drivers/vc5/vc5_screen.h index 710396de999..05a770ba2a1 100644 --- a/src/gallium/drivers/vc5/vc5_screen.h +++ b/src/gallium/drivers/vc5/vc5_screen.h @@ -61,13 +61,6 @@ struct vc5_screen { const char *name; - /** The last seqno we've completed a wait for. - * - * This lets us slightly optimize our waits by skipping wait syscalls - * if we know the job's already done. - */ - uint64_t finished_seqno; - struct slab_parent_pool transfer_pool; struct vc5_bo_cache { |