Hotfix Release: v25.1.0.1 - #3082
Merged
Merged
Conversation
This commit modifies gem5 so that it saves/restores the PMP table when taking/restoring a checkpoint. This resolves issue 2908.
This commit makes it so that all PMP table entries are saved to the checkpoint when one is taken, instead of only the entries from index 0 to index numRules - 1
This commit modifies the `fatal` message to be more clear by printing the PMP table size in both the current simulation and in the checkpoint. Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
This commit modifies PmpEntry so rawAddr is initialized to a default value. This is to prevent an undefined value for rawAddr when saving a checkpoint.
This commit makes it so that numRules and pmpAddr for the pmpTable are not stored when taking a checkpoint, and are instead recomputed using pmpUpdateRule() when restoring from a checkpoint.
Contributor
There was a problem hiding this comment.
Pull request overview
Hotfix release v25.1.0.1 which rolls in two fixes: (1) RISC-V checkpoint correctness by saving/restoring the PMP table and (2) an Arm KVM build fix by including the required misc-reg metadata header.
Changes:
- Bump gem5 version strings / documentation to
25.1.0.1and add hotfix release notes. - RISC-V: Make PMP entries serializable and add
PMP::{serialize,unserialize}to checkpoint PMP configuration/state. - Arm KVM: Add missing
arch/arm/regs/misc_info.hhinclude; update the RISC-V checkpoint example to use a newer resource version.
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
src/base/version.cc |
Updates the reported gem5 version to 25.1.0.1. |
src/arch/riscv/pmp.hh |
Makes PmpEntry serializable and declares PMP checkpoint hooks. |
src/arch/riscv/pmp.cc |
Implements PMP::serialize/unserialize to persist PMP table state and rebuild derived ranges. |
src/arch/arm/kvm/armv8_cpu.cc |
Fixes KVM build by including misc-reg metadata definitions. |
src/Doxyfile |
Updates Doxygen project version to v25.1.0.1. |
configs/example/gem5_library/checkpoints/riscv-hello-restore-checkpoint.py |
Updates referenced checkpoint resource version for the restore example. |
RELEASE-NOTES.md |
Adds release notes entry for hotfix 25.1.0.1 and links the included PRs. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
powerjg
approved these changes
Apr 14, 2026
Member
Author
|
Note: The tests fail in this case due to a PyUnit test that is dependent on the release being "develop". This is a known weakness. I'm therefore ignoring the error and merging the hotfix. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Includes: