panfrost: Workaround -bshadow regression
authorAlyssa Rosenzweig <alyssa@rosenzweig.io>
Sun, 28 Apr 2019 21:39:20 +0000 (21:39 +0000)
committerAlyssa Rosenzweig <alyssa@rosenzweig.io>
Sun, 28 Apr 2019 21:39:20 +0000 (21:39 +0000)
I have *no* idea what's happening here, but let's not regress an app
that used to work in the mean time while we're figuring it out..

Signed-off-by: Alyssa Rosenzweig <alyssa@rosenzweig.io>
src/gallium/drivers/panfrost/pan_context.c

index 9d3d0d2..42db831 100644 (file)
@@ -1591,7 +1591,14 @@ panfrost_create_vertex_elements_state(
         so->num_elements = num_elements;
         memcpy(so->pipe, elements, sizeof(*elements) * num_elements);
 
-        /* Allocate memory for the descriptor state */
+        /* XXX: What the cornball? This is totally, 100%, unapologetically
+         * nonsense. And yet it somehow fixes a regression in -bshadow
+         * (previously, we allocated the descriptor here... a newer commit
+         * removed that allocation, and then memory corruption led to
+         * shader_meta getting overwritten in bad ways and then the whole test
+         * case falling apart . TODO: LOOK INTO PLEASE XXX XXX BAD XXX XXX XXX
+         */
+        panfrost_allocate_chunk(pan_context(pctx), 0, HEAP_DESCRIPTOR);
 
         for (int i = 0; i < num_elements; ++i) {
                 so->hw[i].index = elements[i].vertex_buffer_index;