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

Revisit priorities being 32 bits #717

Open
abouteiller opened this issue Dec 11, 2024 · 0 comments
Open

Revisit priorities being 32 bits #717

abouteiller opened this issue Dec 11, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@abouteiller
Copy link
Contributor

Description

Priorities can overflow for common values of MT/NT (as seen in ICLDisco/dplasma#56). It is quite hard to change the priorities in dplasma to fit in 32 bits, and it may have essentially 0 cost to increase priority field to 64 bit in parsec.

Describe the solution you'd like

increase priority field to 64 bit and maintain the task structure under a cache line.
Maintaining the size under cache line may not be possible, we would have to decide then if the presumably small cost is worth the risk of breaking user code.

@abouteiller abouteiller added the enhancement New feature or request label Dec 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant