linux-arch.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Sam Ravnborg <sam@ravnborg.org>
To: linux arch <linux-arch@vger.kernel.org>, Andi Kleen <ak@suse.de>,
	kbuild devel <kbuild-devel@lists.sourceforge.net>,
	Tony Luck <tony.luck@intel.com>
Cc: Sam Ravnborg <sam@ravnborg.org>
Subject: kbuild: enable possibility to specify xFLAGS on commandline
Date: Sun, 9 Sep 2007 20:02:54 +0200	[thread overview]
Message-ID: <20070909180254.GA20086@uranus.ravnborg.org> (raw)

Following patchset enable the possibility to specify 
CFLAGS, AFLAGS or CPPFLAGS on the commandline to add
additional options while building the kernel.

The patch prefix all uses of CFLAGS, AFLAGS and CPPFLAGS
with KBUILD_ which touches all arch Makefiles.

Regression testing was very simple. Since this patch did not
change behaviour adding the patch should not cause any recompile
and this was tested with defconfig on several architectures.
To be more specific on:
alpha arm i386 mips sparc sparc64 x86_64 ia64 m68k s390

powerpc I dave a toolchain but defconfig seems not to be present.
blackfin and um did not build.

A small cleanup patch for ia64 sneaked in too - to allow the
above mentioned regression test.

The patchset will conflict with a patch from Andi Kleen.
If the patch for x86_64 will be pushed for -rc - no troubles.
If the patch await next mergewindow I could take it in my tree
to avoid the conflict.

The patch in question is:
ftp://ftp.firstfloor.org/pub/ak/x86_64/quilt/patches/cflags-probe


The purpose of this patch is to make it much simpler to try out
different gcc options.
The receipe is the following:

make CFLAGS=-fmuch-quicker-kernel mm/slub.o
make KBUILD_NOCMDDEP=1

The KBUILD_NOCMDDEP=1 tell kbuild not to check any commandline
arguments and can be used to change options AND compiler.
Without this kbuild would detech changes in commandline options
and rebuild mm/slub.o again.

Any objections to this change?

	Sam


             reply	other threads:[~2007-09-09 18:01 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-09-09 18:02 Sam Ravnborg [this message]
2007-09-09 18:04 ` [PATCH 1/4] kbuild: enable 'make CFLAGS=...' to add additional options to CC Sam Ravnborg
2007-09-09 18:05 ` [PATCH 2/4] kbuild: enable 'make AFLAGS=...' to add additional options to AS Sam Ravnborg
2007-09-09 18:06 ` [PATCH 3/4] ia64: fix sn to add include files using EXTRA_CFLAGS Sam Ravnborg
2007-09-09 22:45   ` CPP includes ([PATCH 3/4] ia64: fix sn to add include files using EXTRA_CFLAGS) Oleg Verych
2007-09-11 13:48   ` [PATCH 3/4] ia64: fix sn to add include files using EXTRA_CFLAGS Jes Sorensen
2007-09-09 18:07 ` [PATCH 4/4] kbuild: enable 'make CPPFLAGS=...' to add additional options to CPP Sam Ravnborg
2007-09-09 18:26 ` kbuild: enable possibility to specify xFLAGS on commandline Segher Boessenkool
2007-09-10  1:17   ` Stephen Rothwell

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=20070909180254.GA20086@uranus.ravnborg.org \
    --to=sam@ravnborg.org \
    --cc=ak@suse.de \
    --cc=kbuild-devel@lists.sourceforge.net \
    --cc=linux-arch@vger.kernel.org \
    --cc=tony.luck@intel.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).