Re: [PATCH v2] xen/arm: disable cpuidle and cpufreq when linux is running as dom0

2013-07-21 Thread Stefano Stabellini 
On Sun, 21 Jul 2013, Stefano Stabellini wrote:
> On Fri, 19 Jul 2013, Julien Grall wrote:
> > When linux is running as dom0, Xen doesn't show the physical cpu but a
> > virtual CPU.
> > On some ARM SOC (for instance the exynos 5250), linux registers callbacks
> > for cpuidle and cpufreq. When these callbacks are called, they will modify
> > directly the physical cpu not the virtual one. It can impact the whole board
> > instead of only dom0.
> > 
> > Signed-off-by: Julien Grall 
> > 
> 
> Acked-by: Stefano Stabellini 

I added the patch to my queue
--
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/

Re: [PATCH v2] xen/arm: disable cpuidle and cpufreq when linux is running as dom0

2013-07-21 Thread Stefano Stabellini 
On Fri, 19 Jul 2013, Julien Grall wrote:
> When linux is running as dom0, Xen doesn't show the physical cpu but a
> virtual CPU.
> On some ARM SOC (for instance the exynos 5250), linux registers callbacks
> for cpuidle and cpufreq. When these callbacks are called, they will modify
> directly the physical cpu not the virtual one. It can impact the whole board
> instead of only dom0.
> 
> Signed-off-by: Julien Grall 
> 

Acked-by: Stefano Stabellini 


> Changes in v2:
> - Disable cpufreq
> ---
>  arch/arm/xen/enlighten.c |8 
>  1 file changed, 8 insertions(+)
> 
> diff --git a/arch/arm/xen/enlighten.c b/arch/arm/xen/enlighten.c
> index 49839d8..af82792 100644
> --- a/arch/arm/xen/enlighten.c
> +++ b/arch/arm/xen/enlighten.c
> @@ -24,6 +24,8 @@
>  #include 
>  #include 
>  #include 
> +#include 
> +#include 
>  
>  #include 
>  
> @@ -292,6 +294,12 @@ static int __init xen_pm_init(void)
>  {
>   pm_power_off = xen_power_off;
>   arm_pm_restart = xen_restart;
> + /*
> +  * Making sure board specific code will not set up ops for
> +  * cpu idle and cpu freq.
> +  */
> + disable_cpuidle();
> + disable_cpufreq();
>  
>   return 0;
>  }
> -- 
> 1.7.10.4
> 
--
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/

Re: [PATCH v2] xen/arm: disable cpuidle and cpufreq when linux is running as dom0

2013-07-21 Thread Stefano Stabellini 
On Fri, 19 Jul 2013, Julien Grall wrote:
 When linux is running as dom0, Xen doesn't show the physical cpu but a
 virtual CPU.
 On some ARM SOC (for instance the exynos 5250), linux registers callbacks
 for cpuidle and cpufreq. When these callbacks are called, they will modify
 directly the physical cpu not the virtual one. It can impact the whole board
 instead of only dom0.
 
 Signed-off-by: Julien Grall julien.gr...@linaro.org
 

Acked-by: Stefano Stabellini stefano.stabell...@eu.citrix.com


 Changes in v2:
 - Disable cpufreq
 ---
  arch/arm/xen/enlighten.c |8 
  1 file changed, 8 insertions(+)
 
 diff --git a/arch/arm/xen/enlighten.c b/arch/arm/xen/enlighten.c
 index 49839d8..af82792 100644
 --- a/arch/arm/xen/enlighten.c
 +++ b/arch/arm/xen/enlighten.c
 @@ -24,6 +24,8 @@
  #include linux/of.h
  #include linux/of_irq.h
  #include linux/of_address.h
 +#include linux/cpuidle.h
 +#include linux/cpufreq.h
  
  #include linux/mm.h
  
 @@ -292,6 +294,12 @@ static int __init xen_pm_init(void)
  {
   pm_power_off = xen_power_off;
   arm_pm_restart = xen_restart;
 + /*
 +  * Making sure board specific code will not set up ops for
 +  * cpu idle and cpu freq.
 +  */
 + disable_cpuidle();
 + disable_cpufreq();
  
   return 0;
  }
 -- 
 1.7.10.4
 
