From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lorenzo Pieralisi Subject: Re: [PATCH v8 09/26] kernel/cpu_pm: Manage runtime PM in the idle path for CPUs Date: Wed, 15 Aug 2018 11:44:50 +0100 Message-ID: <20180815104449.GA29108@red-moon> References: <20180620172226.15012-1-ulf.hansson@linaro.org> <2056372.NMt4aPaF4h@aspire.rjw.lan> <2205807.cU2puvubpP@aspire.rjw.lan> <1726374.375PCQfjLZ@aspire.rjw.lan> <20180808105619.GB25150@e107981-ln.cambridge.arm.com> <20180808180248.GC27850@codeaurora.org> <20180809102504.GB13428@e107981-ln.cambridge.arm.com> <20180810201815.GE5081@codeaurora.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20180810201815.GE5081@codeaurora.org> Sender: linux-kernel-owner@vger.kernel.org To: Lina Iyer Cc: "Rafael J. Wysocki" , Ulf Hansson , "Rafael J. Wysocki" , Sudeep Holla , Mark Rutland , Linux PM , Kevin Hilman , Lina Iyer , Rob Herring , Daniel Lezcano , Thomas Gleixner , Vincent Guittot , Stephen Boyd , Juri Lelli , Geert Uytterhoeven , Linux ARM , linux-arm-msm , Linux Kernel Mailing List List-Id: linux-arm-msm@vger.kernel.org On Fri, Aug 10, 2018 at 02:18:15PM -0600, Lina Iyer wrote: [...] > >>But, the OSI feature is critical for QCOM mobile platforms. The > >>last man activities during cpuidle save quite a lot of power. > > > >What I expressed above was that, in PSCI based systems (OSI or PC > >alike), it is up to firmware/hardware to detect "the last man" not > >the kernel. > > > >I need to understand what you mean by "last man activities" to > >provide feedback here. > > > When the last CPU goes down during deep sleep, the following would be > done > - Lower resource requirements for shared resources such as clocks, > busses and regulators that were used by drivers in AP. These shared > resources when not used by other processors in the SoC may be turned > off and put in low power state by a remote processor. [1][2] > - Enable and setup wakeup capable interrupts on an always-on interrupt > controller, so the GIC and the GPIO controllers may be put in low > power state. [3][4] > - Write next known wakeup value to the timer, so the blocks that were > powered off, may be brought back into operational before the wakeup. > [4][5] > > These are commonly done during suspend, but to achieve a good power > efficiency, we have to do this when all the CPUs are just executing CPU > idle. Also, they cannot be done from the firmware (because the data > required for all this is part of Linux). OSI plays a crucial role in > determining when to do all this. No it does not. It is the power domain cpumasks that allow this code to make an educated guess on the last cpu running (the kernel), PSCI OSI is not crucial at all (it is crucial in QC platforms because that's the only mode supported but that's not a reason I accept as valid since it does not comply with the PSCI specifications). As I mentioned in another thread[1] the generic part of this series may be applicable in a platform agnostic way to the CPUidle framework, whether that's beneficial it has to be proven and it is benchmark specific anyway. Lorenzo [1]: https://marc.info/?l=linux-pm&m=153382916513032&w=2 From mboxrd@z Thu Jan 1 00:00:00 1970 From: lorenzo.pieralisi@arm.com (Lorenzo Pieralisi) Date: Wed, 15 Aug 2018 11:44:50 +0100 Subject: [PATCH v8 09/26] kernel/cpu_pm: Manage runtime PM in the idle path for CPUs In-Reply-To: <20180810201815.GE5081@codeaurora.org> References: <20180620172226.15012-1-ulf.hansson@linaro.org> <2056372.NMt4aPaF4h@aspire.rjw.lan> <2205807.cU2puvubpP@aspire.rjw.lan> <1726374.375PCQfjLZ@aspire.rjw.lan> <20180808105619.GB25150@e107981-ln.cambridge.arm.com> <20180808180248.GC27850@codeaurora.org> <20180809102504.GB13428@e107981-ln.cambridge.arm.com> <20180810201815.GE5081@codeaurora.org> Message-ID: <20180815104449.GA29108@red-moon> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Fri, Aug 10, 2018 at 02:18:15PM -0600, Lina Iyer wrote: [...] > >>But, the OSI feature is critical for QCOM mobile platforms. The > >>last man activities during cpuidle save quite a lot of power. > > > >What I expressed above was that, in PSCI based systems (OSI or PC > >alike), it is up to firmware/hardware to detect "the last man" not > >the kernel. > > > >I need to understand what you mean by "last man activities" to > >provide feedback here. > > > When the last CPU goes down during deep sleep, the following would be > done > - Lower resource requirements for shared resources such as clocks, > busses and regulators that were used by drivers in AP. These shared > resources when not used by other processors in the SoC may be turned > off and put in low power state by a remote processor. [1][2] > - Enable and setup wakeup capable interrupts on an always-on interrupt > controller, so the GIC and the GPIO controllers may be put in low > power state. [3][4] > - Write next known wakeup value to the timer, so the blocks that were > powered off, may be brought back into operational before the wakeup. > [4][5] > > These are commonly done during suspend, but to achieve a good power > efficiency, we have to do this when all the CPUs are just executing CPU > idle. Also, they cannot be done from the firmware (because the data > required for all this is part of Linux). OSI plays a crucial role in > determining when to do all this. No it does not. It is the power domain cpumasks that allow this code to make an educated guess on the last cpu running (the kernel), PSCI OSI is not crucial at all (it is crucial in QC platforms because that's the only mode supported but that's not a reason I accept as valid since it does not comply with the PSCI specifications). As I mentioned in another thread[1] the generic part of this series may be applicable in a platform agnostic way to the CPUidle framework, whether that's beneficial it has to be proven and it is benchmark specific anyway. Lorenzo [1]: https://marc.info/?l=linux-pm&m=153382916513032&w=2