<html>
<head>
<base href="https://bugs.freedesktop.org/">
</head>
<body><span class="vcard"><a class="email" href="mailto:idr@freedesktop.org" title="Ian Romanick <idr@freedesktop.org>"> <span class="fn">Ian Romanick</span></a>
</span> changed
<a class="bz_bug_link
bz_status_NEEDINFO "
title="NEEDINFO - High memory usage in Black Mesa"
href="https://bugs.freedesktop.org/show_bug.cgi?id=108946">bug 108946</a>
<br>
<table border="1" cellspacing="0" cellpadding="8">
<tr>
<th>What</th>
<th>Removed</th>
<th>Added</th>
</tr>
<tr>
<td style="text-align:right;">Status</td>
<td>NEW
</td>
<td>NEEDINFO
</td>
</tr>
<tr>
<td style="text-align:right;">CC</td>
<td>
</td>
<td>idr@freedesktop.org
</td>
</tr></table>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEEDINFO "
title="NEEDINFO - High memory usage in Black Mesa"
href="https://bugs.freedesktop.org/show_bug.cgi?id=108946#c6">Comment # 6</a>
on <a class="bz_bug_link
bz_status_NEEDINFO "
title="NEEDINFO - High memory usage in Black Mesa"
href="https://bugs.freedesktop.org/show_bug.cgi?id=108946">bug 108946</a>
from <span class="vcard"><a class="email" href="mailto:idr@freedesktop.org" title="Ian Romanick <idr@freedesktop.org>"> <span class="fn">Ian Romanick</span></a>
</span></b>
<pre>(I'm quite sure I submitted a similar comment earlier today, but it seems to
have just vanished.)
I ran the API trace in Valgrind massif memory profiler. As far as I can tell,
the application never calls glDeleteShader. We don't release any of the memory
because the app hasn't told us that we can. In that trace there is is about
2.6GB (on a 64-bit build of apitrace) of intermediate IR that could be released
by calling glDeleteShader on all the shaders that have been linked.
It's possible that this is a quirk of the API trace. Someone would have to run
the app with Valgrind massif to say for sure.
(In reply to MGG from <a href="show_bug.cgi?id=108946#c0">comment #0</a>)
<span class="quote">> Finally, while running an apitrace I detected a lot this errors: "major
> shader compiler issue 21156: 0:4(12): warning: extension
> `GL_EXT_gpu_shader4' unsupported in vertex shader". Not sure if it could be
> related with any leak though.</span >
This is unrelated. It's a bit misleading that apitrace lists this as a "major
shader compiler issue." It is perfectly valid to enable extensions that the
driver does not support. GLSL is intentionally designed so that shaders can
have multiple code paths that are used based on what extensions are available
at compile time.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
<li>You are the QA Contact for the bug.</li>
</ul>
</body>
</html>