Re: [Qemu-devel] [PATCH v3 0/6] spapr: rework memory nodes

[Alexey Kardashevskiy <aik@ozlabs.ru>]

On 07/03/2014 01:10 PM, Alexey Kardashevskiy wrote:
> c4177479 "spapr: make sure RMA is in first mode of first memory node"
> introduced regression which prevents from running guests with memoryless
> NUMA node#0 which may happen on real POWER8 boxes and which would make
> sense to debug in QEMU.
> 
> This patchset aim is to fix that and also fix various code problems in
> memory nodes generation.
> 
> These 2 patches could be merged (the resulting patch looks rather ugly):
> spapr: Use DT memory node rendering helper for other nodes
> spapr: Move DT memory node rendering to a helper
> 
> 
> Alex, there are "numa: enable sparse node numbering ..." patches from Nish,
> which set can go first so the other could rebase on top of it? Thanks!


Ping. This is for 2.2 indeed.


> 
> 
> 
> Changes:
> v3:
> * fixed bug with ram_size bigger than the only NUMA node
> * fixed bug with 64bit addresses in memory node creation loop
> 
> v2:
> * minor cosmetic change in spapr_node0_size()
> * spapr_populate_memory() fixed to work in a no-numa config
> * patch changing max numa nodes is removed
> 
> Please comment. Thanks!
> 
> 
> 
> 
> Alexey Kardashevskiy (6):
>   spapr: Move DT memory node rendering to a helper
>   spapr: Use DT memory node rendering helper for other nodes
>   spapr: Refactor spapr_populate_memory() to allow memoryless nodes
>   spapr: Split memory nodes to power-of-two blocks
>   spapr: Add a helper for node0_size calculation
>   spapr: Fix ibm,associativity for memory nodes
> 
>  hw/ppc/spapr.c | 111 ++++++++++++++++++++++++++++++++-------------------------
>  1 file changed, 63 insertions(+), 48 deletions(-)
> 


-- 
Alexey