openkylin
/
linux
mirror of https://gitee.com/openkylin/linux.git
9
0
Fork 0
Code Issues Projects Releases Wiki Activity

drm/i915: Kill obj->gtt_offset 

With the getters in place from the previous patch this members serves no
purpose other than saving one spare pointer chase, which will be killed
in the next patch anyway.

Moving to VMAs, this members adds unnecessary confusion since an object
may exist at different offsets in different VMs.

v2: Properly preserve the stolen offset. This code is a bit hacky but it
all goes away when we embed the drm_mm_node and removes the need for the
incorrect patch I submitted previously: "Use gtt_space->start for stolen
reservation"

Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
This commit is contained in:
Ben Widawsky 2013-07-05 14:41:05 -07:00 committed by Daniel Vetter
parent f343c5f647
commit edd41a870f
4 changed files with 15 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
drivers/gpu/drm/i915/i915_drv.h
View File

@ -1201,7 +1201,7 @@ enum hdmi_force_audio {
HDMI_AUDIO_ON, /* force turn on HDMI audio */ HDMI_AUDIO_ON, /* force turn on HDMI audio */
}; };
#define I915_GTT_RESERVED ((struct drm_mm_node *)0x1) #define I915_GTT_RESERVED (1<<0)
#define I915_GTT_OFFSET_NONE ((u32)-1) #define I915_GTT_OFFSET_NONE ((u32)-1)
struct drm_i915_gem_object_ops { struct drm_i915_gem_object_ops {
@ -1329,13 +1329,6 @@ struct drm_i915_gem_object {
unsigned long exec_handle; unsigned long exec_handle;
struct drm_i915_gem_exec_object2 *exec_entry; struct drm_i915_gem_exec_object2 *exec_entry;
/**
* Current offset of the object in GTT space.
*
* This is the same as gtt_space->start
*/
uint32_t gtt_offset;
struct intel_ring_buffer *ring; struct intel_ring_buffer *ring;
/** Breadcrumb of last rendering to the buffer. */ /** Breadcrumb of last rendering to the buffer. */

2
drivers/gpu/drm/i915/i915_gem.c
View File

@ -2634,7 +2634,6 @@ i915_gem_object_unbind(struct drm_i915_gem_object *obj)
drm_mm_put_block(obj->gtt_space); drm_mm_put_block(obj->gtt_space);
obj->gtt_space = NULL; obj->gtt_space = NULL;
obj->gtt_offset = 0;
return 0; return 0;
} }
@ -3164,7 +3163,6 @@ i915_gem_object_bind_to_gtt(struct drm_i915_gem_object *obj,
list_add_tail(&obj->mm_list, &dev_priv->mm.inactive_list); list_add_tail(&obj->mm_list, &dev_priv->mm.inactive_list);
obj->gtt_space = node; obj->gtt_space = node;
obj->gtt_offset = node->start;
fenceable = fenceable =
node->size == fence_size && node->size == fence_size &&

14
drivers/gpu/drm/i915/i915_gem_gtt.c
View File

@ -629,18 +629,20 @@ void i915_gem_setup_global_gtt(struct drm_device *dev,
/* Mark any preallocated objects as occupied */ /* Mark any preallocated objects as occupied */
list_for_each_entry(obj, &dev_priv->mm.bound_list, global_list) { list_for_each_entry(obj, &dev_priv->mm.bound_list, global_list) {
uintptr_t offset = (uintptr_t) obj->gtt_space;
int ret; int ret;
DRM_DEBUG_KMS("reserving preallocated space: %x + %zx\n", DRM_DEBUG_KMS("reserving preallocated space: %lx + %zx\n",
obj->gtt_offset, obj->base.size); offset, obj->base.size);
BUG_ON(obj->gtt_space != I915_GTT_RESERVED); BUG_ON((offset & I915_GTT_RESERVED) != 0);
offset &= ~I915_GTT_RESERVED;
obj->gtt_space = kzalloc(sizeof(*obj->gtt_space), GFP_KERNEL); obj->gtt_space = kzalloc(sizeof(*obj->gtt_space), GFP_KERNEL);
if (!obj->gtt_space) { if (!obj->gtt_space) {
DRM_ERROR("Failed to preserve object at offset %x\n", DRM_ERROR("Failed to preserve object at offset %lx\n",
obj->gtt_offset); offset);
continue; continue;
} }
obj->gtt_space->start = obj->gtt_offset; obj->gtt_space->start = (unsigned long)offset;
obj->gtt_space->size = obj->base.size; obj->gtt_space->size = obj->base.size;
ret = drm_mm_reserve_node(&dev_priv->mm.gtt_space, ret = drm_mm_reserve_node(&dev_priv->mm.gtt_space,
obj->gtt_space); obj->gtt_space);

9
drivers/gpu/drm/i915/i915_gem_stolen.c
View File

@ -389,10 +389,13 @@ i915_gem_object_create_stolen_for_preallocated(struct drm_device *dev,
DRM_DEBUG_KMS("failed to allocate stolen GTT space\n"); DRM_DEBUG_KMS("failed to allocate stolen GTT space\n");
goto free_out; goto free_out;
} }
} else } else {
obj->gtt_space = I915_GTT_RESERVED; if (WARN_ON(gtt_offset & ~PAGE_MASK))
DRM_DEBUG_KMS("Cannot preserve non page aligned offset\n");
obj->gtt_space =
(struct drm_mm_node *)((uintptr_t)(I915_GTT_RESERVED | gtt_offset));
}
obj->gtt_offset = gtt_offset;
obj->has_global_gtt_mapping = 1; obj->has_global_gtt_mapping = 1;
list_add_tail(&obj->global_list, &dev_priv->mm.bound_list); list_add_tail(&obj->global_list, &dev_priv->mm.bound_list);