From mboxrd@z Thu Jan 1 00:00:00 1970 From: Uwe =?iso-8859-1?Q?Kleine-K=F6nig?= Subject: Re: Clocks used by another OS/CPU (was: Re: [RFC PATCH] clk: renesas: cpg-mssr: Add interface for critical core clocks) Date: Sat, 1 Jul 2017 20:14:08 +0200 Message-ID: <20170701181408.yuocymwtj5dgt74d@pengutronix.de> References: <20170630202453.eh6vaehkap3as4np@pengutronix.de> <8d86a9aa-1928-6b52-1487-d5fb9dae17f4@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: 8bit Return-path: Content-Disposition: inline In-Reply-To: <8d86a9aa-1928-6b52-1487-d5fb9dae17f4@gmail.com> Sender: linux-renesas-soc-owner@vger.kernel.org To: Dirk Behme Cc: Rob Herring , Mark Rutland , "devicetree@vger.kernel.org" , Ulf Hansson , Dirk Behme , Linux PM list , "Rafael J. Wysocki" , Michael Turquette , Stephen Boyd , Linux-Renesas , Geert Uytterhoeven , Kevin Hilman , linux-clk , "linux-arm-kernel@lists.infradead.org" List-Id: devicetree@vger.kernel.org Hello, On Sat, Jul 01, 2017 at 07:02:48AM +0200, Dirk Behme wrote: > On 30.06.2017 22:24, Uwe Kleine-König wrote: > > Hello, > > > > On Fri, Jun 30, 2017 at 10:58:26AM -0500, Rob Herring wrote: > > > > TL;DR: Clocks may be in use by another CPU not running Linux, while Linux > > > > disables them as being unused. > > > > not long ago I thought with a few colleagues about this. The scenario is > > to start a Linux kernel on a Cortex-M companion to a Cortex-A. > > > > > > On Mon, Jun 26, 2017 at 1:30 PM, Dirk Behme wrote: > > > > > With commit 72f5df2c2bbb6 ("clk: renesas: cpg-mssr: Migrate to > > > > > CLK_IS_CRITICAL") we are able to handle critical module clocks. > > > > > Introduce the same logic for critical core clocks. > > > > > > > > > > Signed-off-by: Dirk Behme > > > > > --- > > > > > Commit > > > > > > > > > > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/drivers/clk/renesas?id=72f5df2c2bbb66d4a555cb51eb9f412abf1af77f > > > > > > > > > > is quite nice to avoid *module* clocks being disabled. Unfortunately, > > > > > there are *core* clocks, too. E.g. using an other OS on the Cortex R7 > > > > > core of the r8a7795, the 'canfd' is a quite popular core clock which > > > > > shouldn't be disabled by Linux. > > > > > > > > > > Therefore, this patch is a proposal to use the same 'mark clocks as > > > > > critical' logic implemented for the module clocks for the core > > > > > clocks, too. > > > > > > > > > > Opinions? > > > > > > > > On r8a7795, there are several Cortex A cores running Linux, and a Cortex R7 > > > > core which may run another OS. > > > > This is an interesting issue, and relevant to other SoCs, too. > > > > > > > > In this particular case, the "canfd" clock is a core clock used as an > > > > auxiliary clock for the CAN0, CAN1, and CANFD interfaces. This can lead > > > > to three scenarios: > > > > 1. Linux controls all CAN interfaces > > > > => no issue, > > > > 2. The OS on the RT CPU controls all CAN interfaces > > > > => issue, Linux disables the clock > > > > 3. Mix of 1 and 2 > > > > => More issues. > > > > Of course this is not limited to clocks, but also to e.g. PM domains. > > > > > > > > How can this be handled? > > > > I believe just marking the "canfd" clock critical is not the right solution, > > > > as about any clock could be used by the RT CPU. > > > > > > > > Still, Linux needs to be made aware that devices (clocks and PM domains) are > > > > controlled by another CPU/OS. > > > > > > > > Should this be described in DT? It feels like software policy to me. > > > > > > No, it shouldn't. It is Linux policy to disable all unused clocks, so > > > Linux gets to deal with the consequences. > > > > The ideal solution I imagine is to make the other CPU's OS a consumer of > > the Linux clock driver. This would require a generic device driver on the > > companion CPU that forwards clk requests via inter-cpu communication to > > the Linux clk driver. It could be feed with the necessary information by > > the rproc glue. So when the companion cpu is supposed to care for the > > can0 device, the steps that should happen are: > > > > - make sure can0 isn't occupied by the Linux Host > > - reroute the can irq to the companion cpu (if necessary) > > - create a dtb containing something like this for the companion CPU: > > > > clks: virtclk { > > compatible = ??? > > #clock-cells = <1>; > > ... > > }; > > > > can@$address { > > compatible = ... > > regs = ... > > clocks = <&clks 3>; > > clock-names = ... > > ... > > }; > > > > where the driver binding to the virtclk device just forwards clk > > requests to the Linux host side which then knows that clk 3 is the > > can clock and does the necessary stuff. > > > > This way the can clock doesn't need special handling in the host's dtb > > and no clock necessary for the companion is disabled as unused because > > it is requested and enabled. > > > > The only problem I see is that implementing such a driver/protocol > > probably is time consuming. > > > The other problem is security related. If, at all, you have to do it the > other way around, then: > > Make Linux a consumer of the other CPU's (trusted/trustzone/whatever > secured) OS clock driver. That doesn't matter much. Either way the first CPU has to provide the master side of this device (as it needs clks for booting up) and the 2nd gets this virtual clk device that forwards clk requests to the first CPU. On my machine (Udoo Neo, A9 + M4) the A9 is the primary CPU that is started by the bootrom. If I want the M4 being the primary device I'd need support in the bootloader to wait long enough (i.e. until the M4 is up) before letting the A9 jump into Linux. Managable I'd say. This way would even make sense if the M4 runs a rt critical OS that shouldn't be forced to wait on the non-rt A9 to enable a clk. Best regards Uwe -- Pengutronix e.K. | Uwe Kleine-König | Industrial Linux Solutions | http://www.pengutronix.de/ | From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from metis.ext.4.pengutronix.de ([92.198.50.35]:53577 "EHLO metis.ext.4.pengutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751851AbdGASOY (ORCPT ); Sat, 1 Jul 2017 14:14:24 -0400 Date: Sat, 1 Jul 2017 20:14:08 +0200 From: Uwe =?iso-8859-1?Q?Kleine-K=F6nig?= To: Dirk Behme Cc: Rob Herring , Mark Rutland , "devicetree@vger.kernel.org" , Ulf Hansson , Dirk Behme , Linux PM list , "Rafael J. Wysocki" , Michael Turquette , Stephen Boyd , Linux-Renesas , Geert Uytterhoeven , Kevin Hilman , linux-clk , "linux-arm-kernel@lists.infradead.org" Subject: Re: Clocks used by another OS/CPU (was: Re: [RFC PATCH] clk: renesas: cpg-mssr: Add interface for critical core clocks) Message-ID: <20170701181408.yuocymwtj5dgt74d@pengutronix.de> References: <20170630202453.eh6vaehkap3as4np@pengutronix.de> <8d86a9aa-1928-6b52-1487-d5fb9dae17f4@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <8d86a9aa-1928-6b52-1487-d5fb9dae17f4@gmail.com> Sender: linux-renesas-soc-owner@vger.kernel.org List-ID: Hello, On Sat, Jul 01, 2017 at 07:02:48AM +0200, Dirk Behme wrote: > On 30.06.2017 22:24, Uwe Kleine-K�nig wrote: > > Hello, > > > > On Fri, Jun 30, 2017 at 10:58:26AM -0500, Rob Herring wrote: > > > > TL;DR: Clocks may be in use by another CPU not running Linux, while Linux > > > > disables them as being unused. > > > > not long ago I thought with a few colleagues about this. The scenario is > > to start a Linux kernel on a Cortex-M companion to a Cortex-A. > > > > > > On Mon, Jun 26, 2017 at 1:30 PM, Dirk Behme wrote: > > > > > With commit 72f5df2c2bbb6 ("clk: renesas: cpg-mssr: Migrate to > > > > > CLK_IS_CRITICAL") we are able to handle critical module clocks. > > > > > Introduce the same logic for critical core clocks. > > > > > > > > > > Signed-off-by: Dirk Behme > > > > > --- > > > > > Commit > > > > > > > > > > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/drivers/clk/renesas?id=72f5df2c2bbb66d4a555cb51eb9f412abf1af77f > > > > > > > > > > is quite nice to avoid *module* clocks being disabled. Unfortunately, > > > > > there are *core* clocks, too. E.g. using an other OS on the Cortex R7 > > > > > core of the r8a7795, the 'canfd' is a quite popular core clock which > > > > > shouldn't be disabled by Linux. > > > > > > > > > > Therefore, this patch is a proposal to use the same 'mark clocks as > > > > > critical' logic implemented for the module clocks for the core > > > > > clocks, too. > > > > > > > > > > Opinions? > > > > > > > > On r8a7795, there are several Cortex A cores running Linux, and a Cortex R7 > > > > core which may run another OS. > > > > This is an interesting issue, and relevant to other SoCs, too. > > > > > > > > In this particular case, the "canfd" clock is a core clock used as an > > > > auxiliary clock for the CAN0, CAN1, and CANFD interfaces. This can lead > > > > to three scenarios: > > > > 1. Linux controls all CAN interfaces > > > > => no issue, > > > > 2. The OS on the RT CPU controls all CAN interfaces > > > > => issue, Linux disables the clock > > > > 3. Mix of 1 and 2 > > > > => More issues. > > > > Of course this is not limited to clocks, but also to e.g. PM domains. > > > > > > > > How can this be handled? > > > > I believe just marking the "canfd" clock critical is not the right solution, > > > > as about any clock could be used by the RT CPU. > > > > > > > > Still, Linux needs to be made aware that devices (clocks and PM domains) are > > > > controlled by another CPU/OS. > > > > > > > > Should this be described in DT? It feels like software policy to me. > > > > > > No, it shouldn't. It is Linux policy to disable all unused clocks, so > > > Linux gets to deal with the consequences. > > > > The ideal solution I imagine is to make the other CPU's OS a consumer of > > the Linux clock driver. This would require a generic device driver on the > > companion CPU that forwards clk requests via inter-cpu communication to > > the Linux clk driver. It could be feed with the necessary information by > > the rproc glue. So when the companion cpu is supposed to care for the > > can0 device, the steps that should happen are: > > > > - make sure can0 isn't occupied by the Linux Host > > - reroute the can irq to the companion cpu (if necessary) > > - create a dtb containing something like this for the companion CPU: > > > > clks: virtclk { > > compatible = ??? > > #clock-cells = <1>; > > ... > > }; > > > > can@$address { > > compatible = ... > > regs = ... > > clocks = <&clks 3>; > > clock-names = ... > > ... > > }; > > > > where the driver binding to the virtclk device just forwards clk > > requests to the Linux host side which then knows that clk 3 is the > > can clock and does the necessary stuff. > > > > This way the can clock doesn't need special handling in the host's dtb > > and no clock necessary for the companion is disabled as unused because > > it is requested and enabled. > > > > The only problem I see is that implementing such a driver/protocol > > probably is time consuming. > > > The other problem is security related. If, at all, you have to do it the > other way around, then: > > Make Linux a consumer of the other CPU's (trusted/trustzone/whatever > secured) OS clock driver. That doesn't matter much. Either way the first CPU has to provide the master side of this device (as it needs clks for booting up) and the 2nd gets this virtual clk device that forwards clk requests to the first CPU. On my machine (Udoo Neo, A9 + M4) the A9 is the primary CPU that is started by the bootrom. If I want the M4 being the primary device I'd need support in the bootloader to wait long enough (i.e. until the M4 is up) before letting the A9 jump into Linux. Managable I'd say. This way would even make sense if the M4 runs a rt critical OS that shouldn't be forced to wait on the non-rt A9 to enable a clk. Best regards Uwe -- Pengutronix e.K. | Uwe Kleine-K�nig | Industrial Linux Solutions | http://www.pengutronix.de/ | From mboxrd@z Thu Jan 1 00:00:00 1970 From: u.kleine-koenig@pengutronix.de (Uwe =?iso-8859-1?Q?Kleine-K=F6nig?=) Date: Sat, 1 Jul 2017 20:14:08 +0200 Subject: Clocks used by another OS/CPU (was: Re: [RFC PATCH] clk: renesas: cpg-mssr: Add interface for critical core clocks) In-Reply-To: <8d86a9aa-1928-6b52-1487-d5fb9dae17f4@gmail.com> References: <20170630202453.eh6vaehkap3as4np@pengutronix.de> <8d86a9aa-1928-6b52-1487-d5fb9dae17f4@gmail.com> Message-ID: <20170701181408.yuocymwtj5dgt74d@pengutronix.de> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hello, On Sat, Jul 01, 2017 at 07:02:48AM +0200, Dirk Behme wrote: > On 30.06.2017 22:24, Uwe Kleine-K?nig wrote: > > Hello, > > > > On Fri, Jun 30, 2017 at 10:58:26AM -0500, Rob Herring wrote: > > > > TL;DR: Clocks may be in use by another CPU not running Linux, while Linux > > > > disables them as being unused. > > > > not long ago I thought with a few colleagues about this. The scenario is > > to start a Linux kernel on a Cortex-M companion to a Cortex-A. > > > > > > On Mon, Jun 26, 2017 at 1:30 PM, Dirk Behme wrote: > > > > > With commit 72f5df2c2bbb6 ("clk: renesas: cpg-mssr: Migrate to > > > > > CLK_IS_CRITICAL") we are able to handle critical module clocks. > > > > > Introduce the same logic for critical core clocks. > > > > > > > > > > Signed-off-by: Dirk Behme > > > > > --- > > > > > Commit > > > > > > > > > > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/drivers/clk/renesas?id=72f5df2c2bbb66d4a555cb51eb9f412abf1af77f > > > > > > > > > > is quite nice to avoid *module* clocks being disabled. Unfortunately, > > > > > there are *core* clocks, too. E.g. using an other OS on the Cortex R7 > > > > > core of the r8a7795, the 'canfd' is a quite popular core clock which > > > > > shouldn't be disabled by Linux. > > > > > > > > > > Therefore, this patch is a proposal to use the same 'mark clocks as > > > > > critical' logic implemented for the module clocks for the core > > > > > clocks, too. > > > > > > > > > > Opinions? > > > > > > > > On r8a7795, there are several Cortex A cores running Linux, and a Cortex R7 > > > > core which may run another OS. > > > > This is an interesting issue, and relevant to other SoCs, too. > > > > > > > > In this particular case, the "canfd" clock is a core clock used as an > > > > auxiliary clock for the CAN0, CAN1, and CANFD interfaces. This can lead > > > > to three scenarios: > > > > 1. Linux controls all CAN interfaces > > > > => no issue, > > > > 2. The OS on the RT CPU controls all CAN interfaces > > > > => issue, Linux disables the clock > > > > 3. Mix of 1 and 2 > > > > => More issues. > > > > Of course this is not limited to clocks, but also to e.g. PM domains. > > > > > > > > How can this be handled? > > > > I believe just marking the "canfd" clock critical is not the right solution, > > > > as about any clock could be used by the RT CPU. > > > > > > > > Still, Linux needs to be made aware that devices (clocks and PM domains) are > > > > controlled by another CPU/OS. > > > > > > > > Should this be described in DT? It feels like software policy to me. > > > > > > No, it shouldn't. It is Linux policy to disable all unused clocks, so > > > Linux gets to deal with the consequences. > > > > The ideal solution I imagine is to make the other CPU's OS a consumer of > > the Linux clock driver. This would require a generic device driver on the > > companion CPU that forwards clk requests via inter-cpu communication to > > the Linux clk driver. It could be feed with the necessary information by > > the rproc glue. So when the companion cpu is supposed to care for the > > can0 device, the steps that should happen are: > > > > - make sure can0 isn't occupied by the Linux Host > > - reroute the can irq to the companion cpu (if necessary) > > - create a dtb containing something like this for the companion CPU: > > > > clks: virtclk { > > compatible = ??? > > #clock-cells = <1>; > > ... > > }; > > > > can@$address { > > compatible = ... > > regs = ... > > clocks = <&clks 3>; > > clock-names = ... > > ... > > }; > > > > where the driver binding to the virtclk device just forwards clk > > requests to the Linux host side which then knows that clk 3 is the > > can clock and does the necessary stuff. > > > > This way the can clock doesn't need special handling in the host's dtb > > and no clock necessary for the companion is disabled as unused because > > it is requested and enabled. > > > > The only problem I see is that implementing such a driver/protocol > > probably is time consuming. > > > The other problem is security related. If, at all, you have to do it the > other way around, then: > > Make Linux a consumer of the other CPU's (trusted/trustzone/whatever > secured) OS clock driver. That doesn't matter much. Either way the first CPU has to provide the master side of this device (as it needs clks for booting up) and the 2nd gets this virtual clk device that forwards clk requests to the first CPU. On my machine (Udoo Neo, A9 + M4) the A9 is the primary CPU that is started by the bootrom. If I want the M4 being the primary device I'd need support in the bootloader to wait long enough (i.e. until the M4 is up) before letting the A9 jump into Linux. Managable I'd say. This way would even make sense if the M4 runs a rt critical OS that shouldn't be forced to wait on the non-rt A9 to enable a clk. Best regards Uwe -- Pengutronix e.K. | Uwe Kleine-K?nig | Industrial Linux Solutions | http://www.pengutronix.de/ |