On 11/29/2013 02:52 AM, Ilia Mirkin wrote:
> Signed-off-by: Ilia Mirkin <imir...@alum.mit.edu>
> Cc: "10.0" <mesa-sta...@lists.freedesktop.org>
> ---
> 
> Found with valgrind. Don't have the hardware to test a real implementation,
> but with nv50 it seemed to work in that valgrind was no longer marking the
> hash table as leaked.

Reviewed-by: Kenneth Graunke <kenn...@whitecape.org>

and pushed - thanks!

I agree that this is a candidate for 10.0, since it fixes a memory leak
in core Mesa code...even though no driver exposes
AMD_performance_monitor on 10.0 (due to last minute reverts).

Alternatively, one could simply revert the rest of
AMD_performance_monitor from 10.0.  Up to the stable maintainers.

--Ken
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to