Hi Stephane,
Carl Love and I were looking at the code in pfm_create_context in
perfmon_ctx.c today and ran across something that looks like it might be
a typo, but could cause some incorrect behavior:
At line 210, there is this code:
/*
* check if PMU is usable
*/
if (!(ctx->regs.num_pmcs && ctx->regs.num_pmcs)) {
PFM_DBG("no usable PMU registers");
ret = -EBUSY;
goto error_alloc;
}
Should that really be num_pmcs && num_pmcs? or perhaps num_pmcs &&
num_pmds?
Regards,
- Corey
Corey Ashford
Software Engineer
IBM Linux Technology Center, Linux Toolchain
Beaverton, OR
503-578-3507
cjash...@us.ibm.com
------------------------------------------------------------------------------
This SF.net email is sponsored by:
SourcForge Community
SourceForge wants to tell your story.
http://p.sf.net/sfu/sf-spreadtheword
_______________________________________________
perfmon2-devel mailing list
perfmon2-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/perfmon2-devel
