[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CA+icZUU-czmKNm96FDEYFrApWOM6KsdbUY2hcy2w8=7qVg16gA@mail.gmail.com>
Date: Thu, 29 Dec 2016 08:28:04 +0100
From: Sedat Dilek <sedat.dilek@...il.com>
To: "Rafael J. Wysocki" <rafael.j.wysocki@...el.com>,
Daniel Vetter <daniel.vetter@...ll.ch>,
Thomas Gleixner <tglx@...utronix.de>
Cc: Chris Wilson <chris@...is-wilson.co.uk>,
intel-gfx <intel-gfx@...ts.freedesktop.org>,
LKML <linux-kernel@...r.kernel.org>,
"the arch/x86 maintainers" <x86@...nel.org>,
Mika Kuoppala <mika.kuoppala@...ux.intel.com>,
Jani Nikula <jani.nikula@...el.com>, linux-pm@...r.kernel.org,
Pavel Machek <pavel@....cz>
Subject: Re: [Linux v4.10.0-rc1] call-traces after suspend-resume (pm? i915? cpu/hotplug?)
[ Hope I have not dropped people from the CC list ]
Anyway, I tried a bit more with drm-intel-nightly (drm-tip:
2016y-12m-28d-13h-55m-14s UTC integration manifest) on top of Linux
v4.10-rc1.
This has the cpuhp-revert of drm-intel folks but not the
post-cpuhp-fixes of Thomas.
Now, I only see on booting...
[ 29.558350] BUG: sleeping function called from invalid context at
drivers/base/power/runtime.c:1032
[ 29.558356] in_atomic(): 1, irqs_disabled(): 0, pid: 1507, name: Xorg
[ 29.558359] 1 lock held by Xorg/1507:
[ 29.558360] #0: (&dev->struct_mutex){+.+.+.}, at:
[<ffffffffa0660df3>] i915_mutex_lock_interruptible+0x43/0x140 [i915]
[ 29.558456] CPU: 3 PID: 1507 Comm: Xorg Not tainted
4.10.0-rc1-4-iniza-amd64 #1
[ 29.558461] Hardware name: SAMSUNG ELECTRONICS CO., LTD.
530U3BI/530U4BI/530U4BH/530U3BI/530U4BI/530U4BH, BIOS 13XK 03/28/2013
[ 29.558464] Call Trace:
[ 29.558472] dump_stack+0x85/0xc2
[ 29.558478] ___might_sleep+0x196/0x260
[ 29.558484] __might_sleep+0x53/0xb0
[ 29.558488] __pm_runtime_resume+0x7a/0x90
[ 29.558511] intel_runtime_pm_get+0x25/0x90 [i915]
[ 29.558534] aliasing_gtt_bind_vma+0xaa/0xf0 [i915]
[ 29.558558] i915_vma_bind+0xaf/0x1e0 [i915]
[ 29.558580] i915_gem_execbuffer_relocate_entry+0x513/0x6f0 [i915]
[ 29.558584] ? find_get_entry+0x5/0x240
[ 29.558605] i915_gem_execbuffer_relocate_vma.isra.34+0x188/0x250 [i915]
[ 29.558638] ? i915_vma_bind+0xaf/0x1e0 [i915]
[ 29.558673] ? __i915_vma_do_pin+0x30f/0x550 [i915]
[ 29.558706] ? i915_gem_execbuffer_reserve_vma.isra.31+0x152/0x1f0 [i915]
[ 29.558727] ? i915_gem_execbuffer_reserve.isra.32+0x372/0x3a0 [i915]
[ 29.558748] i915_gem_do_execbuffer.isra.38+0xb50/0x1a80 [i915]
[ 29.558752] ? __might_fault+0x4e/0xb0
[ 29.558772] i915_gem_execbuffer2+0xc5/0x260 [i915]
[ 29.558775] ? __might_fault+0x4e/0xb0
[ 29.558791] drm_ioctl+0x20b/0x460 [drm]
[ 29.558811] ? i915_gem_execbuffer+0x340/0x340 [i915]
[ 29.558816] ? __fget+0x5/0x200
[ 29.558819] do_vfs_ioctl+0x91/0x6f0
[ 29.558822] ? __fget+0x111/0x200
[ 29.558824] ? __fget+0x5/0x200
[ 29.558827] SyS_ioctl+0x79/0x90
[ 29.558832] entry_SYSCALL_64_fastpath+0x23/0xc6
...and after suspend/resume (looks like the same call-trace)...
[ 108.502938] PM: resume of devices complete after 792.728 msecs
[ 108.506825] Restarting tasks ... done.
[ 110.410210] ata1: SATA link up 3.0 Gbps (SStatus 123 SControl 300)
[ 110.412520] ata1.00: configured for UDMA/133
[ 110.909622] BUG: sleeping function called from invalid context at
drivers/base/power/runtime.c:1032
[ 110.909639] in_atomic(): 1, irqs_disabled(): 0, pid: 1507, name: Xorg
[ 110.909649] 1 lock held by Xorg/1507:
[ 110.909652] #0: (&dev->struct_mutex){+.+.+.}, at:
[<ffffffffa0660df3>] i915_mutex_lock_interruptible+0x43/0x140 [i915]
[ 110.909775] CPU: 1 PID: 1507 Comm: Xorg Tainted: G W
4.10.0-rc1-4-iniza-amd64 #1
[ 110.909779] Hardware name: SAMSUNG ELECTRONICS CO., LTD.
530U3BI/530U4BI/530U4BH/530U3BI/530U4BI/530U4BH, BIOS 13XK 03/28/2013
[ 110.909783] Call Trace:
[ 110.909800] dump_stack+0x85/0xc2
[ 110.909810] ___might_sleep+0x196/0x260
[ 110.909818] __might_sleep+0x53/0xb0
[ 110.909827] __pm_runtime_resume+0x7a/0x90
[ 110.909902] intel_runtime_pm_get+0x25/0x90 [i915]
[ 110.909976] aliasing_gtt_bind_vma+0xaa/0xf0 [i915]
[ 110.910057] i915_vma_bind+0xaf/0x1e0 [i915]
[ 110.910133] i915_gem_execbuffer_relocate_entry+0x513/0x6f0 [i915]
[ 110.910144] ? free_hot_cold_page+0x1c1/0x390
[ 110.910216] i915_gem_execbuffer_relocate_vma.isra.34+0x188/0x250 [i915]
[ 110.910226] ? free_pages+0x13/0x20
[ 110.910305] ? __i915_vma_do_pin+0x30f/0x550 [i915]
[ 110.910380] ? i915_gem_execbuffer_reserve_vma.isra.31+0x152/0x1f0 [i915]
[ 110.910453] ? i915_gem_execbuffer_reserve.isra.32+0x372/0x3a0 [i915]
[ 110.910506] i915_gem_do_execbuffer.isra.38+0xb50/0x1a80 [i915]
[ 110.910513] ? __might_fault+0x4e/0xb0
[ 110.910559] i915_gem_execbuffer2+0xc5/0x260 [i915]
[ 110.910563] ? __might_fault+0x4e/0xb0
[ 110.910594] drm_ioctl+0x20b/0x460 [drm]
[ 110.910640] ? i915_gem_execbuffer+0x340/0x340 [i915]
[ 110.910646] ? __fget+0x5/0x200
[ 110.910651] do_vfs_ioctl+0x91/0x6f0
[ 110.910655] ? __fget+0x111/0x200
[ 110.910658] ? __fget+0x5/0x200
[ 110.910663] SyS_ioctl+0x79/0x90
[ 110.910671] entry_SYSCALL_64_fastpath+0x23/0xc6
[ 110.910675] RIP: 0033:0x7fb9370e9bb7
[ 110.910678] RSP: 002b:00007fff34e97208 EFLAGS: 00003202 ORIG_RAX:
0000000000000010
[ 110.910684] RAX: ffffffffffffffda RBX: 0000000000000002 RCX: 00007fb9370e9bb7
[ 110.910686] RDX: 00007fff34e97308 RSI: 0000000040406469 RDI: 0000000000000009
[ 110.910689] RBP: 0000563782f58c78 R08: 0000000000000040 R09: 0101010101010101
[ 110.910691] R10: 00007fff34e90000 R11: 0000000000003202 R12: 00007fff34e96420
[ 110.910693] R13: 0000000000000032 R14: 0000000000000010 R15: 0000000000000031
Here the snippet referred above...
[ drivers/base/power/runtime.c ]
...
/**
* __pm_runtime_resume - Entry point for runtime resume operations.
* @dev: Device to resume.
* @rpmflags: Flag bits.
*
* If the RPM_GET_PUT flag is set, increment the device's usage count. Then
* carry out a resume, either synchronous or asynchronous.
*
* This routine may be called in atomic context if the RPM_ASYNC flag is set,
* or if pm_runtime_irq_safe() has been called.
*/
int __pm_runtime_resume(struct device *dev, int rpmflags)
{
unsigned long flags;
int retval;
might_sleep_if(!(rpmflags & RPM_ASYNC) &&
!dev->power.irq_safe); <--- XXX: Line #1032
if (rpmflags & RPM_GET_PUT)
atomic_inc(&dev->power.usage_count);
spin_lock_irqsave(&dev->power.lock, flags);
retval = rpm_resume(dev, rpmflags);
spin_unlock_irqrestore(&dev->power.lock, flags);
return retval;
}
EXPORT_SYMBOL_GPL(__pm_runtime_resume);
...
Hope this helps.
- Sedat -
View attachment "dmesg_4.10.0-rc1-4-iniza-amd64_after-suspend-resume.txt" of type "text/plain" (69309 bytes)
Download attachment "config-4.10.0-rc1-4-iniza-amd64" of type "application/octet-stream" (140058 bytes)
Powered by blists - more mailing lists