Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1308254

Re: [PATCH v15 00/25] Compile-time stack metadata validation

Show all headers | View raw

* Borislav Petkov <bp@alien8.de> wrote:

> On Fri, Dec 18, 2015 at 06:39:14AM -0600, Josh Poimboeuf wrote:
> > This is v15 of the compile-time stack metadata validation patch set,
> > along with proposed fixes for many of the warnings it found.  It's based
> > on the tip/master branch.
> 
> Ok,
> 
> Reviewed-by: Borislav Petkov <bp@suse.de>
> 
> With it applied, my .config gives ~1000 stacktool warnings.

That's way too high!

We should really strive for zero warnings on common configs - otherwise we'll just 
drown in the noise of warnings, and people won't notice (and won't care about) new 
warnings.

With a zero warnings baseline on a reasonable default kernel config people can 
test for new warnings.

> One of the issues that bugs me recently is adding tool-specific markers
> to unrelated code and new tools tend to love doing that. But this makes
> unrelated code ugly and people touching it shouldn't have to know about
> those tools.
> 
> I guess you could improve the analysis of vmlinux as stacktool is
> running post-compile and thus put the onus on the tool to do the right
> thing and not on the code.
> 
> But that's for the TODO.

So if we still have 1,000 warnings, then it's more than a TODO - it's more like a 
MUSTFIX! :-)

Thanks,

	Ingo

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

Re: [PATCH v15 00/25] Compile-time stack metadata validation Borislav Petkov <bp@alien8.de> - 2016-01-12 18:20 +0100
  Re: [PATCH v15 00/25] Compile-time stack metadata validation Josh Poimboeuf <jpoimboe@redhat.com> - 2016-01-12 19:00 +0100
  Re: [PATCH v15 00/25] Compile-time stack metadata validation Ingo Molnar <mingo@kernel.org> - 2016-01-13 11:20 +0100

csiph-web