<html> <head> <base href="https://bugs.freedesktop.org/"> </head> <body> <div> <a class="bz_bug_link bz_status_ASSIGNED " title="ASSIGNED - [CI][DRMTIP] igt@kms_fbcon_fbt@psr(-suspend) - fail - Failed assertion: !feature->wait_until_enabled(drm.debugfs_fd)" href="https://bugs.freedesktop.org/show_bug.cgi?id=107882#c5">Comment # 5</a> on <a class="bz_bug_link bz_status_ASSIGNED " title="ASSIGNED - [CI][DRMTIP] igt@kms_fbcon_fbt@psr(-suspend) - fail - Failed assertion: !feature->wait_until_enabled(drm.debugfs_fd)" href="https://bugs.freedesktop.org/show_bug.cgi?id=107882">bug 107882</a> from <a class="email" href="mailto:dhinakaran.pandiyan@intel.com" title="Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>"> Dhinakaran Pandiyan</a> <pre>We might(In reply to Jose Roberto de Souza from <a href="show_bug.cgi?id=107882#c4">comment #4</a>) > code from: kms_fbcon_fbt.c static void subtest(struct feature *feature, bool > suspend) > ... > kmstest_unset_all_crtcs(drm.fd, drm.res); > wait_user("Modes unset."); > igt_assert(!feature->wait_until_enabled(drm.debugfs_fd)); > ... > wait_user("Back to fbcon."); > igt_assert(!feature->wait_until_enabled(drm.debugfs_fd)); > > So this asserts are expecting that PSR is not enabled when fbcon is active, > my understanding is should check if PSR is not active as the cursor blinking > will disable PSR1, for PSR2 it could go to PSR2 active and update the cursor > with SU. > > What do you think DK? Yes. We'll have to fix this for PSR1 too, since psr_flush() does not disable PSR anymore and the test checks for that.</pre> </div> <hr> You are receiving this mail because: <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>