diff options
author | Eric Anholt <eric@anholt.net> | 2015-05-29 17:27:18 -0700 |
---|---|---|
committer | Eric Anholt <eric@anholt.net> | 2015-06-04 14:15:40 -0700 |
commit | d161078693e34a3315d5fd8bee9916eebfc72df3 (patch) | |
tree | f14b8dbec220872272587fd1b8e0345b6fdf954e | |
parent | a7b0de93b1ecec6219e675a2c43cfb70800f2efd (diff) | |
download | linux-d161078693e34a3315d5fd8bee9916eebfc72df3.tar.gz |
drm/vc4: Return the proper incremented seqno for our job.
I was returning the seqno before the increment, so userspace's waits
on job completion could be too early.
Signed-off-by: Eric Anholt <eric@anholt.net>
-rw-r--r-- | drivers/gpu/drm/vc4/vc4_gem.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/gpu/drm/vc4/vc4_gem.c b/drivers/gpu/drm/vc4/vc4_gem.c index 7e8070668eb4..abc0684d7e28 100644 --- a/drivers/gpu/drm/vc4/vc4_gem.c +++ b/drivers/gpu/drm/vc4/vc4_gem.c @@ -593,9 +593,6 @@ vc4_submit_cl_ioctl(struct drm_device *dev, void *data, if (ret) goto fail; - /* Return the seqno for our job. */ - args->seqno = vc4->emit_seqno; - /* Clear this out of the struct we'll be putting in the queue, * since it's part of our stack. */ @@ -603,6 +600,9 @@ vc4_submit_cl_ioctl(struct drm_device *dev, void *data, vc4_queue_submit(dev, exec); + /* Return the seqno for our job. */ + args->seqno = vc4->emit_seqno; + mutex_unlock(&dev->struct_mutex); return 0; |