Re: [PATCH] hw/arm/boot: Use NUMA node ID in memory node name

2021-06-23 Thread Gavin Shan
On 6/23/21 6:16 PM, Andrew Jones wrote: On Wed, Jun 23, 2021 at 10:07:36AM +0200, Andrew Jones wrote: On Wed, Jun 23, 2021 at 02:43:49PM +1000, Gavin Shan wrote: On 6/22/21 5:13 PM, Andrew Jones wrote: On Tue, Jun 22, 2021 at 06:53:41PM +1000, Gavin Shan wrote: On 6/3/21 2:48 PM, Gavin Shan w

Re: [PATCH] hw/arm/boot: Use NUMA node ID in memory node name

2021-06-23 Thread Andrew Jones
On Wed, Jun 23, 2021 at 10:07:36AM +0200, Andrew Jones wrote: > On Wed, Jun 23, 2021 at 02:43:49PM +1000, Gavin Shan wrote: > > Hi Drew, > > > > On 6/22/21 5:13 PM, Andrew Jones wrote: > > > On Tue, Jun 22, 2021 at 06:53:41PM +1000, Gavin Shan wrote: > > > > On 6/3/21 2:48 PM, Gavin Shan wrote: >

Re: [PATCH] hw/arm/boot: Use NUMA node ID in memory node name

2021-06-23 Thread Andrew Jones
On Wed, Jun 23, 2021 at 02:43:49PM +1000, Gavin Shan wrote: > Hi Drew, > > On 6/22/21 5:13 PM, Andrew Jones wrote: > > On Tue, Jun 22, 2021 at 06:53:41PM +1000, Gavin Shan wrote: > > > On 6/3/21 2:48 PM, Gavin Shan wrote: > > > > On 6/2/21 9:36 PM, Andrew Jones wrote: > > > > > On Wed, Jun 02, 202

Re: [PATCH] hw/arm/boot: Use NUMA node ID in memory node name

2021-06-22 Thread Gavin Shan
Hi Drew, On 6/22/21 5:13 PM, Andrew Jones wrote: On Tue, Jun 22, 2021 at 06:53:41PM +1000, Gavin Shan wrote: On 6/3/21 2:48 PM, Gavin Shan wrote: On 6/2/21 9:36 PM, Andrew Jones wrote: On Wed, Jun 02, 2021 at 11:09:32AM +1000, Gavin Shan wrote: On 6/1/21 5:50 PM, Andrew Jones wrote: On Tue,

Re: [PATCH] hw/arm/boot: Use NUMA node ID in memory node name

2021-06-22 Thread Andrew Jones
On Tue, Jun 22, 2021 at 06:53:41PM +1000, Gavin Shan wrote: > Hi Drew, > > On 6/3/21 2:48 PM, Gavin Shan wrote: > > On 6/2/21 9:36 PM, Andrew Jones wrote: > > > On Wed, Jun 02, 2021 at 11:09:32AM +1000, Gavin Shan wrote: > > > > On 6/1/21 5:50 PM, Andrew Jones wrote: > > > > > On Tue, Jun 01, 2021

Re: [PATCH] hw/arm/boot: Use NUMA node ID in memory node name

2021-06-21 Thread Gavin Shan
Hi Drew, On 6/3/21 2:48 PM, Gavin Shan wrote: On 6/2/21 9:36 PM, Andrew Jones wrote: On Wed, Jun 02, 2021 at 11:09:32AM +1000, Gavin Shan wrote: On 6/1/21 5:50 PM, Andrew Jones wrote: On Tue, Jun 01, 2021 at 03:30:04PM +0800, Gavin Shan wrote: We possibly populate empty nodes where memory is

Re: [PATCH] hw/arm/boot: Use NUMA node ID in memory node name

2021-06-02 Thread Gavin Shan
Hi Drew, On 6/2/21 9:36 PM, Andrew Jones wrote: On Wed, Jun 02, 2021 at 11:09:32AM +1000, Gavin Shan wrote: On 6/1/21 5:50 PM, Andrew Jones wrote: On Tue, Jun 01, 2021 at 03:30:04PM +0800, Gavin Shan wrote: We possibly populate empty nodes where memory isn't included and might be hot added at

Re: [PATCH] hw/arm/boot: Use NUMA node ID in memory node name

2021-06-02 Thread Andrew Jones
On Wed, Jun 02, 2021 at 11:09:32AM +1000, Gavin Shan wrote: > Hi Drew, > > On 6/1/21 5:50 PM, Andrew Jones wrote: > > On Tue, Jun 01, 2021 at 03:30:04PM +0800, Gavin Shan wrote: > > > We possibly populate empty nodes where memory isn't included and might > > > be hot added at late time. The FDT me

Re: [PATCH] hw/arm/boot: Use NUMA node ID in memory node name

2021-06-01 Thread Gavin Shan
Hi Drew, On 6/1/21 5:50 PM, Andrew Jones wrote: On Tue, Jun 01, 2021 at 03:30:04PM +0800, Gavin Shan wrote: We possibly populate empty nodes where memory isn't included and might be hot added at late time. The FDT memory nodes can't be created due to conflicts on their names if multiple empty n

Re: [PATCH] hw/arm/boot: Use NUMA node ID in memory node name

2021-06-01 Thread Andrew Jones
On Tue, Jun 01, 2021 at 03:30:04PM +0800, Gavin Shan wrote: > We possibly populate empty nodes where memory isn't included and might > be hot added at late time. The FDT memory nodes can't be created due > to conflicts on their names if multiple empty nodes are specified. > For example, the VM fail

[PATCH] hw/arm/boot: Use NUMA node ID in memory node name

2021-05-31 Thread Gavin Shan
We possibly populate empty nodes where memory isn't included and might be hot added at late time. The FDT memory nodes can't be created due to conflicts on their names if multiple empty nodes are specified. For example, the VM fails to start with the following error messages. /home/gavin/sandbox