On Wed, 2018-11-21 at 17:20 -0600, Jason Ekstrand wrote: > It affects apply_pipeline_layout. Shaders compiled with the wrong value > will work but they may not be robust as requested by the app. > > Cc: mesa-sta...@lists.freedesktop.org
Hi, Jason. This patch does not apply cleanly in 18.2 branch. As the fix wasn't as trivial as usual, it would be great if you can check the resolution at https://gitlab.freedesktop.org/mesa/mesa/commit/35379ec233353d19d94994dfa43b205a6b5622b2 Thanks in advance! J.A. > --- > src/intel/vulkan/anv_pipeline.c | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/src/intel/vulkan/anv_pipeline.c b/src/intel/vulkan/anv_pipeline.c > index ad0f08253e7..f170366d030 100644 > --- a/src/intel/vulkan/anv_pipeline.c > +++ b/src/intel/vulkan/anv_pipeline.c > @@ -446,6 +446,9 @@ anv_pipeline_hash_graphics(struct anv_pipeline *pipeline, > if (layout) > _mesa_sha1_update(&ctx, layout->sha1, sizeof(layout->sha1)); > > + const bool rba = pipeline->device->robust_buffer_access; > + _mesa_sha1_update(&ctx, &rba, sizeof(rba)); > + > for (unsigned s = 0; s < MESA_SHADER_STAGES; s++) { > if (stages[s].entrypoint) > anv_pipeline_hash_shader(&ctx, &stages[s]); > @@ -466,6 +469,9 @@ anv_pipeline_hash_compute(struct anv_pipeline *pipeline, > if (layout) > _mesa_sha1_update(&ctx, layout->sha1, sizeof(layout->sha1)); > > + const bool rba = pipeline->device->robust_buffer_access; > + _mesa_sha1_update(&ctx, &rba, sizeof(rba)); > + > anv_pipeline_hash_shader(&ctx, stage); > > _mesa_sha1_final(&ctx, sha1_out); _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
