Skip to content

Conversation

@clabby
Copy link
Owner

@clabby clabby commented Sep 12, 2025

Overview

Adds a new option to use an asynchronous Kernel in the StEmu.

Also a few periphery changes:

  • REG_* constants are now usize, better ergonomics given that they're usually used to index.
  • JAL, JALR, AUIPC now use wrapping_add.

@clabby clabby self-assigned this Sep 12, 2025
@clabby clabby added feature ✨ New feature or request A: hw Area: `brisc-hw` crate A: emu Area: `brisc-emu` crate labels Sep 12, 2025
@codecov
Copy link

codecov bot commented Sep 12, 2025

Codecov Report

❌ Patch coverage is 50.66667% with 37 lines in your changes missing coverage. Please review.
✅ Project coverage is 86.0%. Comparing base (8ac0e39) to head (3d9394d).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
crates/emu/src/st/mod.rs 46.6% 24 Missing ⚠️
crates/hw/src/kernel.rs 0.0% 10 Missing ⚠️
crates/emu/src/test_utils.rs 57.1% 3 Missing ⚠️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Removes the `async-kernel` feature flag in favor of a secondary `AsyncKernel` trait.
This API is preferred over the mutual exclusivity of the `async-kernel`
flag.
@clabby clabby marked this pull request as ready for review September 12, 2025 15:04
@clabby clabby merged commit efb208c into main Sep 12, 2025
8 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A: emu Area: `brisc-emu` crate A: hw Area: `brisc-hw` crate feature ✨ New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants