Re: [PATCH 5/5 v2] perf timechart: add emphasize option

From: Arnaldo Carvalho de Melo
Date: Tue Dec 17 2013 - 08:59:35 EST


Em Tue, Dec 17, 2013 at 02:41:37PM +0100, Ingo Molnar escreveu:
> * Arnaldo Carvalho de Melo <acme@xxxxxxxxxxxxxxxxxx> wrote:
> > Em Tue, Dec 17, 2013 at 04:15:22PM +0400, Stanislav Fomichev escreveu:
> > > > Please give a sample usage command line and a sample output. (the
> > > > highlighting can be done using ASCII escape sequences in the changelog
> > > > as well.)
> > > Highlighting is done in the generated SVG, so I can't really show it in
> > > the log or documentation. But I added simple example with -e option.

> > > > -e::
> > > > --emphasize=<duration_nsecs|task_name>::
> > > Thanks, added.

> > I also wonder how to allocate single letter options here... can we live
> > with just --emphasize for now? Wouldn't at some point we maybe want to
> > use -e in 'perf timechart' to pick some specific event, and then, to
> > make it consistent with the other tools, -e would then be used?

> I wanted to raise a similar argument too - so if we don't have to do
> -e then maybe '--grep' would be the better option name?

> If it becomes popular then the pattern matching can be improved,
> without having to change the option name and such.

Humm, 'grep' is a filter, i.e. something that removes parts of the data
being perused, while this case it will not filter anything, it will just
make something that matches some search criteria to stand out from the
rest, that is still there, so perhaps --emphasize is Ok albeit kinda
long...

Do we have any other command line tool that has a long standing
semantics of "highlighting" or "emphasizing" parts of a data stream?

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