Re: [Xen-devel] [PATCH v2] xen/arm: register clocks used by the hypervisor

2016-07-08 Thread Julien Grall
On 08/07/16 07:48, Dirk Behme wrote: #include #include #include @@ -444,6 +445,52 @@ static int __init xen_pm_init(void) } late_initcall(xen_pm_init); +/* + * Check if we want to register some clocks, that they + * are not freed because unused by clk_disable_unused(). + * E.g. the seri

Re: [Xen-devel] [PATCH v2] xen/arm: register clocks used by the hypervisor

2016-07-08 Thread Julien Grall
Hi Dirk, On 08/07/16 06:51, Dirk Behme wrote: On 08.07.2016 04:50, Michael Turquette wrote: Quoting Dirk Behme (2016-07-07 00:32:34) The two issues to resolve are: 1) does consuming these hardware resources from the Linux kernel fit correctly with the Xen model? I think so. Julien, what do

Re: [Xen-devel] [PATCH v2] xen/arm: register clocks used by the hypervisor

2016-07-07 Thread Dirk Behme
Hi, On 06.07.2016 03:34, Michael Turquette wrote: Hi! Quoting Dirk Behme (2016-06-30 03:32:32) Some clocks might be used by the Xen hypervisor and not by the Linux kernel. If these are not registered by the Linux kernel, they might be disabled by clk_disable_unused() as the kernel doesn't know

Re: [Xen-devel] [PATCH v2] xen/arm: register clocks used by the hypervisor

