-
Notifications
You must be signed in to change notification settings - Fork 45
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
resources: ARM Ubuntu 24.04 disk image with SPEC 2017 #52
base: stable
Are you sure you want to change the base?
Conversation
This commit adds the files needed to create an ARM Ubuntu 24.04 disk image with an installation of SPEC 2017.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is the thought that we will always build this disk image from scratch? If so, why? The alternative is to do what we do in npb and gapbs now which is to use the base ubuntu image and build on top of it. I don't have a strong feeling either way, but I would like to understand your thought process.
permalink: resources/arm-ubuntu | ||
shortdoc: > | ||
Resources to build a generic arm-ubuntu disk image. | ||
author: ["Hoa Nguyen", "Kaustav Goswami"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You should include yourself here
|
||
provisioner "file" { | ||
destination = "/home/gem5/" | ||
source = "/home/bees/gem5-bootcamp-private/cpu2017-1.1.0.iso" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can probably have this be part of the initialize
block by loading it as a cdrom drive.
# Check if the specified configuration file exists | ||
if [ -f "$config_file" ]; then | ||
# Install the needed plugins | ||
PACKER_LOG=1 ./packer init "$config_file" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Make sure to remove this logging in the final version
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We probably want to simplify this file for the spec suite. We want to simply say what spec workload to run. Maybe we should only accept parameters to the runspec
script.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We likely don't need this file anymore.
Another to do item before merging this is to figure out how to add ROIs. Zhantong (I can't find her github handle) should be able to help with that. |
To be honest, there wasn't any particular thought process; building it from scratch was just the only way I could figure out how to do it. If the NPB and GAPBS images are built on top of the base disk image, it'll be more consistent and probably be better to build the SPEC disk images the same way. |
Just a thing to note, the size of the SPEC image is around 11.5 GB but the base images we have are only 5GB. We have two options:
|
you can add the following to the post install script to resize the partition and filesystem. You would have to resize the disk by updating the
|
Two comments:
In the comments, you should write what |
I looked into it a little, when I try to resize, it gives a warning that the partition is already in use, as that was stalling the script, I will try to echo the replies in order and see if it works. Also parted is not on the disk by default, so we will have to install it before we update the partition.
I am deleting partition 2 (/dev/vda2 as we have two partitions for the arm image one for boot and one root), and creating a new partition 2 with full size.
I dont understand this, do you mean the output from fdisk for each input I am echoing in? |
This pr adds the files needed to create an ARM Ubuntu 24.04 disk image with an installation of SPEC 2017.
I still need to update the README to reflect the current method of creating disk images.