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

pthread back end hangs if used uninitialized #182

Closed
rppawlo opened this issue Feb 2, 2016 · 5 comments
Closed

pthread back end hangs if used uninitialized #182

rppawlo opened this issue Feb 2, 2016 · 5 comments
Assignees
Labels
Bug Broken / incorrect code; it could be Kokkos' responsibility, or others’ (e.g., Trilinos)
Milestone

Comments

@rppawlo
Copy link
Contributor

rppawlo commented Feb 2, 2016

The other execution spaces seem to exit with an exception reported for attempting to use an execution space that is not yet initialized. However the pthread back end just hangs with no message. Can this be fixed to throw an exception?

@hcedwar hcedwar added the Bug Broken / incorrect code; it could be Kokkos' responsibility, or others’ (e.g., Trilinos) label Feb 4, 2016
@hcedwar
Copy link
Contributor

hcedwar commented Feb 4, 2016

This is a new bug in Kokkos_RangePolicy::set_auto_chunk_size() where the Threads::concurrency() has a value of zero before Threads is initialized(). This function needs to be fixed and reviewed.

@crtrott
Copy link
Member

crtrott commented Feb 4, 2016

Args. I wondered why I was assigned this stuff - I don't like the threads backend ;-)

@crtrott crtrott added this to the GTC 2016 milestone Feb 4, 2016
@crtrott
Copy link
Member

crtrott commented Feb 9, 2016

got it fixed on my machine, will push later. I am trying to get through a number of issues in one go.

@rppawlo
Copy link
Contributor Author

rppawlo commented Feb 10, 2016

Thanks Christian!

crtrott added a commit that referenced this issue Feb 10, 2016
…itialize

This addresses issue #182. Before initialize the thread space returns 0
as concurrency which made a while loop run endless.
@crtrott
Copy link
Member

crtrott commented Mar 14, 2016

I believe this was merged into master and Trilinos.

@crtrott crtrott closed this as completed Mar 14, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Broken / incorrect code; it could be Kokkos' responsibility, or others’ (e.g., Trilinos)
Projects
None yet
Development

No branches or pull requests

3 participants