Fix current_thread_index for masterless arena(1,0) #1128
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
A masterless
arena(max concurrency = 1, master threads = 0)
values might return unexpected tbb::this_task_arena::current_thread_index(), not 0 but 1. That happens because arena preserve additional slot for arenas with max_concurrency == 1 for correct support of enqueue tasks (to ensure that there is a slot for a worker in case of enqueue task submit). However, it is not the case forarena(max concurrency = 1, master threads = 0)
because there is a slot for worker thread.Fixes # - issue number(s) if exists
Type of change
Choose one or multiple, leave empty if none of the other choices apply
Add a respective label(s) to PR if you have permissions
Tests
Documentation
Breaks backward compatibility
Notify the following users
List users with
@
to send notificationsOther information