On Mon, Jan 18, 2016 at 11:54:20PM +0800, Geliang Tang wrote:
Use hlist_for_each_entry() instead of hlist_for_each() to simplify the code.
Signed-off-by: Geliang Tang geliangtang@163.com
Changes in v2:
- Keep head = &eb->buckets[handle & eb->and]
Queued for -next, thanks for the patch. -Daniel
drivers/gpu/drm/i915/i915_gem_execbuffer.c | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_gem_execbuffer.c b/drivers/gpu/drm/i915/i915_gem_execbuffer.c index dccb517..7dc95a7 100644 --- a/drivers/gpu/drm/i915/i915_gem_execbuffer.c +++ b/drivers/gpu/drm/i915/i915_gem_execbuffer.c @@ -193,13 +193,10 @@ static struct i915_vma *eb_get_vma(struct eb_vmas *eb, unsigned long handle) return eb->lut[handle]; } else { struct hlist_head *head;
struct hlist_node *node;
struct i915_vma *vma;head = &eb->buckets[handle & eb->and];
hlist_for_each(node, head) {struct i915_vma *vma;vma = hlist_entry(node, struct i915_vma, exec_node);
hlist_for_each_entry(vma, head, exec_node) { if (vma->exec_handle == handle) return vma;-- 2.5.0
Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx