<html>
<head>
<base href="https://bugs.freedesktop.org/">
</head>
<body><span class="vcard"><a class="email" href="mailto:imre.deak@intel.com" title="Imre Deak <imre.deak@intel.com>"> <span class="fn">Imre Deak</span></a>
</span> changed
<a class="bz_bug_link
bz_status_NEW "
title="NEW - [CI][BAT] igt@kms_chamelium@(hdmi|dp)-hpd-fast|igt@kms_chamelium@dp-edid-change-during-suspend - fail - Failed assertion: igt_hotplug_detected(mon, HOTPLUG_TIMEOUT)"
href="https://bugs.freedesktop.org/show_bug.cgi?id=111045">bug 111045</a>
<br>
<table border="1" cellspacing="0" cellpadding="8">
<tr>
<th>What</th>
<th>Removed</th>
<th>Added</th>
</tr>
<tr>
<td style="text-align:right;">CC</td>
<td>
</td>
<td>daniel@ffwll.ch, ville.syrjala@linux.intel.com
</td>
</tr></table>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - [CI][BAT] igt@kms_chamelium@(hdmi|dp)-hpd-fast|igt@kms_chamelium@dp-edid-change-during-suspend - fail - Failed assertion: igt_hotplug_detected(mon, HOTPLUG_TIMEOUT)"
href="https://bugs.freedesktop.org/show_bug.cgi?id=111045#c27">Comment # 27</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - [CI][BAT] igt@kms_chamelium@(hdmi|dp)-hpd-fast|igt@kms_chamelium@dp-edid-change-during-suspend - fail - Failed assertion: igt_hotplug_detected(mon, HOTPLUG_TIMEOUT)"
href="https://bugs.freedesktop.org/show_bug.cgi?id=111045">bug 111045</a>
from <span class="vcard"><a class="email" href="mailto:imre.deak@intel.com" title="Imre Deak <imre.deak@intel.com>"> <span class="fn">Imre Deak</span></a>
</span></b>
<pre>(In reply to emersion from <a href="show_bug.cgi?id=111045#c26">comment #26</a>)
<span class="quote">> (In reply to Imre Deak from <a href="show_bug.cgi?id=111045#c24">comment #24</a>)
> > Ok, thanks a lot for looking into that. Btw, are you planning to add new
> > subtests that work according to the above (instead of modifying the existing
> > ones) and black list somehow the existing ones (at least for now) for DP-alt
> > connectors? That would be ideal, since we should still keep the existing
> > ones as-is to retain the coverage for all that scenarios.
>
> Hmm, that's a little bit annoying. What about adding a single test that
> doesn't disable an output when unplugged, and converting all existing tests
> to work on DP-alt? IMHO this would make more sense because userspace not
> disabling an output when unplugged is an edge-case.</span >
Changing existing tests has been considered as bad, since that can introduce
new issues into the CI bug-tracking system as regressions, whereas they would
be an issue always present, just not tested before.
So I don't feel easy about changing the existing tests, but I suppose you could
do it if:
- The change is simple like disabling the output only once before the subtest
for the whole duration and not adding more complexity like dynamic
enabling/disabling based on the connector state.
- You've discussed about the policy of changing existing tests with the CI
folks (your team?) and Ville, Daniel, they're the best to decide about this.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the QA Contact for the bug.</li>
<li>You are on the CC list for the bug.</li>
</ul>
</body>
</html>