Commit 39e7f6f8 authored by Maarten Lankhorst's avatar Maarten Lankhorst

drm/radeon: use rcu waits in some ioctls

Signed-off-by: default avatarMaarten Lankhorst <maarten.lankhorst@canonical.com>
Reviewed-by: default avatarChristian König <christian.koenig@amd.com>
parent 59701f96
...@@ -94,7 +94,7 @@ static int radeon_gem_set_domain(struct drm_gem_object *gobj, ...@@ -94,7 +94,7 @@ static int radeon_gem_set_domain(struct drm_gem_object *gobj,
{ {
struct radeon_bo *robj; struct radeon_bo *robj;
uint32_t domain; uint32_t domain;
int r; long r;
/* FIXME: reeimplement */ /* FIXME: reeimplement */
robj = gem_to_radeon_bo(gobj); robj = gem_to_radeon_bo(gobj);
...@@ -110,9 +110,12 @@ static int radeon_gem_set_domain(struct drm_gem_object *gobj, ...@@ -110,9 +110,12 @@ static int radeon_gem_set_domain(struct drm_gem_object *gobj,
} }
if (domain == RADEON_GEM_DOMAIN_CPU) { if (domain == RADEON_GEM_DOMAIN_CPU) {
/* Asking for cpu access wait for object idle */ /* Asking for cpu access wait for object idle */
r = radeon_bo_wait(robj, NULL, false); r = reservation_object_wait_timeout_rcu(robj->tbo.resv, true, true, 30 * HZ);
if (r) { if (!r)
printk(KERN_ERR "Failed to wait for object !\n"); r = -EBUSY;
if (r < 0 && r != -EINTR) {
printk(KERN_ERR "Failed to wait for object: %li\n", r);
return r; return r;
} }
} }
...@@ -449,15 +452,22 @@ int radeon_gem_wait_idle_ioctl(struct drm_device *dev, void *data, ...@@ -449,15 +452,22 @@ int radeon_gem_wait_idle_ioctl(struct drm_device *dev, void *data,
struct drm_radeon_gem_wait_idle *args = data; struct drm_radeon_gem_wait_idle *args = data;
struct drm_gem_object *gobj; struct drm_gem_object *gobj;
struct radeon_bo *robj; struct radeon_bo *robj;
int r; int r = 0;
uint32_t cur_placement = 0; uint32_t cur_placement = 0;
long ret;
gobj = drm_gem_object_lookup(dev, filp, args->handle); gobj = drm_gem_object_lookup(dev, filp, args->handle);
if (gobj == NULL) { if (gobj == NULL) {
return -ENOENT; return -ENOENT;
} }
robj = gem_to_radeon_bo(gobj); robj = gem_to_radeon_bo(gobj);
r = radeon_bo_wait(robj, &cur_placement, false);
ret = reservation_object_wait_timeout_rcu(robj->tbo.resv, true, true, 30 * HZ);
if (ret == 0)
r = -EBUSY;
else if (ret < 0)
r = ret;
/* Flush HDP cache via MMIO if necessary */ /* Flush HDP cache via MMIO if necessary */
if (rdev->asic->mmio_hdp_flush && if (rdev->asic->mmio_hdp_flush &&
radeon_mem_type_to_domain(cur_placement) == RADEON_GEM_DOMAIN_VRAM) radeon_mem_type_to_domain(cur_placement) == RADEON_GEM_DOMAIN_VRAM)
......
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