Commit c2dc6cc9 authored by Chris Wilson's avatar Chris Wilson

drm/i915: Add a test that we terminate the trimmed sgtable as expected

In commit 0c40ce13 ("drm/i915: Trim the object sg table"), we expect
to copy exactly orig_st->nents across and allocate the table thusly.
The copy loop should therefore end with the new_sg being NULL.
Signed-off-by: default avatarChris Wilson <chris@chris-wilson.co.uk>
Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/20161219124346.550-2-chris@chris-wilson.co.ukReviewed-by: default avatarDaniel Vetter <daniel.vetter@ffwll.ch>
Reviewed-by: default avatarTvrtko Ursulin <tvrtko.ursulin@intel.com>
parent d766ef53
...@@ -2333,6 +2333,7 @@ static void i915_sg_trim(struct sg_table *orig_st) ...@@ -2333,6 +2333,7 @@ static void i915_sg_trim(struct sg_table *orig_st)
/* called before being DMA mapped, no need to copy sg->dma_* */ /* called before being DMA mapped, no need to copy sg->dma_* */
new_sg = sg_next(new_sg); new_sg = sg_next(new_sg);
} }
GEM_BUG_ON(new_sg); /* Should walk exactly nents and hit the end */
sg_free_table(orig_st); sg_free_table(orig_st);
......
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