Bug 106602

A CI Bug Log filter associated to this bug has been updated:

{- fi-hsw-peppy: igt@pm_lpsp@edp-panel-fitter - fail - Failed assertion: c->count_modes && c->modes[0].vdisplay > 768 -}
{+ fi-hsw-peppy: igt@pm_lpsp@edp-panel-fitter - fail/dmesg-fail - Failed assertion: c->count_modes && c->modes[0].vdisplay > 768 +}

New failures caught by the filter:

* https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_212/fi-hsw-peppy/igt@pm_lpsp@edp-panel-fitter.html

A CI Bug Log filter associated to this bug has been updated:

{- fi-hsw-peppy: igt@pm_lpsp@edp-panel-fitter - fail/dmesg-fail - Failed assertion: c->count_modes && c->modes[0].vdisplay > 768 -}
{+ fi-hsw-peppy: igt@pm_lpsp@edp-panel-fitter - fail/dmesg-fail - Failed assertion: c->count_modes && c->modes[0].vdisplay > 768 +}

 No new failures caught with the new filter

The CI Bug Log issue associated to this bug has been updated.

### New filters associated

* HSW:  IGT: igt@runner@aborted - fail - Previous test: pm_lpsp (edp-panel-fitter)
  - https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_212/fi-hsw-peppy/igt@runner@aborted.html

A CI Bug Log filter associated to this bug has been updated:

{- HSW:  IGT: igt@runner@aborted - fail - Previous test: pm_lpsp (edp-panel-fitter) -}
{+ HSW:  IGT: igt@runner@aborted - fail -Previous test: pm_lpsp/i915_pm_lpsp +}

New failures caught by the filter:

* https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_232/fi-hsw-peppy/igt@runner@aborted.html

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/

-- GitLab Migration Automatic Message --

This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/drm/intel/issues/120.