public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Ingo Molnar <mingo@elte.hu>
To: "Metzger, Markus T" <markus.t.metzger@intel.com>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>,
	"tglx@linutronix.de" <tglx@linutronix.de>,
	"hpa@zytor.com" <hpa@zytor.com>,
	"markus.t.metzger@gmail.com" <markus.t.metzger@gmail.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [patch] x86, perf_counter, bts: add bts to perf_counter
Date: Fri, 7 Aug 2009 13:29:31 +0200	[thread overview]
Message-ID: <20090807112931.GA31401@elte.hu> (raw)
In-Reply-To: <20090807112634.GC30014@elte.hu>


* Ingo Molnar <mingo@elte.hu> wrote:

> btw., the number of samples seems to be varying too widely:
> 
> titan:~> for ((i=0;i<10;i++)); do perf record -f -e branches:u -c 1 true 2>/dev/null; perf report | head -1; done
> # Samples: 28784
> # Samples: 24063
> # Samples: 22788
> # Samples: 30449
> # Samples: 15335
> # Samples: 30557
> # Samples: 24010
> # Samples: 23866
> # Samples: 24877
> # Samples: 24330
> 
> compared to the branch-stat itself:
> 
> titan:~> perf stat -v --repeat 10 -e branches:u true
> [ perf stat: executing run #1 ... ]
> [ perf stat: executing run #2 ... ]
> [ perf stat: executing run #3 ... ]
> [ perf stat: executing run #4 ... ]
> [ perf stat: executing run #5 ... ]
> [ perf stat: executing run #6 ... ]
> [ perf stat: executing run #7 ... ]
> [ perf stat: executing run #8 ... ]
> [ perf stat: executing run #9 ... ]
> [ perf stat: executing run #10 ... ]
> 
>  Performance counter stats for 'true' (10 runs):
> 
>           23851  branches                   ( +-   0.000% )
> 
>     0.000639653  seconds time elapsed   ( +-   2.474% )
> 
> do we lose records in the recording?

i doubt it's lost records. Even with SCHED_FIFO sampling and with a 
huge, 512 MB mmap ring-buffer we get a BTS sample count variation in 
the +- 10% range:

titan:/home/mingo> for ((i=0;i<10;i++)); do perf record -r 1 -m 
131072 -f -e branches:u -c 1 true 2>/dev/null; perf report | head 
-1; done
# Samples: 24860
# Samples: 24177
# Samples: 26165
# Samples: 25682
# Samples: 29175
# Samples: 23136
# Samples: 27102
# Samples: 29888
# Samples: 25524
# Samples: 24266

(and these are all just user-mode executions)

	Ingo

  reply	other threads:[~2009-08-07 11:29 UTC|newest]

Thread overview: 60+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-07-21 13:56 [patch] x86, perf_counter, bts: add bts to perf_counter Markus Metzger
2009-08-04 10:24 ` Peter Zijlstra
2009-08-04 11:14   ` Ingo Molnar
2009-08-04 11:35     ` Ingo Molnar
2009-08-05 11:10       ` Metzger, Markus T
2009-08-07  7:29       ` Metzger, Markus T
2009-08-07  8:21         ` Peter Zijlstra
2009-08-07 10:18           ` Metzger, Markus T
2009-08-07 10:29             ` Peter Zijlstra
2009-08-07 12:18               ` Metzger, Markus T
2009-08-07 13:05                 ` Peter Zijlstra
2009-08-07 10:31             ` Ingo Molnar
2009-08-07 10:36               ` Ingo Molnar
2009-08-07 10:57                 ` Metzger, Markus T
2009-08-07 11:17                   ` Metzger, Markus T
2009-08-07 11:24                     ` Ingo Molnar
2009-08-07 11:33                       ` Ingo Molnar
2009-08-07 17:49                         ` [PATCH] perf_counter: Fix a race on perf_counter_ctx Peter Zijlstra
2009-08-08 11:52                           ` [tip:perfcounters/core] " tip-bot for Peter Zijlstra
2009-08-08 12:03                           ` [PATCH] " Ingo Molnar
2009-08-10 12:33                             ` Metzger, Markus T
2009-08-10 12:59                               ` Peter Zijlstra
2009-08-10 13:46                               ` Ingo Molnar
2009-08-11  6:33                                 ` Metzger, Markus T
2009-08-11  8:59                                 ` Metzger, Markus T
2009-08-18 12:49                                   ` Metzger, Markus T
2009-08-18 12:59                                     ` Peter Zijlstra
2009-08-18 12:59                                       ` Peter Zijlstra
2009-08-18 13:20                                         ` Metzger, Markus T
2009-08-18 13:37                                           ` Peter Zijlstra
2009-08-18 13:54                                             ` Metzger, Markus T
2009-08-18 14:00                                               ` Ingo Molnar
2009-08-18 14:15                                                 ` Metzger, Markus T
2009-08-28  9:56                                                   ` Metzger, Markus T
2009-09-01 11:17                                                     ` [discuss] BTS overflow handling, was: " Metzger, Markus T
2009-09-01 13:00                                                       ` Peter Zijlstra
2009-09-01 13:09                                                         ` Ingo Molnar
2009-09-01 13:32                                                         ` Metzger, Markus T
2009-09-01 13:53                                                           ` Peter Zijlstra
2009-09-01 14:27                                                             ` Metzger, Markus T
2009-09-01 14:35                                                               ` Peter Zijlstra
2009-09-03 14:25                                                             ` Metzger, Markus T
2009-08-18 14:01                                               ` Peter Zijlstra
2009-08-18 14:19                                                 ` Metzger, Markus T
2009-08-09 11:10                           ` [tip:perfcounters/core] " tip-bot for Peter Zijlstra
2009-08-07 11:26                     ` [patch] x86, perf_counter, bts: add bts to perf_counter Ingo Molnar
2009-08-07 11:29                       ` Ingo Molnar [this message]
2009-08-07 12:14                         ` Metzger, Markus T
2009-08-07 11:19                   ` Ingo Molnar
2009-08-07 11:34                     ` Metzger, Markus T
2009-08-07 11:37                     ` Metzger, Markus T
2009-08-04 11:37 ` [tip:perfcounters/urgent] x86, perf_counter, bts: Add BTS support to perfcounters tip-bot for Markus Metzger
2009-08-07 10:10 ` [tip:perfcounters/core] " tip-bot for Markus Metzger
2009-08-08 11:43 ` tip-bot for Markus Metzger
2009-08-08 11:53 ` tip-bot for Markus Metzger
  -- strict thread matches above, loose matches on Subject: below --
2009-08-07  7:22 [patch] x86, perf_counter, bts: add bts to perf_counter Markus Metzger
2009-08-07 11:45 ` Peter Zijlstra
2009-08-07 11:51   ` Metzger, Markus T
2009-08-07 11:57     ` Peter Zijlstra
2009-08-07 16:39 ` Peter Zijlstra

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20090807112931.GA31401@elte.hu \
    --to=mingo@elte.hu \
    --cc=a.p.zijlstra@chello.nl \
    --cc=hpa@zytor.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=markus.t.metzger@gmail.com \
    --cc=markus.t.metzger@intel.com \
    --cc=tglx@linutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox