- qemu_firmware: Drop support for kernel descriptors (RHEL-82645) - qemu_firmware: Drop 'nvram' local variable (RHEL-82645) - qemu_firmware: Move format=raw compat exception (RHEL-82645) - qemu_firmware: Move copying of nvram.format to loader.format (RHEL-82645) - tests: Add firmware-manual-efi-rw-nvram (RHEL-82645) - domain_validate: Reject NVRAM with read/write firmware (RHEL-82645) - tests: Add firmware-auto-bios-rw (RHEL-82645) - tests: Add firmware-manual-bios-rw (RHEL-82645) - domain_validate: Reject read/write ROMs (RHEL-82645) - tests: Add firmware-auto-efi-format-loader-qcow2-rom (RHEL-82645) - domain_validate: Reject ROMs with format other than raw (RHEL-82645) - qemu_firmware: Ignore stateless/combined when NVRAM is configured (RHEL-82645) - qemu_firmware: Drop fallback for absent nvramTemplateFormat (RHEL-82645) - schemas: Allow templateFormat without template path (RHEL-82645) - tests: Add firmware-manual-efi-nvram-template-nonstandard-format (RHEL-82645) - tests: Add firmware-manual-efi-nvram-template-nonstandard-legacy-paths (RHEL-82645) - tests: Add firmware-auto-efi-format-nvram-raw (RHEL-82645) - tests: Add firmware-auto-efi-format-nvram-raw-loader-path (RHEL-82645) - tests: Add firmware-auto-efi-format-nvram-raw-nvramtemplate-path (RHEL-82645) - tests: Add firmware-auto-efi-format-nvramtemplate-qcow2 (RHEL-82645) - tests: Add firmware-auto-efi-format-mismatch-nvramtemplate (RHEL-82645) - qemu_firmware: Introduce qemuFirmwareFillDomainCustom() (RHEL-82645) - qemu_firmware: Set templateFormat for custom paths (RHEL-82645) - qemu_firmware: Simplify handling of legacy paths (RHEL-82645) - qemu_firmware: Refactor setting NVRAM format (RHEL-82645) - qemu_firmware: Prefer template format to loader format (RHEL-82645) - qemu_firmware: Retain user-specified NVRAM format (RHEL-82645) - qemu_firmware: Take templateFormat into account when matching (RHEL-82645) - qemu_firmware: Take NVRAM format into account when matching (RHEL-82645) - qemu_firmware: Remove NVRAM to loader format copy hack (RHEL-82645) - tests: Add firmware-manual-efi-sev-snp (RHEL-82645) - tests: Add firmware-manual-efi-tdx (RHEL-82645) - qemu_firmware: ROM firmware is always in raw format (RHEL-82645) - qemu_firmware: Don't skip autoselection for ROM (RHEL-82645) - qemu_firmware: Allow matching both UEFI and BIOS for ROM loader (RHEL-82645) - schema: Add firmwareFeatures element for domaincaps (RHEL-82645) - conf: Add firmwareFeatures element for domaincaps (RHEL-82645) - qemu: Fill in firmwareFeature element for domaincaps (RHEL-82645) - docs: Document firmwareFeature element for domaincaps (RHEL-82645) - docs: Rename "BIOS bootloader" section to "guest firmware" (RHEL-82645) - docs: Improvement related to firmware selection (RHEL-82645) - qemu_firmware: Only set format for custom loader if path is present (RHEL-82645) - conf: Move type=rom default for loader to drivers (RHEL-82645) - tests: Rename custom JSON firmware descriptors (RHEL-82645) - schema: Introduce osnvram define (RHEL-82645) - conf: Parse and format varstore element (RHEL-82645) - conf: Update validation to consider varstore element (RHEL-82645) - qemu_capabilities: Introduce QEMU_CAPS_DEVICE_UEFI_VARS (RHEL-82645) - qemu: Validate presence of uefi-vars device (RHEL-82645) - tests: Add firmware-manual-efi-varstore-q35 (RHEL-82645) - tests: Add firmware-manual-efi-varstore-aarch64 (RHEL-82645) - tests: Add firmware-auto-efi-varstore-q35 (RHEL-82645) - tests: Add firmware-auto-efi-varstore-aarch64 (RHEL-82645) - tests: Add firmware-auto-efi-enrolled-keys-aarch64 (RHEL-82645) - qemu_firmware: Parse host-uefi-vars firmware feature (RHEL-82645) - qemu_firmware: Split sanity check (RHEL-82645) - qemu_firmware: Consider host-uefi-vars feature in sanity check (RHEL-82645) - qemu_firmware: Support extended syntax for ROM firmware descriptors (RHEL-82645) - qemu_firmware: Report NVRAM template path for ROMs (RHEL-82645) - conf: Include varstore element in domcaps (RHEL-82645) - qemu: Fill in varstore element in domcaps (RHEL-82645) - qemu_firmware: Use of NVRAM implies stateful firmware (RHEL-82645) - qemu_firmware: Allow matching stateful ROMs (RHEL-82645) - qemu_firmware: Fill in varstore information (RHEL-82645) - qemu: Introduce varstoreDir (RHEL-82645) - qemu_firmware: Generate varstore path when necessary (RHEL-82645) - qemu: Introduce qemuPrepareNVRAMFileCommon() (RHEL-82645) - qemu: Create and delete varstore file (RHEL-82645) - security: Mark ROMs as read only when using AppArmor (RHEL-82645) - security: Handle varstore file (RHEL-82645) - tests: Add firmware descriptors for uefi-vars builds (RHEL-82645) - qemu_command: Use uefi-vars device where appropriate (RHEL-82645) - include: Mention varstore where applicable (RHEL-82645) - virsh: Update for varstore handling (RHEL-82645) - domain_conf: initialize network hostdev private data (RHEL-151916) - qemu_hotplug: enter monitor in order to rollback passed FD (RHEL-151916) Resolves: RHEL-151916, RHEL-82645
152 lines
8.4 KiB
Diff
152 lines
8.4 KiB
Diff
From b4ac6b2fe5cc443ca41eaad3301137c2c47ced2d Mon Sep 17 00:00:00 2001
|
|
Message-ID: <b4ac6b2fe5cc443ca41eaad3301137c2c47ced2d.1772815313.git.jdenemar@redhat.com>
|
|
From: Andrea Bolognani <abologna@redhat.com>
|
|
Date: Fri, 23 Jan 2026 14:34:05 +0100
|
|
Subject: [PATCH] tests: Rename custom JSON firmware descriptors
|
|
MIME-Version: 1.0
|
|
Content-Type: text/plain; charset=UTF-8
|
|
Content-Transfer-Encoding: 8bit
|
|
|
|
Most of the JSON firmware descriptors in our test suite come from
|
|
the Fedora package for edk2, but there are a few additional ones
|
|
that we have created ourselves to ensure coverage of uncommon or
|
|
problematic scenarios.
|
|
|
|
In order to make sure that such descriptors are clearly marked as
|
|
custom, rename them to include the string "libvirt" in the path.
|
|
|
|
Signed-off-by: Andrea Bolognani <abologna@redhat.com>
|
|
Reviewed-by: Michal Privoznik <mprivozn@redhat.com>
|
|
Acked-by: Gerd Hoffmann <kraxel@redhat.com>
|
|
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
|
|
(cherry picked from commit ccdff699ac30fc12691ad45b36951317d37e1030)
|
|
|
|
https://issues.redhat.com/browse/RHEL-82645
|
|
|
|
Signed-off-by: Andrea Bolognani <abologna@redhat.com>
|
|
---
|
|
.../etc/qemu/firmware/20-bios.json | 1 -
|
|
.../etc/qemu/firmware/20-libvirt-bios.json | 1 +
|
|
.../etc/qemu/firmware/59-combined.json | 1 -
|
|
.../etc/qemu/firmware/59-libvirt-combined.json | 1 +
|
|
.../{92-masked.json => 92-libvirt-masked.json} | 0
|
|
.../{10-bios.json => 10-libvirt-bios.json} | 0
|
|
...90-combined.json => 90-libvirt-combined.json} | 0
|
|
.../{91-bios.json => 91-libvirt-bios.json} | 0
|
|
.../{92-masked.json => 92-libvirt-masked.json} | 0
|
|
.../{93-invalid.json => 93-libvirt-invalid.json} | 0
|
|
tests/qemufirmwaretest.c | 16 ++++++++--------
|
|
11 files changed, 10 insertions(+), 10 deletions(-)
|
|
delete mode 120000 tests/qemufirmwaredata/etc/qemu/firmware/20-bios.json
|
|
create mode 120000 tests/qemufirmwaredata/etc/qemu/firmware/20-libvirt-bios.json
|
|
delete mode 120000 tests/qemufirmwaredata/etc/qemu/firmware/59-combined.json
|
|
create mode 120000 tests/qemufirmwaredata/etc/qemu/firmware/59-libvirt-combined.json
|
|
rename tests/qemufirmwaredata/etc/qemu/firmware/{92-masked.json => 92-libvirt-masked.json} (100%)
|
|
rename tests/qemufirmwaredata/home/user/.config/qemu/firmware/{10-bios.json => 10-libvirt-bios.json} (100%)
|
|
rename tests/qemufirmwaredata/usr/share/qemu/firmware/{90-combined.json => 90-libvirt-combined.json} (100%)
|
|
rename tests/qemufirmwaredata/usr/share/qemu/firmware/{91-bios.json => 91-libvirt-bios.json} (100%)
|
|
rename tests/qemufirmwaredata/usr/share/qemu/firmware/{92-masked.json => 92-libvirt-masked.json} (100%)
|
|
rename tests/qemufirmwaredata/usr/share/qemu/firmware/{93-invalid.json => 93-libvirt-invalid.json} (100%)
|
|
|
|
diff --git a/tests/qemufirmwaredata/etc/qemu/firmware/20-bios.json b/tests/qemufirmwaredata/etc/qemu/firmware/20-bios.json
|
|
deleted file mode 120000
|
|
index 2c274dddc2..0000000000
|
|
--- a/tests/qemufirmwaredata/etc/qemu/firmware/20-bios.json
|
|
+++ /dev/null
|
|
@@ -1 +0,0 @@
|
|
-../../../usr/share/qemu/firmware/91-bios.json
|
|
\ No newline at end of file
|
|
diff --git a/tests/qemufirmwaredata/etc/qemu/firmware/20-libvirt-bios.json b/tests/qemufirmwaredata/etc/qemu/firmware/20-libvirt-bios.json
|
|
new file mode 120000
|
|
index 0000000000..fab8877c3e
|
|
--- /dev/null
|
|
+++ b/tests/qemufirmwaredata/etc/qemu/firmware/20-libvirt-bios.json
|
|
@@ -0,0 +1 @@
|
|
+../../../usr/share/qemu/firmware/91-libvirt-bios.json
|
|
\ No newline at end of file
|
|
diff --git a/tests/qemufirmwaredata/etc/qemu/firmware/59-combined.json b/tests/qemufirmwaredata/etc/qemu/firmware/59-combined.json
|
|
deleted file mode 120000
|
|
index da9099ffb7..0000000000
|
|
--- a/tests/qemufirmwaredata/etc/qemu/firmware/59-combined.json
|
|
+++ /dev/null
|
|
@@ -1 +0,0 @@
|
|
-../../../usr/share/qemu/firmware/90-combined.json
|
|
\ No newline at end of file
|
|
diff --git a/tests/qemufirmwaredata/etc/qemu/firmware/59-libvirt-combined.json b/tests/qemufirmwaredata/etc/qemu/firmware/59-libvirt-combined.json
|
|
new file mode 120000
|
|
index 0000000000..74e63c4574
|
|
--- /dev/null
|
|
+++ b/tests/qemufirmwaredata/etc/qemu/firmware/59-libvirt-combined.json
|
|
@@ -0,0 +1 @@
|
|
+../../../usr/share/qemu/firmware/90-libvirt-combined.json
|
|
\ No newline at end of file
|
|
diff --git a/tests/qemufirmwaredata/etc/qemu/firmware/92-masked.json b/tests/qemufirmwaredata/etc/qemu/firmware/92-libvirt-masked.json
|
|
similarity index 100%
|
|
rename from tests/qemufirmwaredata/etc/qemu/firmware/92-masked.json
|
|
rename to tests/qemufirmwaredata/etc/qemu/firmware/92-libvirt-masked.json
|
|
diff --git a/tests/qemufirmwaredata/home/user/.config/qemu/firmware/10-bios.json b/tests/qemufirmwaredata/home/user/.config/qemu/firmware/10-libvirt-bios.json
|
|
similarity index 100%
|
|
rename from tests/qemufirmwaredata/home/user/.config/qemu/firmware/10-bios.json
|
|
rename to tests/qemufirmwaredata/home/user/.config/qemu/firmware/10-libvirt-bios.json
|
|
diff --git a/tests/qemufirmwaredata/usr/share/qemu/firmware/90-combined.json b/tests/qemufirmwaredata/usr/share/qemu/firmware/90-libvirt-combined.json
|
|
similarity index 100%
|
|
rename from tests/qemufirmwaredata/usr/share/qemu/firmware/90-combined.json
|
|
rename to tests/qemufirmwaredata/usr/share/qemu/firmware/90-libvirt-combined.json
|
|
diff --git a/tests/qemufirmwaredata/usr/share/qemu/firmware/91-bios.json b/tests/qemufirmwaredata/usr/share/qemu/firmware/91-libvirt-bios.json
|
|
similarity index 100%
|
|
rename from tests/qemufirmwaredata/usr/share/qemu/firmware/91-bios.json
|
|
rename to tests/qemufirmwaredata/usr/share/qemu/firmware/91-libvirt-bios.json
|
|
diff --git a/tests/qemufirmwaredata/usr/share/qemu/firmware/92-masked.json b/tests/qemufirmwaredata/usr/share/qemu/firmware/92-libvirt-masked.json
|
|
similarity index 100%
|
|
rename from tests/qemufirmwaredata/usr/share/qemu/firmware/92-masked.json
|
|
rename to tests/qemufirmwaredata/usr/share/qemu/firmware/92-libvirt-masked.json
|
|
diff --git a/tests/qemufirmwaredata/usr/share/qemu/firmware/93-invalid.json b/tests/qemufirmwaredata/usr/share/qemu/firmware/93-libvirt-invalid.json
|
|
similarity index 100%
|
|
rename from tests/qemufirmwaredata/usr/share/qemu/firmware/93-invalid.json
|
|
rename to tests/qemufirmwaredata/usr/share/qemu/firmware/93-libvirt-invalid.json
|
|
diff --git a/tests/qemufirmwaretest.c b/tests/qemufirmwaretest.c
|
|
index 2eb9d8e701..e09f50592b 100644
|
|
--- a/tests/qemufirmwaretest.c
|
|
+++ b/tests/qemufirmwaretest.c
|
|
@@ -84,7 +84,7 @@ testFWPrecedence(const void *opaque G_GNUC_UNUSED)
|
|
g_autofree char *fakehome = NULL;
|
|
g_auto(GStrv) fwList = NULL;
|
|
const char *expected[] = {
|
|
- SYSCONFDIR "/qemu/firmware/20-bios.json",
|
|
+ SYSCONFDIR "/qemu/firmware/20-libvirt-bios.json",
|
|
PREFIX "/share/qemu/firmware/30-edk2-ovmf-4m-qcow2-x64-sb-enrolled.json",
|
|
PREFIX "/share/qemu/firmware/31-edk2-ovmf-2m-raw-x64-sb-enrolled.json",
|
|
PREFIX "/share/qemu/firmware/40-edk2-ovmf-4m-qcow2-x64-sb.json",
|
|
@@ -98,12 +98,12 @@ testFWPrecedence(const void *opaque G_GNUC_UNUSED)
|
|
PREFIX "/share/qemu/firmware/51-edk2-ovmf-2m-raw-x64-nosb.json",
|
|
PREFIX "/share/qemu/firmware/52-edk2-aarch64-verbose-qcow2.json",
|
|
PREFIX "/share/qemu/firmware/53-edk2-aarch64-verbose-raw.json",
|
|
- SYSCONFDIR "/qemu/firmware/59-combined.json",
|
|
+ SYSCONFDIR "/qemu/firmware/59-libvirt-combined.json",
|
|
PREFIX "/share/qemu/firmware/60-edk2-ovmf-x64-amdsev.json",
|
|
PREFIX "/share/qemu/firmware/60-edk2-ovmf-x64-inteltdx.json",
|
|
- PREFIX "/share/qemu/firmware/90-combined.json",
|
|
- PREFIX "/share/qemu/firmware/91-bios.json",
|
|
- PREFIX "/share/qemu/firmware/93-invalid.json",
|
|
+ PREFIX "/share/qemu/firmware/90-libvirt-combined.json",
|
|
+ PREFIX "/share/qemu/firmware/91-libvirt-bios.json",
|
|
+ PREFIX "/share/qemu/firmware/93-libvirt-invalid.json",
|
|
NULL
|
|
};
|
|
const char **e;
|
|
@@ -285,9 +285,9 @@ mymain(void)
|
|
DO_PARSE_TEST("usr/share/qemu/firmware/53-edk2-aarch64-verbose-raw.json");
|
|
DO_PARSE_TEST("usr/share/qemu/firmware/60-edk2-ovmf-x64-amdsev.json");
|
|
DO_PARSE_TEST("usr/share/qemu/firmware/60-edk2-ovmf-x64-inteltdx.json");
|
|
- DO_PARSE_TEST("usr/share/qemu/firmware/90-combined.json");
|
|
- DO_PARSE_TEST("usr/share/qemu/firmware/91-bios.json");
|
|
- DO_PARSE_FAILURE_TEST("usr/share/qemu/firmware/93-invalid.json");
|
|
+ DO_PARSE_TEST("usr/share/qemu/firmware/90-libvirt-combined.json");
|
|
+ DO_PARSE_TEST("usr/share/qemu/firmware/91-libvirt-bios.json");
|
|
+ DO_PARSE_FAILURE_TEST("usr/share/qemu/firmware/93-libvirt-invalid.json");
|
|
|
|
if (virTestRun("QEMU FW precedence test", testFWPrecedence, NULL) < 0)
|
|
ret = -1;
|
|
--
|
|
2.53.0
|