<html>
    <head>
      <base href="https://bugs.freedesktop.org/" />
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW --- - [bisected] kernel 3.7.0-rc1 breaks 6950 (boot/grub2 and suspend/resume) (CAYMAN)"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=56139#c37">Comment # 37</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW --- - [bisected] kernel 3.7.0-rc1 breaks 6950 (boot/grub2 and suspend/resume) (CAYMAN)"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=56139">bug 56139</a>
              from <span class="vcard"><a class="email" href="mailto:alexandre.f.demers@gmail.com" title="Alexandre Demers <alexandre.f.demers@gmail.com>"> <span class="fn">Alexandre Demers</span></a>
</span></b>
        <pre>(In reply to <a href="show_bug.cgi?id=56139#c36">comment #36</a>)
<span class="quote">> (In reply to <a href="show_bug.cgi?id=56139#c33">comment #33</a>)
> > > Here is a partial kernel.log tracking what is going on at boot with some
> > > debug printk in evergreen_mc_stop() and evergreen_mc_resume(). Strangely,
> > > only one crtc is saved as enabled in the _stop function, but four are
> > > restored in the _resume function. Pretty sure it's not the problem we are
> > > looking for, but it is abnormal since only one crtc should be restored on
> > > resume. This is a bug by itself showing a problem in the
> > > save(stop)/restore(resume) mechanism.
> > 
> > I should have said "... but six are restored..." instead of "... but four
> > are restored..." I also made a small error in a debug string, but it doesn't
> > change the read value. If you have any question on my patch, let me know.

> That issue is fixed in <span class=""><a href="attachment.cgi?id=69573" name="attach_69573" title="possible fix">attachment 69573</a> <a href="attachment.cgi?id=69573&action=edit" title="possible fix">[details]</a></span> <a href='page.cgi?id=splinter.html&bug=56139&attachment=69573'>[review]</a> [review] which is already
> upstream.</span >

I know it should have been fixed and that's why I added a printk(KERN_INFO
"Restoring crtc[%d] since it was enabled.\n", i) just after it to check it. For
some reason it seems to not be working properly. Have a look at my patch
(<span class=""><a href="attachment.cgi?id=70233" name="attach_70233" title="patch applied on 3.7.0-rc6 for debug output">attachment 70233</a> <a href="attachment.cgi?id=70233&action=edit" title="patch applied on 3.7.0-rc6 for debug output">[details]</a></span>) which was written on a 3.7.0-rc6 kernel. I'll add some other
printk when I'll get home (like a printk just before this check to output its
value).</pre>
        </div>
      </p>
      <hr>
      <span>You are receiving this mail because:</span>
      
      <ul>
          <li>You are the assignee for the bug.</li>
      </ul>
    </body>
</html>