Re: [PATCH v3 1/2] numa: Require distance map when empty node exists

2021-10-15 Thread Gavin Shan
Hi Drew, On 10/15/21 7:33 PM, Andrew Jones wrote: On Fri, Oct 15, 2021 at 07:22:05PM +1100, Gavin Shan wrote: It's possible that the empty NUMA nodes aren't referred by any CPUs, as the following command line indicate. In this case, the empty NUMA node IDs aren't existing in device-tree CPU nod

Re: [PATCH v3 1/2] numa: Require distance map when empty node exists

2021-10-15 Thread Andrew Jones
On Fri, Oct 15, 2021 at 07:22:05PM +1100, Gavin Shan wrote: > It's possible that the empty NUMA nodes aren't referred by any CPUs, > as the following command line indicate. In this case, the empty NUMA > node IDs aren't existing in device-tree CPU nodes. So we still need > the distance-map. Ah, in

Re: [PATCH v3 1/2] numa: Require distance map when empty node exists

2021-10-15 Thread Gavin Shan
Hi Drew, On 10/15/21 2:36 AM, Andrew Jones wrote: On Thu, Oct 14, 2021 at 05:14:17PM +0200, Igor Mammedov wrote: On Wed, 13 Oct 2021 14:28:40 +0200 Andrew Jones wrote: On Wed, Oct 13, 2021 at 02:11:25PM +0200, Andrew Jones wrote: On Wed, Oct 13, 2021 at 01:53:46PM +0200, Igor Mammedov wrote:

Re: [PATCH v3 1/2] numa: Require distance map when empty node exists

2021-10-14 Thread Andrew Jones
On Thu, Oct 14, 2021 at 05:14:17PM +0200, Igor Mammedov wrote: > On Wed, 13 Oct 2021 14:28:40 +0200 > Andrew Jones wrote: > > > On Wed, Oct 13, 2021 at 02:11:25PM +0200, Andrew Jones wrote: > > > On Wed, Oct 13, 2021 at 01:53:46PM +0200, Igor Mammedov wrote: > > > > On Wed, 13 Oct 2021 13:35:44

Re: [PATCH v3 1/2] numa: Require distance map when empty node exists

2021-10-14 Thread Igor Mammedov
On Wed, 13 Oct 2021 14:28:40 +0200 Andrew Jones wrote: > On Wed, Oct 13, 2021 at 02:11:25PM +0200, Andrew Jones wrote: > > On Wed, Oct 13, 2021 at 01:53:46PM +0200, Igor Mammedov wrote: > > > On Wed, 13 Oct 2021 13:35:44 +0200 > > > Andrew Jones wrote: > > > > > > > On Wed, Oct 13, 2021 at

Re: [PATCH v3 1/2] numa: Require distance map when empty node exists

2021-10-13 Thread Andrew Jones
On Wed, Oct 13, 2021 at 02:11:25PM +0200, Andrew Jones wrote: > On Wed, Oct 13, 2021 at 01:53:46PM +0200, Igor Mammedov wrote: > > On Wed, 13 Oct 2021 13:35:44 +0200 > > Andrew Jones wrote: > > > > > On Wed, Oct 13, 2021 at 01:30:11PM +0200, Igor Mammedov wrote: > > > > On Wed, 13 Oct 2021 12:58:

Re: [PATCH v3 1/2] numa: Require distance map when empty node exists

2021-10-13 Thread Andrew Jones
On Wed, Oct 13, 2021 at 01:53:46PM +0200, Igor Mammedov wrote: > On Wed, 13 Oct 2021 13:35:44 +0200 > Andrew Jones wrote: > > > On Wed, Oct 13, 2021 at 01:30:11PM +0200, Igor Mammedov wrote: > > > On Wed, 13 Oct 2021 12:58:04 +0800 > > > Gavin Shan wrote: > > > > > > > The following option is

Re: [PATCH v3 1/2] numa: Require distance map when empty node exists

2021-10-13 Thread Igor Mammedov
On Wed, 13 Oct 2021 13:35:44 +0200 Andrew Jones wrote: > On Wed, Oct 13, 2021 at 01:30:11PM +0200, Igor Mammedov wrote: > > On Wed, 13 Oct 2021 12:58:04 +0800 > > Gavin Shan wrote: > > > > > The following option is used to specify the distance map. It's > > > possible the option isn't provide

Re: [PATCH v3 1/2] numa: Require distance map when empty node exists

2021-10-13 Thread Andrew Jones
On Wed, Oct 13, 2021 at 01:30:11PM +0200, Igor Mammedov wrote: > On Wed, 13 Oct 2021 12:58:04 +0800 > Gavin Shan wrote: > > > The following option is used to specify the distance map. It's > > possible the option isn't provided by user. In this case, the > > distance map isn't populated and expos

Re: [PATCH v3 1/2] numa: Require distance map when empty node exists

2021-10-13 Thread Igor Mammedov
On Wed, 13 Oct 2021 12:58:04 +0800 Gavin Shan wrote: > The following option is used to specify the distance map. It's > possible the option isn't provided by user. In this case, the > distance map isn't populated and exposed to platform. On the > other hand, the empty NUMA node, where no memory r

Re: [PATCH v3 1/2] numa: Require distance map when empty node exists

2021-10-12 Thread Andrew Jones
On Wed, Oct 13, 2021 at 12:58:04PM +0800, Gavin Shan wrote: > The following option is used to specify the distance map. It's > possible the option isn't provided by user. In this case, the > distance map isn't populated and exposed to platform. On the > other hand, the empty NUMA node, where no mem

[PATCH v3 1/2] numa: Require distance map when empty node exists

2021-10-12 Thread Gavin Shan
The following option is used to specify the distance map. It's possible the option isn't provided by user. In this case, the distance map isn't populated and exposed to platform. On the other hand, the empty NUMA node, where no memory resides, is allowed on platforms like ARM64 virt. For these empt