Re: [Qemu-devel] [PATCH V4 01/10] NUMA: Support multiple CPU ranges on -numa option

2013-07-15 Thread Paolo Bonzini
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Il 15/07/2013 23:33, Eric Blake ha scritto: > Newer libvirt can be taught to append 'M' when it detects > it is talking to newer qemu. While you have a point that > it is annoying to force users to upgrade to a newer libvirt > merely b

Re: [Qemu-devel] [PATCH V4 01/10] NUMA: Support multiple CPU ranges on -numa option

2013-07-15 Thread Eric Blake
On 07/14/2013 05:34 AM, Paolo Bonzini wrote: >>> Such an incompatible change is not acceptable, as it would break >>> existing configurations. libvirt doesn't specify any suffix and expects >>> it to always mean "MB". >> >> Newer libvirt can be taught to append 'M' when it detects it is talking >>

Re: [Qemu-devel] [PATCH V4 01/10] NUMA: Support multiple CPU ranges on -numa option

2013-07-14 Thread Paolo Bonzini
Il 08/07/2013 21:02, Eric Blake ha scritto: > On 07/05/2013 12:41 PM, Eduardo Habkost wrote: >> On Thu, Jul 04, 2013 at 05:53:08PM +0800, Wanlong Gao wrote: >>> From: Bandan Das >>> >>> This allows us to use the "cpus" property multiple times >>> to specify multiple cpu (ranges) to the -numa optio

Re: [Qemu-devel] [PATCH V4 01/10] NUMA: Support multiple CPU ranges on -numa option

2013-07-08 Thread Eric Blake
On 07/08/2013 09:28 PM, Wanlong Gao wrote: > > Note that after this patch, the defalut suffix of "-numa node,mem=N" > will no longer be "M". So we must add the suffix "M" like "-numa > node,mem=NM" > when assigning "N MB" of node memory size. > > But if we follow Paolo'

Re: [Qemu-devel] [PATCH V4 01/10] NUMA: Support multiple CPU ranges on -numa option

2013-07-08 Thread Wanlong Gao
On 07/09/2013 03:25 AM, Anthony Liguori wrote: > Eric Blake writes: > >> On 07/05/2013 12:41 PM, Eduardo Habkost wrote: >>> On Thu, Jul 04, 2013 at 05:53:08PM +0800, Wanlong Gao wrote: From: Bandan Das This allows us to use the "cpus" property multiple times to specify multip

Re: [Qemu-devel] [PATCH V4 01/10] NUMA: Support multiple CPU ranges on -numa option

2013-07-08 Thread Anthony Liguori
Eric Blake writes: > On 07/05/2013 12:41 PM, Eduardo Habkost wrote: >> On Thu, Jul 04, 2013 at 05:53:08PM +0800, Wanlong Gao wrote: >>> From: Bandan Das >>> >>> This allows us to use the "cpus" property multiple times >>> to specify multiple cpu (ranges) to the -numa option : >>> >>> -numa node,

Re: [Qemu-devel] [PATCH V4 01/10] NUMA: Support multiple CPU ranges on -numa option

2013-07-08 Thread Eduardo Habkost
On Mon, Jul 08, 2013 at 01:02:41PM -0600, Eric Blake wrote: > On 07/05/2013 12:41 PM, Eduardo Habkost wrote: > > On Thu, Jul 04, 2013 at 05:53:08PM +0800, Wanlong Gao wrote: > >> From: Bandan Das > >> > >> This allows us to use the "cpus" property multiple times > >> to specify multiple cpu (range

Re: [Qemu-devel] [PATCH V4 01/10] NUMA: Support multiple CPU ranges on -numa option

2013-07-08 Thread Eric Blake
On 07/05/2013 12:41 PM, Eduardo Habkost wrote: > On Thu, Jul 04, 2013 at 05:53:08PM +0800, Wanlong Gao wrote: >> From: Bandan Das >> >> This allows us to use the "cpus" property multiple times >> to specify multiple cpu (ranges) to the -numa option : >> >> -numa node,cpus=1,cpus=2,cpus=4 >> or >>

Re: [Qemu-devel] [PATCH V4 01/10] NUMA: Support multiple CPU ranges on -numa option

2013-07-05 Thread Eduardo Habkost
On Thu, Jul 04, 2013 at 05:53:08PM +0800, Wanlong Gao wrote: > From: Bandan Das > > This allows us to use the "cpus" property multiple times > to specify multiple cpu (ranges) to the -numa option : > > -numa node,cpus=1,cpus=2,cpus=4 > or > -numa node,cpus=1-3,cpus=5 > > Note that after this pa

[Qemu-devel] [PATCH V4 01/10] NUMA: Support multiple CPU ranges on -numa option

2013-07-04 Thread Wanlong Gao
From: Bandan Das This allows us to use the "cpus" property multiple times to specify multiple cpu (ranges) to the -numa option : -numa node,cpus=1,cpus=2,cpus=4 or -numa node,cpus=1-3,cpus=5 Note that after this patch, the defalut suffix of "-numa node,mem=N" will no longer be "M". So we must a