Hello,
In testing the perfmon2 and perf_counters substrates for PAPI, I discovered an
error in the Power7 implementation for libpfm. There was an omission from one
of the case statements. The attached patch fixes that.
Thanks for your consideration,
- Corey
Corey Ashford
Software Engineer
IBM Linux Technology Center, Linux Toolchain
Beaverton, OR
503-578-3507
cjash...@us.ibm.com
Index: pfmlib_gen_powerpc.c
===================================================================
RCS file: /cvsroot/perfmon2/libpfm/lib/pfmlib_gen_powerpc.c,v
retrieving revision 1.13
diff -u -p -r1.13 pfmlib_gen_powerpc.c
--- pfmlib_gen_powerpc.c 14 Jul 2009 17:39:49 -0000 1.13
+++ pfmlib_gen_powerpc.c 31 Jul 2009 23:50:50 -0000
@@ -292,6 +292,12 @@ static int pfm_gen_powerpc_get_event_cod
return PFMLIB_SUCCESS;
} else
return PFMLIB_ERR_INVAL;
+ case PFMLIB_POWER7_PMU:
+ if (event < POWER7_PME_EVENT_COUNT) {
+ *code = power7_pe[event].pme_code;
+ return PFMLIB_SUCCESS;
+ } else
+ return PFMLIB_ERR_INVAL;
default:
return PFMLIB_ERR_BADHOST;
}
------------------------------------------------------------------------------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day
trial. Simplify your report design, integration and deployment - and focus on
what you do best, core application coding. Discover what's new with
Crystal Reports now. http://p.sf.net/sfu/bobj-july
_______________________________________________
perfmon2-devel mailing list
perfmon2-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/perfmon2-devel
