On Thu, May 05, 2016 at 04:04:00PM -0700, Andi Kleen wrote:

SNIP

> +
> +EVENT_ATTR_STR_HT(topdown-total-slots, td_total_slots,
> +     "event=0x3c,umask=0x0",                 /* cpu_clk_unhalted.thread */
> +     "event=0x3c,umask=0x0,any=1");          /* cpu_clk_unhalted.thread_any 
> */
> +EVENT_ATTR_STR_HT(topdown-total-slots.scale, td_total_slots_scale, "4", "2");
> +EVENT_ATTR_STR(topdown-slots-issued, td_slots_issued,
> +     "event=0xe,umask=0x1");                 /* uops_issued.any */
> +EVENT_ATTR_STR(topdown-slots-retired, td_slots_retired,
> +     "event=0xc2,umask=0x2");                /* uops_retired.retire_slots */
> +EVENT_ATTR_STR(topdown-fetch-bubbles, td_fetch_bubbles,
> +     "event=0x9c,umask=0x1");                /* idq_uops_not_delivered_core 
> */
> +EVENT_ATTR_STR_HT(topdown-recovery-bubbles, td_recovery_bubbles,
> +     "event=0xd,umask=0x3,cmask=1",          /* int_misc.recovery_cycles */
> +     "event=0xd,umask=0x3,cmask=1,any=1");   /* int_misc.recovery_cycles_any 
> */
> +EVENT_ATTR_STR_HT(topdown-recovery-bubbles.scale, td_recovery_bubbles_scale,
> +     "4", "2");
> +
>  struct attribute *snb_events_attrs[] = {
>       EVENT_PTR(mem_ld_snb),
>       EVENT_PTR(mem_st_snb),
> +     EVENT_PTR(td_slots_issued),
> +     EVENT_PTR(td_slots_retired),
> +     EVENT_PTR(td_fetch_bubbles),
> +     EVENT_PTR(td_total_slots),
> +     EVENT_PTR(td_total_slots_scale),
> +     EVENT_PTR(td_recovery_bubbles),
> +     EVENT_PTR(td_recovery_bubbles_scale),

Peter, Ingo,
any thoughts about adding these events? The rest of the
tooling code is based on them being accepted..

thanks,
jirka

Reply via email to