On 2014/10/27 15:28, Peter Zijlstra wrote: > On Mon, Oct 27, 2014 at 02:27:27PM +0800, Li, Aubrey wrote: >>> Now I suppose the problem is with cpu_pause() which needs IPIs to >>> complete? Do we really need cpuidle_pause() there? >>> cpuidle_uninstall_handlers() seems like a daft thing to call just about >>> there. >> >> Please check the log of 8651f97bd951d0bb1c10fa24e3fa3455193f3548. >> Rafael should know more this question than me. > > That changelog explains its complete bollocks to do it here. We _want_ > to enter and/or remain in deep idle states.
cpuidle_resume() will be called at the end of dpm_resume_noirq(). So we still are able to enter deep idle states after resume. Thanks, -Aubrey > -- > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/ > > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
