drm/radeon: kill radeon_bo->gobj pointer
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch> Signed-off-by: Dave Airlie <airlied@redhat.com>
This commit is contained in:
parent
7e4d15d90a
commit
31c3603d83
2 changed files with 3 additions and 7 deletions
|
@ -258,7 +258,6 @@ struct radeon_bo {
|
||||||
int surface_reg;
|
int surface_reg;
|
||||||
/* Constant after initialization */
|
/* Constant after initialization */
|
||||||
struct radeon_device *rdev;
|
struct radeon_device *rdev;
|
||||||
struct drm_gem_object *gobj;
|
|
||||||
struct drm_gem_object gem_base;
|
struct drm_gem_object gem_base;
|
||||||
};
|
};
|
||||||
#define gem_to_radeon_bo(gobj) container_of((gobj), struct radeon_bo, gem_base)
|
#define gem_to_radeon_bo(gobj) container_of((gobj), struct radeon_bo, gem_base)
|
||||||
|
|
|
@ -127,7 +127,6 @@ retry:
|
||||||
return r;
|
return r;
|
||||||
}
|
}
|
||||||
bo->rdev = rdev;
|
bo->rdev = rdev;
|
||||||
bo->gobj = &bo->gem_base;
|
|
||||||
bo->gem_base.driver_private = NULL;
|
bo->gem_base.driver_private = NULL;
|
||||||
bo->surface_reg = -1;
|
bo->surface_reg = -1;
|
||||||
INIT_LIST_HEAD(&bo->list);
|
INIT_LIST_HEAD(&bo->list);
|
||||||
|
@ -266,7 +265,6 @@ int radeon_bo_evict_vram(struct radeon_device *rdev)
|
||||||
void radeon_bo_force_delete(struct radeon_device *rdev)
|
void radeon_bo_force_delete(struct radeon_device *rdev)
|
||||||
{
|
{
|
||||||
struct radeon_bo *bo, *n;
|
struct radeon_bo *bo, *n;
|
||||||
struct drm_gem_object *gobj;
|
|
||||||
|
|
||||||
if (list_empty(&rdev->gem.objects)) {
|
if (list_empty(&rdev->gem.objects)) {
|
||||||
return;
|
return;
|
||||||
|
@ -274,15 +272,14 @@ void radeon_bo_force_delete(struct radeon_device *rdev)
|
||||||
dev_err(rdev->dev, "Userspace still has active objects !\n");
|
dev_err(rdev->dev, "Userspace still has active objects !\n");
|
||||||
list_for_each_entry_safe(bo, n, &rdev->gem.objects, list) {
|
list_for_each_entry_safe(bo, n, &rdev->gem.objects, list) {
|
||||||
mutex_lock(&rdev->ddev->struct_mutex);
|
mutex_lock(&rdev->ddev->struct_mutex);
|
||||||
gobj = bo->gobj;
|
|
||||||
dev_err(rdev->dev, "%p %p %lu %lu force free\n",
|
dev_err(rdev->dev, "%p %p %lu %lu force free\n",
|
||||||
gobj, bo, (unsigned long)gobj->size,
|
&bo->gem_base, bo, (unsigned long)bo->gem_base.size,
|
||||||
*((unsigned long *)&gobj->refcount));
|
*((unsigned long *)&bo->gem_base.refcount));
|
||||||
mutex_lock(&bo->rdev->gem.mutex);
|
mutex_lock(&bo->rdev->gem.mutex);
|
||||||
list_del_init(&bo->list);
|
list_del_init(&bo->list);
|
||||||
mutex_unlock(&bo->rdev->gem.mutex);
|
mutex_unlock(&bo->rdev->gem.mutex);
|
||||||
radeon_bo_unref(&bo);
|
radeon_bo_unref(&bo);
|
||||||
drm_gem_object_unreference(gobj);
|
drm_gem_object_unreference(&bo->gem_base);
|
||||||
mutex_unlock(&rdev->ddev->struct_mutex);
|
mutex_unlock(&rdev->ddev->struct_mutex);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
Loading…
Add table
Reference in a new issue