[Mesa-stable] [PATCH 2/2] mesa: don't leak performance monitors on context destroy

Kenneth Graunke kenneth at whitecape.org
Wed Dec 4 06:25:37 PST 2013


On 11/29/2013 02:52 AM, Ilia Mirkin wrote:
> Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>
> Cc: "10.0" <mesa-stable at 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 <kenneth at 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


More information about the mesa-stable mailing list