Bug 97027 - [APL] [drm:intel_pipe_update_end [i915]] *ERROR* Potential atomic update failure on pipe
Summary: [APL] [drm:intel_pipe_update_end [i915]] *ERROR* Potential atomic update fail...
Status: CLOSED FIXED
Alias: None
Product: DRI
Classification: Unclassified
Component: DRM/Intel (show other bugs)
Version: DRI git
Hardware: Other Linux (All)
: medium blocker
Assignee: Intel GFX Bugs mailing list
QA Contact: Intel GFX Bugs mailing list
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-07-21 16:20 UTC by yann
Modified: 2017-02-03 03:50 UTC (History)
2 users (show)

See Also:
i915 platform: BXT
i915 features: display/Other


Attachments
Mutex starvation fix with proper patch names (30.00 KB, application/x-tar)
2016-07-21 16:23 UTC, yann
no flags Details

Description yann 2016-07-21 16:20:03 UTC
This to follow-up on 2nd issues linked to bug 96701
Comment 1 yann 2016-07-21 16:23:20 UTC
Created attachment 125235 [details]
Mutex starvation fix with proper patch names

 proposed by Imre to be applied
Comment 2 yann 2016-07-21 16:24:27 UTC
Humberto, can you try ans see if this fix the issue when running
./kms_cursor_legacy --run-subtest basic-flip-vs-cursor --debug
./kms_cursor_legacy --run-subtest basic-cursor-vs-flip --debug
Comment 3 yann 2016-07-21 17:04:42 UTC
Humberto, please discard this comment, I replied on wrong bug please retry on bug 96701
Comment 4 yann 2016-07-21 17:04:47 UTC
From Christophe results running the basic-cursor-vs-flip subtest

--------
Kernel nightly 2fe098f 4.7.0-rc7 from https://cgit.freedesktop.org/drm-intel/
  commit 2fe098ff21273583acc9c232ec120675de9cd811
  Author: Chris Wilson <chris@chris-wilson.co.uk>
  Date:   Fri Jul 15 20:42:25 2016 +0100
  drm-intel-nightly: 2016y-07m-15d-19h-42m-01s UTC integration manifest

root@BXTP6:~/intel-gpu-tools/tests# cat /proc/cmdline
vmlinuz root=UUID=a9394a0f-0785-42fb-b164-d080a278295d initrd=/initrd initcall_debug drm.debug=0xe log_buf_len=10M

root@BXTP6:~/intel-gpu-tools/tests# ./kms_cursor_legacy --r basic-cursor-vs-flip
IGT-Version: 1.15-ge7ed4ef (x86_64) (Linux: 4.7.0-rc7-nightly+ x86_64)
Test requirement not met in function __real_main472, file kms_cursor_legacy.c:493:
Test requirement: !(n >= data.resources->count_crtcs)
Subtest basic-cursor-vs-flip: SUCCESS (1,902s)

root@BXTP6:~/intel-gpu-tools/tests# dmesg |grep atomic
[    2.258842] [drm:intel_modeset_checks] New cdclk calculated to be atomic 288000, actual 288000
[    2.346907] [drm:intel_modeset_checks] New cdclk calculated to be atomic 288000, actual 288000
[   43.255319] [drm:intel_modeset_checks] New cdclk calculated to be atomic 288000, actual 288000
[   43.621526] [drm:intel_pipe_update_start [i915]] *ERROR* Potential atomic update failure on pipe A
[   43.631383] [drm:intel_pipe_update_start [i915]] *ERROR* Potential atomic update failure on pipe A
[   43.641246] [drm:intel_pipe_update_start [i915]] *ERROR* Potential atomic update failure on pipe A
[   43.651095] [drm:intel_pipe_update_start [i915]] *ERROR* Potential atomic update failure on pipe A
[   43.660955] [drm:intel_pipe_update_start [i915]] *ERROR* Potential atomic update failure on pipe A
[   43.670808] [drm:intel_pipe_update_start [i915]] *ERROR* Potential atomic update failure on pipe A
[   43.680669] [drm:intel_pipe_update_start [i915]] *ERROR* Potential atomic update failure on pipe A
[   43.690521] [drm:intel_pipe_update_start [i915]] *ERROR* Potential atomic update failure on pipe A
[   43.700384] [drm:intel_pipe_update_start [i915]] *ERROR* Potential atomic update failure on pipe A
[   43.710235] [drm:intel_pipe_update_start [i915]] *ERROR* Potential atomic update failure on pipe A
[   43.714559] [drm:intel_modeset_checks] New cdclk calculated to be atomic 288000, actual 288000
[   43.719642] [drm:intel_modeset_checks] New cdclk calculated to be atomic 288000, actual 288000
Comment 5 yann 2016-07-21 17:05:43 UTC
Comment on attachment 125235 [details]
Mutex starvation fix with proper patch names

