All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andi Kleen <ak@suse.de>
To: Sam Ravnborg <sam@ravnborg.org>
Cc: linux-kernel@vger.kernel.org, agruen@suse.de
Subject: Re: Building external modules against objdirs
Date: Sun, 30 Jul 2006 20:37:02 +0200	[thread overview]
Message-ID: <200607302037.02559.ak@suse.de> (raw)
In-Reply-To: <20060730183159.GA30278@mars.ravnborg.org>

On Sunday 30 July 2006 20:31, Sam Ravnborg wrote:
> On Sun, Jul 30, 2006 at 07:49:41PM +0200, Andi Kleen wrote:
> > 
> > > Can you check that you really did a 'make prepare' in the relevant
> > > output directory. Previously only the make *config step was needed.
> > 
> > The output directory is a full build (configuration + make without any targets).
> > Is that not enough anymore? 
> > 
> > Anyways after a make prepare it seems to work - thanks - but I think that
> > should be really done as part of the standard build like it was in 2.6.17.
> 'make prepare' is and has always been part of the standard build.
> So I really do not see what is going on.

To reproduce (on x86-64 at least)

mkdir obj
cd obj
make -C ../linux-2.6.18-... O=$(pwd) defconfig
make 



> Can you please check that followign files exists in your output
> directory:
> .config
> include/config/auto.conf.cmd
> include/config/auto.conf
> 
> the latter should be the latest of the three.

-rw-r--r--  1 andi users 28481 2006-07-29 19:01 .config
-rw-r--r--  1 andi users  7739 2006-07-29 19:07 include/config/auto.conf
-rw-r--r--  1 andi users  6867 2006-07-29 19:07 include/config/auto.conf.cmd

 
> Also try applying following patch to reveal why we trigger this rule:
> 
> diff --git a/Makefile b/Makefile
> index 1dd58d3..4c30ed5 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -453,6 +453,7 @@ include/config/auto.conf: $(KCONFIG_CONF
>  ifeq ($(KBUILD_EXTMOD),)
>  	$(Q)$(MAKE) -f $(srctree)/Makefile silentoldconfig
>  else
> +	@echo triggered by - $? -
>  	$(error kernel configuration not valid - run 'make prepare' in $(srctree) to update it)
>  endif


The echo didn't output for some reason, but adding it to the error gives

/home/lsrc/quilt/linux/Makefile:456: *** triggered by /home/lsrc/quilt/linux/drivers/net/wireless/Kconfig /home/lsrc/quilt/linux/drivers/message/fusion/Kconfig /home/lsrc/quilt/linux/net/ieee80211/Kconfig /home/lsrc/quilt/linux/net/netfilter/Kconfig kernel configuration not valid - run 'make prepare' in /home/lsrc/quilt/linux to update it.  Stop.
 
-Andi

  reply	other threads:[~2006-07-30 18:41 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-07-30 16:46 Building external modules against objdirs Andi Kleen
2006-07-30 17:51 ` Sam Ravnborg
2006-07-30 17:49   ` Andi Kleen
2006-07-30 18:31     ` Sam Ravnborg
2006-07-30 18:37       ` Andi Kleen [this message]
2006-07-30 19:17         ` Sam Ravnborg
2006-07-30 20:06           ` Andi Kleen
2006-07-31  9:39           ` Roman Zippel
2006-08-01 19:42             ` Sam Ravnborg
2006-07-30 18:34     ` Sam Ravnborg
2006-07-30 18:42       ` Andi Kleen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=200607302037.02559.ak@suse.de \
    --to=ak@suse.de \
    --cc=agruen@suse.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=sam@ravnborg.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.