From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965181AbdEONwG (ORCPT ); Mon, 15 May 2017 09:52:06 -0400 Received: from mail-dm3nam03on0082.outbound.protection.outlook.com ([104.47.41.82]:46400 "EHLO NAM03-DM3-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S934356AbdEONwD (ORCPT ); Mon, 15 May 2017 09:52:03 -0400 Authentication-Results: arm.com; dkim=none (message not signed) header.d=none;arm.com; dmarc=none action=none header.from=caviumnetworks.com; Date: Mon, 15 May 2017 06:51:57 -0700 From: Vadim Lomovtsev To: Lorenzo Pieralisi Cc: linux-pci@vger.kernel.org, linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org, Sergey Temerkhanov , Sinan Kaya , Zhou Wang , Vadim.Lomovtsev@caviumnetworks.com Subject: Re: [RFC/RFT PATCH v2 0/3] PCI: generic device NUMA node detection Message-ID: <20170515135157.GB9700@localhost.localdomain> References: <20170515132205.19622-1-lorenzo.pieralisi@arm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170515132205.19622-1-lorenzo.pieralisi@arm.com> User-Agent: Mutt/1.6.1 (2016-04-27) X-Originating-IP: [50.233.148.156] X-ClientProxiedBy: MWHPR14CA0030.namprd14.prod.outlook.com (10.171.142.144) To CY1PR07MB1557.namprd07.prod.outlook.com (10.161.168.149) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 52ff27c8-5682-40c1-7993-08d49b998fe3 X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:(22001)(201703131423075)(201703031133081);SRVR:CY1PR07MB1557; X-Microsoft-Exchange-Diagnostics: 1;CY1PR07MB1557;3:XpzJp9jediDV3+eRTmvsLKzmc8xS+XvAvPRu8s38u+PRsfuXq6FklJHnLqnsNw/X8ylZBkvj6ejyhAkxFByy3HoJoF5h7EE3HGxjqUWf/QTLlDls+1KQbibtEDu+CMiSwkRYh0Dn+zxm6BRYr9aDFy3a3jfnT6HVkBMfc5vkhm+AN2LjkGYZpqFXMGtUsfYYpoEUkovC+g72d1tKq/Vnnr+TuRpLneXCeu+WHDjNsgVVEG5OI7KwjnP8K14kQa6g0nZy5PYUfPylNs1/ULXwItpgr9tX6/AVd3NoSkh6USHIzFlzDSAmuaMEy9QwRCoq0Cabk+PLDayDsT3SfsWAJQ==;25:FrCSMZj6tbatFtJTqnG9rq4KGXA9V9jPPFqYb99zIsGp98FVkDmjwKE54lQTBNMW634sqDhZncSDzJ1NU6kkNgd/C3neVF2RxQbRLpqB9CLbLSEyui1R/FYZKxB/8tZJTZvyyY9UTo130Wc6xVoNaEJ/N/1fp4rLNAi9Wwvq36XuEYd9Zg0mDaRQxv3fbG3RzFplvU0BPZ1TmTabict+KP6XEvIMvEC9kpT3Gzk8BKKrxZZpgijaEm2DTgp1Dwpih6HGBp/ff+aNkF6A5sOXDSMIqBMVpQy39lc+NlLpouJb2MIAnoSnoc1yQ4Rclgdyvl3BfNOGfCm29Vjutrkzkzv/YmLtkbhptZ6a7/U2kl6SpM24RnjW72vOw00L9T2nL2VeCJUfbll0DFN8hXh81cKhSnPWKMWfr5VfPMwHNkFBLfh+i097wQoWGnHiWZhJ0+vjeOWPG1inZggnpH0yN7YgF8109roWR/wn2xe7Tc0= X-Microsoft-Exchange-Diagnostics: 1;CY1PR07MB1557;31:GEKY+6uLC6IPeEQSvaPb4W/Bjf0hM5rvA4GYdib86TS7A/ApmYKSpzRT0CqMjn3WTRAs5D+p0RCObrVZafa58jLXMgVGyB+QTUafnDPEZPtTX1nQohIPYuYpsVOJ5JsAIULBNy8aSbZYLegXWZjnOJdFGBp+X3sKHjohqBmgWpa2W/kUJeSr5sbAGCLNDaZLXHZanNuznKq9TP60nfGjM54hfAEl9AdRergrjXr42+c=;20:iEXtNJH2ob6gIzaevsxbHDQOWMe2k+CaQP0mO6KCOMmabMg8TpuBZi1vmyVuL1JAvPER+/xNFx4fVenh4yGIc3NQhT20dYGRqbB8kRUlsJ1mfc4TZANzpQ5FhdP5PU5OH8uqNbUaKt0qUmPKC6D2Oq+85HBV9l/nTlg+4cuTZh+roQwFV0AmrIocB0Xj9rFd3SCefndojTsIJA0VJMTJpoQHSTYpKoOGISXGQI8KrfhOMiFYhSEyg4PhVwE4UxqgIn/tQ/gYLYo1rA7mF5Jd6MhADr3ancCDn7jiK4fGnD2JDIFO9uxOrPnDc3BInWHMV/CqIQCTVAFteaFqKYCiMA782qhXYpeZqKkNhN/ocrFSSb+WKyu8fiwMif6utSCUinXdmPsPY6i49/V/EtoQetjYHA+OVR3sQgwvNrRn3KutFb6T2zGr4iCPBtgkackV1eESyj82gNMXAY/8QPVO8uH4fD/1w0G/Dg9hWfLpSyk35hLaG53rQt6A2BL7H9sdZWPYYLkUDyzqP1dTjyh0PdTIMHQLecCxllQGkezkgcTtRjmCB9xjPKi19tAbwwqIPhNca64L18QzQQdzZysyB31nGpZabwC23H//WJROGNg= X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(6040450)(601004)(2401047)(5005006)(8121501046)(3002001)(93006095)(10201501046)(6041248)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123562025)(20161123564025)(20161123560025)(20161123555025)(20161123558100)(6072148);SRVR:CY1PR07MB1557;BCL:0;PCL:0;RULEID:;SRVR:CY1PR07MB1557; X-Microsoft-Exchange-Diagnostics: 1;CY1PR07MB1557;4:3CnpksroJAwdNQ17jR5KcY1aNzC0V8u9yzP7ebJC+IWljbXHPd7gIf6XPjQ2vv5KQvUGn5Kf9QSbOZkxCfwnc9BvbbsHV5u3LILGSiMANgAheV619BpiNAhUpVHtnvB+RT5KEV15vtjCEn+ih/ms1WVV53qJ6RvID3SZqOo0GSqY3tJwBP6b5E+lyLNQQ9sxNWTeY2WPEYayF8B+dNn8rhM5YC4aY1PGz63TqGpysNFwovAlJkAdxAEak8pC6Z66u61mKhH4gPGq9HsG5zgf3w1dWc7AB3y5rPjeiFqJ7b2Ir9oLBUl630te6ZO1pOroJF0tDyHPPG5m5kjxbyXM5ML9g0Ghg1AjmnINr57TroG+skfeB2ZRJchRxpf5oYWjZZ4f7+NdVuMPvAGa/BB7IrK2OKzUC93MsB/09HIJg4XTGYnlJ3UAP6ZITI2bNZCE1OOEDAQK4Yk8u4H4RSIiDMQFJh/Xy4vsSQvVgjXOeBASOm7vue7YobV1SgUDld7os6wwoVaPL/WPymN3iz7UCybnfQ4HoaRaZ1CRI6nddS1UonOMS4lx+jBBNhhiLl6DVieN8s3TIDgwEkGH9G6/bDIo4+3TMjKvt/8baJbkMDo7KSbBefbq5hz06T4M7gq54h+hKeGE05aWgSRRa+//gjK7hMdjaPv73t+JzUPu3VXF3JWUOO9BiOLI0dlaUDSJSyrd5Yi0ipXlcsB4xpE2EOBjVa11OQ3M4ypqqT+9cYi9ihcHAPTPFda6NyzJRHtPIGUHilDFYHlc1Uak6VG3Fw== X-Forefront-PRVS: 0308EE423E X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10009020)(4630300001)(6009001)(39400400002)(39450400003)(39410400002)(39840400002)(39850400002)(24454002)(81166006)(47776003)(76176999)(54356999)(50986999)(7736002)(305945005)(4001350100001)(61506002)(8676002)(42186005)(66066001)(25786009)(6666003)(6916009)(42882006)(2950100002)(33656002)(478600001)(2906002)(229853002)(1076002)(50466002)(23726003)(53936002)(3846002)(6116002)(72206003)(6306002)(9686003)(55016002)(54906002)(4326008)(5660300001)(6246003)(6506006)(110136004)(107886003)(38730400002)(189998001)(83506001)(18370500001)(217873001);DIR:OUT;SFP:1101;SCL:1;SRVR:CY1PR07MB1557;H:localhost.localdomain;FPR:;SPF:None;MLV:sfv;LANG:en; X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1;CY1PR07MB1557;23:wRq9uqiDH3dpmsetB0SUIZrsG1/2dRGQcedlOquJR?= =?us-ascii?Q?R63sTwGcfLbXBkrmlWtm3HsPE6TcQYAOtxHiDJJIXjzqd053+fJh3FCfXao4?= =?us-ascii?Q?t1KqATDMj1E/CM5pitLGiBKyoNUO3HO8wZgQZBHeI+auFC8QjskjpVsszbhi?= =?us-ascii?Q?QCL7yYKwg6Y9ZjuVgYltnGFlFeIAk9TrJh2+kYlaV1CKgwoIZwPxAQiiWiCw?= =?us-ascii?Q?yJo5rstQqZ3zQSZQ3jWUGuPHknlCI9LGN1N/9ZGzuKZbQW3pin6eImc/SsBd?= =?us-ascii?Q?/xTJFW0jjxDy86txN4Gmvn8YvrweZqXxKBzgk0r7KKukt3tXWOVHYUmxWIBj?= =?us-ascii?Q?9+7ZpL277pVjoc0YP6BUviXRsKf9KAgCWa72D7SQax9iVJ25jQ04r/30XRnG?= =?us-ascii?Q?enFFWDrC00arUYMXW84sqbHh3kkmO6Vx1FCdvEX0mNmHseoCqSK8gi0U+U1X?= =?us-ascii?Q?sktSNZNEZVErcB8R6anL13yeXGhiGxvK25n3m+NokkOZvJrfWjlWKD+2Iy+O?= =?us-ascii?Q?W/DBO1eAaaq7Uvyd86QADCnd9FCr/13OcDFUtwO3azlStC6WvBmhcNq92ab+?= =?us-ascii?Q?2BGN9hqsefvvd3fDo6SOqYRtryppeLMzySvwJWnP48HFp6MwejQwXjQSVbIQ?= =?us-ascii?Q?S4IFyfvFYsJfs82IjjEb0Msx7hzqu2t/k2uqTi8h8/itegMFr28c5G6MqXX3?= =?us-ascii?Q?YG1FDfNhAPqtQeXgWjfx5wr8iAJuq0DuYCkK1zgtLnFuJscLmM0KUzU670zF?= =?us-ascii?Q?RW0l0sW/yIu/Mo+LegPEgokuzGQLOl+QJxkd3BsqVRq9K0xW/MBCz7pq024Y?= =?us-ascii?Q?Q/fLBUk0IkFCMYG3wctrA8xN1pPELKSfYKkR26sUtJO55qVYHTFRltyQP3gO?= =?us-ascii?Q?zonUtPoezU9l31ajtK9mmO/JtIhbdXuE8RJ0gazHV3Ng9SjZS+jyRMQEJyC6?= =?us-ascii?Q?vpk9OqqoCARFwEshyWR0x8G6FNBkRUM13PjIUTAbTNTH7CangSiNlMmq3GZA?= =?us-ascii?Q?xIg10bC87vhn1ygv5SYrPMDesUMkoMDfGcNWGW8DqHH0lLRm8/WCvSOsXUiA?= =?us-ascii?Q?sUasWZnH5h7KG0OKcLeIqV8sliP/4CYKXr/MzjBXWqW9FyVTQfXqmXdlZIiE?= =?us-ascii?Q?OHiXVdmL36hpznYFPK6qU09lNp6Hk4Nom50S7tK4vkhPG9Bae+oooyVWL2e8?= =?us-ascii?Q?kDEdTr+zuCij2E9VeTlu9ZxK2S1rZ5+F7yZmCsG/vwMADcebfOFgnCAImerE?= =?us-ascii?Q?bLUEnfhbkFST88yN3jLKrhktSJnM5tbBTDFcu6o?= X-Microsoft-Exchange-Diagnostics: 1;CY1PR07MB1557;6:EuBntvQpeoow5oOriCRIq+OEGquVJ+d2/XyorHfq16ZHJC427sNZoY3Luqtgt8HlSH08mphu+XaRyGKyvWl3xHiGg2bEtOWDzt7105mnU3O6FyIZopDbvdohTZnXzPLBYsRKAZDVh2kf01w+p+z+VMi6TCMhbVe85jh2c+2vNKZfzxjq28rPmA3X+YjUBG5d9La1Ptnwjd84v5rJkqe4H642+hisCDSjCYo4mSFtlhDPJhzWEunGmB5vsWjgopBQeJj+rRt2beahdqeXGz30Gutgx98/UEaE12PE4M9w2WNfCPbxLik7Fg7MtMCbLWFOFXYHQPIYJN+QTHTweR0zBfTbiZVDJvEWZ9QruJVmvKRof3yRLvnIH+NSCTKt81029DY+AhGXwkUIHy6L6nkzT2OAb35f6JzhjqkxFqm0JgSAI5q5FO7frUTdyGxuCv7xd7dlVkuhZsj8+MiWf3tOMqm+HXYb14e1M8MhUvyakfxUMz0SShOv5e61ySwdgkqf/NPMdGn5Hth7XneF2g4Ung==;5:tCsxZ0NcTWyIQ3juGmvieWeF7D+b2nI8Njs3bjgG+wsn1VYj9fvVPG8iD5023zd4+OGJIsPFc4bUiQ/L7wNR4CQSUJz6t3W74Ya6bx51btjcMFxHG+VfEoW1gXrgm/l3Y3l0Oy8T1HQsnheVMnJ+zA==;24:DVMd2XjTYTpDlO4K/RidkiFUSQFWYqB03kFtvAXDHgmjpLi/dRIWRWb14KyRBqwG9eXV/7IKztCc2ehWGtD8lj8ldGQYK8bIbTZxZElhHoE= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1;CY1PR07MB1557;7:p+PWcK7GGS88QjrCcZLinqyD10t+CcFAW84CfMOBjga0G+WjqR1zupscLnzyXI06ZTtlUIDBlI6Jjv95MCkW2YZuHlbigJj1rSfnQZA1zNBxQt+xpALCBKzNidHnK5B06tipWcEs/38kTPe/ZgRVx5bbRnoAuMulnnb72aLXVIZCTmvbqqbiE0c1OwiAbWiNie4sJL+DgT7CXZV//2qwVsq0HacHpi1Lbp7TCFCrAC0/10/B4kNc2zidmzoE0b5QZxxAk/bZH7P3ySYgVkCpa583Z/QK9uI9/qdpawGQ2jq8p/3Nhu47AJwr/+q1v9WPNTLmH1cEz/E1N//VOZUa3w== X-OriginatorOrg: caviumnetworks.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 May 2017 13:52:00.7509 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY1PR07MB1557 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, May 15, 2017 at 02:22:02PM +0100, Lorenzo Pieralisi wrote: > This patch series is v2 of a previous posting: > > v1 -> v2: > > - Added missing call to acpi_pci_bus_find_numa_node() > - Rebased against v4.12-rc1 > > v1: https://lkml.org/lkml/2017/4/26/211 > > On the ARM64 architecture, the arch specific PCI pcibus_to_node() callback > can be triggered on struct pci_bus.sysdata structures that carry host > bridge specific data through different structures layout (ie sysdata > pointer points at different structs for different host bridges), therefore > it is not possible to have a unified pcibus_to_node() implementation. > > Given that the device NUMA node is a property of the struct pci_bus (and > its associated struct device), move the struct pci_bus node assignment to > generic PCI code, where according to the platform firmware the NUMA node > for the device backing the struct pci_bus is retrieved and propagated > through the PCI bus hierarchy. > > Lorenzo Pieralisi (3): > PCI: Introduce pci_bus_find_numa_node() > PCI: Add call to set-up NUMA node for struct pci_bus devices > PCI/ACPI: Add ACPI pci_bus_find_numa_node() implementation > > drivers/pci/pci-acpi.c | 20 ++++++++++++++++++++ > drivers/pci/pci.c | 6 ++++++ > drivers/pci/probe.c | 3 ++- > include/linux/pci.h | 7 +++++++ > 4 files changed, 35 insertions(+), 1 deletion(-) > Tested at dual-socketed system with Cavium ThunderX SoC. Tested with 4.11 kernel. No issues observed, numa_node value set appropriately for pci devices. Tested-by: Vadim Lomovtsev > -- > 2.10.0 > WBR, Vadim