[Bug 106602] [CI] igt at pm_lpsp@edp-panel-fitter - fail - Failed assertion: c->count_modes && c->modes[0].vdisplay > 768

[bugzilla-daemon at freedesktop.org]

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

Matt Roper <matthew.d.roper at intel.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
           Assignee|intel-gfx-bugs at lists.freede |matthew.d.roper at intel.com
                   |sktop.org                   |

--- Comment #5 from Matt Roper <matthew.d.roper at intel.com> ---
This failure looks like an issue with the test program.  The problematic code
is this section:

   /* This is one of the modes Xorg creates for panels, so
    * it should work just fine. Notice that Gens that
    * support LPSP are too new for panels with native
    * 1024x768 resolution, so this should force the panel
    * fitter. */
   igt_assert(c->count_modes &&
              c->modes[0].hdisplay > 1024);
   igt_assert(c->count_modes &&
              c->modes[0].vdisplay > 768);
   mode = &std_1024_mode;

There are a few problems here:
 * The platform being tested here has a 13x7 panel which fails the second
assert above, even though the panel fitter would still have been used.  We
should check hdisplay and vdisplay together to avoid this.
 * If we do come across a true 1024x768 panel on a platform with LPSP we should
be skipping this test, not marking it as a failure.  I.e., igt_require() rather
than igt_assert().

I think this patch should fix the failure:
https://patchwork.freedesktop.org/series/66015/

-- 
You are receiving this mail because:
You are the assignee for the bug.
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/20190829/bc097b9e/attachment.html>