All of lore.kernel.org
 help / color / mirror / Atom feed
From: Greg KH <gregkh@linuxfoundation.org>
To: Nick Desaulniers <ndesaulniers@google.com>
Cc: stable@vger.kernel.org, Sami Tolvanen <samitolvanen@google.com>,
	Andrey Konovalov <andreyknvl@google.com>
Subject: Re: kasan patches
Date: Tue, 14 Aug 2018 09:24:29 +0200	[thread overview]
Message-ID: <20180814072429.GC24616@kroah.com> (raw)
In-Reply-To: <CAKwvOd=Vt-act-kYwRvFqezihJKjL4EBZbb5LyR8nBWZi1G2rA@mail.gmail.com>

On Mon, Aug 13, 2018 at 02:25:12PM -0700, Nick Desaulniers wrote:
> On Sat, Aug 11, 2018 at 9:40 AM Greg KH <gregkh@linuxfoundation.org> wrote:
> >
> > On Mon, Aug 06, 2018 at 02:08:32PM -0700, Nick Desaulniers wrote:
> > > These patches are needed for kasan+clang support.  I confirmed they
> > > apply cleanly in order (top to bottom):
> > >
> > > 4.9:
> > > commit c5caf21ab0cf8 ("kasan: turn on -fsanitize-address-use-after-scope")
> > > commit 0e410e158e5b ("kasan: don't emit builtin calls when sanitization is off")
> >
> > By applying these, I get a ton of new build warnings, so I can't add
> > them :(
> 
> What? Such innocent patches.
> 
> $ git remote add stable
> git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git
> $ git checkout -b 4.9-stable-kasan stable/linux-4.9.y
> <git log shows 4.9.118>
> $ git cherry-pick c5caf21ab0cf8
> $ git cherry-pick 0e410e158e5b
> $ make defconfig
> $ make -j46
> 
> I get -Wstringop-truncation, -Wstringop-overflow=,
> -Wpacked-not-aligned, -Warray-bounds warnings with gcc-8 with or
> without these patches.
> 
> Can you share your config, or what the warning was?  Did I checkout
> the wrong branch?

I used 'make allmodconfig' on a x86 Fedora 27 release which is using gcc
7.3.1.  Adding those patches adds a ton of warnings.  Let me go do a
test build and send you the full log...

thanks,

greg k-h

  reply	other threads:[~2018-08-14 10:10 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-08-06 21:08 kasan patches Nick Desaulniers
2018-08-11 16:40 ` Greg KH
2018-08-13 21:25   ` Nick Desaulniers
2018-08-14  7:24     ` Greg KH [this message]
2018-08-14  7:37       ` Greg KH
2018-08-14 12:44         ` Andrey Konovalov
2018-08-14 13:51           ` Greg KH
2018-08-14 20:33           ` Nick Desaulniers
2018-08-14 20:56             ` Arnd Bergmann
2018-08-14 22:04               ` Nick Desaulniers

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=20180814072429.GC24616@kroah.com \
    --to=gregkh@linuxfoundation.org \
    --cc=andreyknvl@google.com \
    --cc=ndesaulniers@google.com \
    --cc=samitolvanen@google.com \
    --cc=stable@vger.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 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.