On Thu, Dec 4, 2008 at 3:41 AM, Corey J Ashford <[EMAIL PROTECTED]> wrote:
>
> Stephane,
>
> Thank you.  I think that clears up my misunderstanding.
>
> One thing was throwing me off was this bit of code in pfmlib_gen_ia32.c
>
>                 if (e[j].flags & ~PFMLIB_GEN_IA32_ALL_FLAGS) {
>                         DPRINT("event=%d invalid flags=0x%lx\n", e[j].event,
> e[j].flags);
>                         return PFMLIB_ERR_INVAL;
>                 }
>
> PFMLIB_GEN_IA32_ALL_FLAGS is defined as
>
> #define PFMLIB_GEN_IA32_ALL_FLAGS \
>         (PFM_GEN_IA32_SEL_INV|PFM_GEN_IA32_SEL_EDGE)
>
> e is the events array from the inp parameter.  This made it appear that the
> .flags field of the event was being used for the edge and inv flags.
>
> Is that the bug you were talking about?
>
Exactly! it needs to be cntrs[j].flags. I have the patch ready, will
push it up tomorrow.

-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
perfmon2-devel mailing list
perfmon2-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/perfmon2-devel

Reply via email to