Received: by 2002:a25:7ec1:0:0:0:0:0 with SMTP id z184csp763883ybc; Tue, 19 Nov 2019 08:54:02 -0800 (PST) X-Google-Smtp-Source: APXvYqyvCT5j1DjCav0WUno26K92Kp+SMOawq+OzS/JBxuiKmzmrl5S+IrwcobOtjbadi5wVs0X5 X-Received: by 2002:adf:f744:: with SMTP id z4mr27560234wrp.205.1574182442668; Tue, 19 Nov 2019 08:54:02 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1574182442; cv=none; d=google.com; s=arc-20160816; b=nGZT0UNlr28C0XkQ1HxeL0QZfU9Pt8rzJMb3j/U4lI2bFgG+uBZafTeMXr29JNSbPY FgMbOwd7AAw33e3SY+g3w6NXdkbD7teSHSWO4KeUDNMHIEaTdEY9uaDdVzueUiMbrVuj K9tQQHdDTmf7FSmEMXVm4PvrCDlQt2QF+etHvm/ay4nR0KrmBKP6wT4q8TLF67lwUKji 9a2Da2O58SI0AJSEAFpBK5jFp9xYQVtZ4WSPjituLoOcNQsdCnCeYP+SjA5KcI0npqoh cZ1e94Koip+R+TQmyKEdFEfzr5WMlapul4HokwbI+lNjmh62yUm2D1+UdnHv0knx0rB0 7BfQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:dkim-signature; bh=FcgHggi9Cd+iigCqiWMNDuW0r6cIKaHvaEABrNH/4jI=; b=b0gVBNoZyX1MTYHBsWpeRSN1V5qPZK7+Ek6UHDGLM/F7aTpQtkSz1PqI4f2TA1OMH3 eOA96W+ZYI81aU3WTehb+RGXsW55NuEUeWORTwN3caf/1SJ/Uq//MuZRZkPNoy0prmC5 p4J/r3r9BLDPsa8E/g0rt7XMqbdiC5hlabCDaK73T3sN9T0GVhsZI9mmxTKM7KiBpoft Ts4xEB/w1UYAShvemO/LRHrk3pc5kCNARMwzU3CZTadZy32M0Z+0tA6Qjv9/2T3xVHar 40Wu0g65ViPTm4RDa/w/j815zQnur4Aenv5bNx8EHr01Bc5KSTkeaBV/waTXS4grn+kh WVdw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=e6TrI1GR; 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 Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id c5si15925994edc.246.2019.11.19.08.53.36; Tue, 19 Nov 2019 08:54:02 -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; dkim=pass header.i=@kernel.org header.s=default header.b=e6TrI1GR; 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 S1728300AbfKSQwN (ORCPT + 99 others); Tue, 19 Nov 2019 11:52:13 -0500 Received: from mail.kernel.org ([198.145.29.99]:46680 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727560AbfKSQwM (ORCPT ); Tue, 19 Nov 2019 11:52:12 -0500 Received: from localhost (83-86-89-107.cable.dynamic.v4.ziggo.nl [83.86.89.107]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 00AD2222D1; Tue, 19 Nov 2019 16:52:09 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1574182330; bh=PQE08P8cZPIQVrquCvag3YZlLtDU39GyDrviAt/PyU8=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=e6TrI1GRBWRgsZYJFPhe9V07K6ChCrac2m7Dqfgk53XaPQoocsRZCYLjL/O5lxBb0 evRhJiou6r4rGj6L0bH3eguiEluzt7ujSqmxBJZtAnNvO1ZDKIhk3en01fSPE/xGs7 ZVPQXqU6u1WJEILKbBovdpnI5KluwM0+rCFQZs8o= Date: Tue, 19 Nov 2019 17:52:07 +0100 From: Greg Kroah-Hartman To: Chris Paterson Cc: "linux-kernel@vger.kernel.org" , "cip-dev@lists.cip-project.org" , "torvalds@linux-foundation.org" , "akpm@linux-foundation.org" , "linux@roeck-us.net" , "shuah@kernel.org" , "patches@kernelci.org" , "ben.hutchings@codethink.co.uk" , "lkft-triage@lists.linaro.org" , "stable@vger.kernel.org" Subject: Re: [PATCH 4.19 000/422] 4.19.85-stable review Message-ID: <20191119165207.GA2071545@kroah.com> References: <20191119051400.261610025@linuxfoundation.org> <20191119122909.GC1913916@kroah.com> <20191119154839.GB1982025@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.12.2 (2019-09-21) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Nov 19, 2019 at 04:38:06PM +0000, Chris Paterson wrote: > > From: Greg Kroah-Hartman > > 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 > > > > 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@vger.kernel.org > > 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 > > 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