Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S964961AbcCOXOs (ORCPT ); Tue, 15 Mar 2016 19:14:48 -0400 Received: from mail-qg0-f50.google.com ([209.85.192.50]:34411 "EHLO mail-qg0-f50.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933899AbcCOXOn (ORCPT ); Tue, 15 Mar 2016 19:14:43 -0400 Date: Tue, 15 Mar 2016 19:14:38 -0400 (EDT) From: Nicolas Pitre To: Michal Marek , Michal Marek cc: linux-kbuild@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v6 5/8] kbuild: add fine grained build dependencies for exported symbols In-Reply-To: Message-ID: References: <1457923336-2732-1-git-send-email-nicolas.pitre@linaro.org> <1457923336-2732-6-git-send-email-nicolas.pitre@linaro.org> <56E8718E.9010607@suse.com> User-Agent: Alpine 2.20 (LFD 67 2015-01-07) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3058 Lines: 71 On Tue, 15 Mar 2016, Michal Marek wrote: > Dne 14.3.2016 v 03:42 Nicolas Pitre napsal(a): > > +# Filter out exported kernel symbol names from the preprocessor output. > > +# See also __KSYM_DEPS__ in include/linux/export.h. > > +# We disable the depfile generation here, so as not to overwrite the existing > > +# depfile while fixdep is parsing it > > +flags_nodeps = $(filter-out -Wp$(comma)-M%, $($(1))) > > +ksym_dep_filter = \ > > + case "$(1)" in \ > > + cc_*_c) $(CPP) $(call flags_nodeps,c_flags) -D__KSYM_DEPS__ $< ;; \ > > + as_*_S) $(CPP) $(call flags_nodeps,a_flags) -D__KSYM_DEPS__ $< ;; \ > > + cpp_lds_S) : ;; \ > > + *) echo "Don't know how to preprocess $(1)" >&2; false ;; \ > > + esac | sed -rn 's/^.*=== __KSYM_(.*) ===.*$$/KSYM_\1/p' > > + > > +cmd_and_fixdep = \ > > + $(echo-cmd) $(cmd_$(1)); \ > > + $(ksym_dep_filter) | \ > > + scripts/basic/fixdep -e $(depfile) $@ '$(make-cmd)' \ > > + > $(dot-target).tmp; \ > > + rm -f $(depfile); \ > > + mv -f $(dot-target).tmp $(dot-target).cmd; > > + > > +endif > > Not sure what happened this time, but I got > > drivers/md/.dm-round-robin.mod.o.cmd:5: *** unterminated call to > function 'wildcard': missing ')'. Stop. > > The depfile is indeed corrupt (truncated): > tail drivers/md/.dm-round-robin.mod.o.cmd > arch/x86/include/asm/disabled-features.h \ > $(wildcard include/config/x86/intel/mpx.h) \ > arch/x86/include/asm/rmwcc.h \ > arch/x86/include/asm/barrier.h \ > $(wildcard include/config/x86/ppro/fence.h) \ > arch/x86/include/asm/nops.h \ > $(wildcard include/config/mk7.h) \ > include/asm-generic/barrier.h \ > include/asm-generic/bitops/find.h \ > $(wildcard include/config/generic/ > > The file is exactly 8kB long: > $ ls -l drivers/md/.dm-round-robin.mod.o.cmd > -rw-r--r-- 1 mmarek users 8192 Mar 8 13:33 > drivers/md/.dm-round-robin.mod.o.cmd [...] > Forgot to add: This was an allmodconfig build without > CONFIG_TRIM_UNUSED_SYMS. If CONFIG_TRIM_UNUSED_SYMS is actually not selected, then I really don't see how my changes can be involved in this failure. And I cannot reproduce here so far. I didn't receive anything from kbuild test robot since February 15 either. Is this something that you are able to reproduce on your side? If so, do you get suspicious output on stderr during the build? Maybe fixdep exits prematurelywith "fixdep: bad data on stdin"? However, that cannot happen if CONFIG_TRIM_UNUSED_SYMS=n. Did you test my git branch, or did you apply the posted patches to your tree? If the later, could you publish a branch for it so I could test the same tree as you do? Nicolas