Mesa (master): i965: don't include compute resources in "Combined" limits
GitLab Mirror
gitlab-mirror at kemper.freedesktop.org
Fri Aug 24 00:29:29 UTC 2018
Module: Mesa
Branch: master
Commit: b03dcb1e5f507c5950d0de053a6f76e6306ee71f
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=b03dcb1e5f507c5950d0de053a6f76e6306ee71f
Author: Kenneth Graunke <kenneth at whitecape.org>
Date: Thu Aug 23 17:24:19 2018 -0700
i965: don't include compute resources in "Combined" limits
The combined limits should only include shader stages that can be active
at the same time. We don't need to include compute.
See also cff290df4c09547cd2cb3b129ec59bdebdadba90 for st/mesa.
Unbreaks i965 from assert failing on driver load since Marek's
45f87a48f94148b484961f18a4f1ccf86f066b1c, which dropped the core
Mesa capabilities before adjusting driver limits down to match.
---
src/mesa/drivers/dri/i965/brw_context.c | 22 +++++++++++-----------
1 file changed, 11 insertions(+), 11 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_context.c b/src/mesa/drivers/dri/i965/brw_context.c
index 6ba64e4e06..59871ed84e 100644
--- a/src/mesa/drivers/dri/i965/brw_context.c
+++ b/src/mesa/drivers/dri/i965/brw_context.c
@@ -421,10 +421,10 @@ brw_initialize_context_constants(struct brw_context *brw)
ctx->Const.MaxComputeWorkGroupSize[0] >= 128),
};
- unsigned num_stages = 0;
- for (int i = 0; i < MESA_SHADER_STAGES; i++) {
+ unsigned num_gfx_stages = 0;
+ for (int i = 0; i < MESA_SHADER_COMPUTE; i++) {
if (stage_exists[i])
- num_stages++;
+ num_gfx_stages++;
}
unsigned max_samplers =
@@ -505,14 +505,14 @@ brw_initialize_context_constants(struct brw_context *brw)
MIN2(ctx->Const.MaxTextureCoordUnits,
ctx->Const.Program[MESA_SHADER_FRAGMENT].MaxTextureImageUnits);
- ctx->Const.MaxUniformBufferBindings = num_stages * BRW_MAX_UBO;
- ctx->Const.MaxCombinedUniformBlocks = num_stages * BRW_MAX_UBO;
- ctx->Const.MaxCombinedAtomicBuffers = num_stages * BRW_MAX_ABO;
- ctx->Const.MaxCombinedShaderStorageBlocks = num_stages * BRW_MAX_SSBO;
- ctx->Const.MaxShaderStorageBufferBindings = num_stages * BRW_MAX_SSBO;
- ctx->Const.MaxCombinedTextureImageUnits = num_stages * max_samplers;
- ctx->Const.MaxCombinedImageUniforms = num_stages * BRW_MAX_IMAGES;
-
+ ctx->Const.MaxUniformBufferBindings = num_gfx_stages * BRW_MAX_UBO;
+ ctx->Const.MaxCombinedUniformBlocks = num_gfx_stages * BRW_MAX_UBO;
+ ctx->Const.MaxCombinedAtomicBuffers = num_gfx_stages * BRW_MAX_ABO;
+ ctx->Const.MaxCombinedShaderStorageBlocks = num_gfx_stages * BRW_MAX_SSBO;
+ ctx->Const.MaxCombinedTextureImageUnits = num_gfx_stages * max_samplers;
+ ctx->Const.MaxCombinedImageUniforms = num_gfx_stages * BRW_MAX_IMAGES;
+ ctx->Const.MaxShaderStorageBufferBindings =
+ (num_gfx_stages + stage_exists[MESA_SHADER_COMPUTE]) * BRW_MAX_SSBO;
/* Hardware only supports a limited number of transform feedback buffers.
* So we need to override the Mesa default (which is based only on software
More information about the mesa-commit
mailing list