<html>
<head>
<base href="https://bugs.freedesktop.org/">
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_ASSIGNED "
title="ASSIGNED - memory leak with intel i965 mesa when running android container in Ubuntu"
href="https://bugs.freedesktop.org/show_bug.cgi?id=104884#c12">Comment # 12</a>
on <a class="bz_bug_link
bz_status_ASSIGNED "
title="ASSIGNED - memory leak with intel i965 mesa when running android container in Ubuntu"
href="https://bugs.freedesktop.org/show_bug.cgi?id=104884">bug 104884</a>
from <span class="vcard"><a class="email" href="mailto:lemody@gmail.com" title="Tapani Pälli <lemody@gmail.com>"> <span class="fn">Tapani Pälli</span></a>
</span></b>
<pre>(In reply to Tapani Pälli from <a href="show_bug.cgi?id=104884#c11">comment #11</a>)
<span class="quote">> (In reply to Simon Hausmann from <a href="show_bug.cgi?id=104884#c10">comment #10</a>)
> > Created <span class=""><a href="attachment.cgi?id=137353" name="attach_137353" title="Simple patch that changes glsl_type to manage its own memory">attachment 137353</a> <a href="attachment.cgi?id=137353&action=edit" title="Simple patch that changes glsl_type to manage its own memory">[details]</a></span> <a href='page.cgi?id=splinter.html&bug=104884&attachment=137353'>[review]</a> [review] [review]
> > Simple patch that changes glsl_type to manage its own memory
> >
> > I've made a patch that attempts to simplify the glsl_type memory management
> > and as a bonus also gets rid of the global mutex. Is this patch worth
> > submitting to the list?
>
> Cool, yeah it looks like it would work, I guess strdup needs to be
> util_strdup. I thought about this earlier but I wonder if the reason for
> having the global context is that there are a *lot* of types so now we will
> have loads of ralloc contexts .. not sure. Please send to mesa-dev for
> discussion?</span >
The reason for global context is that we want these glsl_types to live during
the lifetime of context until compiler gets destroyed and only destroy them at
end. This way all shaders will enjoy using the types and not needing to create
the same types all the time. So whatever the fix is, that functionality likely
needs to stay.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the QA Contact for the bug.</li>
</ul>
</body>
</html>