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.

Sign up for GitHub

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

Jump to bottom

Add PowerPC assembly for grabbing clock register in memory pool #511

Closed
gmackey opened this issue Oct 26, 2016 · 0 comments
Closed

Add PowerPC assembly for grabbing clock register in memory pool #511

gmackey opened this issue Oct 26, 2016 · 0 comments
Assignees
@gmackey
Labels
Enhancement Improve existing capability; will potentially require voting
Projects
On-node Task DAG
Milestone
Fall 2016

Comments

@gmackey
Copy link
Contributor

gmackey commented Oct 26, 2016

Si provided assembly to grab the value of the clock register on PowerPC. Add this to MemoryPool::get_clock_register() to speed up allocations on PowerPC.
@gmackey gmackey added the Enhancement Improve existing capability; will potentially require voting label Oct 26, 2016
@gmackey gmackey added this to the Fall 2016 milestone Oct 26, 2016
@gmackey gmackey self-assigned this Oct 26, 2016
gmackey added a commit to gmackey/kokkos that referenced this issue Oct 26, 2016
@gmackey
Fixes MemoryPool bug with GCC 6.1.0 (kokkos#298) by correcting errors…
4105848 
… with atomic loads and memory fences and explicitly initializing all variables. Adds assembly for getting the clock register on PowerPC for MemoryPool (kokkos#511).  Removed a test for swapping the same superblock back as the active as this behavior is okay.  Moved the full fraction defines in MemoryPool to be constants in the constructor.
@gmackey gmackey mentioned this issue Oct 27, 2016
Merged
@crtrott crtrott closed this as completed Oct 30, 2016
@hcedwar hcedwar added this to Backlog in On-node Task DAG Apr 19, 2017
@hcedwar hcedwar moved this from Feature Backlog to Done in On-node Task DAG Apr 19, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gmackey gmackey

Labels
Enhancement Improve existing capability; will potentially require voting
Projects
No open projects
On-node Task DAG
Done
Milestone
Fall 2016
Development

No branches or pull requests
2 participants
@crtrott @gmackey