Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933203AbcJQIvN (ORCPT ); Mon, 17 Oct 2016 04:51:13 -0400 Received: from szxga01-in.huawei.com ([58.251.152.64]:6791 "EHLO szxga01-in.huawei.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932385AbcJQIvL (ORCPT ); Mon, 17 Oct 2016 04:51:11 -0400 Subject: Re: aarch64 ACPI boot regressed by commit 7ba5f605f3a0 ("arm64/numa: remove the limitation that cpu0 must bind to node0") To: Laszlo Ersek , Lorenzo Pieralisi References: <4a64cd93-5ead-aad6-1057-f42224d65b43@redhat.com> <20161014080524.4hm2b4p373r7rhel@hawk.localdomain> <04f22a79-301b-f05b-033d-c7a24c9f4084@redhat.com> <20161014154231.GA4411@red-moon> CC: Andrew Jones , Will Deacon , Hanjun Guo , main kernel list , , "Ard Biesheuvel" , Shannon Zhao , Wei Huang From: "Leizhen (ThunderTown)" Message-ID: <580485F5.9080207@huawei.com> Date: Mon, 17 Oct 2016 16:04:05 +0800 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:38.0) Gecko/20100101 Thunderbird/38.5.1 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit X-Originating-IP: [10.177.23.164] X-CFilter-Loop: Reflected Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1129 Lines: 33 >> diff --git a/arch/arm64/kernel/smp.c b/arch/arm64/kernel/smp.c >> index d3f151cfd4a1..8507703dabe4 100644 >> --- a/arch/arm64/kernel/smp.c >> +++ b/arch/arm64/kernel/smp.c >> @@ -544,6 +544,7 @@ acpi_map_gic_cpu_interface(struct acpi_madt_generic_interrupt *processor) >> return; >> } >> bootcpu_valid = true; >> + early_map_cpu_to_node(0, acpi_numa_get_nid(0, hwid)); >> return; >> } >> > > Anyway, your patch works with both the two-node NUMA configuration Drew suggested for testing, and with the single-node config that I originally used for the bisection. Therefore: > > Tested-by: Laszlo Ersek > Reported-by: Laszlo Ersek > > Thank you very much for the quick bugfix! And, I think your patch (when you send it for real) should carry I'm so sorry about this. My patch series prepared before ACPI NUMA upstreamed, and forgot considering it in later. > > Fixes: 7ba5f605f3a0d9495aad539eeb8346d726dfc183 > > too, because it supplies the cpu#0<->node#xxx association that 7ba5f605f3a0 removed not just for DT, but also for ACPI. > > Cheers! > Laszlo > > . >