[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <7c86c4470812151403s5c8c1fe3u4c7c9e7f67e0055b@mail.gmail.com>
Date: Mon, 15 Dec 2008 23:03:45 +0100
From: "stephane eranian" <eranian@...glemail.com>
To: "Paul Mackerras" <paulus@...ba.org>
Cc: "Vince Weaver" <vince@...ter.net>, "Ingo Molnar" <mingo@...e.hu>,
linux-kernel@...r.kernel.org,
"Thomas Gleixner" <tglx@...utronix.de>,
"Andrew Morton" <akpm@...ux-foundation.org>,
"Eric Dumazet" <dada1@...mosbay.com>,
"Robert Richter" <robert.richter@....com>,
"Arjan van de Ven" <arjan@...radead.org>,
"Peter Anvin" <hpa@...or.com>,
"Peter Zijlstra" <a.p.zijlstra@...llo.nl>,
"David S. Miller" <davem@...emloft.net>,
perfctr-devel@...ts.sourceforge.net
Subject: Re: [patch] Performance Counters for Linux, v4
On Mon, Dec 15, 2008 at 10:42 PM, Paul Mackerras <paulus@...ba.org> wrote:
> Vince Weaver writes:
>
>> I see a large (2300 instruction) fixed overhead when measuring
>> retired instruction count using the "timec" command
>> compared to the "pfmon" tool that comes with perfmon3
>> (the pfmon tool has essentially no overhead when
>> doing aggragate counts).
>
> Looks like timec will be counting the fork() and execvp() system calls
> that are used to run your executable, as well as the executable
> itself. The fork() overhead could be removed fairly easily I think,
> the execvp would be hard to get rid of without using ptrace() - and
> the use of ptrace was one of the things that Ingo et al. objected to
> in perfmon3.
>
Paul, I think your analysis is correct. This is likely what is happening.
Not that timec could not use ptrace() to block the task from executing
its first instruction, but you'd still have a problem because of prctl(ENABLE)
which applies to the current task, not another task, unless I am mistaken.
Prctl() looks odd to me because you have all those supposedly independent
file descriptors to identify events you want to measure, but they are not
used to start/stop. If you are attaching to multiple tasks at the same time
which you can do with the current API, you may not necessarily want to
start/stop all counters at the same time.
Looks like prctl() is not what we want after all...
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists