Skip to content

feat(smcntrpmf): support Smcntrpmf extension #811

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
wissygh merged 1 commit into from
Jan 28, 2026
Merged

feat(smcntrpmf): support Smcntrpmf extension #811

wissygh merged 1 commit into from
Jan 28, 2026

Conversation

@wissygh
Copy link
Contributor

@wissygh wissygh commented Feb 17, 2025 •
edited
Loading

No description provided.

@wissygh wissygh added the do not merge please do not merge it until something label Feb 18, 2025
@wissygh wissygh force-pushed the feat-smcntrpmf branch 2 times, most recently from 853128b to 8777b07 Compare February 19, 2025 07:21
@wissygh wissygh force-pushed the feat-smcntrpmf branch 2 times, most recently from 789ddc2 to f67583c Compare December 29, 2025 02:14
@wissygh wissygh marked this pull request as ready for review January 22, 2026 06:49
@github-actions
Copy link

github-actions bot commented Jan 22, 2026

NEMU Performance Results

Test Instructions Executed Estimated Host Throughput (instr/s) Actual NEMU Throughput (instr/s)
bitmanip.bin 5.151e+07 2.689e+07 3.031e+07
coremark-riscv64-xs-rv64gc-o2.bin 1.710e+08 1.961e+08 2.237e+08
coremark-riscv64-xs-rv64gc-o3.bin 1.705e+08 1.991e+08 2.450e+08
coremark-riscv64-xs-rv64gcb-o3.bin 1.677e+08 1.810e+08 2.230e+08
amtest-riscv64-xs.bin 8.661e+06 1.832e+07 2.637e+07
aliastest-riscv64-xs.bin 7.700e+06 1.787e+06 4.095e+06
softprefetchtest-riscv64-xs.bin 7.746e+06 3.412e+06 1.874e+07
zacas-riscv64-xs.bin 1.208e+07 5.356e+07 2.168e+07
linux-hello 1.843e+10 4.094e+07 4.809e+07
  • Host throughput is estimated based on 4GHz CPU frequency and IPC=2.5.
  • Actual throughput may vary based on the host CPU performance.

@wissygh wissygh requested a review from sinceforYy January 22, 2026 06:58
sinceforYy
sinceforYy approved these changes Jan 22, 2026
View reviewed changes
@github-actions
Copy link

github-actions bot commented Jan 23, 2026

NEMU Performance Results

Test Instructions Executed Estimated Host Throughput (instr/s) Actual NEMU Throughput (instr/s)
bitmanip.bin 5.156e+07 2.686e+07 2.790e+07
coremark-riscv64-xs-rv64gc-o2.bin 1.731e+08 1.938e+08 1.736e+08
coremark-riscv64-xs-rv64gc-o3.bin 1.726e+08 1.967e+08 1.777e+08
coremark-riscv64-xs-rv64gcb-o3.bin 1.697e+08 1.788e+08 1.697e+08
amtest-riscv64-xs.bin 8.672e+06 1.830e+07 1.767e+07
aliastest-riscv64-xs.bin 7.696e+06 1.788e+06 3.457e+06
softprefetchtest-riscv64-xs.bin 7.746e+06 3.412e+06 1.289e+07
zacas-riscv64-xs.bin 1.212e+07 5.341e+07 1.774e+07
linux-hello 1.855e+10 4.068e+07 4.775e+07
  • Host throughput is estimated based on 4GHz CPU frequency and IPC=2.5.
  • Actual throughput may vary based on the host CPU performance.

@wissygh wissygh force-pushed the feat-smcntrpmf branch 2 times, most recently from 4bdb472 to 46cae62 Compare January 23, 2026 06:52
@github-actions
Copy link

github-actions bot commented Jan 23, 2026

NEMU Performance Results

Test Instructions Executed Estimated Host Throughput (instr/s) Actual NEMU Throughput (instr/s)
bitmanip.bin 5.156e+07 2.686e+07 2.789e+07
coremark-riscv64-xs-rv64gc-o2.bin 1.731e+08 1.938e+08 1.749e+08
coremark-riscv64-xs-rv64gc-o3.bin 1.726e+08 1.967e+08 1.806e+08
coremark-riscv64-xs-rv64gcb-o3.bin 1.697e+08 1.788e+08 1.688e+08
amtest-riscv64-xs.bin 8.672e+06 1.830e+07 1.665e+07
aliastest-riscv64-xs.bin 7.698e+06 1.787e+06 3.583e+06
softprefetchtest-riscv64-xs.bin 7.743e+06 3.413e+06 1.460e+07
zacas-riscv64-xs.bin 1.212e+07 5.341e+07 1.980e+07
linux-hello 1.855e+10 4.068e+07 4.798e+07
  • Host throughput is estimated based on 4GHz CPU frequency and IPC=2.5.
  • Actual throughput may vary based on the host CPU performance.

cebarobot
cebarobot approved these changes Jan 23, 2026
View reviewed changes
Copy link
Member

@cebarobot cebarobot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice job!

One point that can be improved: function update_counter_mcountinhibit may have a new name like update_counter_inhibit, as it is not only affected by mcountinhibit but also mcyclecfg and minstretcfg.

@github-actions
Copy link

github-actions bot commented Jan 23, 2026

NEMU Performance Results

Test Instructions Executed Estimated Host Throughput (instr/s) Actual NEMU Throughput (instr/s)
bitmanip.bin 5.157e+07 2.686e+07 3.040e+07
coremark-riscv64-xs-rv64gc-o2.bin 1.731e+08 1.938e+08 1.786e+08
coremark-riscv64-xs-rv64gc-o3.bin 1.726e+08 1.967e+08 1.863e+08
coremark-riscv64-xs-rv64gcb-o3.bin 1.697e+08 1.788e+08 1.682e+08
amtest-riscv64-xs.bin 8.674e+06 1.830e+07 1.625e+07
aliastest-riscv64-xs.bin 7.696e+06 1.788e+06 5.134e+06
softprefetchtest-riscv64-xs.bin 7.746e+06 3.412e+06 5.472e+06
zacas-riscv64-xs.bin 1.212e+07 5.340e+07 1.981e+07
linux-hello 1.855e+10 4.068e+07 4.813e+07
  • Host throughput is estimated based on 4GHz CPU frequency and IPC=2.5.
  • Actual throughput may vary based on the host CPU performance.

@wissygh wissygh removed the do not merge please do not merge it until something label Jan 28, 2026
@wissygh wissygh merged commit b62ec64 into master Jan 28, 2026
14 checks passed
@wissygh wissygh deleted the feat-smcntrpmf branch January 28, 2026 06:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cebarobot cebarobot cebarobot approved these changes

@sinceforYy sinceforYy sinceforYy approved these changes

@good-circle good-circle Awaiting requested review from good-circle good-circle is a code owner

@Anzooooo Anzooooo Awaiting requested review from Anzooooo Anzooooo is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@wissygh @cebarobot @sinceforYy