<html>
<head>
<base href="https://bugs.freedesktop.org/" />
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEEDINFO "
title="NEEDINFO --- - [965gm regression] Black screen when under high load on kernel 3.7 (was OK on 3.6)"
href="https://bugs.freedesktop.org/show_bug.cgi?id=56916#c33">Comment # 33</a>
on <a class="bz_bug_link
bz_status_NEEDINFO "
title="NEEDINFO --- - [965gm regression] Black screen when under high load on kernel 3.7 (was OK on 3.6)"
href="https://bugs.freedesktop.org/show_bug.cgi?id=56916">bug 56916</a>
from <span class="vcard"><a class="email" href="mailto:cedric@belbone.be" title="Cedric Godin <cedric@belbone.be>"> <span class="fn">Cedric Godin</span></a>
</span></b>
<pre>i will try it asap.
I redid several bisects that pointed to
bf7ad8eeab995710c766df49c9c69a8592ca0216 is the first bad commit
commit bf7ad8eeab995710c766df49c9c69a8592ca0216
Author: Michel Lespinasse <<a href="mailto:walken@google.com">walken@google.com</a>>
Date: Mon Oct 8 16:30:37 2012 -0700
rbtree: move some implementation details from rbtree.h to rbtree.c
rbtree users must use the documented APIs to manipulate the tree
structure. Low-level helpers to manipulate node colors and parenthood are
not part of that API, so move them to lib/rbtree.c
it seems to not be the culprit but to expose more the bug.
The only problem I can see (not a de velopper) is that it changes
-static inline void rb_set_parent(struct rb_node *rb, struct rb_node *p)
-{
- rb->rb_parent_color = (rb->rb_parent_color & 3) | (unsigned long)p;
-}
to :
+#define rb_color(r) ((r)->__rb_parent_color & 1)
...
+static inline void rb_set_parent(struct rb_node *rb, struct rb_node *p)
+{
+ rb->__rb_parent_color = rb_color(rb) | (unsigned long)p;
+}
so changing the "& 3" to "& 1".
I tried to apply that change to a working kernel but had no crash and reverting
it from 3.7 didn't make a stable kernel either.</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>