From 3e405afeb9c1cb1182f83e2a1fd6f0beb199df64 Mon Sep 17 00:00:00 2001 From: Icecream95 Date: Thu, 14 Oct 2021 16:38:38 +1300 Subject: [PATCH] panfrost: Don't initialise the trampolines array PIPE_MAX_SHADER_SAMPLER_VIEWS is 128, so we just end up initialising a kilobyte of memory for no reason, when usually only a couple of sampler views are used. Fixes: 53ef20f08d4 ("panfrost: Handle NULL sampler views") Part-of: --- src/gallium/drivers/panfrost/pan_cmdstream.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/gallium/drivers/panfrost/pan_cmdstream.c b/src/gallium/drivers/panfrost/pan_cmdstream.c index 6471bd0..8040040 100644 --- a/src/gallium/drivers/panfrost/pan_cmdstream.c +++ b/src/gallium/drivers/panfrost/pan_cmdstream.c @@ -1401,13 +1401,15 @@ panfrost_emit_texture_descriptors(struct panfrost_batch *batch, return T.gpu; #else - uint64_t trampolines[PIPE_MAX_SHADER_SAMPLER_VIEWS] = { 0 }; + uint64_t trampolines[PIPE_MAX_SHADER_SAMPLER_VIEWS]; for (int i = 0; i < ctx->sampler_view_count[stage]; ++i) { struct panfrost_sampler_view *view = ctx->sampler_views[stage][i]; - if (!view) + if (!view) { + trampolines[i] = 0; continue; + } panfrost_update_sampler_view(view, &ctx->base); -- 2.7.4