<html>
    <head>
      <base href="https://bugs.freedesktop.org/">
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - X server crashes on DP hotplug: radeon_mode_hotplug() in xf86-video-ati does not enumerate second GPU"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=98626#c6">Comment # 6</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - X server crashes on DP hotplug: radeon_mode_hotplug() in xf86-video-ati does not enumerate second GPU"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=98626">bug 98626</a>
              from <span class="vcard"><a class="email" href="mailto:mstaudt@suse.de" title="Max Staudt <mstaudt@suse.de>"> <span class="fn">Max Staudt</span></a>
</span></b>
        <pre>(In reply to Michel Dänzer from <a href="show_bug.cgi?id=98626#c5">comment #5</a>)
<span class="quote">> The purpose of this code is to find all connectors of a single GPU.</span >

Right, I mixed that up.


<span class="quote">> There can only be one or two screens associated with any given GPU
> controlled by this driver.</span >

Okay, just so I understand this: I can have, say, 6 physical displays connected
to one Radeon card (GPU), but I can distribute them between no more than two X
screens (as in :0.0 and :0.1)?


I looked at the code again, and now realized that you're taking the ScrnInfoPtr
(primary_scrn) for the inner loop from pRADEONEnt, which in turn is a child of
scrn.
That's basically doing what I suggested (looping over scrn), so now I
understand how this fixes my bug.

Thank you for the quick fix!</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>