lkml.org 
[lkml]   [2013]   [May]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 1/2] perf report: Add --percent-limit option
On Mon, May 13, 2013 at 05:41:42PM +0900, Namhyung Kim wrote:
> From: Namhyung Kim <namhyung.kim@lge.com>
>
> The --percent-limit option is for not showing small overheaded entries
> in the output. Maybe we want to set a certain default value like 0.1.
>
> Cc: Andi Kleen <andi@firstfloor.org>

I like the option, but 0 is not a good default.
Make it 1 or something.

Perhaps a better heuristic would be
percent < threshold || samples < sample-threshold
with sample-threshold being several hundred at least.

A very small number of hits on something is usually quite much useless,
unless your original event was extremly rare in the beginning
(but then it's unclear if the profile is any useful)

-Andi



\
 
 \ /
  Last update: 2013-05-13 17:21    [W:0.088 / U:1.272 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site