--
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/

Re: [PATCH v2] xen/arm: disable cpuidle and cpufreq when linux is running as dom0

2013-07-21 Thread Stefano Stabellini 
On Sun, 21 Jul 2013, Stefano Stabellini wrote:
 On Fri, 19 Jul 2013, Julien Grall wrote:
  When linux is running as dom0, Xen doesn't show the physical cpu but a
  virtual CPU.
  On some ARM SOC (for instance the exynos 5250), linux registers callbacks
  for cpuidle and cpufreq. When these callbacks are called, they will modify
  directly the physical cpu not the virtual one. It can impact the whole board
  instead of only dom0.
  
  Signed-off-by: Julien Grall julien.gr...@linaro.org
  
 
 Acked-by: Stefano Stabellini stefano.stabell...@eu.citrix.com

I added the patch to my queue
--
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/

Re: [PATCH v2] xen/arm: disable cpuidle and cpufreq when linux is running as dom0

2013-07-19 Thread Konrad Rzeszutek Wilk 
On Fri, Jul 19, 2013 at 12:27:00PM +0100, Julien Grall wrote:
> When linux is running as dom0, Xen doesn't show the physical cpu but a
> virtual CPU.
> On some ARM SOC (for instance the exynos 5250), linux registers callbacks
> for cpuidle and cpufreq. When these callbacks are called, they will modify
> directly the physical cpu not the virtual one. It can impact the whole board
> instead of only dom0.
> 
> Signed-off-by: Julien Grall 

Reviewed-by: Konrad Rzeszutek Wilk 
> 
> ---
> Changes in v2:
> - Disable cpufreq
> ---
>  arch/arm/xen/enlighten.c |8 
>  1 file changed, 8 insertions(+)
> 
> diff --git a/arch/arm/xen/enlighten.c b/arch/arm/xen/enlighten.c
> index 49839d8..af82792 100644
> --- a/arch/arm/xen/enlighten.c
> +++ b/arch/arm/xen/enlighten.c
> @@ -24,6 +24,8 @@
>  #include 
>  #include 
>  #include 
> +#include 
> +#include 
>  
>  #include 
>  
> @@ -292,6 +294,12 @@ static int __init xen_pm_init(void)
>  {
>   pm_power_off = xen_power_off;
>   arm_pm_restart = xen_restart;
> + /*
> +  * Making sure board specific code will not set up ops for
> +  * cpu idle and cpu freq.
> +  */
> + disable_cpuidle();
> + disable_cpufreq();
>  
>   return 0;
>  }
> -- 
> 1.7.10.4
> 
--
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/

[PATCH v2] xen/arm: disable cpuidle and cpufreq when linux is running as dom0

2013-07-19 Thread Julien Grall 
When linux is running as dom0, Xen doesn't show the physical cpu but a
virtual CPU.
On some ARM SOC (for instance the exynos 5250), linux registers callbacks
for cpuidle and cpufreq. When these callbacks are called, they will modify
directly the physical cpu not the virtual one. It can impact the whole board
instead of only dom0.

Signed-off-by: Julien Grall 

---
Changes in v2:
- Disable cpufreq
---
 arch/arm/xen/enlighten.c |8 
 1 file changed, 8 insertions(+)

diff --git a/arch/arm/xen/enlighten.c b/arch/arm/xen/enlighten.c
index 49839d8..af82792 100644
--- a/arch/arm/xen/enlighten.c
+++ b/arch/arm/xen/enlighten.c
@@ -24,6 +24,8 @@
 #include 
 #include 
 #include 
