Skip to content

Analyze SimAI analytical scheduler hang for tp=2 workloads - #304

Draft
tianhao909 with Copilot wants to merge 1 commit into
masterfrom
copilot/fix-simai-analytical-performance
Draft

Analyze SimAI analytical scheduler hang for tp=2 workloads#304
tianhao909 with Copilot wants to merge 1 commit into
masterfrom
copilot/fix-simai-analytical-performance

Conversation

Copilot AI commented Aug 27, 2026

Copy link
Copy Markdown

SimAI_analytical can busy-loop on large or out-of-order event timestamps because it uses FIFO scheduling and advances time one tick at a time.

  • Root cause

    • Events are not ordered by timestamp.
    • Past events cannot be reached by an increment-only clock.
    • Tick delays are narrowed through int, risking overflow.
  • Proposed fix

    • Use a time-ordered event container.
    • Jump directly to the next event timestamp.
    • Preserve delay values as 64-bit Tick.

This PR currently contains analysis and implementation guidance only; no code changes were made.

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Copilot AI linked an issue Aug 27, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Fix performance issue with SimAI_analytical for tp=2 configs Analyze SimAI analytical scheduler hang for tp=2 workloads Aug 27, 2026
Copilot AI requested a review from tianhao909 August 27, 2026 11:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SimAI_analytical hangs / becomes extremely slow for tp=2 configs

3 participants