On Mon, Nov 22, 2010 at 11:53 AM, Rajendra Nayak <rna...@ti.com> wrote: >> -----Original Message----- >> From: linux-omap-ow...@vger.kernel.org > [mailto:linux-omap-ow...@vger.kernel.org] On Behalf Of Jean Pihet >> Sent: Monday, November 22, 2010 4:15 PM >> To: linux-omap@vger.kernel.org >> Cc: Jean Pihet; Kevin Hilman >> Subject: [PATCH] OMAP3: disable idle early in the suspend sequence >> >> Some bad interaction between the idle and the suspend paths has been >> noticed: the idle code is called during the suspend enter and exit >> sequences. This could cause corruption or lock-up of resources. > > Can you elaborate more on what kind of issues were seen?
Trying to get the PRCM registers dump after a suspend/resume does not show the correct registers values, cf. Kevin's patch at http://git.kernel.org/?p=linux/kernel/git/khilman/linux-omap-pm.git;a=commitdiff;h=9fc4891d4a21d2b644a463d62c77ef97da55f091. Digging a bit further I found out that the idle routine is called >50 times while the suspend/resume sequence still is on-going. The root cause is because disable_hlt is called from omap3_pm_prepare which runs after omap3_pm_begin (and the same issue in the resume sequence). This leaves a time window for idle to kick-in while the suspend sequence is busy saving/restoring the system state. This is a potential bug that just waits to show up soon or later, especially if more code is added in the suspend prepare and finish functions. Does the changelog need an update? Regards, Jean -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
