i965: Delete pitch alignment assertion in get_blit_intratile_offset_el.
authorKenneth Graunke <kenneth@whitecape.org>
Tue, 1 Aug 2017 05:04:25 +0000 (22:04 -0700)
committerKenneth Graunke <kenneth@whitecape.org>
Wed, 2 Aug 2017 17:01:34 +0000 (10:01 -0700)
The cacheline alignment restriction is on the base address; the pitch
can be anything.

Fixes assertion failures when using primus (say, on glxgears, which
creates a 300x300 linear BGRX surface with a pitch of 1200):

intel_blit.c:190: get_blit_intratile_offset_el: Assertion `mt->surf.row_pitch % 64 == 0' failed.

Cc: mesa-stable@lists.freedesktop.org
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
src/mesa/drivers/dri/i965/intel_blit.c

index eca87368047c5285f0e8e337447bf236afd1265e..b1db7aa2293b971f81d0347f2918d80f8bbd1a81 100644 (file)
@@ -187,7 +187,6 @@ get_blit_intratile_offset_el(const struct brw_context *brw,
        * The offsets we get from ISL in the tiled case are already aligned.
        * In the linear case, we need to do some of our own aligning.
        */
-      assert(mt->surf.row_pitch % 64 == 0);
       uint32_t delta = *base_address_offset & 63;
       assert(delta % mt->cpp == 0);
       *base_address_offset -= delta;