On Fri, Aug 09, 2013 at 10:12:16PM -0700, Ben Widawsky wrote:
> static int
> i915_gem_execbuffer_move_to_gpu(struct intel_ring_buffer *ring,
> - struct list_head *objects)
> + struct list_head *vmas)
> {
> - struct drm_i915_gem_object *obj;
> + struct i915_vma *vma;
> uint32_t flush_domains = 0;
> int ret;
>
> - list_for_each_entry(obj, objects, exec_list) {
> - ret = i915_gem_object_sync(obj, ring);
> + list_for_each_entry(vma, vmas, exec_list) {
> + ret = i915_gem_object_sync(vma->obj, ring);
> if (ret)
> return ret;
>
> - if (obj->base.write_domain & I915_GEM_DOMAIN_CPU)
> - i915_gem_clflush_object(obj);
> + if (vma->obj->base.write_domain & I915_GEM_DOMAIN_CPU)
> + i915_gem_clflush_object(vma->obj);
>
> - flush_domains |= obj->base.write_domain;
> + flush_domains |= vma->obj->base.write_domain;
This, and a few more places, will be neater with a local
struct drm_i915_gem_object *obj = vma->obj;
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
