From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dmitry Osipenko Subject: Re: [PATCH 0/8] ARM: tegra: support LP1 suspend mode Date: Sat, 27 Jul 2013 22:26:06 +0400 Message-ID: <51F410BE.3000904@gmail.com> References: <1374830110-30685-1-git-send-email-josephl@nvidia.com> <2389689.JgQlW00RdN@ax5200p> <51F3F356.8080709@gmail.com> <4555938.6N2HxzJnNO@ax5200p> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: In-Reply-To: <4555938.6N2HxzJnNO@ax5200p> Sender: linux-tegra-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Marc Dietrich Cc: Joseph Lo , Stephen Warren , linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org List-Id: linux-tegra@vger.kernel.org 27.07.2013 22:09, Marc Dietrich =D0=BF=D0=B8=D1=88=D0=B5=D1=82: > On Saturday 27 July 2013 20:20:38 Dmitry Osipenko wrote: >> 27.07.2013 20:12, Marc Dietrich =D0=BF=D0=B8=D1=88=D0=B5=D1=82: >>> On Friday 26 July 2013 17:15:02 Joseph Lo wrote: >>>> This series adds the support of LP1 suspend mode for Tegra. >>>> >>>> Verified on Seaboard, Cardhu and Dalmore. >>> >>> tested on AC100 and worked fine, except >>> >>> [ 68.448007] PM: Syncing filesystems ... done. >>> [ 71.370086] Freezing user space processes ... (elapsed 0.001 sec= onds) >>> done. [ 71.370088] Freezing remaining freezable tasks ... (elapse= d >>> 0.001 seconds) done. >>> [ 71.371253] Suspending console(s) (use no_console_suspend to deb= ug) >>> [ 71.638481] PM: suspend of devices complete after 266.604 msecs >>> [ 71.638905] PM: late suspend of devices complete after 0.417 mse= cs >>> [ 71.639477] PM: noirq suspend of devices complete after 0.568 ms= ecs >>> [ 71.639480] Disabling non-boot CPUs ... >>> [ 71.640242] CPU1: shutdown >>> [ 71.641216] Entering suspend state LP1 >>> [ 71.641267] Enabling non-boot CPUs ... >>> [ 71.643125] CPU1: Booted secondary processor >>> [ 71.643449] CPU1 is up >>> [ 71.643786] PM: noirq resume of devices complete after 0.331 mse= cs >>> [ 71.644300] PM: early resume of devices complete after 0.364 mse= cs >>> [ 71.648038] ------------[ cut here ]------------ >>> [ 71.648061] WARNING: CPU: 0 PID: 1025 at kernel/irq/manage.c:529 >>> irq_set_irq_wake+0xb8/0xf0() >>> [ 71.648064] Unbalanced IRQ 384 wake disable >>> [ 71.648068] Modules linked in: >>> [ 71.648075] CPU: 0 PID: 1025 Comm: bash Not tainted 3.11.0-rc2- >>> next-20130726-00008-gef01509-dirty #6 >>> [ 71.648107] [] (unwind_backtrace+0x0/0xf8) from [] >>> (show_stack+0x10/0x14) >>> [ 71.648128] [] (show_stack+0x10/0x14) from [= ] >>> (dump_stack+0x80/0xc4) >>> [ 71.648142] [] (dump_stack+0x80/0xc4) from [= ] >>> (warn_slowpath_common+0x64/0x88) >>> [ 71.648151] [] (warn_slowpath_common+0x64/0x88) from >>> [] (warn_slowpath_fmt+0x30/0x40) >>> [ 71.648161] [] (warn_slowpath_fmt+0x30/0x40) from >>> [] >>> (irq_set_irq_wake+0xb8/0xf0) >>> [ 71.648178] [] (irq_set_irq_wake+0xb8/0xf0) from [] >>> (tps6586x_rtc_resume+0x30/0x38) >>> [ 71.648192] [] (tps6586x_rtc_resume+0x30/0x38) from >>> [] (platform_pm_resume+0x2c/0x4c) >>> [ 71.648202] [] (platform_pm_resume+0x2c/0x4c) from >>> [] (dpm_run_callback.isra.5+0x18/0x38) >>> [ 71.648211] [] (dpm_run_callback.isra.5+0x18/0x38) fro= m >>> [] (device_resume+0xb0/0x170) >>> [ 71.648219] [] (device_resume+0xb0/0x170) from [] >>> (dpm_resume+0xe8/0x20c) >>> [ 71.648227] [] (dpm_resume+0xe8/0x20c) from [] >>> (dpm_resume_end+0xc/0x18) >>> [ 71.648245] [] (dpm_resume_end+0xc/0x18) from [] >>> (suspend_devices_and_enter+0xdc/0x2f8) >>> [ 71.648256] [] (suspend_devices_and_enter+0xdc/0x2f8) = from >>> [] (pm_suspend+0x174/0x1e8) >>> [ 71.648265] [] (pm_suspend+0x174/0x1e8) from [] >>> (state_store+0x6c/0xbc) >>> [ 71.648285] [] (state_store+0x6c/0xbc) from [] >>> (kobj_attr_store+0x14/0x20) >>> [ 71.648298] [] (kobj_attr_store+0x14/0x20) from [] >>> (sysfs_write_file+0x168/0x198) >>> [ 71.648312] [] (sysfs_write_file+0x168/0x198) from >>> [] (vfs_write+0xb0/0x188) >>> [ 71.648321] [] (vfs_write+0xb0/0x188) from [= ] >>> (SyS_write+0x3c/0x70) >>> [ 71.648330] [] (SyS_write+0x3c/0x70) from [] >>> (ret_fast_syscall+0x0/0x30) >>> [ 71.648334] ---[ end trace e5ed8212ac4db5da ]--- >>> [ 72.063578] mmc1: power class selection to bus width 8 failed >>> [ 72.297753] usb 2-1: reset high-speed USB device number 2 using >>> tegra-ehci [ 72.567736] usb 3-1: reset high-speed USB device numb= er 2 >>> using tegra-ehci [ 73.022058] usb 3-1.3: reset high-speed USB dev= ice >>> number 4 using tegra- ehci >>> [ 73.216064] usb 3-1.2: reset high-speed USB device number 3 usin= g >>> tegra- >>> ehci >>> [ 73.360225] PM: resume of devices complete after 1715.937 msecs >>> [ 73.361008] Restarting tasks ... done. >>> >>> and sdcard being offline after resume (fixed by >>> CONFIG_MMC_UNSAFE_RESUME=3Dy). Anyway, thanks for your great work! >>> >>> Tested-By: Marc Dietrich >> >> I had same warn on 3.8. >> https://bitbucket.org/digetx/picasso-kernel/commits/a86dc95ab48dd77e= aa046384 >> 4cdf88a5acf9d842 >=20 > Stephen Warren said on IRC that this is known and on the internal bug= tracker=20 > already. I just posted it so others are not surprised. Well, I should= have=20 > added that it's a know problem. Sorry for that. >=20 > Marc >=20 >=20 >=20 >=20 It might be a known problem only for those who faced with it. Since it'= s on their internal irc, it wasn't a known issue. It also would be interesting to know if suspend with thumb2 works corre= ctly. On 3.8 I've ported lp0 suspend and it works fine with arm compiled kernel,= thumb2 also works but only for about 5-10 hours (resuming on each 5-30 minutes= , default android behavior) and after I'm getting pmd corruption. I have no idea = from where that corruption comes from without ability for hw debugging.