+#include 
+#include 
 
 #include 
 
@@ -292,6 +294,12 @@ static int __init xen_pm_init(void)
 {
pm_power_off = xen_power_off;
arm_pm_restart = xen_restart;
+   /*
+* Making sure board specific code will not set up ops for
+* cpu idle and cpu freq.
+*/
+   disable_cpuidle();
+   disable_cpufreq();
 
return 0;
 }
-- 
1.7.10.4

--
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/

[PATCH v2] xen/arm: disable cpuidle and cpufreq when linux is running as dom0

2013-07-19 Thread Julien Grall 
When linux is running as dom0, Xen doesn't show the physical cpu but a
virtual CPU.
On some ARM SOC (for instance the exynos 5250), linux registers callbacks
for cpuidle and cpufreq. When these callbacks are called, they will modify
directly the physical cpu not the virtual one. It can impact the whole board
instead of only dom0.

Signed-off-by: Julien Grall julien.gr...@linaro.org

---
Changes in v2:
- Disable cpufreq
---
 arch/arm/xen/enlighten.c |8 
 1 file changed, 8 insertions(+)

diff --git a/arch/arm/xen/enlighten.c b/arch/arm/xen/enlighten.c
index 49839d8..af82792 100644
--- a/arch/arm/xen/enlighten.c
+++ b/arch/arm/xen/enlighten.c
@@ -24,6 +24,8 @@
 #include linux/of.h
 #include linux/of_irq.h
 #include linux/of_address.h
+#include linux/cpuidle.h
+#include linux/cpufreq.h
 
 #include linux/mm.h
 
@@ -292,6 +294,12 @@ static int __init xen_pm_init(void)
 {
pm_power_off = xen_power_off;
arm_pm_restart = xen_restart;
+   /*
+* Making sure board specific code will not set up ops for
+* cpu idle and cpu freq.
+*/
+   disable_cpuidle();
+   disable_cpufreq();
 
return 0;
 }
-- 
1.7.10.4

--
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/

Re: [PATCH v2] xen/arm: disable cpuidle and cpufreq when linux is running as dom0

2013-07-19 Thread Konrad Rzeszutek Wilk 
On Fri, Jul 19, 2013 at 12:27:00PM +0100, Julien Grall wrote:
 When linux is running as dom0, Xen doesn't show the physical cpu but a
 virtual CPU.
 On some ARM SOC (for instance the exynos 5250), linux registers callbacks
 for cpuidle and cpufreq. When these callbacks are called, they will modify
 directly the physical cpu not the virtual one. It can impact the whole board
 instead of only dom0.
 
 Signed-off-by: Julien Grall julien.gr...@linaro.org

Reviewed-by: Konrad Rzeszutek Wilk konrad.w...@oracle.com
 
 ---
 Changes in v2:
 - Disable cpufreq
 ---
  arch/arm/xen/enlighten.c |8 
  1 file changed, 8 insertions(+)
 
 diff --git a/arch/arm/xen/enlighten.c b/arch/arm/xen/enlighten.c
 index 49839d8..af82792 100644
 --- a/arch/arm/xen/enlighten.c
 +++ b/arch/arm/xen/enlighten.c
 @@ -24,6 +24,8 @@
  #include linux/of.h
  #include linux/of_irq.h
  #include linux/of_address.h
 +#include linux/cpuidle.h
 +#include linux/cpufreq.h
  
  #include linux/mm.h
  
 @@ -292,6 +294,12 @@ static int __init xen_pm_init(void)
  {
   pm_power_off = xen_power_off;
   arm_pm_restart = xen_restart;
 + /*
 +  * Making sure board specific code will not set up ops for
 +  * cpu idle and cpu freq.
 +  */
 + disable_cpuidle();
 + disable_cpufreq();
  
   return 0;
  }
 -- 
 1.7.10.4
 
--
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/