Pushed, thanks! Marek
On Thu, Dec 19, 2019 at 2:22 PM Neha Bhende <bhen...@vmware.com> wrote: > Since we are using st_common_variant while creating variant for vertext > program, we can release tokens created in st_create_vp_variant which > are already stored in respective states. > This fix memory leak found with piglit tests > > Fixes bc99b22a305b ('st/mesa: use a separate VS variant for the draw > module') > > Reviewed-by: Charmaine Lee <charmai...@vmware.com> > --- > src/mesa/state_tracker/st_program.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/src/mesa/state_tracker/st_program.c > b/src/mesa/state_tracker/st_program.c > index a9ff68c1f50..ef10399fa18 100644 > --- a/src/mesa/state_tracker/st_program.c > +++ b/src/mesa/state_tracker/st_program.c > @@ -694,6 +694,10 @@ st_create_vp_variant(struct st_context *st, > else > vpv->base.driver_shader = pipe->create_vs_state(pipe, &state); > > + if (state.tokens) { > + tgsi_free_tokens(state.tokens); > + } > + > return vpv; > } > > -- > 2.17.1 > > _______________________________________________ > mesa-stable mailing list > mesa-sta...@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/mesa-stable >
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev