Re: [PATCH/RFC 00/11] perf/s390/kvm: trace events, perf kvm stat

2014-05-02 Thread Jiri Olsa
On Fri, Apr 25, 2014 at 11:12:22AM +0200, Christian Borntraeger wrote: The goal of this series is to - enable perf kvm stat on s390 - get rid of a perf warning insn_to_mnemonic not found when it tries to decode s390 perf trace events perf tools changes look good to me Review feedback is

Re: [PATCH] perf/tool: Fix usage of trace events with '-' in trace system name.

2014-04-23 Thread Jiri Olsa
On Mon, Apr 21, 2014 at 07:43:50PM +0400, Alexander Yarygin wrote: SNIP --- a/tools/perf/tests/parse-events.c +++ b/tools/perf/tests/parse-events.c @@ -1346,6 +1346,12 @@ static struct evlist_test test__events[] = { .name = {cycles,cache-misses,branch-misses}:D,

Re: [PATCH] perf/tool: Fix usage of trace events with '-' in trace system name.

2014-04-17 Thread Jiri Olsa
in this part Acked-by: Jiri Olsa jo...@redhat.com it'd be nice to add test to tests/parse-events.c, probably s390 specific, because the parsing code touches the tracepoint format file thanks, jirka -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to majord

Re: [PATCH] perf/tool: Fix usage of trace events with '-' in trace system name.

2014-04-17 Thread Jiri Olsa
On Thu, Apr 17, 2014 at 01:41:56PM +0200, Christian Borntraeger wrote: On 17/04/14 13:32, Jiri Olsa wrote: On Tue, Mar 25, 2014 at 11:15:29AM +0100, Paolo Bonzini wrote: Il 24/03/2014 21:49, Christian Borntraeger ha scritto: event_legacy_tracepoint: +PE_NAME '-' PE_NAME ':' PE_NAME

Re: [PATCH] perf/tool: Fix usage of trace events with '-' in trace system name.

2014-04-17 Thread Jiri Olsa
On Mon, Mar 24, 2014 at 09:49:00PM +0100, Christian Borntraeger wrote: From: Alexander Yarygin yary...@linux.vnet.ibm.com Trace events potentially can have a '-' in their trace system name, e.g. kvm on s390 defines kvm-s390:* tracepoints. tools/perf could not parse them, because there was no

Re: [PULL 1/2] perf/tool: Fix usage of trace events with '-' in trace system name.

2014-04-16 Thread Jiri Olsa
On Wed, Apr 09, 2014 at 04:21:58PM +0200, Christian Borntraeger wrote: From: Alexander Yarygin yary...@linux.vnet.ibm.com Trace events potentially can have a '-' in their trace system name, e.g. kvm on s390 defines kvm-s390:* tracepoints. tools/perf could not parse them, because there was no

Re: [PULL 1/2] perf/tool: Fix usage of trace events with '-' in trace system name.

2014-04-16 Thread Jiri Olsa
On Wed, Apr 16, 2014 at 03:31:15PM +0200, Christian Borntraeger wrote: On 16/04/14 15:28, Christian Borntraeger wrote: On 16/04/14 15:00, Jiri Olsa wrote: On Wed, Apr 09, 2014 at 04:21:58PM +0200, Christian Borntraeger wrote: From: Alexander Yarygin yary...@linux.vnet.ibm.com Trace

Re: [PULL 0/2] small perf fixes

2014-04-15 Thread Jiri Olsa
On Tue, Apr 15, 2014 at 03:05:02PM +0200, Christian Borntraeger wrote: On 09/04/14 16:21, Christian Borntraeger wrote: Ingo, Arnaldo, two fixes indentified during kvm on s390 development are available since commit 538592ff0b008237ae88f5ce5fb1247127dc3ce5: Merge tag

Re: [PATCH 2/3] kernel,cred,kvm,security - removing superfluous rcu_read_lock_held check

2010-11-02 Thread Jiri Olsa
On Mon, Nov 01, 2010 at 11:42:23PM +0100, Paolo Bonzini wrote: On 11/01/2010 08:15 PM, Jiri Olsa wrote: diff --git a/include/linux/kvm_host.h b/include/linux/kvm_host.h index a055742..a90a7e3 100644 --- a/include/linux/kvm_host.h +++ b/include/linux/kvm_host.h @@ -256,7 +256,6 @@ void

[PATCH 2/3] kernel,cred,kvm,security - removing superfluous rcu_read_lock_held check

2010-11-01 Thread Jiri Olsa
hi, the rcu_dereference_check is defined as #define rcu_dereference_check(p, c) \ __rcu_dereference_check((p), rcu_read_lock_held() || (c), __rcu) so the caller does not need to specify rcu_read_lock_held() condition. wbr, jirka Signed-off-by: Jiri Olsa jo...@redhat.com

[PATCH 3/3] net - removing superfluous rcu_read_lock_held check

2010-11-01 Thread Jiri Olsa
hi, the rcu_dereference_check is defined as #define rcu_dereference_check(p, c) \ __rcu_dereference_check((p), rcu_read_lock_held() || (c), __rcu) so the caller does not need to specify rcu_read_lock_held() condition. wbr, jirka Signed-off-by: Jiri Olsa jo...@redhat.com

[PATCH 1/3] cgroup - removing superfluous rcu_read_lock_held check

2010-11-01 Thread Jiri Olsa
hi, the rcu_dereference_check is defined as #define rcu_dereference_check(p, c) \ __rcu_dereference_check((p), rcu_read_lock_held() || (c), __rcu) so the caller does not need to specify rcu_read_lock_held() condition. wbr, jirka Signed-off-by: Jiri Olsa jo...@redhat.com

[PATCH 0/3] rcu - removing superfluous rcu_read_lock_held check

2010-11-01 Thread Jiri Olsa
somebody wrongly to recipients wbr, jirka Signed-off-by: Jiri Olsa jo...@redhat.com --- include/linux/cgroup.h |1 - include/linux/cred.h |1 - include/linux/fdtable.h|1 - include/linux/kvm_host.h |1 - include/linux/rtnetlink.h