Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752671AbcKHBub (ORCPT ); Mon, 7 Nov 2016 20:50:31 -0500 Received: from mail-pf0-f195.google.com ([209.85.192.195]:35826 "EHLO mail-pf0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751454AbcKHBu1 (ORCPT ); Mon, 7 Nov 2016 20:50:27 -0500 Date: Tue, 8 Nov 2016 12:33:34 +1100 From: Nicholas Piggin To: Peter Wu Cc: Vince Weaver , Michal Marek , Al Viro , Oliver Hartkopp , linux-kernel@vger.kernel.org, linux-kbuild@vger.kernel.org, linux-arch@vger.kernel.org Subject: Re: 'kbuild' merge before 4.9-rc1 breaks build and boot Message-ID: <20161108123334.3df2dcc9@roar.ozlabs.ibm.com> In-Reply-To: <20161107213907.GB18092@al> References: <4c04b846-3da8-dc42-4fa6-30f72a5c2437@hartkopp.net> <20161027154202.GE27017@al> <20161107213907.GB18092@al> Organization: IBM X-Mailer: Claws Mail 3.14.1 (GTK+ 2.24.31; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2944 Lines: 66 On Mon, 7 Nov 2016 22:39:07 +0100 Peter Wu wrote: > On Mon, Nov 07, 2016 at 02:10:12PM -0500, Vince Weaver wrote: > > On Thu, 27 Oct 2016, Peter Wu wrote: > > > > > I can confirm Olivers issue, the current mainline kernel fails to boot > > > on kernels with CONFIG_MODVERSIONS=y. Bisection points to: > > > > > > commit 784d5699eddc55878627da20d3fe0c8542e2f1a2 > > > Author: Al Viro > > > Date: Mon Jan 11 11:04:34 2016 -0500 > > > > WARNING: "memset" [sound/usb/snd-usbmidi-lib.ko] has no CRC! > > > > WARNING: "__fentry__" [sound/usb/snd-usbmidi-lib.ko] has no CRC! > > > > WARNING: "memcpy" [sound/usb/snd-usbmidi-lib.ko] has no CRC! > > > > WARNING: "__sw_hweight32" [sound/usb/snd-usbmidi-lib.ko] has no CRC! > > > > Has any progress been made with this problem? > > > > I'm also encountering it on my debian-unstable box on any kernel more > > recent than 4.9-rc1 (up to and including 4.9-rc4). I am glad someone > > managed to isolate it as I was unable to get a clean bisect. > > > > Vince > > The original kbuild issue went in via > merge commit 84d69848c97faab0c25aa2667b273404d2e2a64a which notes: > > - EXPORT_SYMBOL for asm source by Al Viro. > > This does bring a regression, because genksyms no longer generates > checksums for these symbols (CONFIG_MODVERSIONS). Nick Piggin is > working on a patch to fix this. > > Plus, we are talking about functions like strcpy(), which rarely > change prototypes. > > Adding Nicholas in the cc, hopefully he can give a status update. I think Michal has everything needed now for the kbuild bits. The arch specific patches can go via arch trees quite easily (there is no hard dependency either way). This is the kbuild bit: https://git.kernel.org/cgit/linux/kernel/git/mmarek/kbuild.git/commit/?h=rc-fixes&id=4efca4ed05cbdfd13ec3e8cb623fb77d6e4ab187 And it also needs this incremental bit not in Michal's tree yet: --- scripts/Makefile.build | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/Makefile.build b/scripts/Makefile.build index 3e223c2..05c6bb4 100644 --- a/scripts/Makefile.build +++ b/scripts/Makefile.build @@ -332,7 +332,7 @@ cmd_gensymtypes_S = \ (echo "\#include " ; \ echo "\#include " ; \ $(CPP) $(a_flags) $< | \ - grep ^___EXPORT_SYMBOL | \ + grep ___EXPORT_SYMBOL | \ sed 's/___EXPORT_SYMBOL \([a-zA-Z0-9_]*\),.*/EXPORT_SYMBOL(\1);/' ) | \ $(CPP) -D__GENKSYMS__ $(c_flags) -xc - | \ $(GENKSYMS) $(if $(1), -T $(2)) \ -- 2.9.3