[Intel-gfx] [PATCH 2/2] drm/i915: Warn if ring tail is not qword aligned

[Ville Syrjälä]

On Mon, Nov 26, 2012 at 04:25:47PM +0000, Chris Wilson wrote:
> On Mon, 26 Nov 2012 14:48:19 +0200, ville.syrjala at linux.intel.com wrote:
> > From: Ville Syrjälä <ville.syrjala at linux.intel.com>
> > 
> > Ringbuffer tail pointer must be qword aligned. Warn if someone
> > makes a mistake and forgets to pad the ring when the commands
> > inserted into the ring don't align to qword naturally.
> 
> The assertion should be that we wrote precisely the number of dwords we
> declared in intel_ring_begin(). Which is one of the important factors to
> check whenever reviewing such code. This assertion (which should be a
> BUG_ON) is no substitute for such review.

Yeah. I was considering adding some reserved_space field to the ring,
populate it in ring_begin(), and and make sure it was correctly
consumed at ring_advance(). If you think that sounds good, I can cook
up a patch for it.

-- 
Ville Syrjälä
Intel OTC