From: Jason Ekstrand Date: Fri, 15 Dec 2017 19:27:39 +0000 (-0800) Subject: anv/pipeline: Don't assert on more than 32 samplers X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=4b69ba381766cd911eb1284f1b0332a139ec8a75;p=mesa.git anv/pipeline: Don't assert on more than 32 samplers This prevents an assert when running one unreleased Vulkan game. Tested-by: Józef Kucia Reviewed-by: Topi Pohjolainen Cc: "18.0" --- diff --git a/src/intel/vulkan/genX_pipeline.c b/src/intel/vulkan/genX_pipeline.c index cfc3bea4265..82fdf206a95 100644 --- a/src/intel/vulkan/genX_pipeline.c +++ b/src/intel/vulkan/genX_pipeline.c @@ -1081,7 +1081,13 @@ emit_3dstate_streamout(struct anv_pipeline *pipeline, static uint32_t get_sampler_count(const struct anv_shader_bin *bin) { - return DIV_ROUND_UP(bin->bind_map.sampler_count, 4); + uint32_t count_by_4 = DIV_ROUND_UP(bin->bind_map.sampler_count, 4); + + /* We can potentially have way more than 32 samplers and that's ok. + * However, the 3DSTATE_XS packets only have 3 bits to specify how + * many to pre-fetch and all values above 4 are marked reserved. + */ + return MIN2(count_by_4, 4); } static uint32_t