<html>
<head>
<base href="https://bugs.freedesktop.org/" />
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_RESOLVED bz_closed"
title="RESOLVED FIXED - Missing RandR events on hotplug"
href="https://bugs.freedesktop.org/show_bug.cgi?id=91929#c9">Comment # 9</a>
on <a class="bz_bug_link
bz_status_RESOLVED bz_closed"
title="RESOLVED FIXED - Missing RandR events on hotplug"
href="https://bugs.freedesktop.org/show_bug.cgi?id=91929">bug 91929</a>
from <span class="vcard"><a class="email" href="mailto:chris@chris-wilson.co.uk" title="Chris Wilson <chris@chris-wilson.co.uk>"> <span class="fn">Chris Wilson</span></a>
</span></b>
<pre>Hmm, nasty. So,
diff --git a/src/sna/sna_display.c b/src/sna/sna_display.c
index 1b1a125..0a59d1d 100644
--- a/src/sna/sna_display.c
+++ b/src/sna/sna_display.c
@@ -4900,6 +4900,7 @@ void sna_mode_discover(struct sna *sna)
DBG(("%s: output %s (id=%d), changed state,
reprobing\n",
__FUNCTION__, output->name,
sna_output->id));
sna_output->last_detect = 0;
+ changed |= 4;
}
continue;
}
@@ -4918,7 +4919,8 @@ void sna_mode_discover(struct sna *sna)
changed |= 2;
}
- if (changed) {
+ /* Have the list of available outputs been updated? */
+ if (changed & 3) {
DBG(("%s: outputs changed, broadcasting\n", __FUNCTION__));
sna_mode_set_primary(sna);
@@ -4933,7 +4935,9 @@ void sna_mode_discover(struct sna *sna)
xf86RandR12TellChanged(screen);
}
- RRGetInfo(screen, TRUE);
+ /* If anything has changed, refresh the RandR information */
+ if (changed)
+ RRGetInfo(screen, TRUE);
RRTellChanged(screen);
}</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>