website: Add documentation for supported kernels and disk images#177
website: Add documentation for supported kernels and disk images#177Harshil2107 wants to merge 4 commits intogem5:stablefrom
Conversation
erin-le
left a comment
erin-le
left a comment
There was a problem hiding this comment.
@Harshil2107: I took a look at this PR by running the website locally, and I don't think this documentation is actually reachable by clicking through links on the website. I've attached a Markdown file with commands to run the website locally in a Docker container. Hopefully it'll help you double-check if the documentation is or isn't unreachable.
erin-le
left a comment
erin-le
left a comment
There was a problem hiding this comment.
Generally, this looks good. I had to modify a permalink to get the documentation to show up on the website, though.
_pages/documentation/general_docs/fullsystem/supported_disks_and_kernel.md
Outdated
Show resolved
Hide resolved
| * **Ubuntu 22.04 disk images** use **kernel 5.15.168** | ||
| * **Ubuntu 24.04 disk images** (including `npb` and `gapbs` variants) use **kernel 6.8.12** |
There was a problem hiding this comment.
It's not a big deal, but could you double check the sub-versions for these? On the gem5 resources website, we have version 5.15.180 kernels, but not version 5.15.168 kernels.
There was a problem hiding this comment.
The sub version keep on changing as we remake the disk images as we get the source via apt, I think I will update them to be 5.15 as the minor version is consistent.
erin-le
left a comment
erin-le
left a comment
There was a problem hiding this comment.
I think the documentation is fine as it is, but also left some comments on how it might be improved.
| The following kernel versions are used with each base disk image: | ||
|
|
||
| * **Ubuntu 22.04 disk images** use **kernel 5.15** | ||
| * **Ubuntu 24.04 disk images** (including `npb` and `gapbs` variants) use **kernel 6.8.12** |
There was a problem hiding this comment.
The minor versions also change when building the kernel for Ubuntu 24.04, right? It might be good to change instances of 6.8.12 to 6.8 as well.
I think it'd also be good to (1) link to the source (in the gem5-resources repository) and (2) also warn that the sub-versions change and could cause issues with the kernel build process, if the readers build the kernel/disk images themselves.
- I haven't built the disk images/kernels recently, but I remember running into issues with kernel sub-versions. Please let me know if this is no longer an issue.
You could also include links to the relevant resources on the gem5 Resources website.
3 participants
No description provided.