From patchwork Wed Mar 2 09:40:54 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Arnd Bergmann X-Patchwork-Id: 102633 Delivered-To: patch@linaro.org Received: by 10.112.199.169 with SMTP id jl9csp2286681lbc; Wed, 2 Mar 2016 01:41:59 -0800 (PST) X-Received: by 10.98.67.195 with SMTP id l64mr420761pfi.111.1456911719683; Wed, 02 Mar 2016 01:41:59 -0800 (PST) Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id k68si32647858pfj.138.2016.03.02.01.41.59; Wed, 02 Mar 2016 01:41:59 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754534AbcCBJl5 (ORCPT + 30 others); Wed, 2 Mar 2016 04:41:57 -0500 Received: from mout.kundenserver.de ([217.72.192.73]:52698 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752727AbcCBJly (ORCPT ); Wed, 2 Mar 2016 04:41:54 -0500 Received: from wuerfel.lan. ([78.42.132.4]) by mrelayeu.kundenserver.de (mreue103) with ESMTPA (Nemesis) id 0LcWHu-1a8pWo0tmg-00jqJP; Wed, 02 Mar 2016 10:41:12 +0100 From: Arnd Bergmann To: Jiri Pirko , "David S. Miller" Cc: linux-arm-kernel@lists.infradead.org, Arnd Bergmann , Yishai Hadas , Doug Ledford , Sean Hefty , Hal Rosenstock , Ido Schimmel , linux-rdma@vger.kernel.org, linux-kernel@vger.kernel.org, netdev@vger.kernel.org Subject: [PATCH] net: mellanox: add DEVLINK dependencies Date: Wed, 2 Mar 2016 10:40:54 +0100 Message-Id: <1456911665-1424389-1-git-send-email-arnd@arndb.de> X-Mailer: git-send-email 2.7.0 X-Provags-ID: V03:K0:Ptp/+Sz7Kv9S6JXiW2CyxHDQsARytVgKnd0hplNHiDjvip/hL9Y LDjIE9q0B9uwP491UZIx4VOzaAEeyKoA24Ve/c3e44cit6l6H5Lsa2SZZAM0HDH/WotiIfI n7AiwQq52uzIq7QHQ1bpkbd6Tl+7bWF1mRiJzUnczyZRwWHlCFOUD2L+rWf2rfgPZrEYfAG g6lnHjYQ1T7zN5nkLkd5A== X-UI-Out-Filterresults: notjunk:1; V01:K0:/KgR9StR6Mg=:sqlZaO5Nr18ddgaAUvOEwm JO95NONvFq5LMgLPcCSI+PC9CUIIC7c5rWpKyELuWX2nkQf+LsXzw660TjxBfbtxAweI0RwGl 67K/3IyFBoefw/7emZ/drR2UIWk47qT0NMdT9Ndeg/pCF9+LDqVxDDNqCZNvGsd5N92kJ1F3w pmPoACcFuFp2dRDxhWSP79gV0HS0OlF2fFAOPXCzbZjb49ObZvAaJ6bU4Y65etLU+hWgO1zMg 86U52xdVCEqiyNsDEeCyyt7OubCVjE7of2lRqFLIiECbZgvduxEtEnqx98qOcBA6g/q+agnxk oZiek5oFMa9AH3RV5UBo5OEvLNouolmy+UHMt4mnFNNDCvf3CbT9x3HNxnmCqLFBa/FZVd1o6 6C1NeNh1tsp4dGrxRddkC2ebPT6DZKaORcHdo35MbgSlk6oYuHoPWoK8lacmcH96txrT2UmIP DqLSslgr/DPj4ildjkTSLNxsmaD22btVxjTeC6SLQcxByupc4XVJGyyW0O5ID1xPr8rC8pYGy kdeuKW3VVvvE3jkZ9idXmqZ28KAROT7n/+q0ImliGgi+YGNr4NKqHE5ZEVySA1ldVsH8zn5pO l12ez5Hhe1xulrKq9klzokCWVtOknE8o9QqBGm8CuPAcUrf+h9CUsapTrddb+Y7YSY/um8JeV DP7vfk4zv9BRRA3DuDFTxLmqrzL2HouxKsB5ak1o8xAYicqNbvKF8q9yCi4dA8jGoEik= Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org The new NET_DEVLINK infrastructure can be a loadable module, but the drivers using it might be built-in, which causes link errors like: drivers/net/built-in.o: In function `mlx4_load_one': :(.text+0x2fbfda): undefined reference to `devlink_port_register' :(.text+0x2fc084): undefined reference to `devlink_port_unregister' drivers/net/built-in.o: In function `mlxsw_sx_port_remove': :(.text+0x33a03a): undefined reference to `devlink_port_type_clear' :(.text+0x33a04e): undefined reference to `devlink_port_unregister' There are multiple ways to avoid this: a) add 'depends on NET_DEVLINK || !NET_DEVLINK' dependencies for each user b) use 'select NET_DEVLINK' from each driver that uses it and hide the symbol in Kconfig. c) make NET_DEVLINK a 'bool' option so we don't have to list it as a dependency, and rely on the APIs to be stubbed out when it is disabled d) use IS_REACHABLE() rather than IS_ENABLED() to check for NET_DEVLINK in include/net/devlink.h This implements a variation of approach a) by adding an intermediate symbol that drivers can depend on, and changes the three drivers using it. Signed-off-by: Arnd Bergmann Fixes: 09d4d087cd48 ("mlx4: Implement devlink interface") Fixes: c4745500e988 ("mlxsw: Implement devlink interface") --- drivers/infiniband/hw/mlx4/Kconfig | 1 + drivers/net/ethernet/mellanox/mlx4/Kconfig | 1 + drivers/net/ethernet/mellanox/mlxsw/Kconfig | 1 + net/Kconfig | 9 +++++++++ 4 files changed, 12 insertions(+) -- 2.7.0 diff --git a/drivers/infiniband/hw/mlx4/Kconfig b/drivers/infiniband/hw/mlx4/Kconfig index fc01deac1d3c..db4aa13ebae0 100644 --- a/drivers/infiniband/hw/mlx4/Kconfig +++ b/drivers/infiniband/hw/mlx4/Kconfig @@ -1,6 +1,7 @@ config MLX4_INFINIBAND tristate "Mellanox ConnectX HCA support" depends on NETDEVICES && ETHERNET && PCI && INET + depends on MAY_USE_DEVLINK select NET_VENDOR_MELLANOX select MLX4_CORE ---help--- diff --git a/drivers/net/ethernet/mellanox/mlx4/Kconfig b/drivers/net/ethernet/mellanox/mlx4/Kconfig index 1486ce902a56..9ca3734ebb6b 100644 --- a/drivers/net/ethernet/mellanox/mlx4/Kconfig +++ b/drivers/net/ethernet/mellanox/mlx4/Kconfig @@ -4,6 +4,7 @@ config MLX4_EN tristate "Mellanox Technologies 1/10/40Gbit Ethernet support" + depends on MAY_USE_DEVLINK depends on PCI select MLX4_CORE select PTP_1588_CLOCK diff --git a/drivers/net/ethernet/mellanox/mlxsw/Kconfig b/drivers/net/ethernet/mellanox/mlxsw/Kconfig index ce26adcb4988..2ad7f67854d5 100644 --- a/drivers/net/ethernet/mellanox/mlxsw/Kconfig +++ b/drivers/net/ethernet/mellanox/mlxsw/Kconfig @@ -4,6 +4,7 @@ config MLXSW_CORE tristate "Mellanox Technologies Switch ASICs support" + depends on MAY_USE_DEVLINK ---help--- This driver supports Mellanox Technologies Switch ASICs family. diff --git a/net/Kconfig b/net/Kconfig index 6c9cfb0d7639..2760825e53fa 100644 --- a/net/Kconfig +++ b/net/Kconfig @@ -403,6 +403,15 @@ config NET_DEVLINK infrastructure to support access to physical chip-wide config and monitoring. +config MAY_USE_DEVLINK + tristate + default m if NET_DEVLINK=m + default y if NET_DEVLINK=y || NET_DEVLINK=n + help + Drivers using the devlink infrastructure should have a dependency + on MAY_USE_DEVLINK to ensure they do not cause link errors when + devlink is a loadable module and the driver using it is built-in. + endif # if NET # Used by archs to tell that they support BPF_JIT