<html> <head> <base href="https://bugs.freedesktop.org/"> </head> <body> <div> <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#c4">Comment # 4</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 <a class="email" href="mailto:mstaudt@suse.de" title="Max Staudt <mstaudt@suse.de>"> Max Staudt</a> <pre>Yes, these patches fix the crash. However, as far as I can see, they do not allow for arbitrary number of Radeon cards in arbitrary primary/secondary configurations. So it's not a final fix... As far as I can see, instead of going through all screens, we could walk only over the outputs attached to 'scrn', or maybe restore the behavior before c801f9f10a5d and not walk over screens at all. Hoever that would necessitate a different way of enumerating outputs. Is it just an issue of giving every output a unique name? Maybe, instead of having globally incrementing counters per connector type, we could name the connectors "radeon-0-DisplayPort-1" or similar? Then we wouldn't have to count them globally. Also, having the driver name in the connector name prevents clashes with other drivers that have a similar naming scheme.</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are the assignee for the bug.</li> </ul> </body> </html>