<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Nov 26, 2014 at 10:39 AM, Matt Turner <span dir="ltr"><<a href="mailto:mattst88@gmail.com" target="_blank">mattst88@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">The i965 backends pass something out of 'screen', which is allocated<br>
per-process, making using this as a ralloc context not thread-safe.<br>
<br>
All callers ra_alloc_interference_graph() already ralloc_free() its<br>
return value.<br>
---<br>
 src/util/register_allocate.c | 2 +-<br>
 1 file changed, 1 insertion(+), 1 deletion(-)<br>
<br>
diff --git a/src/util/register_allocate.c b/src/util/register_allocate.c<br>
index 6cf7ce7..1cfd66f 100644<br>
--- a/src/util/register_allocate.c<br>
+++ b/src/util/register_allocate.c<br>
@@ -374,7 +374,7 @@ ra_alloc_interference_graph(struct ra_regs *regs, unsigned int count)<br>
    struct ra_graph *g;<br>
    unsigned int i;<br>
<br>
-   g = rzalloc(regs, struct ra_graph);<br>
+   g = rzalloc(NULL, struct ra_graph);<br></blockquote><div><br></div><div>Why not make ra_alloc_interference_graph take a ralloc context?<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
    g->regs = regs;<br>
    g->nodes = rzalloc_array(g, struct ra_node, count);<br>
    g->count = count;<br>
<span class="HOEnZb"><font color="#888888">--<br>
2.0.4<br>
<br>
_______________________________________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/mesa-dev" target="_blank">http://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</font></span></blockquote></div><br></div></div>