[Mesa-stable] [Mesa-dev] [PATCH] i965: Delete pitch alignment assertion in get_blit_intratile_offset_el.

Chris Wilson chris at chris-wilson.co.uk
Wed Aug 2 10:58:25 UTC 2017


Quoting Kenneth Graunke (2017-08-02 02:14:22)
> The cacheline alignment restriction is on the base address; the pitch
> can be anything.

From my understanding of the bug, this is correct and the workaround is
to adjust the xoffset as you have done. (And since the chunk is limited
to less than 32k pixels wide, adjusting the chunk width is not going to
cause an overflow.)

The code was correct and the new assertion was a little overzealous,
Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk>

The 64B alignment for pitch is only applicable to linear scanout, iirc.
-Chris


More information about the mesa-stable mailing list