Re: [PATCH 3/5] perf diff: Print diff result more precisely

[Namhyung Kim]

On Mon, Jan 5, 2015 at 1:47 AM, Jiri Olsa <jolsa@xxxxxxxxxx> wrote:
> On Sat, Dec 27, 2014 at 02:06:31PM +0900, Namhyung Kim wrote:
>> Current perf diff result is somewhat confusing since it sometimes hide
>> small result and sometimes there's no result.  So do not hide small
>> result (less than 0.01%) and print "N/A" if baseline is not
>> recorded (for ratio and wdiff only).  Blank means the baseline is
>> available but its pairs are not.
>
> I think we should document this, especialy when diff has different
> behaviour from the rest of the computation.. :-\

Well, probably I'm missing something but I don't know what needs to be
documented.  Currently perf diff doesn't show small result so it's
hard to distinguish entries that have small result and no result.  But
to sort the output properly, it needs to show the results and I think
it's natural to not hide the result to user.


>
> anyway for the change itself:
>
> Acked-by: Jiri Olsa <jolsa@xxxxxxxxxx>

Thanks,
Namhyung
--
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/