Navigation Menu

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

HPX-thread cumulative count performance counters report incorrect value #1028

Closed
brycelelbach opened this issue Dec 3, 2013 · 4 comments
Closed
Assignees
Milestone

Comments

@brycelelbach
Copy link
Member

The cumulative HPX-thread count performance counters do not correctly count the number of executed threads; in the scheduling loop, after an HPX-thread yields or terminates back to the scheduler, the state is not checked before the counter is incremented (the state could be suspended or even pending).

@hkaiser
Copy link
Member

hkaiser commented Dec 3, 2013

Huh? Your fix changes the semantics of the performance counter. It was counting the number of executed threads before, now it counts the number of terminated threads, which is not the same. What good is it to know how many threads got eventually terminated (you have this value by querying the number of created threads)? The purpose of this was to be able to tell how often threads are invoked (executed). Please revert this change.

@hkaiser hkaiser reopened this Dec 3, 2013
@ghost ghost assigned brycelelbach Dec 3, 2013
@pagrubel
Copy link
Member

pagrubel commented Dec 3, 2013

It is my understanding that this will be corrected by adding a counter?

@hkaiser
Copy link
Member

hkaiser commented Dec 4, 2013

Yes, we will:

  • fix the documentation of the existing counter
  • add a new counter returning the number of terminated (created) threads

@ghost ghost assigned hkaiser Dec 4, 2013
@hkaiser
Copy link
Member

hkaiser commented Dec 10, 2013

There are still the two tasks to be fixed (see above). Thus I'm reopening the ticket.

@hkaiser hkaiser reopened this Dec 10, 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

3 participants