<html> <head> <base href="https://bugs.freedesktop.org/" /> </head> <body> <div> <a class="bz_bug_link bz_status_NEW " title="NEW --- - Regression: Crash in 3Dmark2001" href="https://bugs.freedesktop.org/show_bug.cgi?id=82828#c16">Comment # 16</a> on <a class="bz_bug_link bz_status_NEW " title="NEW --- - Regression: Crash in 3Dmark2001" href="https://bugs.freedesktop.org/show_bug.cgi?id=82828">bug 82828</a> from <a class="email" href="mailto:maraeo@gmail.com" title="Marek Olšák <maraeo@gmail.com>"> Marek Olšák</a> <pre>(In reply to <a href="show_bug.cgi?id=82828#c14">comment #14</a>) > (In reply to <a href="show_bug.cgi?id=82828#c13">comment #13</a>) > > (In reply to <a href="show_bug.cgi?id=82828#c12">comment #12</a>) > > > Created <a href="attachment.cgi?id=105630" name="attach_105630" title="another debugging patch">attachment 105630</a> <a href="attachment.cgi?id=105630&action=edit" title="another debugging patch">[details]</a> <a href='page.cgi?id=splinter.html&bug=82828&attachment=105630'>[review]</a> [review] [review] [review] > > > another debugging patch > > > > > > Ok, it looks like the problem is that node 0's q_total is bogus, which means > > > it never even gets considered for optimistic coloring. To help me figure out > > > why this is, can you apply this patch to master (not on top of the other > > > patch) and tell me the output of the piglit test now? > > > > On (In reply to <a href="show_bug.cgi?id=82828#c12">comment #12</a>) > > > Created <a href="attachment.cgi?id=105630" name="attach_105630" title="another debugging patch">attachment 105630</a> <a href="attachment.cgi?id=105630&action=edit" title="another debugging patch">[details]</a> <a href='page.cgi?id=splinter.html&bug=82828&attachment=105630'>[review]</a> [review] [review] [review] > > > another debugging patch > > > > > > Ok, it looks like the problem is that node 0's q_total is bogus, which means > > > it never even gets considered for optimistic coloring. To help me figure out > > > why this is, can you apply this patch to master (not on top of the other > > > patch) and tell me the output of the piglit test now? > > > > I'm not sure if this matters, but r300g pre-allocates the input registers > > before calling ra_allocate_no_spills(). > > I think there are no input registers in this case (there's a NumInputs = 0 > somewhere in the backtrace) so there aren't any pre-allocated nodes here. What Tom probably meant is that inputs are loaded to temps before the fragment shader starts, so inputs and temps pretty much share the temporary file. Not sure how relevant it is to this issue, but obviously you can't rename the temps which are supposed to contain inputs.</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are the assignee for the bug.</li> </ul> </body> </html>