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


Groups > linux.kernel > #1244451 > unrolled thread

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

Started byIngo Molnar <mingo@kernel.org>
First post2015-10-12 09:50 +0200
Last post2015-10-12 16:30 +0200
Articles 2 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH v13 00/23] Compile-time stack metadata validation Ingo Molnar <mingo@kernel.org> - 2015-10-12 09:50 +0200
    Re: [PATCH v13 00/23] Compile-time stack metadata validation Josh Poimboeuf <jpoimboe@redhat.com> - 2015-10-12 16:30 +0200

#1244451 — Re: [PATCH v13 00/23] Compile-time stack metadata validation

FromIngo Molnar <mingo@kernel.org>
Date2015-10-12 09:50 +0200
SubjectRe: [PATCH v13 00/23] Compile-time stack metadata validation
Message-ID<qiG1P-MT-15@gated-at.bofh.it>
* Josh Poimboeuf <jpoimboe@redhat.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: 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 ...

It's an 100% equivalent superset of your current stacktool's parameter structure, 
so none of your prior objections apply.

Thanks,

	Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]


#1244747

FromJosh Poimboeuf <jpoimboe@redhat.com>
Date2015-10-12 16:30 +0200
Message-ID<qiMgX-1zC-27@gated-at.bofh.it>
In reply to#1244451
On Mon, Oct 12, 2015 at 09:41:11AM +0200, Ingo Molnar wrote:
> 
> * Josh Poimboeuf <jpoimboe@redhat.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'?

-- 
Josh
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web