From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from cantor2.suse.de ([195.135.220.15]:32999 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1759956AbaGDWPj (ORCPT ); Fri, 4 Jul 2014 18:15:39 -0400 Message-ID: <53B72780.4000302@suse.cz> Date: Sat, 05 Jul 2014 00:15:28 +0200 From: Michal Marek MIME-Version: 1.0 Subject: Re: [PATCH] kbuild: Do not run modules_install and install in paralel References: <1402652395-28779-1-git-send-email-mmarek@suse.cz> <539AC82B.4070505@suse.cz> In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Sender: linux-kbuild-owner@vger.kernel.org List-ID: To: Andy Lutomirski Cc: "H. Peter Anvin" , X86 ML , "linux-kbuild@vger.kernel.org" , "linux-kernel@vger.kernel.org" Dne 13.6.2014 19:29, Andy Lutomirski napsal(a): > On Fri, Jun 13, 2014 at 2:45 AM, Michal Marek wrote: >> Dne 13.6.2014 11:39, Michal Marek napsal(a): >>> Based on a x86-only patch by Andy Lutomirski >>> >>> With modular kernels, 'make install' is going to need the installed >>> modules at some point to generate the initramfs. >>> >>> Signed-off-by: Michal Marek >>> --- >>> Makefile | 6 ++++++ >>> 1 file changed, 6 insertions(+) >>> >>> diff --git a/Makefile b/Makefile >>> index 7680d7c..7e5e483 100644 >>> --- a/Makefile >>> +++ b/Makefile >>> @@ -503,6 +503,12 @@ ifeq ($(KBUILD_EXTMOD),) >>> endif >>> endif >>> endif >>> +# install and module_install need also be processed one by one >>> +ifneq ($(filter install,$(MAKECMDGOALS)),) >>> + ifneq ($(filter modules_install,$(MAKECMDGOALS)),) >>> + mixed-targets := 1 >>> + endif >>> +endif >> >> Note that this version does not enforce the ordering, it just avoids the >> interleaved execution. It can be added if desired. > > Hmm. This will fix 'make modules_install install' but will not fix > 'make install modules_install'. I don't know how many people would > type the latter. Resuming a thread from last month -- As I said, it does not enforce the ordering. So it's not as bullet-proof as your patch, but it works on all architectures. I'm thinking about adding the patch to kbuild.git for 3.17. But if you want to fix the ordering as well, feel free to change the __build_one_by_one rule to do this. Thanks, Michal