Re: [Qemu-devel] [PATCH v3] vl.c: Support multiple CPU ranges on -numa option

2013-06-21 Thread Bandan Das
Wanlong Gao writes: > On 06/21/2013 12:02 AM, Bandan Das wrote: >> Paolo Bonzini writes: >> >>> Il 20/06/2013 15:26, Eduardo Habkost ha scritto: On Thu, Jun 20, 2013 at 11:52:42AM +0200, Paolo Bonzini wrote: > Il 20/06/2013 11:30, Igor Mammedov ha scritto: So, basically th

Re: [Qemu-devel] [PATCH v3] vl.c: Support multiple CPU ranges on -numa option

2013-06-21 Thread Wanlong Gao
On 06/21/2013 12:02 AM, Bandan Das wrote: > Paolo Bonzini writes: > >> Il 20/06/2013 15:26, Eduardo Habkost ha scritto: >>> On Thu, Jun 20, 2013 at 11:52:42AM +0200, Paolo Bonzini wrote: Il 20/06/2013 11:30, Igor Mammedov ha scritto: >>> So, basically the format seemed easier to work

Re: [Qemu-devel] [PATCH v3] vl.c: Support multiple CPU ranges on -numa option

2013-06-20 Thread Bandan Das
Paolo Bonzini writes: > Il 20/06/2013 15:26, Eduardo Habkost ha scritto: >> On Thu, Jun 20, 2013 at 11:52:42AM +0200, Paolo Bonzini wrote: >>> Il 20/06/2013 11:30, Igor Mammedov ha scritto: >> So, basically the format seemed easier to work with if we are >> thinking >> o

Re: [Qemu-devel] [PATCH v3] vl.c: Support multiple CPU ranges on -numa option

2013-06-20 Thread Paolo Bonzini
Il 20/06/2013 15:26, Eduardo Habkost ha scritto: > On Thu, Jun 20, 2013 at 11:52:42AM +0200, Paolo Bonzini wrote: >> Il 20/06/2013 11:30, Igor Mammedov ha scritto: > So, basically the format seemed easier to work with if we are > thinking > of using QemuOpts for -numa. Usi

Re: [Qemu-devel] [PATCH v3] vl.c: Support multiple CPU ranges on -numa option

2013-06-20 Thread Eduardo Habkost
On Thu, Jun 20, 2013 at 11:52:42AM +0200, Paolo Bonzini wrote: > Il 20/06/2013 11:30, Igor Mammedov ha scritto: > > > > So, basically the format seemed easier to work with if we are > > > > thinking > > > > of using QemuOpts for -numa. Using -cpu rather than cpus probably > >

Re: [Qemu-devel] [PATCH v3] vl.c: Support multiple CPU ranges on -numa option

2013-06-20 Thread Paolo Bonzini
Il 20/06/2013 13:34, Michael Tokarev ha scritto: > 20.06.2013 13:52, Paolo Bonzini wrote: >> Il 20/06/2013 11:30, Igor Mammedov ha scritto: > libvirt uses the "cpus" option already, so we have to keep it working. >>> Sure, we can leave it as it's now for some time while a new interface is >>> i

Re: [Qemu-devel] [PATCH v3] vl.c: Support multiple CPU ranges on -numa option

2013-06-20 Thread Michael Tokarev
20.06.2013 13:52, Paolo Bonzini wrote: > Il 20/06/2013 11:30, Igor Mammedov ha scritto: libvirt uses the "cpus" option already, so we have to keep it working. >> Sure, we can leave it as it's now for some time while a new interface is >> introduced/adopted. And than later deprecate "cpus". >

Re: [Qemu-devel] [PATCH v3] vl.c: Support multiple CPU ranges on -numa option

2013-06-20 Thread Igor Mammedov
On Wed, 19 Jun 2013 10:26:42 -0300 Eduardo Habkost wrote: > On Wed, Jun 19, 2013 at 01:42:52PM +0200, Igor Mammedov wrote: > > On Tue, 18 Jun 2013 16:09:49 -0400 > > Bandan Das wrote: > > > > > > > > This allows us to use the cpu property multiple times > > > to specify multiple cpu (ranges) t

Re: [Qemu-devel] [PATCH v3] vl.c: Support multiple CPU ranges on -numa option

2013-06-20 Thread Paolo Bonzini
Il 20/06/2013 11:30, Igor Mammedov ha scritto: > > > So, basically the format seemed easier to work with if we are > > > thinking > > > of using QemuOpts for -numa. Using -cpu rather than cpus probably > > > makes it less ambiguous as well IMO. However, it's probably not a >>

Re: [Qemu-devel] [PATCH v3] vl.c: Support multiple CPU ranges on -numa option

2013-06-19 Thread Eduardo Habkost
On Wed, Jun 19, 2013 at 01:42:52PM +0200, Igor Mammedov wrote: > On Tue, 18 Jun 2013 16:09:49 -0400 > Bandan Das wrote: > > > > > This allows us to use the cpu property multiple times > > to specify multiple cpu (ranges) to the -numa option : > > > > -numa node,cpu=1,cpu=2,cpu=4 > > or > > -num

Re: [Qemu-devel] [PATCH v3] vl.c: Support multiple CPU ranges on -numa option

2013-06-19 Thread Igor Mammedov
On Tue, 18 Jun 2013 16:09:49 -0400 Bandan Das wrote: > > This allows us to use the cpu property multiple times > to specify multiple cpu (ranges) to the -numa option : > > -numa node,cpu=1,cpu=2,cpu=4 > or > -numa node,cpu=1-3,cpu=5 > > Signed-off-by: Bandan Das > --- > v3: Convert to using

[Qemu-devel] [PATCH v3] vl.c: Support multiple CPU ranges on -numa option

2013-06-18 Thread Bandan Das
This allows us to use the cpu property multiple times to specify multiple cpu (ranges) to the -numa option : -numa node,cpu=1,cpu=2,cpu=4 or -numa node,cpu=1-3,cpu=5 Signed-off-by: Bandan Das --- v3: Convert to using QemuOpts Use -cpu rather than -cpus which probably probably makes it more m