-
Notifications
You must be signed in to change notification settings - Fork 20
Priorities are 32bit and overflow too easily #738
Copy link
Copy link
Open
Milestone
Description
Followup to #706
Will still need to revisit the priorities being 32 bit later, because dplasma has priorities overflow for 'normal' MT/NT values, and changing the formulas to fit in 32 bits is not super obvious, while the cost of having a 64 bit compare is probably 0.
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.
Originally posted by @abouteiller in #706 (comment)
Duplicate of #717
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels