Bug 108364

Summary: [CI][BAT] igt@kms_flip@basic-flip-vs-modeset - dmesg-warn - RPM wakelock ref not held during HW access
Product: DRI Reporter: Martin Peres <martin.peres>
Component: DRM/IntelAssignee: Intel GFX Bugs mailing list <intel-gfx-bugs>
Status: CLOSED FIXED QA Contact: Intel GFX Bugs mailing list <intel-gfx-bugs>
Severity: normal    
Priority: highest CC: intel-gfx-bugs
Version: XOrg git   
Hardware: Other   
OS: All   
Whiteboard: ReadyForDev
i915 platform: ICL i915 features:

Description Martin Peres 2018-10-15 08:51:44 UTC 
https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_4977/fi-icl-u2/igt@kms_flip@basic-flip-vs-modeset.html

<4> [262.375989] ------------[ cut here ]------------
<4> [262.375992] RPM wakelock ref not held during HW access
<4> [262.376074] WARNING: CPU: 2 PID: 2753 at drivers/gpu/drm/i915/intel_drv.h:2011 intel_uncore_forcewake_get+0x73/0xa0 [i915]
<4> [262.376078] Modules linked in: vgem snd_hda_codec_hdmi snd_hda_codec_realtek snd_hda_codec_generic i915 x86_pkg_temp_thermal coretemp crct10dif_pclmul crc32_pclmul ghash_clmulni_intel snd_hda_intel snd_hda_codec snd_hwdep snd_hda_core e1000e snd_pcm btusb btrtl btbcm btintel cdc_ether usbnet mii bluetooth ecdh_generic prime_numbers
<4> [262.376159] CPU: 2 PID: 2753 Comm: kms_flip Tainted: G     U  W         4.19.0-rc7-CI-CI_DRM_4977+ #1
<4> [262.376162] Hardware name: Intel Corporation Ice Lake Client Platform/IceLake U DDR4 SODIMM PD RVP TLC, BIOS ICLSFWR1.R00.2352.A01.1808281852 08/28/2018
<4> [262.376211] RIP: 0010:intel_uncore_forcewake_get+0x73/0xa0 [i915]
<4> [262.376216] Code: e7 5b 5d 41 5c 41 5d e9 2b c0 6a e1 f3 c3 80 3d e0 c3 18 00 00 75 c8 48 c7 c7 90 ce 38 a0 c6 05 d0 c3 18 00 01 e8 dd f9 de e0 <0f> 0b eb b1 80 3d c1 c3 18 00 00 75 9e 48 c7 c7 38 cd 38 a0 c6 05
<4> [262.376220] RSP: 0018:ffffc90000ae3d10 EFLAGS: 00010286
<4> [262.376226] RAX: 0000000000000000 RBX: ffff880496b80000 RCX: 0000000000000001
<4> [262.376230] RDX: 0000000080000001 RSI: ffffffff820c2e6e RDI: 00000000ffffffff
<4> [262.376233] RBP: 00000000000001ff R08: 00000000501ba55a R09: 0000000000000000
<4> [262.376237] R10: ffff8804ae818358 R11: 0000000000000000 R12: ffff880496b877e8
<4> [262.376240] R13: ffff880496b877a8 R14: ffff880496b80000 R15: ffffffffa0246e30
<4> [262.376244] FS:  00007fad09164980(0000) GS:ffff8804b0700000(0000) knlGS:0000000000000000
<4> [262.376248] CS:  0010 DS: 0000 ES: 0000 CR0: 0000000080050033
<4> [262.376251] CR2: 0000559219c08240 CR3: 00000004a6e80006 CR4: 0000000000760ee0
<4> [262.376254] DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000
<4> [262.376258] DR3: 0000000000000000 DR6: 00000000fffe0ff0 DR7: 0000000000000400
<4> [262.376262] PKRU: 55555554
<4> [262.376265] Call Trace:
<4> [262.376311]  i915_gem_reset_prepare_engine+0x11/0x40 [i915]
<4> [262.376352]  i915_gem_set_wedged+0xce/0x200 [i915]
<4> [262.376359]  ? __drm_printfn_info+0x20/0x20
<4> [262.376403]  i915_drop_caches_set+0x167/0x270 [i915]
<4> [262.376413]  simple_attr_write+0xb0/0xd0
<4> [262.376422]  full_proxy_write+0x51/0x80
<4> [262.376432]  __vfs_write+0x31/0x180
<4> [262.376438]  ? rcu_lockdep_current_cpu_online+0x8f/0xd0
<4> [262.376445]  ? rcu_read_lock_sched_held+0x6f/0x80
<4> [262.376450]  ? rcu_sync_lockdep_assert+0x29/0x50
<4> [262.376455]  ? __sb_start_write+0x152/0x1f0
<4> [262.376460]  ? __sb_start_write+0x168/0x1f0
<4> [262.376469]  vfs_write+0xbd/0x1b0
<4> [262.376476]  ksys_write+0x50/0xc0
<4> [262.376486]  do_syscall_64+0x55/0x190
<4> [262.376493]  entry_SYSCALL_64_after_hwframe+0x49/0xbe
<4> [262.376497] RIP: 0033:0x7fad0819c281
<4> [262.376502] Code: c3 0f 1f 84 00 00 00 00 00 48 8b 05 59 8d 20 00 c3 0f 1f 84 00 00 00 00 00 8b 05 8a d1 20 00 85 c0 75 16 b8 01 00 00 00 0f 05 <48> 3d 00 f0 ff ff 77 57 f3 c3 0f 1f 44 00 00 41 54 55 49 89 d4 53
<4> [262.376505] RSP: 002b:00007ffcf5787938 EFLAGS: 00000246 ORIG_RAX: 0000000000000001
<4> [262.376512] RAX: ffffffffffffffda RBX: 0000000000000001 RCX: 00007fad0819c281
<4> [262.376516] RDX: 0000000000000005 RSI: 00007ffcf57879c0 RDI: 0000000000000008
<4> [262.376519] RBP: 00007ffcf5787960 R08: 0000000000000000 R09: 0000000000000000
<4> [262.376523] R10: 0000000000000000 R11: 0000000000000246 R12: 00007fad08185718
<4> [262.376526] R13: 0000000000000003 R14: 00007fad0818a628 R15: 00007fad08186d80
<4> [262.376543] irq event stamp: 1218820
<4> [262.376550] hardirqs last  enabled at (1218819): [<ffffffff810f826a>] console_unlock+0x3fa/0x5f0
<4> [262.376554] hardirqs last disabled at (1218820): [<ffffffff81001930>] trace_hardirqs_off_thunk+0x1a/0x1c
<4> [262.376559] softirqs last  enabled at (1218756): [<ffffffff81c0031d>] __do_softirq+0x31d/0x483
<4> [262.376565] softirqs last disabled at (1218749): [<ffffffff8108c539>] irq_exit+0xa9/0xc0
<4> [262.376620] WARNING: CPU: 2 PID: 2753 at drivers/gpu/drm/i915/intel_drv.h:2011 intel_uncore_forcewake_get+0x73/0xa0 [i915]
<4> [262.376623] ---[ end trace c299f0aa28526b0e ]---
Comment 1 Chris Wilson 2018-10-15 19:35:09 UTC 
commit 9d3eb2c33f03432a25a6a3ab3177f839f25cbaf5 (HEAD -> drm-intel-next-queued, drm-intel/drm-intel-next-queued)
Author: Chris Wilson <chris@chris-wilson.co.uk>
Date:   Mon Oct 15 12:58:56 2018 +0100

    drm/i915: Hold rpm wakeref for debugfs/i915_drop_caches_set
    
    Since we peek into HW state and poke around, it behoves us to acquire a
    runtime pm wakeref beforehand.
    
    References: https://bugs.freedesktop.org/show_bug.cgi?id=108343
    Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=108364
    Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
    Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
    Link: https://patchwork.freedesktop.org/patch/msgid/20181015115856.18590-1-chris@chris-wilson.co.uk
Comment 2 Lakshmi 2018-10-23 13:12:41 UTC 
This issue occurred only once CI_DRM_4977 (1 week, 3 days / 114 runs ago).
Comment 3 Martin Peres 2018-10-30 16:47:44 UTC 
(In reply to Chris Wilson from comment #1)
> commit 9d3eb2c33f03432a25a6a3ab3177f839f25cbaf5 (HEAD ->
> drm-intel-next-queued, drm-intel/drm-intel-next-queued)
> Author: Chris Wilson <chris@chris-wilson.co.uk>
> Date:   Mon Oct 15 12:58:56 2018 +0100
> 
>     drm/i915: Hold rpm wakeref for debugfs/i915_drop_caches_set
>     
>     Since we peek into HW state and poke around, it behoves us to acquire a
>     runtime pm wakeref beforehand.
>     
>     References: https://bugs.freedesktop.org/show_bug.cgi?id=108343
>     Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=108364
>     Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
>     Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
>     Link:
> https://patchwork.freedesktop.org/patch/msgid/20181015115856.18590-1-
> chris@chris-wilson.co.uk

Thanks, closing.

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.