Skip to content

xen: use SeaBIOS' passthru.firmware attribute#345324

Merged
emilazy merged 1 commit into
NixOS:masterfrom
SigmaSquadron:seabios
Oct 18, 2024
Merged

xen: use SeaBIOS' passthru.firmware attribute#345324
emilazy merged 1 commit into
NixOS:masterfrom
SigmaSquadron:seabios

Conversation

@SigmaSquadron

@SigmaSquadron SigmaSquadron commented Sep 29, 2024

Copy link
Copy Markdown
Contributor

Description of changes

Requires #342692
Closes #345172

Things done

  • Built on platform(s)
    • x86_64-linux
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review pr 345324". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

Add a 👍 reaction to pull requests you find important.

@SigmaSquadron SigmaSquadron requested review from a team and AndersonTorres September 29, 2024 16:45
@github-actions github-actions Bot added the 6.topic: xen-project Issues and PRs related to the Xen Project Hypervisor. label Sep 29, 2024
@SigmaSquadron SigmaSquadron mentioned this pull request Sep 29, 2024
13 tasks
@ofborg ofborg Bot added 11.by: package-maintainer This PR was created by a maintainer of all the package it changes. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. labels Sep 29, 2024
Comment thread pkgs/by-name/se/seabios/package.nix Outdated
Comment thread pkgs/applications/virtualization/xen/generic/default.nix Outdated
@SigmaSquadron SigmaSquadron added the 12.approvals: 1 This PR was reviewed and approved by one person. label Sep 30, 2024
@ofborg ofborg Bot requested a review from CertainLach September 30, 2024 02:11
@AndersonTorres

Copy link
Copy Markdown
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.
So I think the better idea is to create split packages, one for each blob.
I will do it asap.

@SigmaSquadron SigmaSquadron removed the 12.approvals: 1 This PR was reviewed and approved by one person. label Sep 30, 2024
@SigmaSquadron

SigmaSquadron commented Sep 30, 2024 via email

Copy link
Copy Markdown
Contributor Author

@SigmaSquadron SigmaSquadron changed the title seabios: add passthru.firmware attribute xen: use SeaBIOS' passthru.firmware attribute Oct 2, 2024
@SigmaSquadron SigmaSquadron changed the title xen: use SeaBIOS' passthru.firmware attribute xen: use SeaBIOS' passthru.firmware attribute Oct 2, 2024
@SigmaSquadron

Copy link
Copy Markdown
Contributor Author

We should also probably change our SeaBIOS to seabios-qemu.

@SigmaSquadron

SigmaSquadron commented Oct 18, 2024

Copy link
Copy Markdown
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.5

cc @tylerjl


nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 345324


x86_64-linux

⏩ 1 package marked as broken and skipped:
  • qubes-core-vchan-xen
❌ 1 package failed to build:
  • vagrant
✅ 57 packages built:
  • appvm
  • collectd
  • diffoscope
  • diffoscope.dist
  • diffoscope.man
  • docker-machine-kvm2
  • gnome-boxes
  • guestfs-tools
  • htcondor
  • libguestfs
  • libguestfs-with-appliance
  • librenms
  • libvirt
  • libvirt-glib
  • libvirt-glib.dev
  • libvirt-glib.devdoc
  • libvmi
  • libvmi.dev
  • libvmi.lib
  • mgmt
  • minikube
  • multipass
  • ocamlPackages.ocaml_libvirt
  • perl538Packages.SysVirt
  • perl538Packages.SysVirt.devdoc
  • perl540Packages.SysVirt
  • perl540Packages.SysVirt.devdoc
  • python311Packages.guestfs
  • python311Packages.guestfs.dist
  • python311Packages.libvirt
  • python311Packages.libvirt.dist
  • python311Packages.xen
  • python311Packages.xen.boot
  • python311Packages.xen.dev
  • python311Packages.xen.doc
  • python311Packages.xen.man
  • python312Packages.guestfs
  • python312Packages.guestfs.dist
  • python312Packages.libvirt
  • python312Packages.libvirt.dist
  • xen (python312Packages.xen)
  • xen.boot (python312Packages.xen.boot)
  • xen.dev (python312Packages.xen.dev)
  • xen.doc (python312Packages.xen.doc)
  • xen.man (python312Packages.xen.man)
  • qemu_xen
  • qemu_xen.debug
  • qemu_xen.ga
  • rubyPackages.ruby-libvirt
  • rubyPackages_3_1.ruby-libvirt
  • rubyPackages_3_2.ruby-libvirt
  • virt-manager
  • virt-manager-qt
  • virt-manager.dist
  • virt-top
  • virt-viewer
  • xen-guest-agent

@ofborg ofborg Bot requested a review from hehongbo October 18, 2024 05:36
@ofborg ofborg Bot added the 10.rebuild-linux: 11-100 This PR causes between 11 and 100 packages to rebuild on Linux. label Oct 18, 2024
@ofborg ofborg Bot removed the 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. label Oct 18, 2024
Related: NixOS#342692
Signed-off-by: Fernando Rodrigues <alpha@sigmasquadron.net>
@ofborg ofborg Bot requested a review from CertainLach October 18, 2024 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

6.topic: xen-project Issues and PRs related to the Xen Project Hypervisor. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 11-100 This PR causes between 11 and 100 packages to rebuild on Linux. 11.by: package-maintainer This PR was created by a maintainer of all the package it changes.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants