Re: [Qemu-devel] [PATCH 7/7] numa: Allow empty nodes

2014-06-16 Thread Nishanth Aravamudan
On 16.06.2014 [17:31:24 -0300], Eduardo Habkost wrote: > On Mon, Jun 16, 2014 at 05:11:40PM -0300, Eduardo Habkost wrote: > [...] > > Wait, is the node ID visible to the guest at all? I believe it is a > > QEMU-internal thing, just to allow the NUMA nodes to be ordered in the > > command-line. I wo

Re: [Qemu-devel] [PATCH 7/7] numa: Allow empty nodes

2014-06-16 Thread Nishanth Aravamudan
On 16.06.2014 [17:11:40 -0300], Eduardo Habkost wrote: > On Mon, Jun 16, 2014 at 11:49:46AM -0700, Nishanth Aravamudan wrote: > > On 16.06.2014 [13:15:00 -0300], Eduardo Habkost wrote: > > > On Mon, Jun 16, 2014 at 05:53:53PM +1000, Alexey Kardashevskiy wrote: > > > > Currently NUMA nodes must go c

Re: [Qemu-devel] [PATCH 7/7] numa: Allow empty nodes

2014-06-16 Thread Eduardo Habkost
On Mon, Jun 16, 2014 at 05:11:40PM -0300, Eduardo Habkost wrote: [...] > Wait, is the node ID visible to the guest at all? I believe it is a > QEMU-internal thing, just to allow the NUMA nodes to be ordered in the > command-line. I would even claim that the parameter is useless and > shouldn't have

Re: [Qemu-devel] [PATCH 7/7] numa: Allow empty nodes

2014-06-16 Thread Eduardo Habkost
On Mon, Jun 16, 2014 at 11:49:46AM -0700, Nishanth Aravamudan wrote: > On 16.06.2014 [13:15:00 -0300], Eduardo Habkost wrote: > > On Mon, Jun 16, 2014 at 05:53:53PM +1000, Alexey Kardashevskiy wrote: > > > Currently NUMA nodes must go consequently and QEMU ignores nodes > > > with @nodeid bigger th

Re: [Qemu-devel] [PATCH 7/7] numa: Allow empty nodes

2014-06-16 Thread Nishanth Aravamudan
On 16.06.2014 [13:15:00 -0300], Eduardo Habkost wrote: > On Mon, Jun 16, 2014 at 05:53:53PM +1000, Alexey Kardashevskiy wrote: > > Currently NUMA nodes must go consequently and QEMU ignores nodes > > with @nodeid bigger than the number of NUMA nodes in the command line. > > Why would somebody need

Re: [Qemu-devel] [PATCH 7/7] numa: Allow empty nodes

2014-06-16 Thread Eduardo Habkost
On Mon, Jun 16, 2014 at 05:53:53PM +1000, Alexey Kardashevskiy wrote: > Currently NUMA nodes must go consequently and QEMU ignores nodes > with @nodeid bigger than the number of NUMA nodes in the command line. Why would somebody need a NUMA node with nodeid bigger than the number of NUMA nodes? NU

[Qemu-devel] [PATCH 7/7] numa: Allow empty nodes

2014-06-16 Thread Alexey Kardashevskiy
Currently NUMA nodes must go consequently and QEMU ignores nodes with @nodeid bigger than the number of NUMA nodes in the command line. This prevents us from creating memory-less nodes which is possible situation in POWERPC under pHyp or Sapphire. This makes nb_numa_nodes a total number of nodes o