mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Jiri Kosina <jikos@kernel.org>
To: "Toralf Förster" <toralf.foerster@gmx.de>
Cc: Alexander Tsoy <alexander@tsoy.me>,
	Andy Lutomirski <luto@amacapital.net>,
	Linus Torvalds <torvalds@linux-foundation.org>,
	stable <stable@vger.kernel.org>,
	Linux Kernel <linux-kernel@vger.kernel.org>,
	the arch/x86 maintainers <x86@kernel.org>,
	jpoimboe@redhat.com
Subject: Re: 4.14.9 doesn't boot (regression)
Date: Sat, 30 Dec 2017 14:14:57 +0100 (CET)	[thread overview]
Message-ID: <alpine.LRH.2.00.1712301404570.1449@gjva.wvxbf.pm> (raw)
In-Reply-To: <755e371f-576c-419d-b8fe-66232e8de680@gmx.de>

On Sat, 30 Dec 2017, Toralf Förster wrote:

> This made the issue go away :
> 
> diff --git a/Makefile b/Makefile
> index ac8c441866b7..11a12947c550 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -414,7 +414,7 @@ LINUXINCLUDE    := \
>  
>  KBUILD_AFLAGS   := -D__ASSEMBLY__
>  KBUILD_CFLAGS   := -Wall -Wundef -Wstrict-prototypes -Wno-trigraphs \
> -                  -fno-strict-aliasing -fno-common -fshort-wchar \
> +                  -fno-strict-aliasing -fno-common -fshort-wchar -fstack-check=no \
>                    -Werror-implicit-function-declaration \
>                    -Wno-format-security \
>                    -std=gnu89
> 
> But this doesn't solve the root cause, right ? So if the root cause is 
> "Gentoo hardened GCC is broken" please just let me know this - FWIW I'm 
> in #gentoo-dev on freenode.

-fstack-check for kernel is never going to work properly.

That option is purely for userspace, and assumes all the logic around 
'stack guard gap' and the auto-growing semantics being in place; which is 
there for user stack VMA, but definitely not for kernel stack.

It's probably the "hardened" flavor of your distro trying to push 
'-fstack-check' to everything it compiles; so I actually think the 
Makefile patch, sanitizing CFLAGS by force-disabling -fstack-check is 
exactly what we should be doing.

Thanks,

-- 
Jiri Kosina
SUSE Labs

  reply	other threads:[~2017-12-30 13:15 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-12-29 11:14 Toralf Förster
2017-12-29 13:33 ` Sebastian Gottschall
2017-12-29 13:38   ` Toralf Förster
2017-12-29 15:29     ` Andy Shevchenko
2017-12-29 15:48 ` Alexander Tsoy
2017-12-29 15:52   ` Toralf Förster
2017-12-29 20:12 ` Linus Torvalds
2017-12-29 20:21   ` Ingo Molnar
2017-12-29 21:02   ` Toralf Förster
2017-12-29 21:17     ` Linus Torvalds
2017-12-29 21:39       ` Linus Torvalds
2017-12-29 22:04         ` Alexander Tsoy
2017-12-29 22:30       ` Toralf Förster
2017-12-29 22:53         ` Linus Torvalds
2017-12-29 23:14           ` Toralf Förster
2017-12-30  0:10           ` Andy Lutomirski
2017-12-30  1:00             ` Linus Torvalds
2017-12-30  1:34               ` Linus Torvalds
2017-12-30  9:14                 ` Alexander Tsoy
2017-12-30  9:21                   ` Toralf Förster
2017-12-30  1:13             ` Alexander Tsoy
2017-12-30 12:58               ` Toralf Förster
2017-12-30 13:14                 ` Jiri Kosina [this message]
2017-12-30  3:49             ` Josh Poimboeuf
2017-12-30  8:45               ` Alexander Tsoy
2017-12-30 17:09                 ` Josh Poimboeuf
2017-12-30 17:57                   ` Josh Poimboeuf
2017-12-30 22:03                     ` Alexander Tsoy
2017-12-30 22:16                       ` Josh Poimboeuf
2017-12-30  9:30               ` Toralf Förster
2017-12-30  8:32             ` Toralf Förster
2017-12-30  9:53           ` Jiri Kosina

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=alpine.LRH.2.00.1712301404570.1449@gjva.wvxbf.pm \
    --to=jikos@kernel.org \
    --cc=alexander@tsoy.me \
    --cc=jpoimboe@redhat.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=luto@amacapital.net \
    --cc=stable@vger.kernel.org \
    --cc=toralf.foerster@gmx.de \
    --cc=torvalds@linux-foundation.org \
    --cc=x86@kernel.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 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®