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

2013-07-16 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 because they

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 to newer qemu.

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 b...@redhat.com This allows us to use the cpus property multiple times to specify multiple cpu (ranges) to the -numa option

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 b...@redhat.com 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-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 b...@redhat.com This allows us to use the cpus property multiple times to specify multiple cpu

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

2013-07-08 Thread Anthony Liguori
Eric Blake ebl...@redhat.com 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 b...@redhat.com This allows us to use the cpus property multiple times to specify multiple cpu (ranges) to the -numa option :

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 ebl...@redhat.com 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 b...@redhat.com This allows us to use the cpus property multiple times to

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's suggestion like: -numa

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 b...@redhat.com 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

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

2013-07-04 Thread Wanlong Gao
From: Bandan Das b...@redhat.com 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