[Mesa-dev] [PATCH 2/2] st/mesa: fix CTS regression caused by fcbb93e86024

Emil Velikov emil.l.velikov at gmail.com
Fri Aug 11 20:32:56 UTC 2017


On 1 August 2017 at 08:35, Timothy Arceri <tarceri at itsqueeze.com> wrote:
> When generation the storage offset for struct members we need
> to skip opaque types as they no longer have backing storage.
>
> Fixes: fcbb93e86024 ("mesa: stop assigning unused storage for non-bindless opaque types")
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=101983

Tim, did you went with another solution or the patches are waiting for review?
I was about to pick the revert for stable, but would need a reference
for the fix in master.

> ---
>  src/mesa/Makefile.sources                  |   2 +
>  src/mesa/state_tracker/st_glsl_to_tgsi.cpp |  16 ++++-
>  src/mesa/state_tracker/st_glsl_types.cpp   | 105 +++++++++++++++++++++++++++++
>  src/mesa/state_tracker/st_glsl_types.h     |  44 ++++++++++++
I think the st_glsl_types.* are revert conflicts and should not be needed.

If that's correct, this series is overall smaller and preferable for
stable, since it does not diverge from master.
It's your call though.

Thanks
Emil


More information about the mesa-dev mailing list