Skip to content

Fix/prologue synchronization#296

Open
guosran wants to merge 2 commits into
tancheng:masterfrom
guosran:fix/prologue-synchronization
Open

Fix/prologue synchronization#296
guosran wants to merge 2 commits into
tancheng:masterfrom
guosran:fix/prologue-synchronization

Conversation

@guosran

@guosran guosran commented Jun 19, 2026

Copy link
Copy Markdown
Collaborator

Summary

Addresses #292 with a minimized prologue fix. This PR only updates FlexibleFuRTL.py and its focused test.

Changes

  • During prologue-skipped const operations, keep the const queue aligned with the control stream.
  • Override only the selected FU operation to OPT_NAH for the skipped cycle.
  • Add a concrete example as an inline test comment.

No new ports or interfaces are introduced.

@tancheng tancheng requested review from tancheng and yyan7223 June 21, 2026 06:30
Comment thread fu/flexible/FlexibleFuRTL.py Outdated
Comment thread noc/CrossbarRTL.py Outdated
@guosran guosran force-pushed the fix/prologue-synchronization branch from 92bd204 to 4d0f9ae Compare June 23, 2026 20:41
@tancheng tancheng requested a review from Jackcuii June 24, 2026 22:05
@guosran guosran force-pushed the fix/prologue-synchronization branch from 4d0f9ae to 07a826a Compare June 25, 2026 14:43
@guosran guosran changed the base branch from kernel-submit to master June 25, 2026 14:52
@guosran guosran marked this pull request as draft June 25, 2026 14:54
@guosran

guosran commented Jun 26, 2026

Copy link
Copy Markdown
Collaborator Author

Concrete example:
During prologue, the FU should not execute the real operation yet.

cycle 0: ctrl = ADD, prologue_count = 1
FlexibleFuRTL sends OPT_NAH to inner FU
no ADD result is produced

cycle 1: prologue_count = 0
FlexibleFuRTL forwards ADD
FU executes normally

@guosran guosran marked this pull request as ready for review June 26, 2026 00:32
@tancheng tancheng removed the request for review from Jackcuii June 26, 2026 20:07
@tancheng

Copy link
Copy Markdown
Owner

Not sure why it shows "Expected — Waiting for status to be reported"

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.

3 participants