2016-07-07 Thread Dirk Behme
On 08.07.2016 04:50, Michael Turquette wrote: Quoting Dirk Behme (2016-07-07 00:32:34) Hi Michael, On 06.07.2016 22:42, Michael Turquette wrote: Hi Julien, Quoting Julien Grall (2016-07-06 06:10:52) On 06/07/16 02:34, Michael Turquette wrote: Hi! Hello Michael, Quoting Dirk Behme (2016-

Re: [Xen-devel] [PATCH v2] xen/arm: register clocks used by the hypervisor

2016-07-07 Thread Michael Turquette
Quoting Dirk Behme (2016-07-07 00:32:34) > Hi Michael, > > On 06.07.2016 22:42, Michael Turquette wrote: > > Hi Julien, > > > > Quoting Julien Grall (2016-07-06 06:10:52) > >> On 06/07/16 02:34, Michael Turquette wrote: > >>> Hi! > >> > >> Hello Michael, > >> > >>> Quoting Dirk Behme (2016-06-30 0

Re: [Xen-devel] [PATCH v2] xen/arm: register clocks used by the hypervisor

2016-07-07 Thread Dirk Behme
Hi Michael, On 06.07.2016 22:42, Michael Turquette wrote: Hi Julien, Quoting Julien Grall (2016-07-06 06:10:52) On 06/07/16 02:34, Michael Turquette wrote: Hi! Hello Michael, Quoting Dirk Behme (2016-06-30 03:32:32) Some clocks might be used by the Xen hypervisor and not by the Linux ker

Re: [Xen-devel] [PATCH v2] xen/arm: register clocks used by the hypervisor

2016-07-06 Thread Michael Turquette
Quoting Mark Rutland (2016-07-05 04:07:37) > Hi, > > On Tue, Jul 05, 2016 at 12:45:34PM +0200, Dirk Behme wrote: > > On 05.07.2016 12:39, Mark Rutland wrote: > > >On Tue, Jul 05, 2016 at 08:50:23AM +0200, Dirk Behme wrote: > > >>+- clocks: one or more clocks to be registered. > > >>+ Xen hypervis

Re: [Xen-devel] [PATCH v2] xen/arm: register clocks used by the hypervisor

2016-07-06 Thread Michael Turquette
Hi Julien, Quoting Julien Grall (2016-07-06 06:10:52) > On 06/07/16 02:34, Michael Turquette wrote: > > Hi! > > Hello Michael, > > > Quoting Dirk Behme (2016-06-30 03:32:32) > >> Some clocks might be used by the Xen hypervisor and not by the Linux > >> kernel. If these are not registered by the

Re: [Xen-devel] [PATCH v2] xen/arm: register clocks used by the hypervisor

2016-07-06 Thread Mark Rutland
On Wed, Jul 06, 2016 at 02:16:18PM +0100, Stefano Stabellini wrote: > On Wed, 6 Jul 2016, Julien Grall wrote: > > On 06/07/16 02:34, Michael Turquette wrote: > > > Hi! > > > > Hello Michael, > > > > > Quoting Dirk Behme (2016-06-30 03:32:32) > > > > Some clocks might be used by the Xen hypervisor

Re: [Xen-devel] [PATCH v2] xen/arm: register clocks used by the hypervisor

2016-07-06 Thread Julien Grall
On 06/07/16 14:16, Stefano Stabellini wrote: On Wed, 6 Jul 2016, Julien Grall wrote: On 06/07/16 02:34, Michael Turquette wrote: Hi! Hello Michael, Quoting Dirk Behme (2016-06-30 03:32:32) Some clocks might be used by the Xen hypervisor and not by the Linux kernel. If these are not regis

Re: [Xen-devel] [PATCH v2] xen/arm: register clocks used by the hypervisor

2016-07-06 Thread Stefano Stabellini
On Wed, 6 Jul 2016, Julien Grall wrote: > On 06/07/16 02:34, Michael Turquette wrote: > > Hi! > > Hello Michael, > > > Quoting Dirk Behme (2016-06-30 03:32:32) > > > Some clocks might be used by the Xen hypervisor and not by the Linux > > > kernel. If these are not registered by the Linux kernel,

Re: [Xen-devel] [PATCH v2] xen/arm: register clocks used by the hypervisor

2016-07-06 Thread Julien Grall
On 06/07/16 02:34, Michael Turquette wrote: Hi! Hello Michael, Quoting Dirk Behme (2016-06-30 03:32:32) Some clocks might be used by the Xen hypervisor and not by the Linux kernel. If these are not registered by the Linux kernel, they might be disabled by clk_disable_unused() as the kernel d

Re: [Xen-devel] [PATCH v2] xen/arm: register clocks used by the hypervisor

2016-07-06 Thread Michael Turquette
Hi! Quoting Dirk Behme (2016-06-30 03:32:32) > Some clocks might be used by the Xen hypervisor and not by the Linux > kernel. If these are not registered by the Linux kernel, they might be > disabled by clk_disable_unused() as the kernel doesn't know that they > are used. The clock of the serial c

Re: [Xen-devel] [PATCH v2] xen/arm: register clocks used by the hypervisor

2016-07-05 Thread Stefano Stabellini
On Tue, 5 Jul 2016, Julien Grall wrote: > On 05/07/16 15:04, Stefano Stabellini wrote: > > On Tue, 5 Jul 2016, Julien Grall wrote: > > > On 05/07/16 14:53, Stefano Stabellini wrote: > > > > On Thu, 30 Jun 2016, Dirk Behme wrote: > > > > > +- clocks: one or more clocks to be registered. > > > > > +

Re: [Xen-devel] [PATCH v2] xen/arm: register clocks used by the hypervisor

2016-07-05 Thread Julien Grall
On 05/07/16 15:04, Stefano Stabellini wrote: On Tue, 5 Jul 2016, Julien Grall wrote: On 05/07/16 14:53, Stefano Stabellini wrote: On Thu, 30 Jun 2016, Dirk Behme wrote: +- clocks: one or more clocks to be registered. + Xen hypervisor drivers might replace native drivers, resulting in + clo

Re: [Xen-devel] [PATCH v2] xen/arm: register clocks used by the hypervisor

2016-07-05 Thread Stefano Stabellini
On Tue, 5 Jul 2016, Julien Grall wrote: > On 05/07/16 14:53, Stefano Stabellini wrote: > > On Thu, 30 Jun 2016, Dirk Behme wrote: > > > +- clocks: one or more clocks to be registered. > > > + Xen hypervisor drivers might replace native drivers, resulting in > > > + clocks not registered by these

Re: [Xen-devel] [PATCH v2] xen/arm: register clocks used by the hypervisor

2016-07-05 Thread Julien Grall
On 05/07/16 14:54, Julien Grall wrote: On 05/07/16 14:53, Stefano Stabellini wrote: On Thu, 30 Jun 2016, Dirk Behme wrote: +- clocks: one or more clocks to be registered. + Xen hypervisor drivers might replace native drivers, resulting in + clocks not registered by these native drivers. T

Re: [Xen-devel] [PATCH v2] xen/arm: register clocks used by the hypervisor

2016-07-05 Thread Julien Grall
On 05/07/16 14:53, Stefano Stabellini wrote: On Thu, 30 Jun 2016, Dirk Behme wrote: +- clocks: one or more clocks to be registered. + Xen hypervisor drivers might replace native drivers, resulting in + clocks not registered by these native drivers. To avoid that these + unregistered clocks

Re: [Xen-devel] [PATCH v2] xen/arm: register clocks used by the hypervisor

2016-07-05 Thread Stefano Stabellini
On Thu, 30 Jun 2016, Dirk Behme wrote: > Some clocks might be used by the Xen hypervisor and not by the Linux > kernel. If these are not registered by the Linux kernel, they might be > disabled by clk_disable_unused() as the kernel doesn't know that they > are used. The clock of the serial console

Re: [Xen-devel] [PATCH v2] xen/arm: register clocks used by the hypervisor

2016-07-05 Thread Mark Rutland
Hi, On Tue, Jul 05, 2016 at 12:45:34PM +0200, Dirk Behme wrote: > On 05.07.2016 12:39, Mark Rutland wrote: > >On Tue, Jul 05, 2016 at 08:50:23AM +0200, Dirk Behme wrote: > >>+- clocks: one or more clocks to be registered. > >>+ Xen hypervisor drivers might replace native drivers, resulting in > >

Re: [Xen-devel] [PATCH v2] xen/arm: register clocks used by the hypervisor

2016-07-05 Thread Dirk Behme
Hi Mark, On 05.07.2016 12:39, Mark Rutland wrote: Hi, On Tue, Jul 05, 2016 at 08:50:23AM +0200, Dirk Behme wrote: Some clocks might be used by the Xen hypervisor and not by the Linux kernel. If these are not registered by the Linux kernel, they might be disabled by clk_disable_unused() as the

Re: [Xen-devel] [PATCH v2] xen/arm: register clocks used by the hypervisor

2016-07-05 Thread Mark Rutland
Hi, On Tue, Jul 05, 2016 at 08:50:23AM +0200, Dirk Behme wrote: > Some clocks might be used by the Xen hypervisor and not by the Linux > kernel. If these are not registered by the Linux kernel, they might be > disabled by clk_disable_unused() as the kernel doesn't know that they > are used. The cl

[Xen-devel] [PATCH v2] xen/arm: register clocks used by the hypervisor

2016-07-04 Thread Dirk Behme
Some clocks might be used by the Xen hypervisor and not by the Linux kernel. If these are not registered by the Linux kernel, they might be disabled by clk_disable_unused() as the kernel doesn't know that they are used. The clock of the serial console handled by Xen is one example for this. It migh

Re: [Xen-devel] [PATCH v2] xen/arm: register clocks used by the hypervisor

2016-06-30 Thread Julien Grall
Hi, On 30/06/16 16:18, Mark Rutland wrote: On Thu, Jun 30, 2016 at 04:56:40PM +0200, Dirk Behme wrote: On 30.06.2016 16:21, Mark Rutland wrote: On Thu, Jun 30, 2016 at 12:32:32PM +0200, Dirk Behme wrote: +- clocks: one or more clocks to be registered. + Xen hypervisor drivers might replace n

Re: [Xen-devel] [PATCH v2] xen/arm: register clocks used by the hypervisor

2016-06-30 Thread Mark Rutland
Hi, On Thu, Jun 30, 2016 at 04:56:40PM +0200, Dirk Behme wrote: > On 30.06.2016 16:21, Mark Rutland wrote: > >On Thu, Jun 30, 2016 at 12:32:32PM +0200, Dirk Behme wrote: > >>+- clocks: one or more clocks to be registered. > >>+ Xen hypervisor drivers might replace native drivers, resulting in > >

Re: [Xen-devel] [PATCH v2] xen/arm: register clocks used by the hypervisor

2016-06-30 Thread Dirk Behme
On 30.06.2016 16:21, Mark Rutland wrote: On Thu, Jun 30, 2016 at 12:32:32PM +0200, Dirk Behme wrote: Some clocks might be used by the Xen hypervisor and not by the Linux kernel. If these are not registered by the Linux kernel, they might be disabled by clk_disable_unused() as the kernel doesn't

Re: [Xen-devel] [PATCH v2] xen/arm: register clocks used by the hypervisor

2016-06-30 Thread Mark Rutland
On Thu, Jun 30, 2016 at 12:32:32PM +0200, Dirk Behme wrote: > Some clocks might be used by the Xen hypervisor and not by the Linux > kernel. If these are not registered by the Linux kernel, they might be > disabled by clk_disable_unused() as the kernel doesn't know that they > are used. The clock o

[Xen-devel] [PATCH v2] xen/arm: register clocks used by the hypervisor

2016-06-30 Thread Dirk Behme
Some clocks might be used by the Xen hypervisor and not by the Linux kernel. If these are not registered by the Linux kernel, they might be disabled by clk_disable_unused() as the kernel doesn't know that they are used. The clock of the serial console handled by Xen is one example for this. It migh