intel/blorp: Fix the early return condition in convert_to_single_slice
authorJason Ekstrand <jason.ekstrand@intel.com>
Wed, 31 Aug 2016 19:58:54 +0000 (12:58 -0700)
committerJason Ekstrand <jason.ekstrand@intel.com>
Tue, 13 Sep 2016 02:42:57 +0000 (19:42 -0700)
commit6da968b651f7c3f3475f68c80756adce441754e1
tree64a8c8ce18203076d8d75f8a5feabbfbdc405da3
parentec7e0d62c5fe5b17e833c7ab91d0b753f0d02918
intel/blorp: Fix the early return condition in convert_to_single_slice

The convert_to_single_slice operation is *mostly* idempotent.  The only
non-repeatable thing it does is that, when it sets the intratile offset
fields, it just overwrites them instead of doing a += operation.  This is
supposed to be ok because we have an early return at the top that should
make it bail of the surface is already a single slice.  Unfortunately, the
if condition has been broken ever since it was first added in 96fa98c18.
This commit fixes the condition and adds an assert to ensure we don't stomp
any non-zero intratile offsets.

Signed-off-by: Jason Ekstrand <jason@jlekstrand.net>
Reviewed-by: Topi Pohjolainen <topi.pohjolainen@intel.com>
src/intel/blorp/blorp_blit.c