attachment to wrong bug
Comment 6 cprigent 2016-09-22 12:25:38 UTC
I don't see anymore the error "Potential atomic update failure on pipe".

root@BXTP6:/opt/X11R7/src/intel-gpu-tools/tests# ./kms_cursor_legacy --r basic-flip-before-cursor-legacy
IGT-Version: 1.16-g200237a (x86_64) (Linux: 4.8.0-rc7-nightly+ x86_64)
Subtest basic-flip-before-cursor-legacy: SUCCESS (1.523s)
root@BXTP6:/opt/X11R7/src/intel-gpu-tools/tests# ./kms_cursor_legacy --r basic-flip-after-cursor-legacy
IGT-Version: 1.16-g200237a (x86_64) (Linux: 4.8.0-rc7-nightly+ x86_64)
Subtest basic-flip-after-cursor-legacy: SUCCESS (1.469s)
root@BXTP6:/opt/X11R7/src/intel-gpu-tools/tests# ./kms_cursor_legacy --r basic-flip-before-cursor-varying-size
IGT-Version: 1.16-g200237a (x86_64) (Linux: 4.8.0-rc7-nightly+ x86_64)
Subtest basic-flip-before-cursor-varying-size: SUCCESS (1.485s)
root@BXTP6:/opt/X11R7/src/intel-gpu-tools/tests# ./kms_cursor_legacy --r basic-flip-after-cursor-varying-size
IGT-Version: 1.16-g200237a (x86_64) (Linux: 4.8.0-rc7-nightly+ x86_64)
Subtest basic-flip-after-cursor-varying-size: SUCCESS (1.500s)
root@BXTP6:/opt/X11R7/src/intel-gpu-tools/tests# ./kms_cursor_legacy --r basic-flip-before-cursor-atomic
IGT-Version: 1.16-g200237a (x86_64) (Linux: 4.8.0-rc7-nightly+ x86_64)
Test requirement not met in function basic_flip_cursor, file kms_cursor_legacy.c:469:
Test requirement: display->is_atomic
Subtest basic-flip-before-cursor-atomic: SKIP (0.000s)
root@BXTP6:/opt/X11R7/src/intel-gpu-tools/tests# ./kms_cursor_legacy --r basic-flip-after-cursor-atomic
IGT-Version: 1.16-g200237a (x86_64) (Linux: 4.8.0-rc7-nightly+ x86_64)
Test requirement not met in function basic_flip_cursor, file kms_cursor_legacy.c:469:
Test requirement: display->is_atomic
Subtest basic-flip-after-cursor-atomic: SKIP (0.000s)
root@BXTP6:/opt/X11R7/src/intel-gpu-tools/tests#
root@BXTP6:/opt/X11R7/src/intel-gpu-tools/tests#
root@BXTP6:/opt/X11R7/src/intel-gpu-tools/tests# dmesg |grep atomic
[  618.257262] [drm:intel_modeset_checks] New cdclk calculated to be atomic 288000, actual 288000
[  618.280505] [drm:intel_modeset_checks] New cdclk calculated to be atomic 144000, actual 144000
[  619.429625] [drm:intel_modeset_checks] New cdclk calculated to be atomic 144000, actual 144000
[  619.705505] [drm:intel_modeset_checks] New cdclk calculated to be atomic 288000, actual 288000
[  627.696042] [drm:intel_modeset_checks] New cdclk calculated to be atomic 288000, actual 288000
[  627.719754] [drm:intel_modeset_checks] New cdclk calculated to be atomic 144000, actual 144000
[  628.829613] [drm:intel_modeset_checks] New cdclk calculated to be atomic 144000, actual 144000
[  629.108178] [drm:intel_modeset_checks] New cdclk calculated to be atomic 288000, actual 288000
[  639.464369] [drm:intel_modeset_checks] New cdclk calculated to be atomic 288000, actual 288000
[  639.486520] [drm:intel_modeset_checks] New cdclk calculated to be atomic 144000, actual 144000
[  640.629920] [drm:intel_modeset_checks] New cdclk calculated to be atomic 144000, actual 144000
[  640.906207] [drm:intel_modeset_checks] New cdclk calculated to be atomic 288000, actual 288000
[  649.965661] [drm:intel_modeset_checks] New cdclk calculated to be atomic 288000, actual 288000
[  649.990105] [drm:intel_modeset_checks] New cdclk calculated to be atomic 144000, actual 144000
[  651.129557] [drm:intel_modeset_checks] New cdclk calculated to be atomic 144000, actual 144000
[  651.405965] [drm:intel_modeset_checks] New cdclk calculated to be atomic 288000, actual 288000
[  660.331193] kms_cursor_legacy: starting subtest basic-flip-before-cursor-atomic
[  671.761603] kms_cursor_legacy: starting subtest basic-flip-after-cursor-atomic

