Re: [Qemu-devel] [PATCH v2] Fixes related to processing of qemu's -numa option

2012-06-25 Thread Vinod, Chegu
-Original Message- From: Eduardo Habkost [mailto:ehabk...@redhat.com] Sent: Monday, June 25, 2012 1:01 PM To: Vinod, Chegu Cc: Hada, Craig M; Hull, Jim; qemu-devel@nongnu.org; k...@vger.kernel.org Subject: Re: [Qemu-devel] [PATCH v2] Fixes related to processing of qemu's -numa o

Re: [Qemu-devel] [PATCH v2] Fixes related to processing of qemu's -numa option

2012-06-25 Thread Eduardo Habkost
Just found another issue: On Wed, Jun 20, 2012 at 05:33:29PM -0300, Eduardo Habkost wrote: [...] > > @@ -970,27 +974,24 @@ static void numa_add(const char *optarg) > > } > > node_mem[nodenr] = sval; > > } > > -if (get_param_value(option, 128, "cpus", opta

Re: [Qemu-devel] [PATCH v2] Fixes related to processing of qemu's -numa option

2012-06-20 Thread Eduardo Habkost
(removed r...@hydra11.kio from CC) On Tue, Jun 19, 2012 at 03:45:27PM -0700, Chegu Vinod wrote: > From: root > > Changes since v1: > > > - Use bitmap functions that are already in qemu (instead >of cpu_set_t macro's) > - Added a check for endvalue >= max_cpus. > - Fix to

[Qemu-devel] [PATCH v2] Fixes related to processing of qemu's -numa option

2012-06-19 Thread Chegu Vinod
From: root Changes since v1: - Use bitmap functions that are already in qemu (instead of cpu_set_t macro's) - Added a check for endvalue >= max_cpus. - Fix to address the round-robbing assignment (for the case when cpu's are not explicitly specified) Note: Continuing t