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

Add global start/stop API for performance counters #731

Closed
hkaiser opened this issue Feb 28, 2013 · 4 comments
Closed

Add global start/stop API for performance counters #731

hkaiser opened this issue Feb 28, 2013 · 4 comments

Comments

@hkaiser
Copy link
Member

hkaiser commented Feb 28, 2013

Add global start/stop API for performance counters allowing to explicitly instrument application code.

@ghost ghost assigned hkaiser Feb 28, 2013
@hkaiser
Copy link
Member Author

hkaiser commented Feb 28, 2013

We will add 4 new API functions:

hpx::start_active_counters(hpx::error_code& ec = throws);
hpx::stop_active_counters(hpx::error_code& ec = throws);
hpx::reset_active_counters(hpx::error_code& ec = throws);
hpx::evaluate_active_counters(bool reset = false, char const* description = 0, 
    hpx::error_code& ec = throws);

This should allow for arbitrary application specific instrumentation. These API functions will refer to the counters which are specified on the command lines using the command line option --hpx:print-counter.

If no counters are being specified on the command line these functions will have no effect.

@pagrubel
Copy link
Member

pagrubel commented Mar 1, 2013

And if none of these API's are used the counters will be active for the entire program, right?

@hkaiser
Copy link
Member Author

hkaiser commented Mar 1, 2013

Yes, same as before.

hkaiser added a commit that referenced this issue Mar 1, 2013
hkaiser added a commit that referenced this issue Mar 1, 2013
@hkaiser
Copy link
Member Author

hkaiser commented Mar 1, 2013

This has been fixed by dffb281

@hkaiser hkaiser closed this as completed Mar 1, 2013
hkaiser added a commit that referenced this issue Mar 26, 2013
hkaiser added a commit that referenced this issue Apr 3, 2013
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

2 participants