All of lore.kernel.org
 help / color / mirror / Atom feed
From: Namhyung Kim <namhyung.kim@lge.com>
To: Peter Zijlstra <peterz@infradead.org>
Cc: Ingo Molnar <mingo@kernel.org>,
	Linus Torvalds <torvalds@linux-foundation.org>,
	Arnaldo Carvalho de Melo <acme@infradead.org>,
	linux-kernel@vger.kernel.org,
	Arjan van de Ven <arjan@infradead.org>,
	David Ahern <dsahern@gmail.com>,
	Frederic Weisbecker <fweisbec@gmail.com>,
	Mike Galbraith <efault@gmx.de>, Paul Mackerras <paulus@samba.org>,
	Stephane Eranian <eranian@google.com>,
	arnaldo.melo@gmail.com,
	Arnaldo Carvalho de Melo <acme@redhat.com>
Subject: Re: [GIT PULL 0/8] Annotation weekly ponies delivery
Date: Wed, 16 May 2012 14:57:49 +0900	[thread overview]
Message-ID: <87liksd5wi.fsf@sejong.aot.lge.com> (raw)
In-Reply-To: <1337101633.27694.92.camel@twins> (Peter Zijlstra's message of "Tue, 15 May 2012 19:07:13 +0200")

Hi,

On Tue, 15 May 2012 19:07:13 +0200, Peter Zijlstra wrote:
> This does indeed make it work again:
>
> pre patch:
>
> [ perf record: Woken up 1 times to write data ]
> [ perf record: Captured and wrote 0.005 MB perf.data (~212 samples) ]
>
> post patch:
>
> [ perf record: Woken up 209 times to write data ]
> [ perf record: Captured and wrote 54.931 MB perf.data (~2399974 samples) ]
>
> So please do send as a proper patch, and you can add:
>
> Acked-and-tested-by: Peter Zijlstra <a.p.zijlstra@chello.nl>

Hmm.. but it seems this breaks my 'perf stat -g' - it doesn't
respond any more (even for ^C):

  namhyung@sejong:perf$ ./perf stat -g sleep 1
  ^C^C^C^C^\Quit (core dumped)

  namhyung@sejong:perf$ gdb ./perf core.11964
  [SNIP]
  Core was generated by `./perf stat -g sleep 1'.
  Program terminated with signal 3, Quit.
  #0  0x0000003fd800e090 in __read_nocancel () from /lib64/libpthread.so.0
  [SNIP]
  (gdb) bt
  #0  0x0000003fd800e090 in __read_nocancel () from /lib64/libpthread.so.0
  #1  0x000000000047fa00 in read (__nbytes=24, __buf=0x7fffc5e1d0b0, __fd=0) at /usr/include/bits/unistd.h:45
  #2  readn (fd=0, buf=<optimized out>, buf@entry=0x7fffc5e1d0b0, n=n@entry=24) at util/util.c:140
  #3  0x000000000043e4bb in __perf_evsel__read (evsel=evsel@entry=0xc7e700, ncpus=<optimized out>, nthreads=1, scale=true)
      at util/evsel.c:267
  #4  0x0000000000422777 in read_counter_aggr (counter=0xc7e700) at builtin-stat.c:360
  #5  run_perf_stat (argv=0x7fffc5e1d510, argc=2) at builtin-stat.c:528
  #6  cmd_stat (argc=2, argv=0x7fffc5e1d510, prefix=<optimized out>) at builtin-stat.c:1251
  #7  0x0000000000414723 in run_builtin (p=p@entry=0x77e1d8, argc=argc@entry=4, argv=argv@entry=0x7fffc5e1d510) at perf.c:273
  #8  0x0000000000413f3b in handle_internal_command (argv=0x7fffc5e1d510, argc=4) at perf.c:345
  #9  run_argv (argv=0x7fffc5e1d300, argcp=0x7fffc5e1d30c) at perf.c:389
  #10 main (argc=4, argv=0x7fffc5e1d510) at perf.c:487


In fact, my perf stat -g was already broken - it cannot count at
all on my SNB box. I'll investigate it more.

Any thoughts?

Namhyung

  reply	other threads:[~2012-05-16  5:59 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-05-12 19:52 [GIT PULL 0/8] Annotation weekly ponies delivery Arnaldo Carvalho de Melo
2012-05-12 19:53 ` [PATCH 1/8] perf annotate: Use raw form for register indirect call instructions Arnaldo Carvalho de Melo
2012-05-12 19:53 ` [PATCH 2/8] perf annotate: Resolve symbols using objdump comment Arnaldo Carvalho de Melo
2012-05-12 19:53 ` [PATCH 3/8] perf annotate: Resolve symbols using objdump comment for single op ins Arnaldo Carvalho de Melo
2012-05-12 19:53 ` [PATCH 4/8] perf annotate: Augment lock instruction output Arnaldo Carvalho de Melo
2012-05-12 19:53 ` [PATCH 5/8] perf annotate: Introduce ->free() method in ins_ops Arnaldo Carvalho de Melo
2012-05-12 19:53 ` [PATCH 6/8] perf annotate browser: Count the numbers of jump sources to a target Arnaldo Carvalho de Melo
2012-05-12 19:53 ` [PATCH 7/8] perf annotate browser: Show 'jumpy' functions Arnaldo Carvalho de Melo
2012-05-12 19:53 ` [PATCH 8/8] perf annotate browser: Add key bindings help window Arnaldo Carvalho de Melo
2012-05-12 20:40 ` [GIT PULL 0/8] Annotation weekly ponies delivery Linus Torvalds
2012-05-12 21:25   ` Arnaldo Carvalho de Melo
2012-05-12 22:29     ` Linus Torvalds
2012-05-14  7:05       ` Ingo Molnar
2012-05-14 11:13   ` Peter Zijlstra
2012-05-14 11:55     ` Ingo Molnar
2012-05-14 15:06       ` Namhyung Kim
2012-05-15  4:48         ` Namhyung Kim
2012-05-15 10:32         ` Peter Zijlstra
2012-05-15 14:44           ` Namhyung Kim
2012-05-15 17:07             ` Peter Zijlstra
2012-05-16  5:57               ` Namhyung Kim [this message]
2012-05-16  8:19                 ` Namhyung Kim
2012-05-17  0:13                 ` Namhyung Kim

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=87liksd5wi.fsf@sejong.aot.lge.com \
    --to=namhyung.kim@lge.com \
    --cc=acme@infradead.org \
    --cc=acme@redhat.com \
    --cc=arjan@infradead.org \
    --cc=arnaldo.melo@gmail.com \
    --cc=dsahern@gmail.com \
    --cc=efault@gmx.de \
    --cc=eranian@google.com \
    --cc=fweisbec@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@kernel.org \
    --cc=paulus@samba.org \
    --cc=peterz@infradead.org \
    --cc=torvalds@linux-foundation.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.