[Intel-gfx] ✗ Fi.CI.BAT: failure for acpi/pwm/i915: Convert pwm-crc and i915 driver's PWM code to use the atomic PWM API

Tue Aug 25 11:10:03 UTC 2020

Hi,

On 8/25/20 12:44 PM, Patchwork wrote:
> *Patch Details*
> *Series:*	acpi/pwm/i915: Convert pwm-crc and i915 driver's PWM code to use the atomic PWM API
> *URL:*	https://patchwork.freedesktop.org/series/80976/
> *State:*	failure
> *Details:*	https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_18396/index.html
> 
> 
>   CI Bug Log - changes from CI_DRM_8924 -> Patchwork_18396
> 
> 
>     Summary
> 
> *FAILURE*
> 
> Serious unknown changes coming with Patchwork_18396 absolutely need to be
> verified manually.
> 
> If you think the reported changes have nothing to do with the changes
> introduced in Patchwork_18396, please notify your bug team to allow them
> to document this new failure mode, which will reduce false positives in CI.
> 
> External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_18396/index.html
> 
> 
>     Possible new issues
> 
> Here are the unknown changes that may have been introduced in Patchwork_18396:
> 
> 
>       IGT changes
> 
> 
>         Possible regressions
> 
>   * igt at gem_exec_parallel@engines at contexts:
>       o fi-kbl-x1275: PASS <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8924/fi-kbl-x1275/igt@gem_exec_parallel@engines@contexts.html> -> INCOMPLETE <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_18396/fi-kbl-x1275/igt@gem_exec_parallel@engines@contexts.html>

v6 of this patch-set did pass the BAT tests yesterday and nothing has really
changed other then a single cast to avoid a 64 bit divide which only does
anything on 32 bit archs. And that divide is a param to a drm_dbg statement.

So this seems to be a false-positive. I guess that this test sometimes
failing should be added to the known issues list ?

Anyways since v6 did pass and nothing has really changed I'm going to
ignore this test result.

Regards,

Hans

