[Bug 111045] [CI][BAT] igt at kms_chamelium@(hdmi|dp)-hpd-fast|igt at kms_chamelium@dp-edid-change-during-suspend - fail - Failed assertion: igt_hotplug_detected(mon, HOTPLUG_TIMEOUT)

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=111045

Imre Deak <imre.deak at intel.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |daniel at ffwll.ch,
                   |                            |ville.syrjala at linux.intel.c
                   |                            |om

--- Comment #27 from Imre Deak <imre.deak at intel.com> ---
(In reply to emersion from comment #26)
> (In reply to Imre Deak from comment #24)
> > 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.

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.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/intel-gfx-bugs/attachments/20190710/fe3aca7c/attachment.html>