Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Papi counters don't work with reset and evaluate API's #755

Closed
pagrubel opened this issue Apr 22, 2013 · 4 comments
Closed

Papi counters don't work with reset and evaluate API's #755

pagrubel opened this issue Apr 22, 2013 · 4 comments

Comments

@pagrubel
Copy link
Member

A segmentation fault occurs if the API's for resetting and evaluating the counters is used as in hpx_homogenous_timed_task_spawn
// Reset performance counters (if specified on command line)
reset_active_counters();
.....do some work......
// Evaluate Performance Counters
evaluate_active_counters();
(This is probably not needed in the benchmark but I'm going to leave it there and make a new one without it so we have something to test for using them.)
Here are results for hpx_homogeneous_task_spawn hpx_homogeneous_timed_task_spawn

bin/hpx_homogeneous_timed_task_spawn --hpx:print-counter=/papi{locality#0/worker-thread#0}/PAPI_TLB_DM
Segmentation fault

bin/hpx_homogeneous_task_spawn --hpx:print-counter=/papi{locality#0/worker-thread#0}/PAPI_TLB_DM
OS-threads,Tasks,Delay (iterations),Total Walltime (seconds),Walltime per Task (seconds)
1, 500000, 0, 1.27074, 2.54148e-06
/papi{locality#0/worker-thread#0}/PAPI_TLB_DM,1,1.311261,[s],1.79657e+06

@ghost ghost assigned maciekab Apr 22, 2013
@hkaiser
Copy link
Member

hkaiser commented Apr 22, 2013

Maciek, I assigned you for the sole purpose to get you in the loop. Feel free to un-assign yourself, if appropriate. In any case, any help would be very much appreciated.

@maciekab
Copy link
Member

OK, will take a look at it...

On 04/22/2013 06:08 PM, Hartmut Kaiser wrote:

Maciek, I assigned you for the sole purpose to get you in the loop.
Feel free to un-assign yourself, if appropriate. In any case, any help
would be very much appreciated.


Reply to this email directly or view it on GitHub
#755 (comment).

maciekab added a commit that referenced this issue Apr 24, 2013
@hkaiser
Copy link
Member

hkaiser commented Apr 24, 2013

Thanks Maciek!

@pagrubel
Copy link
Member Author

thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants