Re: [PATCH] perf tools: Fix reading of perf.data file header

From: Ingo Molnar
Date: Fri Aug 07 2009 - 03:45:38 EST



* Brice Goglin <Brice.Goglin@xxxxxxxx> wrote:

> Ingo Molnar wrote:
> > Ok, there's indeed a flag clash with -s/--sort as you noticed.
> >
> > -S looks good to me, how about going one step further and changing
> > perf record to use -S/--stat as well, to make the flag consistent
> > across all tools?
>
> Sure, but perf stat already uses -S for --scale :)

heh :-)

I completely forgot about it. And since we now have --scale enabled
by default (and it doesnt make much sense to disable it i guess), it
would not be a big issue to rename that -c/--scale to free up that
flag for -S/--stat?

Ingo
--
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/