<html>
<head>
<base href="https://bugs.freedesktop.org/">
</head>
<body><span class="vcard"><a class="email" href="mailto:martin.peres@free.fr" title="Martin Peres <martin.peres@free.fr>"> <span class="fn">Martin Peres</span></a>
</span> changed
<a class="bz_bug_link
bz_status_ASSIGNED "
title="ASSIGNED - [IGT][CI] Change kms_sysfs_edid_timing test into a performance test"
href="https://bugs.freedesktop.org/show_bug.cgi?id=100047">bug 100047</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;">Summary</td>
<td>[IGT][CI] kms_sysfs_edid_timing test Failed assertion: mean.mean < (THRESHOLD_TOTAL * 1e6)
</td>
<td>[IGT][CI] Change kms_sysfs_edid_timing test into a performance test
</td>
</tr>
<tr>
<td style="text-align:right;">QA Contact</td>
<td>intel-gfx-bugs@lists.freedesktop.org
</td>
<td>
</td>
</tr>
<tr>
<td style="text-align:right;">Component</td>
<td>DRM/Intel
</td>
<td>IGT
</td>
</tr></table>
<p>
<div>
<b><a class="bz_bug_link
bz_status_ASSIGNED "
title="ASSIGNED - [IGT][CI] Change kms_sysfs_edid_timing test into a performance test"
href="https://bugs.freedesktop.org/show_bug.cgi?id=100047#c56">Comment # 56</a>
on <a class="bz_bug_link
bz_status_ASSIGNED "
title="ASSIGNED - [IGT][CI] Change kms_sysfs_edid_timing test into a performance test"
href="https://bugs.freedesktop.org/show_bug.cgi?id=100047">bug 100047</a>
from <span class="vcard"><a class="email" href="mailto:martin.peres@free.fr" title="Martin Peres <martin.peres@free.fr>"> <span class="fn">Martin Peres</span></a>
</span></b>
<pre>This test is asking the kernel to reprobe all connectors by writing 'detect' to
/sys/class/drm/$name/status, which blocks until all connectors are reprobed.
This does not verify that the state is valid or anything, it just checks the
performance.
EDID reading cannot be tested properly without controling the end-point, which
is what we already do with the igt@kms_chamelium@(dp|hdmi)-edid-read test. EDID
reading failures are anyway tested by pretty much every KMS test, which would
generate dmesg messages in case of failures.
So I agree with Petri's comment, this test does not increase our test coverage,
and we need to remove this test by making it a performance test instead.
User impact of a potentially slow EDID read is a high latency between hotplug
and changes in the Desktop Environment.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are on the CC list for the bug.</li>
<li>You are the QA Contact for the bug.</li>
</ul>
</body>
</html>