lkml.org 
[lkml]   [2011]   [Feb]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH 1/6] perf-events: Add support for supplementary event registers v5
From
Date
On Tue, 2011-02-22 at 23:20 +0800, Peter Zijlstra wrote:
> On Mon, 2011-02-21 at 00:56 +0800, Lin Ming wrote:
> > @@ -225,7 +225,10 @@ struct perf_event_attr {
> > };
> >
> > __u32 bp_type;
> > - __u64 bp_addr;
> > + union {
> > + __u64 bp_addr;
> > + __u64 event_extra; /* Extra for some events */
> > + };
> > __u64 bp_len;
> > };
>
> I did s/event_extra/config1/g and:
>
> Index: linux-2.6/include/linux/perf_event.h
> ===================================================================
> --- linux-2.6.orig/include/linux/perf_event.h
> +++ linux-2.6/include/linux/perf_event.h
> @@ -227,9 +230,12 @@ struct perf_event_attr {
> __u32 bp_type;
> union {
> __u64 bp_addr;
> - __u64 config1; /* Extra for some events */
> + __u64 config1; /* extension of config0 */
> + };
> + union {
> + __u64 bp_len;
> + __u64 config2; /* extension of config1 */
> };
> - __u64 bp_len;
> };
>
> /*
>
> Giving us basically 192 bits of config{,1,2} space and the possibility
> of eventually deprecating the bp_ stuff.

OK,

Do you mean it's possible to encode bp_type/bp_addr/bp_len into
config1/config2?




\
 
 \ /
  Last update: 2011-02-23 02:11    [W:0.292 / U:0.012 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site