Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id ; Sat, 25 Jan 2003 21:22:32 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id ; Sat, 25 Jan 2003 21:22:32 -0500 Received: from chaos.physics.uiowa.edu ([128.255.34.189]:53446 "EHLO chaos.physics.uiowa.edu") by vger.kernel.org with ESMTP id ; Sat, 25 Jan 2003 21:22:31 -0500 Date: Sat, 25 Jan 2003 20:31:27 -0600 (CST) From: Kai Germaschewski X-X-Sender: kai@chaos.physics.uiowa.edu To: Sam Ravnborg cc: linux-kernel@vger.kernel.org, Rusty Russell Subject: Re: [RFC] [PATCH] new modversions implementation In-Reply-To: <20030125225613.GA9429@mars.ravnborg.org> Message-ID: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org On Sat, 25 Jan 2003, Sam Ravnborg wrote: > On Sat, Jan 25, 2003 at 10:56:37PM +0100, Sam Ravnborg wrote: > > + > > +# Don't rebuilt vermagic.o unless we actually are in the init/ dir > > +ifneq ($(obj),init) > > +init/vermagic.o: ; > > +endif > > > > The above magic does not look safe to me when utilising parrallel builds. > > At least init/vermagic.o needs to be in the prepare: rule. > > Checked, it is in the prepare: rule, but it looks ugly anyway. > Does it really need to be a prerequisite? Yes, because we need to rebuild the module when vermagic.o changed, otherwise we'd end up with stale information in modules. --Kai - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/