-
-
Notifications
You must be signed in to change notification settings - Fork 2
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
SeaBIOS support #574
Open
pietrushnic
wants to merge
33
commits into
develop
Choose a base branch
from
seabios-support
base: develop
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
SeaBIOS support #574
Changes from all commits
Commits
Show all changes
33 commits
Select commit
Hold shift + click to select a range
2a1aba8
add support for BIOS_LIB uefi or seabios
pietrushnic 38dbe77
add lib/bios/seabios.robot, remove resource menus.robot
pietrushnic 44b5018
platform-configs/include/default.robot: disable OPTIONS_LIB, which de…
pietrushnic 55e14fa
platform-configs/qemu-selftests.robot: change configuration to one us…
pietrushnic 9fcaf08
scripts/ci/qemu-run.sh: give ability to provide QEMU_FW_FILE through …
pietrushnic bca7507
scripts/ci/qemu-run.sh: enabled kvm only if it supported
pietrushnic 9fed386
scripts/ci/qemu-self-test-seabios.sh: initial commit
pietrushnic bba6a4d
self-tests/setup-and-boot-seabios.robot: initial commit
pietrushnic d6d1c32
scripts: add support for Dasharo (coreboot+SeaBIOS) for QEMU Q35
pietrushnic 87e2b6f
lib/bios/seabios.robot: add more keywords and its tests
pietrushnic 9d3a4d6
.github/workflows/qemu-self-test-seabios.yml: initial commit
pietrushnic 13fa3f6
platform-configs/qemu: swap release and selftests configs
macpijan 329f474
platform-configs/qemu-selftests-uefi.robot: use common part from qemu…
macpijan 48a7ffe
keywords.robot: do not import lib/bios/menus.robot twice
pietrushnic f15c763
platform-configs/qemu-selftests-uefi.robot: fix resource order
pietrushnic 3fc5a86
platform-configs,scripts: make BIOS_LIB=uefi default and override onl…
pietrushnic 13b6871
platform-configs/qemu-selftests-seabios.robot: import common qemu.rob…
pietrushnic e84c0ea
docs,scripts: keep firmware option for qemu-run.sh as backward compat…
pietrushnic c6de4d6
rename lib/bios/menus.robot -> lib/bios/edk2.robot
pietrushnic b2aeac0
use platform-defined FW_STRING instead of {SEABIOS,TIANOCORE}_STRING
pietrushnic 3fa1672
lib/bios/common.robot: initial support for common Enter Boot Menu kwd
pietrushnic ea76664
add unified Enter Boot Menu and Return Construction
pietrushnic db3a1ad
lib/bios/seabios.robot: remove unused Get IPXE Boot Menu Construction
pietrushnic ced49a6
lib/bios/seabios.robot: remove unused Get RTC Clock Submenu Construction
pietrushnic de96c46
unify Enter Setup Menu kwd for uefi and seabios
pietrushnic fcdf8fa
lib/bios: user Get Menu Construction instead of SeaBIOS dedicated wra…
pietrushnic 57289d9
lib/bios/seabios.robot: remove unused keywords
pietrushnic 880043d
lib/bios/edk2.robot: remove Get Menu Consutruction use common implmen…
pietrushnic d8b7190
lib/bios/seabios.robot: align API with edk2 lib
pietrushnic c7f0aeb
lib/bios: move Get Index Of Matching Option In Menu to common
pietrushnic 728bf25
lib/bios: move Boot System Or From Connected Disk to common lib
filipleple b193b25
platform-configs/optiplex-7010-seabios.robot: add
filipleple 76ccdd9
lib/bios: make Boot System Or From Connected Disk more generic
filipleple File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,53 @@ | ||
name: Keywords self-tests for Dasharo (coreboot+SeaBIOS) on QEMU Q35 | ||
|
||
on: [push, pull_request] | ||
|
||
jobs: | ||
qemu: | ||
runs-on: ubuntu-latest | ||
|
||
steps: | ||
- name: Checkout repository and submodules | ||
uses: actions/checkout@v2 | ||
with: | ||
submodules: 'recursive' | ||
|
||
- name: Set up QEMU | ||
run: | | ||
sudo apt-get update | ||
sudo apt-get install qemu-system-x86-64 swtpm | ||
|
||
# Based on: https://github.blog/changelog/2023-02-23-hardware-accelerated-android-virtualization-on-actions-windows-and-linux-larger-hosted-runners/ | ||
# It mentiones enteprise large-runners, let's see if it works | ||
# on regular public runners as well | ||
- name: Enable KVM group perms | ||
run: | | ||
echo 'KERNEL=="kvm", GROUP="kvm", MODE="0666", OPTIONS+="static_node=kvm"' | sudo tee /etc/udev/rules.d/99-kvm4all.rules | ||
sudo udevadm control --reload-rules | ||
sudo udevadm trigger --name-match=kvm | ||
|
||
- name: Start QEMU in background | ||
run: | | ||
./scripts/ci/qemu-run.sh nographic seabios & | ||
|
||
- name: Set up Python | ||
uses: actions/setup-python@v2 | ||
with: | ||
python-version: '3.11' | ||
|
||
- name: Install dependencies | ||
run: | | ||
python -m pip install --upgrade pip | ||
pip install -r requirements.txt | ||
|
||
- name: Start keywords self-tests with QEMU | ||
run: | | ||
./scripts/ci/qemu-self-test-seabios.sh | ||
|
||
- name: Save artifacts | ||
uses: actions/upload-artifact@v4 | ||
with: | ||
name: "qemu-logs" | ||
path: | | ||
./logs/ | ||
retention-days: 30 |
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
Oops, something went wrong.
Oops, something went wrong.
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.
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.
Most of this is copied from existing file, so we could avoid duplication: https://docs.github.com/en/actions/sharing-automations/reusing-workflows
Not a high priority, we can also leave it in issues.
I can also imagine, we would like to reuse this workflow not only in this repo, but use very similar config in e.g. coreboot or edk2 repos in the future.
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 should stick to GH's actions. The question is if this would be reliable. Sooner or later, we will exhaust the pool of free actions and storage. I don't see ourselves switching to paid GH, so I would switch to our Woodpecker CI instead.
But I get your point about reusability. My gh-actions-fu was not good enough to be aware of that possibility. There is no space for such work for now, but we should track it as a potential improvement. Based on your comment, I will create an issue for the following PC Engines release.