Commit 33a63e68 authored by Lucas Stach's avatar Lucas Stach

drm/etnaviv: move object unpinning to submit cleanup

This is safe to call in all paths, as the BO_PINNED flag tells us if the BO
needs unpinning.
Signed-off-by: default avatarLucas Stach <l.stach@pengutronix.de>
Reviewed-by: default avatarPhilipp Zabel <p.zabel@pengutronix.de>
parent 9efabd73
...@@ -207,19 +207,6 @@ static void submit_attach_object_fences(struct etnaviv_gem_submit *submit) ...@@ -207,19 +207,6 @@ static void submit_attach_object_fences(struct etnaviv_gem_submit *submit)
} }
} }
static void submit_unpin_objects(struct etnaviv_gem_submit *submit)
{
int i;
for (i = 0; i < submit->nr_bos; i++) {
if (submit->bos[i].flags & BO_PINNED)
etnaviv_gem_mapping_unreference(submit->bos[i].mapping);
submit->bos[i].mapping = NULL;
submit->bos[i].flags &= ~BO_PINNED;
}
}
static int submit_pin_objects(struct etnaviv_gem_submit *submit) static int submit_pin_objects(struct etnaviv_gem_submit *submit)
{ {
int i, ret = 0; int i, ret = 0;
...@@ -362,6 +349,13 @@ static void submit_cleanup(struct etnaviv_gem_submit *submit) ...@@ -362,6 +349,13 @@ static void submit_cleanup(struct etnaviv_gem_submit *submit)
for (i = 0; i < submit->nr_bos; i++) { for (i = 0; i < submit->nr_bos; i++) {
struct etnaviv_gem_object *etnaviv_obj = submit->bos[i].obj; struct etnaviv_gem_object *etnaviv_obj = submit->bos[i].obj;
/* unpin all objects */
if (submit->bos[i].flags & BO_PINNED) {
etnaviv_gem_mapping_unreference(submit->bos[i].mapping);
submit->bos[i].mapping = NULL;
submit->bos[i].flags &= ~BO_PINNED;
}
/* if the GPU submit failed, objects might still be locked */ /* if the GPU submit failed, objects might still be locked */
submit_unlock_object(submit, i); submit_unlock_object(submit, i);
drm_gem_object_put_unlocked(&etnaviv_obj->base); drm_gem_object_put_unlocked(&etnaviv_obj->base);
...@@ -508,23 +502,23 @@ int etnaviv_ioctl_gem_submit(struct drm_device *dev, void *data, ...@@ -508,23 +502,23 @@ int etnaviv_ioctl_gem_submit(struct drm_device *dev, void *data,
ret = submit_pin_objects(submit); ret = submit_pin_objects(submit);
if (ret) if (ret)
goto out; goto err_submit_objects;
ret = submit_reloc(submit, stream, args->stream_size / 4, ret = submit_reloc(submit, stream, args->stream_size / 4,
relocs, args->nr_relocs); relocs, args->nr_relocs);
if (ret) if (ret)
goto out; goto err_submit_objects;
ret = submit_perfmon_validate(submit, cmdbuf, pmrs, args->nr_pmrs); ret = submit_perfmon_validate(submit, cmdbuf, pmrs, args->nr_pmrs);
if (ret) if (ret)
goto out; goto err_submit_objects;
memcpy(cmdbuf->vaddr, stream, args->stream_size); memcpy(cmdbuf->vaddr, stream, args->stream_size);
cmdbuf->user_size = ALIGN(args->stream_size, 8); cmdbuf->user_size = ALIGN(args->stream_size, 8);
ret = etnaviv_gpu_submit(gpu, submit, cmdbuf); ret = etnaviv_gpu_submit(gpu, submit, cmdbuf);
if (ret) if (ret)
goto out; goto err_submit_objects;
submit_attach_object_fences(submit); submit_attach_object_fences(submit);
...@@ -540,7 +534,7 @@ int etnaviv_ioctl_gem_submit(struct drm_device *dev, void *data, ...@@ -540,7 +534,7 @@ int etnaviv_ioctl_gem_submit(struct drm_device *dev, void *data,
sync_file = sync_file_create(submit->out_fence); sync_file = sync_file_create(submit->out_fence);
if (!sync_file) { if (!sync_file) {
ret = -ENOMEM; ret = -ENOMEM;
goto out; goto err_submit_objects;
} }
fd_install(out_fence_fd, sync_file->file); fd_install(out_fence_fd, sync_file->file);
} }
...@@ -548,9 +542,6 @@ int etnaviv_ioctl_gem_submit(struct drm_device *dev, void *data, ...@@ -548,9 +542,6 @@ int etnaviv_ioctl_gem_submit(struct drm_device *dev, void *data,
args->fence_fd = out_fence_fd; args->fence_fd = out_fence_fd;
args->fence = submit->out_fence->seqno; args->fence = submit->out_fence->seqno;
out:
submit_unpin_objects(submit);
err_submit_objects: err_submit_objects:
submit_cleanup(submit); submit_cleanup(submit);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment