On Tuesday 13 May 2014 04:10 AM, Roger Quadros wrote: > On 05/13/2014 01:07 AM, Tony Lindgren wrote: >> * Santosh Shilimkar <santosh.shilimkar@xxxxxx> [140512 14:41]: >>> On Sunday 11 May 2014 11:55 AM, Tony Lindgren wrote: >>>> * Kevin Hilman <khilman@xxxxxxxxxx> [140509 16:46]: >>>>> Roger Quadros <rogerq@xxxxxx> writes: >>>>> >>>>>> Kevin, >>>>>> >>>>>> On 05/09/2014 01:15 AM, Kevin Hilman wrote: >>>>>>> Tony Lindgren <tony@xxxxxxxxxxx> writes: >>>>>>> >>>>>>> [...] >>>>>>> >>>>>>>> ..but I think I found the cause for recent hangs on panda, just a wild >>>>>>>> guess based on looking at the recent cpuidle patches after v3.14. >>>>>>>> >>>>>>>> Looks like reverting 0b89e9aa2856 (cpuidle: delay enabling interrupts >>>>>>>> until all coupled CPUs leave idle) makes booting work reliably again >>>>>>>> on panda. >>>>>>>> >>>>>>>> Can you guys confirm, so far no issues here after few boot tests, >>>>>>>> but it might be too early to tell. >>>>>>> >>>>>>> Reverting that makes things a bit more stable, but it still eventually >>>>>>> fails in the same way. For me it took 8 boots for it to eventually >>>>>>> fail. >>>>>>> >>>>>>> However, if I build with CONFIG_CPU_IDLE=n, it becomes much more stable >>>>>>> (20+ boots in a row and still going.) >>>>>>> >>>>>> >>>>>> Can you please test with CPU_IDLE enabled but C3 disabled as in below patch? >>>>>> It worked for me 10/10 boots. >>>>> >>>>> Yup, it worked for me too for 10/10 boots in a row. >>>> >>>> But what has caused this regression, does it work reliably with let's >>>> say v3.13 or v3.12? >>>> >>> IIRC things were stable till some CPUIDLE code consolidation happened. >>> I don't recall exactly but some one did discuss about it a while back. >> >> OK that's good to hear. >> >>> Can you re-run your test-cases with patch at end of the email. This >>> is just a hunch so don't blame me if I waste your time testing the >>> patch. >> >> Seems to work after adding "#include <linux/clockchips.h>". I did about 10 >> reboots and they all succeeded for me. Without your revert, I'm getting >> a hang (with sysrq not working) about 1/3 of the boots. >> >> Kevin, Roger, does the revert from Santosh work for you too? >> > > next-20140508 worked for me 10/10 times with Santosh's patch. > The heartbeat LED behaves normally as well. So I like it :). > Great. Will post the patch with change log updated and cc you guys. Regards, Santosh -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
