linux-kernel - Re: [PATCH v13 00/23] Compile-time stack metadata validation

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Wed, 11 Nov 2015 12:14:58 -0600
From:	Josh Poimboeuf <jpoimboe@...hat.com>
To:	Ingo Molnar <mingo@...nel.org>
Cc:	Ingo Molnar <mingo@...hat.com>, linux-kernel@...r.kernel.org,
	live-patching@...r.kernel.org, Michal Marek <mmarek@...e.cz>,
	Peter Zijlstra <peterz@...radead.org>,
	Andy Lutomirski <luto@...nel.org>,
	Borislav Petkov <bp@...en8.de>,
	Linus Torvalds <torvalds@...ux-foundation.org>,
	Andi Kleen <andi@...stfloor.org>,
	Pedro Alves <palves@...hat.com>,
	Namhyung Kim <namhyung@...il.com>,
	Bernd Petrovitsch <bernd@...rovitsch.priv.at>,
	Chris J Arges <chris.j.arges@...onical.com>,
	Andrew Morton <akpm@...ux-foundation.org>,
	David Vrabel <david.vrabel@...rix.com>,
	Konrad Rzeszutek Wilk <konrad.wilk@...cle.com>,
	Boris Ostrovsky <boris.ostrovsky@...cle.com>,
	Jeremy Fitzhardinge <jeremy@...p.org>,
	Chris Wright <chrisw@...s-sol.org>,
	Alok Kataria <akataria@...are.com>,
	Rusty Russell <rusty@...tcorp.com.au>,
	Herbert Xu <herbert@...dor.apana.org.au>,
	"David S. Miller" <davem@...emloft.net>,
	Pavel Machek <pavel@....cz>,
	"Rafael J. Wysocki" <rjw@...ysocki.net>,
	Len Brown <len.brown@...el.com>,
	Matt Fleming <matt.fleming@...el.com>,
	Thomas Gleixner <tglx@...utronix.de>,
	"H. Peter Anvin" <hpa@...or.com>, x86@...nel.org
Subject: Re: [PATCH v13 00/23] Compile-time stack metadata validation

Hi Ingo,

Do you still have objections to my proposed command-line interface for
stacktool?

As I described below, I still don't think subcommands are a good fit for
stacktool.  However, if you strongly disagree, I can change it.

The patch set is now quite mature at v14.  Further, it's holding up work
on several other items:

- more frame pointer fixes
- CFI generation and validation
- livepatch consistency model
- DWARF unwinder

So I'm wondering if it would be possible for you to merge stacktool in
time for the next merge window (4.5).

Thanks!
Josh

On Mon, Oct 12, 2015 at 09:23:14AM -0500, Josh Poimboeuf wrote:
> On Mon, Oct 12, 2015 at 09:41:11AM +0200, Ingo Molnar wrote:
> > 
> > * Josh Poimboeuf <jpoimboe@...hat.com> wrote:
> > 
> > > Hi Ingo,
> > > 
> > > Do you have any more objections to these patches?  Would you be willing
> > > to apply them?
> > 
> > So I still don't like the tool namespace you picked: Git-alike generic
> > naming plus subcommands work so much better that I'm not sure why we
> > are even having that discussion:
> 
> Because subcommands are useful in _some_ cases, but they aren't a
> panacea that should be blindly applied everywhere.
> 
> > if you name your tool 'stacktool' and
> > alias everything you have today to under 'stacktool run ...' and add
> > 'stacktool help' as a second, obvious subcommand then you'll have your
> > current syntax and a lot more future flexibility and ability to branch
> > off various functionality a'la Git, perf or kvmtool ...
> 
> Sure, subcommands work great for monolithic framework tools like git,
> perf, yum, docker, etc.  But stacktool is not (and never will be) a
> monolithic framework type of tool.
> 
> The suggestion to put 100% of the functionality under 'stacktool run
> [options]' is certainly possible.  But 'run' is so broad.  What else
> could the tool ever do but 'run'?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/