summaryrefslogtreecommitdiffstats
path: root/firmware
diff options
context:
space:
mode:
authorFrediano Ziglio2015-06-03 13:09:17 +0200
committerDave Airlie2015-06-05 03:00:51 +0200
commite07154e28cb079562fcc0be6915cb7d4890fdba1 (patch)
treea3a566b77ad1db29f87be44eb98433cdcbf6f411 /firmware
parentdrm/qxl: Remove format string errors (diff)
downloadkernel-qcow2-linux-e07154e28cb079562fcc0be6915cb7d4890fdba1.tar.gz
kernel-qcow2-linux-e07154e28cb079562fcc0be6915cb7d4890fdba1.tar.xz
kernel-qcow2-linux-e07154e28cb079562fcc0be6915cb7d4890fdba1.zip
drm/qxl: Move main reference counter to GEM object instead of TTM ones
qxl_bo structure has two reference counters, one in the GEM object and another in the TTM object. The GEM object keep a counter to the TTM object so when GEM counter reached zero the TTM counter (using qxl_bo_unref) was decremented. The qxl object is fully freed (both GEM and TTM part are cleaned) when the TTM counter reach zero. One issue was that surface idr structure has no owning on qxl_bo objects however it contains a pointer to qxl_bo object. This caused some nasty race condition for instance qxl_bo object was reaped even after counter was already zero. This patch fix these races moving main counter (the one used by qxl_bo_(un)ref) to GEM object which cleanup routine (qxl_gem_object_free) remove the idr pointer (using qxl_surface_evict) when the counters are still valid. Signed-off-by: Frediano Ziglio <fziglio@redhat.com> Reviewed-by: Dave Airlie <airlied@redhat.com> Signed-off-by: Dave Airlie <airlied@redhat.com>
Diffstat (limited to 'firmware')
0 files changed, 0 insertions, 0 deletions