Skip to content

Use Queue instead of ConcurrentQueue in single-threaded thread pool #111245

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
Merged
merged 12 commits into from
Jan 20, 2025
Merged

Use Queue instead of ConcurrentQueue in single-threaded thread pool #111245

merged 12 commits into from
Jan 20, 2025

Conversation

pavelsavara
Copy link
Member

@pavelsavara pavelsavara commented Jan 9, 2025 •
edited
Loading

Motivation: performance and size improvements for single-threaded WASM

  • introduce FEATURE_SINGLE_THREAD define symbol
  • use Queue instead of ConcurrentQueue in single-threaded thread pool
  • make ThreadPoolTypedWorkItemQueue #if TARGET_WINDOWS and non-generic

@pavelsavara
- introduce FEATURE_SINGLE_THREAD define symbol
843c4e1 
- use Queue instead of ConcurrentQueue in single-threaded thread pool
- protect ConcurrentQueue ctor from trimming when used with `JsonSerializer.Deserialize`
@pavelsavara pavelsavara added arch-wasm WebAssembly architecture area-VM-threading-mono size-reduction Issues impacting final app size primary for size sensitive workloads labels Jan 9, 2025
@pavelsavara pavelsavara added this to the 10.0.0 milestone Jan 9, 2025
@pavelsavara pavelsavara self-assigned this Jan 9, 2025
@pavelsavara pavelsavara changed the title [browser][ST] Use Queue instead of ConcurrentQueue in single-threaded thread pool [WASM][ST] Use Queue instead of ConcurrentQueue in single-threaded thread pool Jan 9, 2025
This was referenced Jan 9, 2025
Open
Closed
@SingleAccretion SingleAccretion mentioned this pull request Jan 10, 2025
Open
This was referenced Jan 10, 2025
Open
Open
This was referenced Jan 11, 2025
Closed
Closed
@build-analysis build-analysis bot mentioned this pull request Jan 14, 2025
Open
@build-analysis build-analysis bot mentioned this pull request Jan 16, 2025
Closed
@pavelsavara pavelsavara marked this pull request as ready for review January 16, 2025 20:34
@pavelsavara pavelsavara changed the title [WASM][ST] Use Queue instead of ConcurrentQueue in single-threaded thread pool Use Queue instead of ConcurrentQueue in single-threaded thread pool Jan 20, 2025
@pavelsavara pavelsavara merged commit 8c4c381 into dotnet:main Jan 20, 2025
138 of 140 checks passed
@pavelsavara pavelsavara deleted the browser_trim_st_concurrent_queue branch January 20, 2025 10:45
@github-actions github-actions bot locked and limited conversation to collaborators Feb 20, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@marek-safar marek-safar marek-safar left review comments

@eerhardt eerhardt eerhardt left review comments

@maraf maraf maraf approved these changes

@steveisok steveisok Awaiting requested review from steveisok steveisok is a code owner

@vitek-karas vitek-karas Awaiting requested review from vitek-karas vitek-karas is a code owner

+1 more reviewer

@SingleAccretion SingleAccretion SingleAccretion left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@pavelsavara pavelsavara

Labels

arch-wasm WebAssembly architecture area-VM-threading-mono size-reduction Issues impacting final app size primary for size sensitive workloads

Projects

None yet

Milestone

10.0.0

Development

Successfully merging this pull request may close these issues.

5 participants

@pavelsavara @marek-safar @eerhardt @maraf @SingleAccretion