Re: [PATCH v3 1/6] drm/i915/gen8: Add infrastructure to initialize WA batch buffers

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Jun 05, 2015 at 11:34:01AM +0100, Arun Siluvery wrote:
> +	/* FIXME: fill unused locations with NOOPs.
> +	 * Replace these instructions with WA
> +	 */
> +        while (index < end)
> +		reg_state[index++] = MI_NOOP;

I found calling it reg_state was very confusing. Maybe batch, bb, data or cmd?
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx





[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux