Bug 89792 - [BSW/SKL Debugfs] cat /sys/kernel/debug/dri/0/i915_sr_status showing self-refresh: disabled
Summary: [BSW/SKL Debugfs] cat /sys/kernel/debug/dri/0/i915_sr_status showing self-ref...
Status: CLOSED INVALID
Alias: None
Product: DRI
Classification: Unclassified
Component: DRM/Intel (show other bugs)
Version: unspecified
Hardware: Other Linux (All)
: low normal
Assignee: Intel GFX Bugs mailing list
QA Contact: Intel GFX Bugs mailing list
URL:
Whiteboard: ReadyForDev
Keywords:
Depends on:
Blocks:
 
Reported: 2015-03-27 08:49 UTC by xubin
Modified: 2018-05-04 07:53 UTC (History)
2 users (show)

See Also:
i915 platform: BSW/CHT, SKL
i915 features: power/Other


Attachments
Dmesg log (250.90 KB, text/plain)
2017-03-30 20:17 UTC, Elio
no flags Details
dmesg.log (290.31 KB, text/plain)
2017-08-14 14:13 UTC, Ricardo Madrigal
no flags Details

Description xubin 2015-03-27 08:49:45 UTC
==System Environment==
--------------------------
Regression: No. This is the first time to do the case.
Non-working platforms: BYT/BSW/SKL 
Working platforms: BDW

==kernel==
--------------------------
-nightly: 60c5b7f582e8ced10dcd557db4ea66665e6a4dff (fails)
    drm-intel-nightly: 2015y-03m-26d-21h-48m-18s UTC integration manifest
-queued: d21bd67bbbc635424b336b7c881dbcdf6098f9f5 (fails)
    drm/i915: Don't use encoder->new_crtc in intel_lvds_compute_config()
-fixes: 5f407751b0ca9bd876fe8f15ff28153661c6ba0a (fails)
    drm/i915: Fixup legacy plane->crtc link for initial fb config

==Bug detailed description==
-----------------------------
Consult Damien, as far as he knows: we don't have anything to do on kernel driver on SKL for (system) memory self-refresh and the WM programming is enough.

The bug just debugfs file does not update to support new platforms.

If this debugfs file can be enable, will greatly help QA verify the MSR/WM funcitonality.


==Reproduce steps==
---------------------------- 
1. Boot system into kernel and keep idleness
2. cat /sys/kernel/debug/dri/0/i915_sr_status
3. Debugfs file show: self-refresh: disabled
Comment 1 Jesse Barnes 2015-04-02 18:21:06 UTC
Should be fixed on BYT and BSW by

http://lists.freedesktop.org/archives/intel-gfx/2015-April/063840.html
Comment 2 xubin 2015-04-03 11:08:33 UTC
Tested it on BYT and BSW, cat /sys/kernel/debug/dri/0/i915_sr_status 