>     Known issues
> 
> Here are the changes found in Patchwork_18396 that come from known issues:
> 
> 
>       IGT changes
> 
> 
>         Issues hit
> 
>   *
> 
>     igt at i915_module_load@reload:
> 
>       o fi-bxt-dsi: PASS <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8924/fi-bxt-dsi/igt@i915_module_load@reload.html> -> TIMEOUT <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_18396/fi-bxt-dsi/igt@i915_module_load@reload.html> (i915#1418 <https://gitlab.freedesktop.org/drm/intel/issues/1418> / i915#1635 <https://gitlab.freedesktop.org/drm/intel/issues/1635>)
>   *
> 
>     igt at i915_pm_rpm@module-reload:
> 
>       o fi-bxt-dsi: PASS <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8924/fi-bxt-dsi/igt@i915_pm_rpm@module-reload.html> -> SKIP <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_18396/fi-bxt-dsi/igt@i915_pm_rpm@module-reload.html> (fdo#109271 <https://bugs.freedesktop.org/show_bug.cgi?id=109271> / i915#1635 <https://gitlab.freedesktop.org/drm/intel/issues/1635>)
>   *
> 
>     igt at kms_flip@basic-flip-vs-wf_vblank at c-edp1:
> 
>       o fi-icl-u2: PASS <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8924/fi-icl-u2/igt@kms_flip@basic-flip-vs-wf_vblank@c-edp1.html> -> DMESG-WARN <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_18396/fi-icl-u2/igt@kms_flip@basic-flip-vs-wf_vblank@c-edp1.html> (i915#1982 <https://gitlab.freedesktop.org/drm/intel/issues/1982>) +1 similar issue
> 
> 
>         Possible fixes
> 
>   *
> 
>     igt at i915_pm_rpm@basic-pci-d3-state:
> 
>       o fi-bsw-kefka: DMESG-WARN <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8924/fi-bsw-kefka/igt@i915_pm_rpm@basic-pci-d3-state.html> (i915#1982 <https://gitlab.freedesktop.org/drm/intel/issues/1982>) -> PASS <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_18396/fi-bsw-kefka/igt@i915_pm_rpm@basic-pci-d3-state.html>
>   *
> 
>     igt at kms_cursor_legacy@basic-busy-flip-before-cursor-atomic:
> 
>       o fi-byt-j1900: DMESG-WARN <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8924/fi-byt-j1900/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-atomic.html> (i915#1982 <https://gitlab.freedesktop.org/drm/intel/issues/1982>) -> PASS <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_18396/fi-byt-j1900/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-atomic.html> +1 similar issue
>   *
> 
>     igt at kms_cursor_legacy@basic-busy-flip-before-cursor-legacy:
> 
>       o fi-icl-u2: DMESG-WARN <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8924/fi-icl-u2/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy.html> (i915#1982 <https://gitlab.freedesktop.org/drm/intel/issues/1982>) -> PASS <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_18396/fi-icl-u2/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy.html>
>   *
> 
>     igt at kms_flip@basic-flip-vs-wf_vblank at c-hdmi-a2:
> 
>       o fi-skl-guc: DMESG-WARN <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8924/fi-skl-guc/igt@kms_flip@basic-flip-vs-wf_vblank@c-hdmi-a2.html> (i915#2203 <https://gitlab.freedesktop.org/drm/intel/issues/2203>) -> PASS <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_18396/fi-skl-guc/igt@kms_flip@basic-flip-vs-wf_vblank@c-hdmi-a2.html>
>   *
> 
>     igt at kms_pipe_crc_basic@suspend-read-crc-pipe-a:
> 
>       o fi-apl-guc: INCOMPLETE <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8924/fi-apl-guc/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a.html> (i915#1635 <https://gitlab.freedesktop.org/drm/intel/issues/1635> / i915#337 <https://gitlab.freedesktop.org/drm/intel/issues/337>) -> PASS <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_18396/fi-apl-guc/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a.html>
> 
> 
>         Warnings
> 
>   *
> 
>     igt at i915_pm_rpm@module-reload:
> 
>       o fi-kbl-x1275: DMESG-FAIL <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8924/fi-kbl-x1275/igt@i915_pm_rpm@module-reload.html> (i915#62 <https://gitlab.freedesktop.org/drm/intel/issues/62> / i915#95 <https://gitlab.freedesktop.org/drm/intel/issues/95>) -> DMESG-FAIL <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_18396/fi-kbl-x1275/igt@i915_pm_rpm@module-reload.html> (i915#62 <https://gitlab.freedesktop.org/drm/intel/issues/62>)
>   *
> 
>     igt at kms_cursor_legacy@basic-flip-after-cursor-legacy:
> 
>       o fi-kbl-x1275: DMESG-WARN <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8924/fi-kbl-x1275/igt@kms_cursor_legacy@basic-flip-after-cursor-legacy.html> (i915#62 <https://gitlab.freedesktop.org/drm/intel/issues/62> / i915#92 <https://gitlab.freedesktop.org/drm/intel/issues/92>) -> DMESG-WARN <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_18396/fi-kbl-x1275/igt@kms_cursor_legacy@basic-flip-after-cursor-legacy.html> (i915#62 <https://gitlab.freedesktop.org/drm/intel/issues/62> / i915#92 <https://gitlab.freedesktop.org/drm/intel/issues/92> / i915#95
>         <https://gitlab.freedesktop.org/drm/intel/issues/95>) +2 similar issues
>   *
> 
>     igt at kms_force_connector_basic@force-edid:
> 
>       o fi-kbl-x1275: DMESG-WARN <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8924/fi-kbl-x1275/igt@kms_force_connector_basic@force-edid.html> (i915#62 <https://gitlab.freedesktop.org/drm/intel/issues/62> / i915#92 <https://gitlab.freedesktop.org/drm/intel/issues/92> / i915#95 <https://gitlab.freedesktop.org/drm/intel/issues/95>) -> DMESG-WARN <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_18396/fi-kbl-x1275/igt@kms_force_connector_basic@force-edid.html> (i915#62 <https://gitlab.freedesktop.org/drm/intel/issues/62> / i915#92 <https://gitlab.freedesktop.org/drm/intel/issues/92>) +5
>         similar issues
> 
> 
>     Participating hosts (37 -> 34)
> 
> Missing (3): fi-byt-clapper fi-byt-squawks fi-bsw-cyan
> 
> 
>     Build changes
> 
>   * Linux: CI_DRM_8924 -> Patchwork_18396
> 
> CI-20190529: 20190529
> CI_DRM_8924: a8c0611e412aab46eab5475b0117d074892b96e2 @ git://anongit.freedesktop.org/gfx-ci/linux
> IGT_5770: f1d0c240ea2e631dfb9f493f37f8fb61cb2b1cf2 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
> Patchwork_18396: 871bbce513b9511e674714dec71406955d950ecd @ git://anongit.freedesktop.org/gfx-ci/linux
> 
> == Linux commits ==
> 
> 871bbce513b9 drm/i915: panel: Use atomic PWM API for devs with an external PWM controller
> 5cba5633c584 drm/i915: panel: Honor the VBT PWM min setting for devs with an external PWM controller
> 6c5200023ed2 drm/i915: panel: Honor the VBT PWM frequency for devs with an external PWM controller
> ace83a7f4e48 drm/i915: panel: Add get_vbt_pwm_freq() helper
> 9a833ba135f1 pwm: crc: Implement get_state() method
> f0758f16468b pwm: crc: Implement apply() method to support the new atomic PWM API
> 04e01429ab69 pwm: crc: Enable/disable PWM output on enable/disable
> 2e80db8eb3ff pwm: crc: Fix period changes not having any effect
> 2061f911c93f pwm: crc: Fix off-by-one error in the clock-divider calculations
> 2dda19e1952a pwm: crc: Fix period / duty_cycle times being off by a factor of 256
> 9fb3d675a6b6 pwm: lpss: Use pwm_lpss_apply() when restoring state on resume
> 0d3912600257 pwm: lpss: Add pwm_lpss_prepare_enable() helper
> a4e4727f40e6 pwm: lpss: Add range limit check for the base_unit register value
> 5dd4db0dc219 pwm: lpss: Fix off by one error in base_unit math in pwm_lpss_prepare()
> 1d26b9fe569e ACPI / LPSS: Save Cherry Trail PWM ctx registers only once (at activation)
> ef85c003b00a ACPI / LPSS: Resume Cherry Trail PWM controller in no-irq phase
> 