Re: [Xen-devel] [PATCH v6 04/31] xen/arm: Set nr_cpu_ids to available number of cpus

2015-09-09 Thread Ian Campbell 
On Mon, 2015-08-31 at 15:25 +0100, Julien Grall wrote:
> title: Set nr_cpu_ids to the number of CPUs available
> 
> On 31/08/2015 12:06, vijay.kil...@gmail.com wrote:
> > From: Vijaya Kumar K 
> > 
> > nr_cpu_ids for arm platforms is set to NR_CPUS irrespective of
> 
> s/is set/is incorrectly set/ to make clear that it's the previous 
> behavior and not the one that you are implemented.
> 
> > number of cpus supported by platform.
> 
> ^ the number
> 
> > 
> > Signed-off-by: Vijaya Kumar K 
> 
> With the typos mentioned:
> 
> Reviewed-by: Julien Grall 

Agreed, with those fixed: Acked-by: Ian Campbell 


___
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

[Xen-devel] [PATCH v6 04/31] xen/arm: Set nr_cpu_ids to available number of cpus

2015-08-31 Thread vijay . kilari 
From: Vijaya Kumar K 

nr_cpu_ids for arm platforms is set to NR_CPUS irrespective of
number of cpus supported by platform.

Signed-off-by: Vijaya Kumar K 
---
v6: - Updated nr_cpu_ids in setup.c instead of creating
  a helper function
---
 xen/arch/arm/setup.c |2 ++
 1 file changed, 2 insertions(+)

diff --git a/xen/arch/arm/setup.c b/xen/arch/arm/setup.c
index 6626eba..67b5f14 100644
--- a/xen/arch/arm/setup.c
+++ b/xen/arch/arm/setup.c
@@ -772,6 +772,8 @@ void __init start_xen(unsigned long boot_phys_offset,
 
 smp_init_cpus();
 cpus = smp_get_max_cpus();
+printk(XENLOG_INFO "SMP: Allowing %u CPUs\n", cpus);
+nr_cpu_ids = cpus;
 
 init_xen_time();
 
-- 
1.7.9.5


___
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v6 04/31] xen/arm: Set nr_cpu_ids to available number of cpus

2015-08-31 Thread Julien Grall 

title: Set nr_cpu_ids to the number of CPUs available

On 31/08/2015 12:06, vijay.kil...@gmail.com wrote:

From: Vijaya Kumar K 

nr_cpu_ids for arm platforms is set to NR_CPUS irrespective of


s/is set/is incorrectly set/ to make clear that it's the previous 
behavior and not the one that you are implemented.



number of cpus supported by platform.


^ the number



Signed-off-by: Vijaya Kumar K 


With the typos mentioned:

Reviewed-by: Julien Grall 

Regards,

--
Julien Grall

___
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel