Skip to content
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.

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

fix(MMU): Unify latency in different CSR bundles #4389

Merged
merged 1 commit into from
Mar 11, 2025
Merged

Conversation

good-circle
Copy link
Contributor

In the previous design, since we don't flush pipeline after modifying satp, we need to set whether or not increase the delay for different signals in the csr bundle, such as priv and satp, respectively.

Currently, CSR will add send redirect signal to refresh the pipeline after modifying satp, so we can unify latency in different CSR bundles to fenceDelay.

In the previous design, since we don't flush pipeline after modifying `satp`, we need to set whether or not increase the delay for different signals in the csr bundle, such as priv and satp, respectively.

Currently, CSR will add send redirect signal to refresh the pipeline after modifying `satp`, so we can unify latency in different CSR bundles to fenceDelay.
@XiangShanRobot
Copy link

[Generated by IPC robot]
commit: b348db6

commit astar copy_and_run coremark gcc gromacs hmmer-Vector lbm linux mcf microbench milc namd povray wrf xalancbmk
b348db6 1.806 0.442 2.641 1.232 2.150 1.676 2.149 2.364 0.933 1.389 2.010 3.108 2.538 2.274 3.309

master branch:

commit astar copy_and_run coremark gcc gromacs hmmer-Vector lbm linux mcf microbench milc namd povray wrf xalancbmk
fad7c42 0.442 2.641 1.228 2.158 1.676 2.144 2.364 0.940 1.389 1.972 3.112 2.538 3.300
dfb03ba 1.822 0.442 2.641 1.234 2.158 1.676 2.144 2.364 0.940 1.378 1.972 3.112 2.538 2.272 3.300
d6b0a27 1.823 0.442 2.641 1.226 2.158 1.676 2.144 2.364 0.940 1.378 1.972 3.112 2.538 2.272 3.300
591ae1c 1.822 0.442 2.641 1.236 2.158 1.676 2.144 2.364 0.940 1.389 1.972 3.112 2.538 2.272 3.300
11269ca 1.823 0.442 2.641 1.231 2.158 1.676 2.144 2.364 0.940 1.389 1.972 3.112 2.538 2.272 3.300
7fbc1cb 1.823 0.442 2.641 1.233 2.158 1.676 2.144 2.364 0.940 1.378 1.972 3.112 2.538 2.272 3.300
0bac66c 1.823 0.442 2.641 1.223 2.158 1.676 2.144 2.364 0.940 1.378 1.972 3.112 2.538 2.272 3.300
0837330 1.822 0.442 2.641 1.231 2.158 1.676 2.144 2.364 0.940 1.378 1.972 3.112 2.538 2.272 3.300
ba0bece 1.823 0.442 2.641 1.231 2.158 1.676 2.144 2.364 0.940 1.378 1.972 3.112 2.538 2.272 3.300
db3923f 1.822 0.442 2.641 1.237 2.158 1.676 2.144 2.364 0.940 1.389 1.972 3.112 2.538 2.272 3.300

@good-circle good-circle marked this pull request as ready for review March 10, 2025 23:23
@good-circle good-circle merged commit c1eb288 into master Mar 11, 2025
10 checks passed
@good-circle good-circle deleted the mmu-delay branch March 11, 2025 02:38
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