Hi Olof, On Sat, Dec 21, 2013 at 2:52 AM, Olof Johansson <[email protected]> wrote: > On Fri, Dec 20, 2013 at 3:38 AM, Abhilash Kesavan > <[email protected]> wrote: >> Hi Sunil, >> >> On Fri, Dec 20, 2013 at 3:56 PM, sunil joshi <[email protected]> wrote: >>> Hi Abhilash, >>> I saw another patch in chrome tree ..by Andrew Bresticker >>> which may be relevant here .. >>> >>> Just wondering if you missed adding this ? or this is not needed ? >>> You did not face any issue in getting core to suspend ? >> >> This has not been added for Exynos5250 in mainline yet. We did notice >> that the system would fail to suspend on the rare occasion (~1% of the >> time on exynos5250) when we repeatedly suspended and resumed the >> system. >> >> I have not run s2r stress tests, but have had no issues suspending the >> system in my limited testing. We can probably let this be for now as >> the system would resume fine even on a failed suspend. I will do some >> stress testing and then post if needed. > > Hold on, you're claiming that this patch you've posted has only seen > very limited testing, and that you are aware of failures? > > Don't merge known-broken code into the mainline kernel. I have tested this patch on an 5250-snow and SMDK5420 through 30 cycles without any failure. Yes, I have seen failures on a 3.4 chromium kernel on an exynos5250 during stress testing. Mainline kernel is significantly different from what we are carrying in 3.4 or 3.8 chromium and the reason for me not adding the GIC patch was that it was a hack in the first place. The basic S2R functionality works fine, but if you'd rather have it pass a stress test of say a 1000 cycles then let me do the same and provide you with the results.
> > > -Olof -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
