Re: [Qemu-devel] [PATCH v2] numa, spapr: align default numa node memory size to 256MB

2017-03-21 Thread Eduardo Habkost
On Tue, Mar 21, 2017 at 12:03:21PM +1100, David Gibson wrote: > On Mon, Mar 20, 2017 at 04:11:14PM -0300, Eduardo Habkost wrote: > > On Mon, Mar 20, 2017 at 03:12:44PM +0100, Laurent Vivier wrote: > > > Since commit 224245b ("spapr: Add LMB DR connectors"), NUMA node > > > memory size must be align

Re: [Qemu-devel] [PATCH v2] numa, spapr: align default numa node memory size to 256MB

2017-03-20 Thread David Gibson
On Mon, Mar 20, 2017 at 04:11:14PM -0300, Eduardo Habkost wrote: > On Mon, Mar 20, 2017 at 03:12:44PM +0100, Laurent Vivier wrote: > > Since commit 224245b ("spapr: Add LMB DR connectors"), NUMA node > > memory size must be aligned to 256MB (SPAPR_MEMORY_BLOCK_SIZE). > > > > But when "-numa" optio

Re: [Qemu-devel] [PATCH v2] numa, spapr: align default numa node memory size to 256MB

2017-03-20 Thread Eduardo Habkost
On Mon, Mar 20, 2017 at 03:12:44PM +0100, Laurent Vivier wrote: > Since commit 224245b ("spapr: Add LMB DR connectors"), NUMA node > memory size must be aligned to 256MB (SPAPR_MEMORY_BLOCK_SIZE). > > But when "-numa" option is provided without "mem" parameter, > the memory is equally divided betw

[Qemu-devel] [PATCH v2] numa, spapr: align default numa node memory size to 256MB

2017-03-20 Thread Laurent Vivier
Since commit 224245b ("spapr: Add LMB DR connectors"), NUMA node memory size must be aligned to 256MB (SPAPR_MEMORY_BLOCK_SIZE). But when "-numa" option is provided without "mem" parameter, the memory is equally divided between nodes, but 8MB aligned. This can be not valid for pseries. In that ca