drm/i915: Extract compute_partial_view()
In order to reuse the partial view for selftesting, extract the common function for computing the view. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Reviewed-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com> Link: http://patchwork.freedesktop.org/patch/msgid/20170110095633.6612-1-chris@chris-wilson.co.uk
This commit is contained in:
@@ -1749,6 +1749,31 @@ int i915_gem_mmap_gtt_version(void)
|
|||||||
return 1;
|
return 1;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
static inline struct i915_ggtt_view
|
||||||
|
compute_partial_view(struct drm_i915_gem_object *obj,
|
||||||
|
struct vm_area_struct *area,
|
||||||
|
pgoff_t page_offset,
|
||||||
|
unsigned int chunk)
|
||||||
|
{
|
||||||
|
struct i915_ggtt_view view;
|
||||||
|
|
||||||
|
if (i915_gem_object_is_tiled(obj))
|
||||||
|
chunk = roundup(chunk, tile_row_pages(obj));
|
||||||
|
|
||||||
|
memset(&view, 0, sizeof(view));
|
||||||
|
view.type = I915_GGTT_VIEW_PARTIAL;
|
||||||
|
view.params.partial.offset = rounddown(page_offset, chunk);
|
||||||
|
view.params.partial.size =
|
||||||
|
min_t(unsigned int, chunk,
|
||||||
|
vma_pages(area) - view.params.partial.offset);
|
||||||
|
|
||||||
|
/* If the partial covers the entire object, just create a normal VMA. */
|
||||||
|
if (chunk >= obj->base.size >> PAGE_SHIFT)
|
||||||
|
view.type = I915_GGTT_VIEW_NORMAL;
|
||||||
|
|
||||||
|
return view;
|
||||||
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* i915_gem_fault - fault a page into the GTT
|
* i915_gem_fault - fault a page into the GTT
|
||||||
* @area: CPU VMA in question
|
* @area: CPU VMA in question
|
||||||
@@ -1825,26 +1850,10 @@ int i915_gem_fault(struct vm_area_struct *area, struct vm_fault *vmf)
|
|||||||
/* Now pin it into the GTT as needed */
|
/* Now pin it into the GTT as needed */
|
||||||
vma = i915_gem_object_ggtt_pin(obj, NULL, 0, 0, flags);
|
vma = i915_gem_object_ggtt_pin(obj, NULL, 0, 0, flags);
|
||||||
if (IS_ERR(vma)) {
|
if (IS_ERR(vma)) {
|
||||||
struct i915_ggtt_view view;
|
|
||||||
unsigned int chunk_size;
|
|
||||||
|
|
||||||
/* Use a partial view if it is bigger than available space */
|
/* Use a partial view if it is bigger than available space */
|
||||||
chunk_size = MIN_CHUNK_PAGES;
|
struct i915_ggtt_view view =
|
||||||
if (i915_gem_object_is_tiled(obj))
|
compute_partial_view(obj, area,
|
||||||
chunk_size = roundup(chunk_size, tile_row_pages(obj));
|
page_offset, MIN_CHUNK_PAGES);
|
||||||
|
|
||||||
memset(&view, 0, sizeof(view));
|
|
||||||
view.type = I915_GGTT_VIEW_PARTIAL;
|
|
||||||
view.params.partial.offset = rounddown(page_offset, chunk_size);
|
|
||||||
view.params.partial.size =
|
|
||||||
min_t(unsigned int, chunk_size,
|
|
||||||
vma_pages(area) - view.params.partial.offset);
|
|
||||||
|
|
||||||
/* If the partial covers the entire object, just create a
|
|
||||||
* normal VMA.
|
|
||||||
*/
|
|
||||||
if (chunk_size >= obj->base.size >> PAGE_SHIFT)
|
|
||||||
view.type = I915_GGTT_VIEW_NORMAL;
|
|
||||||
|
|
||||||
/* Userspace is now writing through an untracked VMA, abandon
|
/* Userspace is now writing through an untracked VMA, abandon
|
||||||
* all hope that the hardware is able to track future writes.
|
* all hope that the hardware is able to track future writes.
|
||||||
|
Reference in New Issue
Block a user