Comment 42 for bug 1173787

Revision history for this message
In , Micael Dias (kam1kaz3) wrote :

(In reply to comment #32)
> Created an attachment (id=48680) [details]
> crash fix
>
> Here's a patch fixing (hopefully) the crash reported in comment #22
>
> It add some logic to r600g query handling, to avoid unnecessary flushes and
> infinite loop (though the crash is less easy to reproduce with git master
> because commit 61c976c3ccc815aaf0128eea835aee9667cd8bbe reduces flush count)

Looks good to me. There's just a typo in the comment you added r600_context_queries_resume(), should be "doesn't need" instead of "does need".

I think you should send it to the list :)