pan/decode: Remove size/stride divisibility check
authorAlyssa Rosenzweig <alyssa.rosenzweig@collabora.com>
Thu, 22 Aug 2019 20:09:00 +0000 (13:09 -0700)
committerAlyssa Rosenzweig <alyssa.rosenzweig@collabora.com>
Thu, 22 Aug 2019 20:31:39 +0000 (13:31 -0700)
The hardware doesn't care, and a lot of Panfrost code relies on an
oversized buffer. The important part is that (stride *
padded_num_vertices) is no greater than size, which we'll need to check
once we validate instancing.

Signed-off-by: Alyssa Rosenzweig <alyssa.rosenzweig@collabora.com>
src/panfrost/pandecode/decode.c

index 456d0fd15c82140fe566a38b3311c63f4b469adf..2de376d46042917a4915c13c4bbc0b5bb7132198 100644 (file)
@@ -1276,13 +1276,9 @@ pandecode_attributes(const struct pandecode_mapped_memory *mem,
                 /* Print the stride and size */
                 pandecode_log_cont("<%u>[%u]", attr[i].stride, attr[i].size);
 
-                /* Check: the size must be divisible by the stride */
-                if (attr[i].size % attr[i].stride)
-                        pandecode_msg("XXX: size not divisible by stride\n");
-
-                /* TODO: Sanity check the quotient itself -- it should equal
-                 * vertex count (or something computed from it for instanced)
-                 * which means we can check and elide */
+                /* TODO: Sanity check the quotient itself. It must be equal to
+                 * (or be greater than, if the driver added padding) the padded
+                 * vertex count. */
 
                 /* Finally, print the pointer */
                 mali_ptr raw_elements = attr[i].elements & ~7;