mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Arjan van de Ven <arjan@infradead.org>
To: Andrew Morton <akpm@linux-foundation.org>
Cc: Ingo Molnar <mingo@elte.hu>, Thomas Gleixner <tglx@linutronix.de>,
	sam@ravnborg.org, arjan@linux.intel.com,
	linux-kernel@vger.kernel.org
Subject: Stack protector build failure (was Re: 2.6.25-mm1: not looking good)
Date: Fri, 18 Apr 2008 06:58:06 -0700	[thread overview]
Message-ID: <20080418065806.35c73f96@laptopd505.fenrus.org> (raw)
In-Reply-To: <20080418002858.de236663.akpm@linux-foundation.org>

On Fri, 18 Apr 2008 00:28:58 -0700
Andrew Morton <akpm@linux-foundation.org> wrote:


> 
> > No harm done on a 
> > perfectly bug-free system - but once a bug happens that SELinux
> > should have mitigated, the breakage becomes real. Having a
> > prominent warning is the _minimum_.
> > 
> > having a build failure would be nice too because this is a build 
> > environment problem. (not a build warning - warnings can easily be 
> > missed because on a typical kernel build there's so many false
> > positives that get emitted by various other warning mechanisms)
> > Arjan?
> > 
> 
> Yeah, #error would work too.

I'm totally fine with that, but I think I need Sam's help on making that happen
the right way; this is going to need makefile fu L(

Sam:
Basically what I need is that if the
scripts/gcc-x86_64-has-stack-protector.sh script fails, the build aborts with
a message/#error that says that the compiler is not capable of supporting this feature.

Right now the script is used like this:

	stackp := $(CONFIG_SHELL) $(srctree)/scripts/gcc-x86_64-has-stack-protector.sh
        stackp-$(CONFIG_CC_STACKPROTECTOR) := $(shell $(stackp) \
                "$(CC)" -fstack-protector )

It's obviously easy to make this script print a warning.. but how do we make it stop the build?

-- 
If you want to reach me at my work email, use arjan@linux.intel.com
For development, discussion and tips for power savings, 
visit http://www.lesswatts.org

  parent reply	other threads:[~2008-04-18 13:58 UTC|newest]

Thread overview: 54+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-04-17 23:03 2.6.25-mm1: not looking good Andrew Morton
2008-04-17 23:24 ` Greg KH
2008-04-18  0:48   ` Kay Sievers
2008-04-18  1:12     ` Andrew Morton
2008-04-18  4:07     ` Andrew Morton
2008-04-17 23:24 ` Dan Williams
2008-04-17 23:40 ` Andrew Morton
2008-04-18  0:14   ` Andrew Morton
2008-04-18  3:05     ` Jason Wessel
2008-04-18  7:37       ` Ingo Molnar
2008-04-18 11:46         ` Vegard Nossum
2008-04-18 12:34           ` Ingo Molnar
2008-04-18 12:41             ` Vegard Nossum
2008-04-18 13:02               ` Jason Wessel
2008-04-18 13:22                 ` Vegard Nossum
2008-04-18 13:27                   ` Jason Wessel
2008-04-18 14:47                     ` Vegard Nossum
2008-04-18 16:02                       ` Vegard Nossum
2008-04-18 21:54         ` Jason Wessel
2008-04-17 23:55 ` Paul Moore
2008-04-18  0:04   ` Andrew Morton
2008-04-18 14:55     ` Paul Moore
2008-04-18  1:35   ` Andrew Morton
2008-04-18 14:57     ` Paul Moore
2008-04-18  5:49 ` Arjan van de Ven
2008-04-18  6:10   ` Andrew Morton
2008-04-18  7:19     ` Ingo Molnar
2008-04-18  7:28       ` Andrew Morton
2008-04-18  9:28         ` Ingo Molnar
2008-04-18 13:58         ` Arjan van de Ven [this message]
2008-04-18 16:57           ` Stack protector build failure (was Re: 2.6.25-mm1: not looking good) Arjan van de Ven
2008-04-18  6:40 ` 2.6.25-mm1: not looking good Pekka Enberg
2008-04-18  6:56   ` Andrew Morton
2008-04-18  7:24   ` Ingo Molnar
2008-04-18  7:25     ` Pekka Enberg
2008-04-18 10:32     ` James Morris
2008-04-18  7:09 ` Ingo Molnar
2008-04-18  7:50 ` Andrew Morton
2008-04-18  7:53   ` Andrew Morton
2008-04-18  7:57     ` Andrew Morton
2008-04-18  9:22       ` Ingo Molnar
2008-04-18 12:18         ` Ingo Molnar
2008-04-18  9:42     ` Pavel Machek
2008-04-18 15:22       ` Alan Stern
2008-04-18 11:07     ` Pavel Machek
2008-04-28 16:42 ` 2.6.25-mm1: Failing to probe IDE interface Mel Gorman
2008-04-28 16:59   ` Andrew Morton
2008-04-29  9:39     ` Mel Gorman
2008-04-28 18:44   ` Bartlomiej Zolnierkiewicz
2008-04-29  9:43     ` Mel Gorman
2008-04-29 15:49       ` Mel Gorman
2008-04-29 16:58         ` Mel Gorman
2008-04-29 21:37           ` Bartlomiej Zolnierkiewicz
2008-04-30 11:16             ` Mel Gorman

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=20080418065806.35c73f96@laptopd505.fenrus.org \
    --to=arjan@infradead.org \
    --cc=akpm@linux-foundation.org \
    --cc=arjan@linux.intel.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@elte.hu \
    --cc=sam@ravnborg.org \
    --cc=tglx@linutronix.de \
    /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

all inboxes | Powered by JetHome®