> 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 Kind regards, Chris > > If so, I'll modify the file to have that, seems to be some sort of > include file mess :( > > thanks, > > greg k-h
