xen: use SeaBIOS' passthru.firmware attribute#345324
Merged
Merged
Conversation
CertainLach
reviewed
Sep 29, 2024
CertainLach
approved these changes
Sep 29, 2024
16bc5e0 to
42252e9
Compare
Member
|
By reading the Makefile from SeaBIOS project I found this: https://github.com/coreboot/seabios/blob/2424e4c04aa30d90e85073ea41d18a7845460783/Makefile#L98 target-y :=
target-$(CONFIG_QEMU) += $(OUT)bios.bin
target-$(CONFIG_CSM) += $(OUT)Csm16.bin
target-$(CONFIG_COREBOOT) += $(OUT)bios.bin.elf
target-$(CONFIG_BUILD_VGABIOS) += $(OUT)vgabios.bin
all: $(target-y)In other words, it can build at least four different files, possibly at the same time. |
hehongbo
approved these changes
Sep 30, 2024
Contributor
Author
|
I should note that this was partially superseded by #342692. I just haven't deleted the SeaBIOS commits yet.
|
42252e9 to
10d085c
Compare
passthru.firmware attribute
Contributor
Author
|
We should also probably change our SeaBIOS to |
5 tasks
10d085c to
91e6f3d
Compare
Contributor
Author
|
Vagrant fails to build, but I'm not sure why; seems unrelated to Xen: Running phase: installCheckPhase
Vagrant failed to initialize at a very early stage:
The plugins failed to initialize correctly. This may be due to manual
modifications made within the Vagrant home directory. Vagrant can
attempt to automatically correct this issue by running:
vagrant plugin repair
If Vagrant was recently updated, this error may be due to incompatible
versions of dependencies. To fix this problem please remove and re-install
all plugins. Vagrant can attempt to do this automatically by running:
vagrant plugin expunge --reinstall
Or you may want to try updating the installed plugins to their latest
versions:
vagrant plugin update
Error message given during initialization: can't activate rexml-3.3.6, already activated rexml-3.2.5cc @tylerjl
|
Related: NixOS#342692 Signed-off-by: Fernando Rodrigues <alpha@sigmasquadron.net>
91e6f3d to
d175e88
Compare
CertainLach
approved these changes
Oct 18, 2024
emilazy
approved these changes
Oct 18, 2024
13 tasks
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.
Description of changes
firmwareinstead of hard-coding a directory path.seabiosis overridden byqubes-seabiosinqubes-vmm-xen(qubes-vmm-xen: init at 4.19.0-5, qubes-seabios: init at 4.0.2 #341429).Requires #342692
Closes #345172
Things done
nix-shell -p nixpkgs-review --run "nixpkgs-review pr 345324". Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/)Add a 👍 reaction to pull requests you find important.