r600g: make sure async blit is done 8 * pitch at a time v2
authorJerome Glisse <jglisse@redhat.com>
Wed, 6 Feb 2013 20:03:17 +0000 (15:03 -0500)
committerJerome Glisse <jglisse@redhat.com>
Mon, 11 Feb 2013 23:44:18 +0000 (18:44 -0500)
The blit must be aligned on 8 horizontal block.

v2: no need to align the reminder

Signed-off-by: Jerome Glisse <jglisse@redhat.com>
src/gallium/drivers/r600/r600_state.c

index 67c4d99..5322850 100644 (file)
@@ -3068,14 +3068,15 @@ static boolean r600_dma_copy_tile(struct r600_context *rctx,
                return FALSE;
        }
 
-       size = (copy_height * pitch) >> 2;
-       ncopy = (size / 0x0000ffff) + !!(size % 0x0000ffff);
+       /* It's a r6xx/r7xx limitation, the blit must be on 8 boundary for number
+        * line in the blit. Compute max 8 line we can copy in the size limit
+        */
+       cheight = ((0x0000ffff << 2) / pitch) & 0xfffffff8;
+       ncopy = (copy_height / cheight) + !!(copy_height % cheight);
        r600_need_dma_space(rctx, ncopy * 7);
+
        for (i = 0; i < ncopy; i++) {
-               cheight = copy_height;
-               if (((cheight * pitch) >> 2) > 0x0000ffff) {
-                       cheight = (0x0000ffff << 2) / pitch;
-               }
+               cheight = cheight > copy_height ? copy_height : cheight;
                size = (cheight * pitch) >> 2;
                /* emit reloc before writting cs so that cs is always in consistent state */
                r600_context_bo_reloc(rctx, &rctx->rings.dma, &rsrc->resource, RADEON_USAGE_READ);