Skip to content

Refresh P2E runtime before workloads - #22

Merged
daiyongyuan merged 2 commits into
mainfrom
fix/p2e-diff-build-dir
Sep 25, 2026
Merged

daiyongyuan merged 2 commits into
mainfrom
fix/p2e-diff-build-dir

Conversation

@daiyongyuan

@daiyongyuan daiyongyuan commented Sep 25, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Name P2E --diff bitstream build directories <chip>-diff-<timestamp> while retaining <chip>-<timestamp> for normal builds and honoring an explicit output directory.
  • Before --runworkload, let Cargo rebuild the selected case's P2E runtime when software sources change, then install the updated executable and, for DiffTest, libriscv.so.
  • Build and run DiffTest with the same <chip>-p2e-diff target directory so vdbg loads the newly built libbebop_p2e.so. Cargo skips compilation when its inputs are unchanged. A build failure stops before FPGA simulation.

The stale-runtime trigger was a W0-only DPI change: rerunning LeNet against an existing bitstream reused a bebop-p2e executable built before the Rust change and loaded a DPI library from the ordinary target directory.

Verification

  • Python AST syntax check and git diff --check for the runworkload handler.
  • CI Build and GitGuardian checks passed.
  • A full P2E workload rerun was not available in this session.

This change is required by DangoSys/buckyball#125 alongside DangoSys/bebop#24.

@daiyongyuan daiyongyuan changed the title Name P2E DiffTest bitstream directories distinctly Refresh P2E runtime before workloads Sep 25, 2026
@daiyongyuan
daiyongyuan merged commit 46ab401 into main Sep 25, 2026
2 checks passed
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.

1 participant