From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932643AbcFJKli (ORCPT ); Fri, 10 Jun 2016 06:41:38 -0400 Received: from mail-bn1bon0072.outbound.protection.outlook.com ([157.56.111.72]:56736 "EHLO na01-bn1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S932123AbcFJKlf (ORCPT ); Fri, 10 Jun 2016 06:41:35 -0400 Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Robert.Richter@cavium.com; Date: Fri, 10 Jun 2016 12:26:15 +0200 From: Robert Richter To: "Rafael J. Wysocki" CC: David Daney , Will Deacon , , Mark Rutland , Catalin Marinas , Tony Luck , Fenghua Yu , Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" , , Len Brown , Rob Herring , Frank Rowand , Grant Likely , Robert Moore , Lv Zheng , Hanjun Guo , Marc Zyngier , , , , , David Daney Subject: Re: [PATCH v7 00/15] ACPI NUMA support for ARM64 Message-ID: <20160610102615.GI1535@rric.localdomain> References: <1464129345-18985-1-git-send-email-ddaney.cavm@gmail.com> <6187968.ZZVJRAFzpv@vostro.rjw.lan> <20160610102005.GH1535@rric.localdomain> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <20160610102005.GH1535@rric.localdomain> User-Agent: Mutt/1.5.23 (2014-03-12) X-Originating-IP: [92.224.192.171] X-ClientProxiedBy: AM5PR0901CA0014.eurprd09.prod.outlook.com (10.164.186.152) To SN1PR07MB2351.namprd07.prod.outlook.com (10.169.127.17) X-MS-Office365-Filtering-Correlation-Id: 175a4494-eb57-4e7b-0d6b-08d39119afb3 X-Microsoft-Exchange-Diagnostics: 1;SN1PR07MB2351;2:hK/JBffJG6N7E3IVqyEIQppeuQH5nmXgDLajtVS2SNAM3F+rIVvhQe8p0YOlZ4TXfbQpvC31vvAXK61e4idDTA9jtt1m39DKjq3OkdRZUkNcdxnySifDCalqmGAsfc2LygZeyE/PbaczoqRNhGhVXsVVsabbgHM9tGx2icAGWR+4wr3ZG5mx0y7iULVAP2kG;3:l+OCW35WQ5hvgzzG4qGM9vfWB5PNN9rQXhCQVApY6CTAS/FWRJD/kvnYNjaax1r1uohWjyGGNlwON6oapvqcL3pjQ1mpUj7rJWo+kO8aCXrauGfi9DBz+Ciohxf7CemH;25:z+OXUMxVp0/1207AcfvDcoSM6iO0u1G5f7Zoazl/n5iXvgG72h+fZHjY3ckN+Rm0rSk+WQDmBYdFTm795BKO5XcbOskcdBRE/QTC/TtnNpeMFnSgotJblqPUP1RygqusmAESuPn8E7IebK/cUD0LiF/GsrB+R8pOsHeUHh3qngM/11vRzZyKXFwGq88X7zKDiFk6TVGrD26CrPhuEJTqcaNWvUj1bU9/J3zpzxHAjWaDBoGIybh9vNREebalBuF/oldtuHMKJDi6H1dDb0HLhaz0i7XYlP+15bzi9VKfMkuIkoiWmsc60OPUdxtUTLK3DNGzHrw6nZANnSDKB2igoQ8xXX1/3Y+JebKiOa5ZFfg9WDzjrLkZe8s7HWsCgZpRNVRFZvs2cgs6caSdVKxl1YqABwBYfPx3FpCP1f7+CNo= X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:SN1PR07MB2351; X-Microsoft-Exchange-Diagnostics: 1;SN1PR07MB2351;20:DZce81kSZZNyDCZVFJ16hqfY4B1gjdwDR5o6kVHsk/NMMuwSe87t77a0E37kKSVpH/RZ0PjiemuBUfNUg2cPYFs5TeDgZB91bNPoRqsfuh9GLajm4dy27djnVLWv27jIyl7jhQekePoF9PcHboOS8ms4X1dFay9koqF35I5cNih4bjlJsuaetRYHHoqEQhIRVw+SWlHC1wQZkVyxV4gPJKwqjbrjQorJUgcHN09+tCfMKnTo2THDE/0F/jePI43+vcrjMOhwPgbBFDVf4cRgQrTloqiA0VS5YXIoECxMK9MWLnpafwMvkxRwwdEACnVJbuum2gFjpL02sDCvq+7Cx0vDCrdyDtlKbQZ7HK42/6oFnPsUd1wrYsMfU+WfqY8wINM6uu1zxB3c9dac0tQsEODXS9wdFu428NxeM1wPel3YGMe/kldBiayqWHHq/TiRlT2rDXYnzDJUQxRp1ebAERwgZGrKZfETx619LZZfooQQOKaYFcDi/SeKW508ueSD;4:9FbwtdlUat/wK+0x1aM7lB5bBa/JRsZodk6sh7gNO35BatEJjUyT9bcYpWMAzW96JtmzIJS1WNx1p2Iq9wEcA1qkk86MICVOoAqsOmqrvPgYDfarPlFfMRHlCvA6Ci4S0NWZY2svLq2YcjhZ9+958V3CoG5aS8JfP9IN1agvxHvdR+WnSTITkSFU8HSZUqlbzHB/LUOsSxmHrdiJp/1GOS+K+u6Jes8iubQmE0I9BEe2LZoWOuHISK67PVqSAokEcaxmKFlM77Ne2qexPUYm7kvC8Zbvqn8BB+d53KVDbNHhXjAkSyEtwn2dpL6+NJ//bzCz5xkGcrzSOh45z1+71RSWZ/OQv8kDA8JgXxrSWkp/LT70N254UIcacnJjHhwlMYSvkSsBxnWQLlScRACqrCPH8oKjHYTP++hDv1Ztgo8= X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(788757137089); X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(601004)(2401047)(5005006)(8121501046)(10201501046)(3002001);SRVR:SN1PR07MB2351;BCL:0;PCL:0;RULEID:;SRVR:SN1PR07MB2351; X-Forefront-PRVS: 096943F07A X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10009020)(4630300001)(6009001)(24454002)(199003)(189002)(377424004)(377454003)(42186005)(50466002)(81156014)(81166006)(66066001)(8676002)(9686002)(76176999)(4326007)(4001350100001)(23726003)(47776003)(1076002)(77096005)(2950100001)(586003)(92566002)(3846002)(19580395003)(97736004)(83506001)(101416001)(68736007)(6116002)(19580405001)(86362001)(107886002)(46406003)(4001430100002)(110136002)(189998001)(33656002)(2906002)(97756001)(5004730100002)(50986999)(54356999)(105586002)(5008740100001)(106356001)(217873001);DIR:OUT;SFP:1101;SCL:1;SRVR:SN1PR07MB2351;H:rric.localdomain;FPR:;SPF:None;PTR:InfoNoRecords;A:1;MX:1;CAT:NONE;LANG:en;CAT:NONE; X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1;SN1PR07MB2351;23:9dYKD8HNtw8buAdpl/Zu3M+DF3o0XBZWk4nPBF+hi?= =?us-ascii?Q?NVX0gwSH5k8JPWwUOOxZ96z9WV/XHK4tJ9aJcJffd2iXUDsW0Fsq+OwQDxjs?= =?us-ascii?Q?Km9k0rIdpE5nviV9uvf9I/wcncHFgoP1nengapdclnoPK5UNvzdMZGYn+id2?= =?us-ascii?Q?BaRvMTLUieWZczVx2+fQ4yj3w0w4bzkwenztzgyQcmU8RfhH/ZjBra5pmXkm?= =?us-ascii?Q?ZT53zHJn8KsHBam+lbwTBiNf27BCS78qLh/chYRdzCdSChPtE4OfAvSP3iHk?= =?us-ascii?Q?o8tZdxLsfBhWWGvvMN+sAYn6NlbLKtwzSUuhLVF0C+jfBS/wuE+jLX94Yc04?= =?us-ascii?Q?e4koq1dYDaSOSRUd9FO1t6DQ+XC/XXHUM7q/2cOnEzOMfz/1h7yNJAfHO8i5?= =?us-ascii?Q?79Hf6tE9o8qUKx4q3b98Z1DF05VeNJFKAYc36uo20Dfo/4HBYezqtn6nW31C?= =?us-ascii?Q?Yt7bGa4VUB+d4TT1TLwk2+F59UmahhRqg+uBSFCNhtattI5H32G4toWvrLrM?= =?us-ascii?Q?/hkyVprS9F7OcGZPeJ76grxiDxtIyIWXcD5l4+mrzSNajYJK5KMpBawnB0ig?= =?us-ascii?Q?zdXQhnL2dfEGLNOUGClLeWnt/HiHnZekf00ofovMnepb04vqC1mVFDOwXQbl?= =?us-ascii?Q?kqmNzZHK9txdo5L1tTjt0wyZTtDCKyhKuKfW+T1smZRX38Ot7Uog6RTLd0rW?= =?us-ascii?Q?fB0DzKalIcesufYO2LN9pfSNYnlWeqSi5AMst23e+XXszuNC98SCkCYk5Ksr?= =?us-ascii?Q?ycM8FVOfazYchfLWm+rEEEAHhxxkx6dbZmID8ss8OgSzcPB/qAMnzUEJha28?= =?us-ascii?Q?iQoj+sEK+wwglhN8nsnew0oVc8RGc0tD+n90Y9xdEU8Q0a2HkIXMsJOxOGPQ?= =?us-ascii?Q?qKuGBSy+4kdgpOgFRzXwWVbsejsdAwaK0ZMQ24XNt6UvyvYInVmMhJt323Ur?= =?us-ascii?Q?DCub3s4Y+0mNP/0EeNVyCkLxI830cJPmJ6/2RjLo26UH1v+cp2ii1afTr2Ie?= =?us-ascii?Q?KN1d+eR17jJ6ruXt8MECpE+bKDdrKDBbsmSQTTe+mcMD+EpCYaOLfcYjJC6j?= =?us-ascii?Q?oU82V8UNQLwB7f62e5E4k8toM4qcbeQTbUqgUBc6Fs4zLHhVuHWqIDYd9SsS?= =?us-ascii?Q?ccrrM8RNYgUUh0ddOkQ3QhBc2Gw8m91Rud7dVcXxv8c7+V3PpFx+HTPiwjik?= =?us-ascii?Q?fYUQqIeJmFDgXKewBlwvXfYxTayp3qHxik9Pl7gvSRRQoZfKSujDW90rSOjh?= =?us-ascii?Q?O8w5su9Q/fOSyQNros=3D?= X-Microsoft-Exchange-Diagnostics: 1;SN1PR07MB2351;6:HwvD2NVOXJU6W9VK1BQToWwJyFDXrZdwiceXJbe1HxGVXYv1E3P/w4xPOcZY6ORfzP0Ruufa4lO70VXh/Ao7B/acagWX5p5ixOAB+RpaKpwxpoIYQVbb0HeV2GQA+FaXO8w5TQUlY+s8VTfIjCG48QBqzmsdLmEkUx/dSmz+mij9PAK5CEeMWdXo+BpqFbmqoYtNcRifrH+uGWnt2SVNXAXY0S5N43AB/L35TED8nbCA4Wzzi41YQkpEPKxAVvy6TMyVVBBRKph45gOR8sEjuA==;5:cl4vhjHS5uCk5dFvrUQln6/rc1QFrgiVxDjad9HjJrPtt+qZY2gJEzocr024mujB36ET3lKjTKhkLf9MbeQKVgxJNFtdTDjcVzBnPIBmyJFT8M4GaKTE3t+/vUfBBjLK+yhjzbyk7xOfEwjw4pZThQ==;24:+e2NC9nRrDnBGisqsU7lbzW4IK/dNNAQXdrtDL5BAYiXOZDJDfmRMySzbmkhJ5VUHWW8GWqpGni1X6jiU49ilJ1+H0AWXqy7cZwqXzaDpd8=;7:EmNw71gWdZGY4fv9M1yRSjhbfgea4dz8UyKP1oSRiSJSUIEf2PIeDtpTZ4eX4o4zgOqqQKcXIT4Yct6qKzat2wFZCp2qNOh6nb5lNLCegPxbn8GHx6iz0IzGMsR07U0p0Od9uAqw+hYCxxy1AoRyglSyXrOJsCvLD9qLDKet2Jq8OApG+p7LBp4hZry7/nDXpXUAHobxB5QPJY0pOtsL6Q== SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: cavium.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 10 Jun 2016 10:26:25.5452 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN1PR07MB2351 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 10.06.16 12:20:05, Robert Richter wrote: > On 04.06.16 00:07:04, Rafael J. Wysocki wrote: > > On Tuesday, May 24, 2016 03:35:30 PM David Daney wrote: > > > From: David Daney > > > > > > Rebased to Linus' master branch at commit 1d6da87a3241 ("Merge branch > > > 'drm-next' of git://people.freedesktop.org/~airlied/linux") > > > > > > ACPI 5.1 already introduced NUMA support for ARM64, which can get the > > > NUMA domain information from SRAT and SLIT table, so parse those two > > > tables to get mappings from cpu/mem to numa node configuration and > > > system locality. > > > > > > v7 updates: > > > > > > - Rebased to resolve one merge conflict. > > > > > > - New patch 13/15 to enable use of existing MADT parsing code instead > > > of adding an additional implementation of the parser. > > > > > > - Added some Acked-by headers to patches in arm64 arch directories. > > > > > > v6 updates: > > > > > > - Changed message printed in 13/14 to omit meaningless "cpu" > > > number. (noted by Dennis Chen and Hanjun Guo) > > > > > > - New patch 7/14 to print different messages for NUMA disabled case. > > > (noted by Dennis Chen) > > > > > > - Squashed two patches into 14/14. > > > > > > - Added Reviewed-by to 4/14 > > > > > > v5 updates: > > > > > > - Fixed ia64 build failure by gating some unused functions with #if > > > CONFIG_{X86,ARM64}. > > > > > > - Fixed section mismatch errors for X86 case. > > > > > > - Removed unneeded #include from some files. > > > > > > - Tested to build cleanly on ARM64, X86_64, IA64 > > > > > > v4 updates: > > > > > > - Updated from Hanjun Guo's v3 patches. > > > > > > - Rebased on top of v16 of device-tree NUMA patches. > > > > > > - Reordered some of the changes so that we don't introduce code and > > > then change it several times in the patch set. New code is > > > introduced in its final form. Code reused from x86 is first moved > > > with no change, and then a separate patch to make any needed > > > changes. > > > > > > - code that is used only by ia64, moved to architecture specific > > > files. > > > > > > v3 updates: > > > - Deep investigation about the ACPI_DEBUG_PRINT() and remvoe > > > that for acpi/numa.c (patch 2/12) > > > > > > - Remove the duplicate NULL check for table print (patch 3/12) > > > > > > - Introduce CONFIG_ACPI_HAS_NUMA_ARCH_FIXUP to remove duplicate > > > dummy function for acpi_numa_arch_fixup() > > > > > > - Solve the problem that the mapping from logical cpu to numa node > > > is wrong which spotted out by Lorenzo > > > > > > - cleanups for x86 and move acpi_numa_slit_init() and some other > > > functions to common place, then reduce the duplicate of x86 > > > and arm64 (patch 7-12/12). > > > > > > - rebased on top of 4.4 and Ganapat's v9 patch set. > > > > > > David Daney (4): > > > arm64, numa: Cleanup NUMA disabled messages. > > > acpi, numa: move bad_srat() and srat_disabled() to > > > drivers/acpi/numa.c > > > acpi, numa, srat: Improve SRAT error detection and add messages. > > > ACPI / processor: Add acpi_map_madt_entry(). > > > > > > Hanjun Guo (10): > > > acpi, numa: Use pr_fmt() instead of printk > > > acpi, numa: Replace ACPI_DEBUG_PRINT() with pr_debug() > > > acpi, numa: remove duplicate NULL check > > > acpi, numa: move acpi_numa_slit_init() to drivers/acpi/numa.c > > > arm64, numa: rework numa_add_memblk() > > > x86, acpi, numa: cleanup acpi_numa_processor_affinity_init() > > > acpi, numa: remove unneeded acpi_numa=1 > > > acpi, numa: Move acpi_numa_memory_affinity_init() to > > > drivers/acpi/numa.c > > > arm64, acpi, numa: NUMA support based on SRAT and SLIT > > > acpi, numa: Enable ACPI based NUMA on ARM64 > > > > > > Robert Richter (1): > > > acpi, numa: Move acpi_numa_arch_fixup() to ia64 only > > > > > > arch/arm64/include/asm/acpi.h | 8 ++ > > > arch/arm64/include/asm/numa.h | 2 + > > > arch/arm64/kernel/Makefile | 1 + > > > arch/arm64/kernel/acpi_numa.c | 112 +++++++++++++++++++++ > > > arch/arm64/kernel/smp.c | 2 + > > > arch/arm64/mm/numa.c | 28 ++++-- > > > arch/ia64/include/asm/acpi.h | 3 + > > > arch/ia64/kernel/acpi.c | 2 +- > > > arch/ia64/kernel/setup.c | 1 + > > > arch/x86/include/asm/acpi.h | 1 - > > > arch/x86/mm/numa.c | 2 +- > > > arch/x86/mm/srat.c | 116 +--------------------- > > > drivers/acpi/Kconfig | 4 +- > > > drivers/acpi/numa.c | 226 ++++++++++++++++++++++++++++++++---------- > > > drivers/acpi/processor_core.c | 26 ++++- > > > drivers/of/of_numa.c | 4 +- > > > include/acpi/acpi_numa.h | 4 + > > > include/acpi/processor.h | 1 + > > > include/linux/acpi.h | 18 +++- > > > 19 files changed, 369 insertions(+), 192 deletions(-) > > > create mode 100644 arch/arm64/kernel/acpi_numa.c > > > > All [1-15/15] queued up for 4.8, thanks! > > Rafael, > > I noticed patch #15 missing in your today's tree, I only found: > > d8b47fca8c23 arm64, ACPI, NUMA: NUMA support based on SRAT and SLIT > > Is this intended? It still cleanly applies. Ah, just seen Matthias' comment, I guess we need to resend a fixed version. -Robert