Commit 74d9a633 authored by Frediano Ziglio's avatar Frediano Ziglio Committed by Dave Airlie

drm/qxl: Simplify cleaning qxl processing command

In qxlhw_handle_to_bo we incremented counters twice, one time for release object
and one for reloc_info.
In the main function however reloc_info references was drop much earlier than
release so keeping the pointer only on release is safe and make cleaning
process easier.
Signed-off-by: default avatarFrediano Ziglio <fziglio@redhat.com>
Reviewed-by: default avatarDave Airlie <airlied@redhat.com>
Signed-off-by: default avatarDave Airlie <airlied@redhat.com>
parent e07154e2
...@@ -122,10 +122,9 @@ static struct qxl_bo *qxlhw_handle_to_bo(struct qxl_device *qdev, ...@@ -122,10 +122,9 @@ static struct qxl_bo *qxlhw_handle_to_bo(struct qxl_device *qdev,
qobj = gem_to_qxl_bo(gobj); qobj = gem_to_qxl_bo(gobj);
ret = qxl_release_list_add(release, qobj); ret = qxl_release_list_add(release, qobj);
if (ret) { drm_gem_object_unreference_unlocked(gobj);
drm_gem_object_unreference_unlocked(gobj); if (ret)
return NULL; return NULL;
}
return qobj; return qobj;
} }
...@@ -145,7 +144,7 @@ static int qxl_process_single_command(struct qxl_device *qdev, ...@@ -145,7 +144,7 @@ static int qxl_process_single_command(struct qxl_device *qdev,
struct qxl_release *release; struct qxl_release *release;
struct qxl_bo *cmd_bo; struct qxl_bo *cmd_bo;
void *fb_cmd; void *fb_cmd;
int i, j, ret, num_relocs; int i, ret, num_relocs;
int unwritten; int unwritten;
switch (cmd->type) { switch (cmd->type) {
...@@ -269,12 +268,6 @@ static int qxl_process_single_command(struct qxl_device *qdev, ...@@ -269,12 +268,6 @@ static int qxl_process_single_command(struct qxl_device *qdev,
qxl_release_fence_buffer_objects(release); qxl_release_fence_buffer_objects(release);
out_free_bos: out_free_bos:
for (j = 0; j < num_relocs; j++) {
if (reloc_info[j].dst_bo != cmd_bo)
drm_gem_object_unreference_unlocked(&reloc_info[j].dst_bo->gem_base);
if (reloc_info[j].src_bo && reloc_info[j].src_bo != cmd_bo)
drm_gem_object_unreference_unlocked(&reloc_info[j].src_bo->gem_base);
}
out_free_release: out_free_release:
if (ret) if (ret)
qxl_release_free(qdev, release); qxl_release_free(qdev, release);
......
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