Platform: APL system
CPU Name : Intel(R) Genuine Processor @ 1.1 GHz (family: 6, model: 12, stepping: 9) 4 cores
QDF : Q6HE
SoC : B1
CRB : Apollo Lake DDR3L RVP1A FAB2
Reworks: R19, R20

Software 
Bios: 144_B10 APLK_B0_IFWI_X64_R_2016_06_27_0956_SPI_RVP1.bin from \\gar\ec\proj\ba\CCG\APL BIOS\External\BIOS_Release\Daily\v144_10_2016_WW27.1\IFWI\IFWI_RVP1_Release\IFWI
KSC: 1.15
Linux distribution: Ubuntu 16.04 64 bits
DMC 1.07
GuC 8.7
Kernel: 4.8.0-rc7 0e34cb5 from http://cgit.freedesktop.org/drm-intel/
commit 0e34cb5b35f0f837219495c402073141481b1b90
  Author: Jani Nikula <jani.nikula@intel.com>
  Date:   Mon Sep 19 18:39:27 2016 +0300
  drm-intel-nightly: 2016y-09m-19d-15h-38m-53s UTC integration manifest
libdrm-2.4.70-14 0659558 from git://anongit.freedesktop.org/mesa/drm
mesa: mesa-12.0.0 8b06176 from git://anongit.freedesktop.org/mesa/mesa
cairo 1.15.2 db8a7f1 from git://anongit.freedesktop.org/cairo
xorg-server-1.18.99-2 3ef51c5 from git://git.freedesktop.org/git/xorg/xserver
xf86-video-intel 2.99.917-703 15c5ff1 from git://git.freedesktop.org/git/xorg/driver/xf86-video-intel
libva-1.7.0-50 7aa2dd9 from git://git.freedesktop.org/git/vaapi/libva 
vaapi-intel-driver: 1.7.0-136 36fbd81 from git://git.freedesktop.org/git/vaapi/intel-driver
IGT: intel-gpu-tools-1.16-22 200237a from http://anongit.freedesktop.org/git/xorg/app/intel-gpu-tools.git

External screens: ASUS PB238Q (HDMI), LG 25UM55D (DP)
Comment 7 cprigent 2016-09-22 12:30:07 UTC
So closed


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.