Bug 88913 - [ILK bisected]igt/gem_workarounds/suspend-resume cause dmesg warn.
Summary: [ILK bisected]igt/gem_workarounds/suspend-resume cause dmesg warn.
Status: CLOSED DUPLICATE of bug 88911
Alias: None
Product: DRI
Classification: Unclassified
Component: DRM/Intel (show other bugs)
Version: DRI git
Hardware: Other All
: medium normal
Assignee: Mika Kuoppala
QA Contact: Intel GFX Bugs mailing list
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-02-02 07:53 UTC by Ding Heng
Modified: 2017-09-04 10:29 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:


Attachments
dmesg for case igt/gem_workarounds/suspend-resume (64.28 KB, text/plain)
2015-02-02 07:53 UTC, Ding Heng
no flags Details

Description Ding Heng 2015-02-02 07:53:40 UTC
Created attachment 113039 [details]
dmesg for case igt/gem_workarounds/suspend-resume

==System Environment==
--------------------------
Regression: yes
Non-working platforms:  ILK==kernel==
--------------------------
drm-intel-nightly/8b4216f91c7bf8d3459cadf9480116220bd6545e(2015-02-02)


==Bug detailed description==
-----------------------------
igt/gem_workarounds/suspend-resume case success, but there is dmesg warn after run this case:
dmesg -r|egrep "<[1-4]>"|grep drm
<4>[    1.427143] WARNING: CPU: 0 PID: 1219 at drivers/gpu/drm/i915/intel_uncore.c:169 intel_uncore_forcewake_reset+0x18b/0x27e [i915]()
<4>[    1.427150] Modules linked in: i915(+) button video drm_kms_helper drm cfbfillrect cfbimgblt cfbcopyarea
<4>[    1.427152] CPU: 0 PID: 1219 Comm: udevd Not tainted 3.19.0-rc6_drm-intel-nightly_8b4216_20150202+ #126
<4>[    1.427315]  [<ffffffffa00135ac>] ? drm_dev_register+0x74/0xe6 [drm]
<4>[    1.427320]  [<ffffffffa0015952>] ? drm_get_pci_dev+0xf7/0x1b2 [drm]
<4>[  279.286823] WARNING: CPU: 1 PID: 4052 at drivers/gpu/drm/i915/intel_uncore.c:169 intel_uncore_forcewake_reset+0x18b/0x27e [i915]()
<4>[  279.286839] Modules linked in: ip6table_filter ip6_tables ebtable_nat ebtables nf_conntrack_ipv4 nf_defrag_ipv4 xt_conntrack nf_conntrack ipt_REJECT nf_reject_ipv4 iptable_mangle xt_tcpudp iptable_filter ip_tables x_tables bridge stp llc ipv6 dm_mod iTCO_wdt iTCO_vendor_support snd_hda_codec_hdmi ppdev snd_hda_codec_realtek snd_hda_codec_generic dcdbas serio_raw pcspkr i2c_i801 snd_hda_intel lpc_ich snd_hda_controller mfd_core snd_hda_codec snd_hwdep snd_pcm snd_timer snd soundcore wmi parport_pc parport tpm_tis tpm acpi_cpufreq i915 button video drm_kms_helper drm cfbfillrect cfbimgblt cfbcopyarea
<4>[  279.286841] CPU: 1 PID: 4052 Comm: kworker/u16:13 Tainted: G        W      3.19.0-rc6_drm-intel-nightly_8b4216_20150202+ #126
<4>[  279.286886]  [<ffffffffa00858dd>] ? i915_drm_suspend+0x17d/0x1b3 [i915]
<4>[  279.695643] WARNING: CPU: 3 PID: 967 at drivers/gpu/drm/i915/intel_uncore.c:169 intel_uncore_forcewake_reset+0x18b/0x27e [i915]()
<4>[  279.695664] Modules linked in: ip6table_filter ip6_tables ebtable_nat ebtables nf_conntrack_ipv4 nf_defrag_ipv4 xt_conntrack nf_conntrack ipt_REJECT nf_reject_ipv4 iptable_mangle xt_tcpudp iptable_filter ip_tables x_tables bridge stp llc ipv6 dm_mod iTCO_wdt iTCO_vendor_support snd_hda_codec_hdmi ppdev snd_hda_codec_realtek snd_hda_codec_generic dcdbas serio_raw pcspkr i2c_i801 snd_hda_intel lpc_ich snd_hda_controller mfd_core snd_hda_codec snd_hwdep snd_pcm snd_timer snd soundcore wmi parport_pc parport tpm_tis tpm acpi_cpufreq i915 button video drm_kms_helper drm cfbfillrect cfbimgblt cfbcopyarea
<4>[  279.695667] CPU: 3 PID: 967 Comm: kworker/u16:3 Tainted: G        W      3.19.0-rc6_drm-intel-nightly_8b4216_20150202+ #126
<4>[  279.695708]  [<ffffffffa00863a3>] ? i915_drm_resume_early+0x94/0x94 [i915]
<4>[  279.695721]  [<ffffffffa008636c>] ? i915_drm_resume_early+0x5d/0x94 [i915]

==Reproduce steps==
---------------------------- 

./gem_workarounds --run-subtest suspend-resume
Comment 1 Ding Heng 2015-02-02 07:57:02 UTC
f9b3927afbb97649f5c89f0815820167b7c5cba8 is the first bad commit

commit f9b3927afbb97649f5c89f0815820167b7c5cba8
Author:     Mika Kuoppala <mika.kuoppala@linux.intel.com>
AuthorDate: Wed Jan 28 14:43:24 2015 +0200
Commit:     Daniel Vetter <daniel.vetter@ffwll.ch>
CommitDate: Fri Jan 30 17:15:31 2015 +0100

    drm/i915: Do uncore early sanitize after domain init
    
    intel_uncore_early_sanitize() will reset the forcewake registers. When
    forcewake domains were introduced, the domain init was done after the
    sanitization of the forcewake registers. And as the resetting of
    registers use the domain accessors, we tried to reset the forcewake
    registers with unitialized forcewake domains and failed.
    
    Fix this by sanitizing after all the domains have been initialized. Do
    per domain clearing of forcewake register on domain init so that
    IVB can do early access to ECOBUS do determine the final configuration.
    
    This regression was introduced in
    
    commit 05a2fb157e44a53c79133805d30eaada43911941
    Author: Mika Kuoppala <mika.kuoppala@linux.intel.com>
    Date:   Mon Jan 19 16:20:43 2015 +0200
    
        drm/i915: Consolidate forcewake code
    
    v2: Carve out ellc detect, fw_domain_reset for ivb/ecobus (Chris)
    
    Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=88805
    Cc: Chris Wilson <chris@chris-wilson.co.uk>
    Reported-by: Olof Johansson <olof@lixom.net>
    Tested-by: Darren Hart <dvhart@linux.intel.com> (v1)
    Signed-off-by: Mika Kuoppala <mika.kuoppala@intel.com>
    Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
    Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Comment 2 Jani Nikula 2015-02-03 12:49:01 UTC
Likely dupe of bug 88911.
Comment 3 Mika Kuoppala 2015-02-03 18:04:01 UTC

*** This bug has been marked as a duplicate of bug 88911 ***
Comment 4 Jari Tahvanainen 2017-09-04 10:29:30 UTC
Closing old verified+duplicate as duplicate of closed+fixed.


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.