Skip to content

Add per-VM execution timing to main.rs#9

Merged
gpBlockchain merged 3 commits intogp/rv64imfrom
copilot/sub-pr-5-yet-again
Mar 4, 2026
Merged

Add per-VM execution timing to main.rs#9
gpBlockchain merged 3 commits intogp/rv64imfrom
copilot/sub-pr-5-yet-again

Conversation

Copy link

Copilot AI commented Mar 4, 2026

Track and report cumulative execution time for both patch and sched VMs in src/main.rs to enable performance comparison between the two implementations when running against corpus files.

Changes

  • Timing instrumentation in src/main.rs — each call to patch::run() and sched::run() (for both version 0 and version 2) is wrapped with std::time::Instant, accumulating nanoseconds into patch_vm_total_ns and sched_vm_total_ns
  • Per-directory reporting — after processing each corpus directory, the totals are printed:
patch_vm_total=123ms sched_vm_total=456ms

Timing uses u128 accumulators to safely handle large cumulative nanosecond values across all corpus files.


🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

…ry 1000 runs

Co-authored-by: gpBlockchain <32102187+gpBlockchain@users.noreply.github.com>
Copy link
Author

Copilot AI commented Mar 4, 2026

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • https://api.github.com/repos/gpBlockchain/ckb/commits/copilot/update-ckb-vm-to-asm-rv64im
    • Triggering command: /home/REDACTED/.rustup/toolchains/stable-x86_64-REDACTED-linux-gnu/bin/cargo /home/REDACTED/.rustup/toolchains/stable-x86_64-REDACTED-linux-gnu/bin/cargo metadata --format-version 1 --all-features --manifest-path /home/REDACTED/work/schedfuzz/schedfuzz/fuzz/Cargo.toml --filter-platform x86_64-REDACTED-linux-gnu --lockfile-path /tmp/rust-analyzerfea-0/Cargo.lock -Zunstable-options (http block)
  • https://api.github.com/repos/mohanson-fork/ckb-vm/commits/asm_rv64im
    • Triggering command: /home/REDACTED/.rustup/toolchains/stable-x86_64-REDACTED-linux-gnu/bin/cargo /home/REDACTED/.rustup/toolchains/stable-x86_64-REDACTED-linux-gnu/bin/cargo metadata --format-version 1 --all-features --manifest-path /home/REDACTED/work/schedfuzz/schedfuzz/fuzz/Cargo.toml --filter-platform x86_64-REDACTED-linux-gnu --lockfile-path /tmp/rust-analyzerfea-0/Cargo.lock -Zunstable-options (http block)
  • https://api.github.com/repos/nervosnetwork/ckb/commits/v0.204.0
    • Triggering command: /home/REDACTED/.rustup/toolchains/stable-x86_64-REDACTED-linux-gnu/bin/cargo /home/REDACTED/.rustup/toolchains/stable-x86_64-REDACTED-linux-gnu/bin/cargo metadata --format-version 1 --all-features --manifest-path /home/REDACTED/work/schedfuzz/schedfuzz/fuzz/Cargo.toml --filter-platform x86_64-REDACTED-linux-gnu --lockfile-path /tmp/rust-analyzerfea-0/Cargo.lock -Zunstable-options (http block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI changed the title [WIP] Fix CI workflow issues in stacked pull request Add per-VM execution timing to fuzz_tx_consistency Mar 4, 2026
@gpBlockchain
Copy link
Owner

不是补充在fuzz里,是补充在main.rs @copilot

Co-authored-by: gpBlockchain <32102187+gpBlockchain@users.noreply.github.com>
Copy link
Author

Copilot AI commented Mar 4, 2026

不是补充在fuzz里,是补充在main.rs @copilot

Done in commit b65e8b5. Timing is now in src/main.rs — each patch::run() and sched::run() call is wrapped with std::time::Instant, and after processing each corpus directory the totals are printed:

patch_vm_total=123ms sched_vm_total=456ms

Copilot AI changed the title Add per-VM execution timing to fuzz_tx_consistency Add per-VM execution timing to main.rs Mar 4, 2026
@gpBlockchain gpBlockchain marked this pull request as ready for review March 4, 2026 02:31
@gpBlockchain gpBlockchain merged commit 2d9b148 into gp/rv64im Mar 4, 2026
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.

2 participants