showing self-refresh: enabled.
Comment 3 liulei 2015-04-29 12:26:18 UTC
This issue still exists on SKL.
Kernel version:
uname :4.0.0_drm-intel-testing_43d9c2_20150424
git tag:drm-intel-testing-2015-04-23 drm-intel-nightly: 2015y-04m-23d-19h-58m-37s UTC integration manifest
Comment 4 xubin 2015-05-14 11:19:40 UTC
Tested on SKLY03 with the testing kernel drm-intel-testing-2015-05-08.
this problem still exists.
Comment 5 xubin 2015-05-15 11:21:01 UTC
(In reply to Jesse Barnes from comment #1)
> Should be fixed on BYT and BSW by
> http://lists.freedesktop.org/archives/intel-gfx/2015-April/063840.html

Tested on BYT with the testing kernel drm-intel-testing-2015-05-08,the issue still exist.

It was fixed on BYT and BSW by 
http://lists.freedesktop.org/archives/intel-gfx/2015-April/063840.html

Doesn't it  merge into drm-intel-nightly branch?
Comment 6 ye.tian 2015-05-26 08:16:04 UTC
This problem still exists on SKL with drm-intel-testing-2015-05-22+ kernel.
Comment 7 cprigent 2015-08-03 22:18:17 UTC
Bug scrub: assigned to Rami.
Rami, please check if still reproduced in SKL
Comment 8 Rami 2015-08-21 15:31:33 UTC
Bug still exist on skl.

Hardware
------------------
Platform: SKY LAKE Y A0
CPU : Intel(R) Core(TM) m3-6Y30 CPU @ 0.8GHz 4MB (family: 6, model: 78  stepping: 3)
MCP : SKL-Y  D1  2+2
QDF : QVY3 
CPU : SKL D1
Chipset PCH: Sunrise Point LP C1       
CRB : SKY LAKE Y LPDDR3 RVP3 CRB FAB2
Reworks : All Mandatories + FBS02 & FBS03, O-06

Software
------------------
BIOS : SKLSE2R1.R00.X093.B02.1507222151
ME FW : 11.0.0.1163
Ksc (EC FW): 1.15
Linux distribution: Ubuntu 14.04 LTS 64 bits
Kernel: drm-intel-nightly 30263ef782e0548c114fb6f2771b58b27c56ba0d 4.2.0-rc7 from git://anongit.freedesktop.org/drm-intel 
Mesa: mesa-10.6.3 ddc976368fef367e464472ebcc2ac4fd89eb9fd8 from http://cgit.freedesktop.org/mesa/mesa/ 
Xf86_video_intel: 2.99.917 baec802b21387d04aebb10ac29e719a1800c5aa0 from http://cgit.freedesktop.org/xorg/driver/xf86-video-intel/ 
Libdrm: libdrm-2.4.62 ba4b5ac010ab85406ec52e3906e13d58cd9aa782 from http://cgit.freedesktop.org/mesa/drm/ 
Cairo: 1.14.2 from 93422b3cb5e0ef8104b8194c8873124ce2f5ea2d http://cgit.freedesktop.org/cairo 
libva: libva-1.6.0 a8008998bc0d4a76ae6927607c048e52ba50fd0e from http://cgit.freedesktop.org/libva/ 
intel-driver: 1.6.0 32268c46d538667d437dc9266aa4c183e51c1286 from http://cgit.freedesktop.org/vaapi/intel-driver 
Xserver: xorg-server-1.17.2 2123f7682d522619f101b05fb75efa75dabbe371 from http://cgit.freedesktop.org/xorg/xserver
Comment 9 Elio 2016-01-04 22:59:11 UTC
Confirming: This issue is not present in BYT
Comment 10 Ricardo 2017-02-20 18:35:50 UTC
Elio can you check this bug in SKL and BSW looks like it was solved in BYT (by your last comment)
Comment 11 Elio 2017-03-30 20:16:55 UTC
The problem seems to exist with SKL and BSW:
cat /sys/kernel/debug/dri/0/i915_sr_status
self-refresh: disabled

Configuration:

Component         : drm
	url       : http://cgit.freedesktop.org/mesa/drm
	tag       : libdrm-2.4.75
	commit    : d4b8344363b4e0f0e831e5722b6df5cc0bb08df8
	author    : Chad Versace <chadversary@chromium.org>
	age       : 6 weeks ago
	comment   : Bump version for 2.4.75 release



Component         : mesa
	url       : http://cgit.freedesktop.org/mesa/mesa
	tag       : mesa-17.0.1
	commit    : 8fee1d348cc3d91a88319c0d72689acabaa2bf47
	author    : Emil Velikov <emil.velikov@collabora.com>
	age       : 9 days ago
	comment   : docs: add release notes for 17.0.1


Component         : xf86-video-intel
	url       : http://cgit.freedesktop.org/xorg/driver/xf86-video-intel
	tag       : 2.99.917-767-g7e9e92c
	commit    : 7e9e92c86b0fc4c848d164fe571798add5e1e36e
	author    : Chris Wilson <chris@chris-wilson.co.uk>
	age       : 4 hours ago
	comment   : sna: No hw scaling for Baytrail sprites



Component         : libva
	url       : http://cgit.freedesktop.org/libva
	tag       : 1.8.0.pre1
	commit    : 156c3d8ce747f00e6b886af247f69552650a36d9
	author    : Xiang Haihao <haihao.xiang@intel.com>
	age       : 8 days ago
	comment   : Update NEWS for 1.8.0.pre1

Signed-off-by: Xiang, Haihao <haihao.xiang@intel.com> 

Component         : intel-driver
	url       : http://cgit.freedesktop.org/vaapi/intel-driver
	tag       : 1.8.0.pre1
	commit    : d24d79f0999e3a161c52ac98eb96d8511cfb0716
	author    : Xiang Haihao <haihao.xiang@intel.com>
	age       : 7 days ago
	comment   : Update NEWS for 1.8.0.pre1


Component         : cairo
	url       : http://cgit.freedesktop.org/cairo
	tag       : 1.15.4
	commit    : 9fe6683cb105354e86ea649ba7a13052c7edc757
	author    : Bryce Harrington <bryce@osg.samsung.com>
	age       : 3 months ago
	comment   : 1.15.4 release 

Component         : xserver
	url       : http://cgit.freedesktop.org/xorg/xserver
	tag       : xorg-server-1.19.2
	commit    : f23e65f96365706c69fa781b2c6cbf3203619c9f
	author    : Adam Jackson <ajax@redhat.com>
	age       : 11 days ago
	comment   : xserver 1.19.2, no, really



Component         : macros
	url       : https://cgit.freedesktop.org/xorg/util/macros
	tag       : util-macros-1.19.1-2-g39f07f7
	commit    : 39f07f7db58ebbf3dcb64a2bf9098ed5cf3d1223
	author    : Emil Velikov <emil.veliko@collabora.com>
	age       : 4 weeks ago
	comment   : Rework INSTALL_CMD to touch/echo >&2 only as needed




Component         : intel-gpu-tools
	url       : https://cgit.freedesktop.org/xorg/app/intel-gpu-tools
	tag       : intel-gpu-tools-1.18
	commit    : bb97a0542adf2ae2adab946b2e3fee396d54063a
	author    : Petri Latvala <petri.latvala@intel.com>
	age       : 6 hours ago
	comment   : Update NEWS, bump version to 1.18.


Component         : rendercheck
	url       : https://cgit.freedesktop.org/xorg/app/rendercheck
	tag       : rendercheck-1.5
	commit    : 604e3bbe
	author    : Dave Airlie <airlied@redhat.com>
	age       : 1 year, 9 months ago
	comment   : rendercheck 1.5



Component         : piglit
	url       : https://cgit.freedesktop.org/piglit
	tag       : piglit-v1
	commit    : 7a6474d3a2fd58ada4bd003e309b4bf018d28ea0
	author    : Constantine Charlamov <Hi-Angel@yandex.ru>
	age       : 10 days ago
	comment   : README: correct flags for 32-on-64 build
Comment 12 Elio 2017-03-30 20:17:22 UTC
Created attachment 130592 [details]
Dmesg log
Comment 13 Elio 2017-07-17 21:01:33 UTC
Still disabled using following configuration:
Ubuntu 17.04
Kernel version: 4.12.0
Same status
Comment 14 Ricardo Madrigal 2017-08-11 20:00:43 UTC
Still disabled using following configuration:
Ubuntu 17.04
Kernel version: 4.13.0-rc4-drm-tip-ww32-commit-fbb8288+
Same status
Comment 15 Ricardo Madrigal 2017-08-14 14:13:35 UTC
Created attachment 133496 [details]
dmesg.log
Comment 16 Elizabeth 2017-12-07 21:12:57 UTC
BSW and SKL with 
$ : uname -a
Linux SKL-1-NUC6i7KYB 4.15.0-rc2-drm-intel-qa-ww49-commit-bdf9b36+ #1 SMP Thu Dec 7 10:29:10 CST 2017 x86_64 x86_64 x86_64 GNU/Linux

$ : uname -a
Linux BSW-1-BRASWELL 4.15.0-rc2-drm-intel-qa-ww49-commit-bdf9b36+ #1 SMP Thu Dec 7 10:29:10 CST 2017 x86_64 x86_64 x86_64 GNU/Linux

Same status:
$ : sudo cat /sys/kernel/debug/dri/0/i915_sr_status
self-refresh: disabled

$ : sudo cat /sys/kernel/debug/dri/0/i915_sr_status
self-refresh: disabled
Comment 17 Hector Velazquez 2018-03-14 22:49:40 UTC
continue disabled on SKL and BSW

SKL:
. . .
$ : uname -a
Linux gfx-desktop 4.16.0-rc4-drm-tip-ww11-commit-1bf8f00+ #1 SMP Mon Mar 12 05:57:41 CST 2018 x86_64 x86_64 x86_64 GNU/Linux
$ : sudo cat /sys/kernel/debug/dri/0/i915_sr_status
self-refresh: disabled
. . .

BSW:
. . .
$ : uname -a
Linux BSW-2-BRASWELL 4.16.0-rc5-drm-intel-qa-ww11-commit-307515c+ #1 SMP Wed Mar 14 04:15:45 CST 2018 x86_64 x86_64 x86_64 GNU/Linux
$ :  sudo cat /sys/kernel/debug/dri/0/i915_sr_status
self-refresh: disabled
. . .
Comment 18 Jani Saarinen 2018-03-29 07:11:29 UTC
First of all. Sorry about spam.
This is mass update for our bugs. 

Sorry if you feel this annoying but with this trying to understand if bug still valid or not.
If bug investigation still in progress, please ignore this and I apologize!

If you think this is not anymore valid, please comment to the bug that can be closed.
If you haven't tested with our latest pre-upstream tree(drm-tip), can you do that also to see if issue is valid there still and if you cannot see issue there, please comment to the bug.
Comment 19 Elizabeth 2018-04-06 16:49:12 UTC
Not sure of the validity of this tests, but if I do the "cat" I always get "disable".

[gfx@BSW-1-BRASWELL] [~]$ : uname -r
4.16.0-rc7-drm-intel-qa-ww14-commit-f818c2b+
[gfx@BSW-1-BRASWELL] [~]$ : sudo cat /sys/kernel/debug/dri/0/i915_sr_status
self-refresh: disabled

[gfx@SKL-3-NUC6i7KYB] [~]$ : sudo cat /sys/kernel/debug/dri/0/i915_sr_status
self-refresh: disabled
[gfx@SKL-3-NUC6i7KYB] [~]$ : uname -r
4.16.0-rc7-drm-intel-qa-ww14-commit-f818c2b+
Comment 20 Jani Saarinen 2018-04-25 06:23:49 UTC
Please correct me if I am wrong here. This haven't really moved, should it? 
Lets resolve this now?
Comment 21 Imre Deak 2018-04-25 11:30:11 UTC
(In reply to Elizabeth from comment #19)
> Not sure of the validity of this tests, but if I do the "cat" I always get
> "disable".
> 
> [gfx@BSW-1-BRASWELL] [~]$ : uname -r
> 4.16.0-rc7-drm-intel-qa-ww14-commit-f818c2b+
> [gfx@BSW-1-BRASWELL] [~]$ : sudo cat /sys/kernel/debug/dri/0/i915_sr_status
> self-refresh: disabled

This should show enabled if you have any output enabled on pipe A or B. Can you make sure this is the case before reading i915_sr_status?

> 
> [gfx@SKL-3-NUC6i7KYB] [~]$ : sudo cat /sys/kernel/debug/dri/0/i915_sr_status
> self-refresh: disabled
> [gfx@SKL-3-NUC6i7KYB] [~]$ : uname -r
> 4.16.0-rc7-drm-intel-qa-ww14-commit-f818c2b+

This is expected, on GEN9+ there is no global memory self-refresh state, so i915_sr_status will always return disabled on these platforms.
Comment 22 Jani Saarinen 2018-05-04 07:53:32 UTC
Lets close this now.


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.