From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932586AbcFJKfR (ORCPT ); Fri, 10 Jun 2016 06:35:17 -0400 Received: from mail-by2on0070.outbound.protection.outlook.com ([207.46.100.70]:55648 "EHLO na01-by2-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S932495AbcFJKfL (ORCPT ); Fri, 10 Jun 2016 06:35:11 -0400 X-Greylist: delayed 518 seconds by postgrey-1.27 at vger.kernel.org; Fri, 10 Jun 2016 06:35:10 EDT Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Robert.Richter@cavium.com; Date: Fri, 10 Jun 2016 12:20:05 +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: <20160610102005.GH1535@rric.localdomain> References: <1464129345-18985-1-git-send-email-ddaney.cavm@gmail.com> <6187968.ZZVJRAFzpv@vostro.rjw.lan> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <6187968.ZZVJRAFzpv@vostro.rjw.lan> User-Agent: Mutt/1.5.23 (2014-03-12) X-Originating-IP: [92.224.192.171] X-ClientProxiedBy: AM3PR03CA048.eurprd03.prod.outlook.com (10.141.191.176) To BL2PR07MB2340.namprd07.prod.outlook.com (10.167.101.18) X-MS-Office365-Filtering-Correlation-Id: 5baf1930-3790-4ec1-843d-08d39118d258 X-Microsoft-Exchange-Diagnostics: 1;BL2PR07MB2340;2:WJv7xE/aZuPYeutkWYNlx5+tf2xhyhLnSKoXlxMDmRsLz6ECK+IqkUJzLEQBxWcblRtxvtjOHzOe2aOuF1FOvENGiL1+F6PfwtGalQkNzrVmm94BM4wtu7Q1ipHXlGG25ZSG1J9hLfMn9+K0chLsEZli/d6ECyye8/edrK12ilr8jNoYOZSd2sRzYDlxwDmR;3:hghDIXNzUXdY+NVmu+GaSQ0XgEln07OCSV/fQ0hDrrCujB7WDHfdfdgfvYPtPcwgjW2qQG0koIdjZBqsdi+Enb3vFR7+ef0ti2U6lSidzUWM/CPgaqDUxYrk0k6L4o9n;25:8BxuufXosXbIXspXS3MGU0gU9e3fj+F0cAVj6WlhsigBpIz5OKWB8uc8rPV8qvvm7u4kQTxPkxPlemcNvaEesI50g5JK//PLBw9ShszqZxDHJ3J9XlwwL5RncjEgd//2YD4AZZ3mYo+S7InyZiJYuTB2Ybd/XItUbtzGoLxNMOl5XtwnPUk5hoBB+puC6PSftZjxakKNxgcoe1Gcp8waG/az+uEZhwH+JOX2x+PxZ7NSjOkfY0gL7M/ntzljuH/3XFxwXXF8ecRVTl4WeV2Axvs6zKRQQNiWUjHi37sH58hbU0Y0YvWVlBD6WlspZNLFMBpmqatRMvNZBpBBY5w82tGoHMipukAIf+wWZ10yMDI540Q1xKiKReUo3vujSnfac9GhdhFDacUxXC02Xf//MnOi22WAEFs9B8kqiYhZFx4= X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:BL2PR07MB2340; X-Microsoft-Exchange-Diagnostics: 1;BL2PR07MB2340;20:bok9jTWzkOSE3KvJ5XPrNTcWTe16Zh+oUMJy0YDgvEl4BGoaz07IMx5Li84C2UA6BZ9WgF3brzWjEAY5/Vp4JwOt6HfVNAlcQwnHdYiz2AYZJmaxR6h08lRDqIPdIBLIrqAiZ2cfPpN9Xl4K0HfAlRFIGhdRgL7PVDvkT1N9vGEX+P/b1IdWn5AA0TElFkOU6KA01McWJSRP7C4QHWu50hIEpoHjFIcZzKeuM9VQH7WESD3BnE2/B8W94jDypHIomrNWGQtbzHKnVeT3zNPUPzb5NT98KcXs2gFxL5gFWpNqWCzaqKm2Mo5coZmNBS6pWBug1AvBbNtGdfCWPUZ3eTJVBu5AziV5E2l6m8FmEwz9kFQ9BHptThATHEl5LQHleFPAHZflQrMqsrKAld2s2ljZb4ldvl+vFNr8UyK1iyleqaqE//tp9wbyYoACC0QlxoFYGsV1W+Yr6R/9hczj0PynH+L1A8VoBX93T7nNZ3D8u9lBV+Kpv6FGxs3SE1o0;4:LRCvtu8H+tbVRORyKc1BPyiJDWxnofFR8Eo0uHQeTYygA/eX2OuBusm+gpqdRfYnb/sQxwrWoaIwW9yBhY+eAzWtsZQKOqlVBy7WSolWVVzivreMDdna75Wr+chxbEoOrA58vau8nAJ6Hgatu5Fvp0OONti5ql2iAThlSCQKRxzqNdY9laky6Bkqu4dx7J2dnk6YUWE15jrVoalHGm52ZokUsoLAYBp4VeiNzSn3rzzrjOYLboDUPHLukbEoWT/LR9xZLeKIzItDF9alrNMllsNUx/1gyals0q4ar7JBdxKGeYj4TCjK5hYDL27qcSjei0iJeR/XBZ7Do+mqHldjV0Pimjwy8ifl6gMd521/SGP8wN0wOdFcXpdw/xAFET5V8G0YqEtLx/2qTWmWKsuyZ7zsxMgVv6CLSAUbF76dvQo= 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:BL2PR07MB2340;BCL:0;PCL:0;RULEID:;SRVR:BL2PR07MB2340; X-Forefront-PRVS: 096943F07A X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10009020)(4630300001)(6009001)(189002)(377454003)(24454002)(199003)(377424004)(101416001)(81166006)(47776003)(97736004)(110136002)(77096005)(2906002)(1076002)(4326007)(107886002)(5004730100002)(50986999)(76176999)(68736007)(83506001)(9686002)(54356999)(50466002)(19580405001)(4001430100002)(19580395003)(97756001)(3846002)(6116002)(586003)(86362001)(33656002)(4001350100001)(189998001)(42186005)(23726003)(5008740100001)(81156014)(105586002)(8676002)(92566002)(66066001)(106356001)(2950100001)(46406003)(217873001);DIR:OUT;SFP:1101;SCL:1;SRVR:BL2PR07MB2340;H:rric.localdomain;FPR:;SPF:None;PTR:InfoNoRecords;MX:1;A:1;LANG:en; X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1;BL2PR07MB2340;23:dLLn5lBkFK+eaKHRFVuy1F9CuDpEJ3vBcaBWKqB2u?= =?us-ascii?Q?zYNBNVexav3o2RSYARtROdzNgl2X3gztcq4OJkTBprcYtrpJooHUQ6uaSA6u?= =?us-ascii?Q?FOJR9AmoCHlZUuiYRua+V5RguemeeH9okKdqNIeCwzdRsiihixOfBZ2mTG7O?= =?us-ascii?Q?hV4zlVSyWMvoOsLNiX54BdziAJ9IdoAotuSBkmyDgkg2Ecji3YL+dizRN4UH?= =?us-ascii?Q?gZ3CUd+C8K7kK0EKcDAxD/nuIrQVTVZ+3WKw6HMqMtu05Kdgyc1D3FL0JyCQ?= =?us-ascii?Q?cjGUO3j7uSsmHyup/+IEthK2aPkKHIbfNKciFxEA49Jdg+w/g0dqqk4FH/ID?= =?us-ascii?Q?AbrBoLPFQMgKC8B0IqxK/lYQshHgRwMLGPfQLx9BpBUoCvIcvo2BU2sAFtFb?= =?us-ascii?Q?pu5RaCvA2N1t6yygf4kSYkkqyTwPGbLkenkhAmC3i1a4CiqAlpdJgFYMxZAH?= =?us-ascii?Q?BgRP6ib/djCYkUU7wnl8rC5072da3/CY+R0k5k6CNodHL36UhX6sXLMU9Rx9?= =?us-ascii?Q?wNSlzAWhjMrSj7hIw/8TT3jx+XEr7ViAQ32qY67IbO5qsLlWl7RPzV/hn7z3?= =?us-ascii?Q?Iz67kOhO8s9KiF1MbDympBqsqO5bphL4feG8+TtoSPJytUapAPCDrB/foKmy?= =?us-ascii?Q?JIoy4R07Q3CTxXIANolBcXE8js0YC/cLOfeXv7k6mVWy+qkJMNrzAYQ3AGAW?= =?us-ascii?Q?8p72dam8JTGik4RCRbfTyA2JBt7GBW7xg4PnLNs8/TIqXCaNJwDocxTMsY6L?= =?us-ascii?Q?z49xIWJTVyoTPDoT9o85U9eA8Zs4ldAnGPDtx/uFug00pP1ZDsQlNsZfAKGQ?= =?us-ascii?Q?kIFBISCEfahYkvE7+U4Vx3UtFYeH39vxFdmrdPzWyEP4gg2YZ8EwtHzz20Ps?= =?us-ascii?Q?Slbo+JNAwY5Ith8x2TwzKKjJRyIVYxhmxiuBzy7J9QENt1sr/AhXfNQ94qJK?= =?us-ascii?Q?n4WN3w5eliscmCgPqy2aHY0vz6o3edsJ8plPGRmn5ZUqVfa70/SgMYtcFGyS?= =?us-ascii?Q?zqQf7BHaAhCvCFPGP9BvjW7FjxTBuAHV0NkIu9xzWXoaXTPVclLLm85vKbSM?= =?us-ascii?Q?2aZ7/W+5xpTiUpw98nZd5sBlv8sLi0HSMw259LzOc+d0CfB9v3Gy5AyRu4Tb?= =?us-ascii?Q?cMp1ZULO1uTLbEawzWv4AXaxwtgByfY0jQDpt54j2QoZFwAtxLBsQ5clNGpP?= =?us-ascii?Q?G1ngJoS4DFd6loxYrNLsA+l78kbTsdEdB7ts78WvGIdgVD5NZZQL4EMBQAN4?= =?us-ascii?Q?UuptGz4hCQ6YHB95A0=3D?= X-Microsoft-Exchange-Diagnostics: 1;BL2PR07MB2340;5:YydS1rcby6f50Cg7vvcyrSLmIyC9LgSPawU+hn1MbEDnMXpPWYCMl15JZ5qNSuzJNJ1v6yzoyroSHjhojpgWbl7xFXding8Zh+p1awvkaVvVL7iv99UjsrtjZqwFI/cXxDVGyPWHlRIo0PMnXtVhLQ==;24:zohdw86u52XZ166TaMEf1kjlEgzxPG6g8IQ64WntWvhsGECJG0bJ/3ZFxAsOo4A/kNHUvdwh3GmJiTWZ6Z5yQKPNodzVxc+uwcc276v+U4o=;7:BApXpUf7EFr7hSHBOdOmlP4daB/DYDuS9M1scvPaRcTZsF5JaQGf6/AIQoRhbIiiw2D4BIRJyLNqA/BkQJ/G1aCOYUEeCBbWlZzmc+GZU4mkzjBHWRrbKu1Yi2Gsmufymu0ddNZ/pKmXmRG60qFyhWBU7MmJ52miKPbfnUqrRa0wOyfxzr/MV1teG0NogrpMg+LgS8zJdNCF0GigYKQ3OQZTUQ/BBhTVjMngvMti+Wc= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: cavium.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 10 Jun 2016 10:20:15.0504 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL2PR07MB2340 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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. Thanks, -Robert