G
YouTube TikTok Games
Skip to content

TE Fortunes regression on Windows #103256

Closed
Closed

Description

@sebastienros

Only on Windows
Isolated to a single PR: Add three-stage enqueuer/dispatcher scheme to SocketAsyncEngine, ThreadPoolWorkQueue and ThreadPoolTypedWorkItemQueue

I see this regression was mentioned in the PR but I think it's better to create an issue to raise awareness and/or officially accepting it.

Regression in RPS
image

CPU has less usage (not a bad thing per se, compromise or correlation with loss of RPS)
image

Max TP items /s also lower
image

@eduardo-vp
@kouvel

Activity

  1. dotnet-policy-service commented on Jun 10, 2024

    @dotnet-policy-service
    Contributor

    Tagging subscribers to this area: @mangod9
    See info in area-owners.md if you want to be subscribed.

  2. added this to the 9.0.0 milestone on Jun 13, 2024
  3. self-assigned this
    on Aug 15, 2024
  4. kouvel commented on Aug 26, 2024

    @kouvel
    Contributor

    The change reduces excessive parallelization, which is not a bad thing. There can be some minor tradeoffs based on timing. Excessive parallelization can be beneficial for some situations, but typically it just adds to CPU time without any other perf benefit. Maybe there are other ways to recover the throughput regression.

  5. modified the milestones: 9.0.0, 10.0.0 on Aug 26, 2024
  6. mangod9 commented on Aug 13, 2025

    @mangod9
    Member

    Closing since some regressions are expected here

  7. locked and limited conversation to collaborators on Sep 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions