On Tue, Nov 19, 2019 at 04:38:06PM +0000, Chris Paterson wrote: > > From: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> > > Sent: 19 November 2019 15:49 > > > > On Tue, Nov 19, 2019 at 02:44:12PM +0000, Chris Paterson wrote: > > > Hi Greg, > > > > > > > From: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> > > > > Sent: 19 November 2019 12:29 > > > > > > > > On Tue, Nov 19, 2019 at 08:54:25AM +0000, Chris Paterson wrote: > > > > > Hello Greg, all, > > > > > > > > > > > From: stable-owner@xxxxxxxxxxxxxxx <stable-owner@xxxxxxxxxxxxxxx> > > On > > > > > > Behalf Of Greg Kroah-Hartman > > > > > > Sent: 19 November 2019 05:13 > > > > > > > > > > > > This is the start of the stable review cycle for the 4.19.85 release. > > > > > > There are 422 patches in this series, all will be posted as a response > > > > > > to this one. If anyone has any issues with these being applied, please > > > > > > let me know. > > > > > > > > > > I'm seeing some build issues with module compilation with this release > > > > (1b1960cc Linux 4.19.85-rc1), I also saw them with the previous two versions > > of > > > > Linux 4.19.85-rc1 (cd21ecdb and 1fd0ac64). > > > > > > > > > > Full log available on GitLab [0]. Build conf [1]. > > > > > [0] https://gitlab.com/cip-playground/linux-stable-rc-ci/-/jobs/354591285 > > > > > [1] https://gitlab.com/cip-playground/linux-stable-rc-ci/- > > > > /jobs/354591285/artifacts/file/output/4.19.85- > > > > rc1_1b1960cc7/x86/siemens_iot2000.config/config/.config > > > > > > > > > > Main error below: > > > > > > > > > > 3907 CC [M] drivers/net/ethernet/mellanox/mlx4/main.o > > > > > 3908 LD [M] fs/ntfs/ntfs.o > > > > > 3909 CC [M] drivers/net/ethernet/intel/i40evf/i40e_txrx.o > > > > > 3910 CC [M] drivers/usb/musb/musb_core.o > > > > > 3911 CC [M] drivers/net/ethernet/nvidia/forcedeth.o > > > > > 3912 CC [M] fs/udf/balloc.o > > > > > 3913 CC [M] drivers/net/ethernet/intel/fm10k/fm10k_debugfs.o > > > > > 3914 CC [M] fs/udf/dir.o > > > > > 3915 CC [M] drivers/net/ethernet/broadcom/bnx2x/bnx2x_vfpf.o > > > > > 3916 CC [M] drivers/net/ethernet/intel/i40e/i40e_ptp.o > > > > > 3917 drivers/net/ethernet/mellanox/mlx4/main.c: In function > > 'mlx4_init_one': > > > > > 3918 drivers/net/ethernet/mellanox/mlx4/main.c:3985:2: error: implicit > > > > declaration of function 'devlink_reload_enable'; did you mean > > > > 'devlink_region_create'? [-Werror=implicit-function-declaration] > > > > > 3919 devlink_reload_enable(devlink); > > > > > 3920 ^~~~~~~~~~~~~~~~~~~~~ > > > > > 3921 devlink_region_create > > > > > 3922 CC [M] drivers/net/ethernet/chelsio/cxgb4/cxgb4_cudbg.o > > > > > 3923 drivers/net/ethernet/mellanox/mlx4/main.c: In function > > > > 'mlx4_remove_one': > > > > > 3924 drivers/net/ethernet/mellanox/mlx4/main.c:4097:2: error: implicit > > > > declaration of function 'devlink_reload_disable'; did you mean > > > > 'devlink_region_destroy'? [-Werror=implicit-function-declaration] > > > > > 3925 devlink_reload_disable(devlink); > > > > > 3926 ^~~~~~~~~~~~~~~~~~~~~~ > > > > > 3927 devlink_region_destroy > > > > > 3928 CC [M] drivers/net/ethernet/packetengines/hamachi.o > > > > > 3929 CC [M] fs/udf/file.o > > > > > 3930 LD [M] drivers/net/ethernet/intel/fm10k/fm10k.o > > > > > > > > > > I haven't tried to trace the issue further yet, sorry. > > > > > > > > Any chance you can bisect this? I don't see any obvious reason why this > > > > error should be happening, and it isn't showing up here :( > > > > > > Looking through the commit history, the issue seems to be related to: > > > 672cf82122be ("devlink: disallow reload operation during device cleanup") > > > > > > I've reverted this commit and Linux 4.19.85-rc2 (af1bb7db before revert) will > > build with the configuration I'm using [2]. > > > I haven't looked further yet though, sorry. > > > > > > [2] https://gitlab.com/cip-project/cip-kernel/cip-kernel- > > config/raw/master/4.19.y-cip/x86/siemens_iot2000.config > > > > If you add: > > #include <net/devlink.h> > > to the top of drivers/net/ethernet/mellanox/mlx4/main.c, does it fix the > > issue for you? > > This is already defined: > https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git/tree/drivers/net/ethernet/mellanox/mlx4/main.c?h=linux-4.19.y#n47 Ah, ok, the issue is that CONFIG_NET_DEVLINK is not enabled, the driver now requires this. This was resolved by adding the dependancy to the driver itself, and then just punting and always enabling it over time. We can backport part of f6b19b354d50 ("net: devlink: select NET_DEVLINK from drivers") if you want, but that feels messy. For now, if you enable that option, does it build for you? thanks, greg k-h
