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(Unprivileged): wait a cycle to update time when nextV =/= v #4132

Merged
merged 1 commit into from
Jan 5, 2025

Conversation

Tang-Haojin
Copy link
Member

When virt mode changed, time should be updated because we may need a offset version with htimedelta. This commit delays a cycle to update time register because v is actually changed a cycle after nextV =/= v.

When virt mode changed, `time` should be updated because
we may need a offset version with `htimedelta`. This commit
delays a cycle to update `time` register because `v` is
actually changed a cycle after `nextV =/= v`.
@XiangShanRobot
Copy link

[Generated by IPC robot]
commit: 2fbb217

commit astar copy_and_run coremark gcc gromacs lbm linux mcf microbench milc namd povray wrf xalancbmk
2fbb217 1.814 0.442 2.639 1.231 2.152 2.141 2.361 0.933 1.403 1.985 3.107 2.558 2.267 3.308

master branch:

commit astar copy_and_run coremark gcc gromacs lbm linux mcf microbench milc namd povray wrf xalancbmk
718a93f 1.814 0.442 2.639 1.231 2.152 2.141 2.361 0.933 1.407 1.985 3.107 2.558 2.267 3.308
2f0227e 1.814 0.442 2.640 1.231 2.152 2.141 2.361 0.933 1.406 1.985 3.107 2.558 2.267 3.308
ca892e7 1.814 0.442 2.639 1.231 2.152 2.141 2.360 0.933 1.406 1.985 3.107 2.558 2.267 3.308
51aa1b6 1.814 0.442 2.639 1.231 2.152 2.141 2.360 0.933 1.403 1.985 3.107 2.558 2.267 3.308

@Tang-Haojin Tang-Haojin merged commit d94fbff into master Jan 5, 2025
9 checks passed
@Tang-Haojin Tang-Haojin deleted the fix-time-virt-change branch January 5, 2025 18:32
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