From f607d0143d1173152d90343feb93849b95f2de15 Mon Sep 17 00:00:00 2001
From: Philipp Schuster <philipp.schuster@cyberus-technology.de>
Date: Tue, 2 Jun 2026 06:44:17 +0200
Subject: [PATCH] arch: preserve error chain for memmap table

On-behalf-of: SAP philipp.schuster@sap.com
Signed-off-by: Philipp Schuster <philipp.schuster@cyberus-technology.de>
---
 arch/src/lib.rs        | 4 +++-
 arch/src/x86_64/mod.rs | 9 +++++----
 2 files changed, 8 insertions(+), 5 deletions(-)

diff --git a/arch/src/lib.rs b/arch/src/lib.rs
index 79d5c510c..8c067c302 100644
--- a/arch/src/lib.rs
+++ b/arch/src/lib.rs
@@ -35,7 +35,9 @@ pub enum Error {
     #[error("The memory map table extends past the end of guest memory")]
     MemmapTablePastRamEnd,
     #[error("Error writing memory map table to guest memory")]
-    MemmapTableSetup,
+    MemmapTableSetup(#[source] vm_memory::GuestMemoryError),
+    #[error("Error generating memory map table")]
+    MemmapTableGeneration,
     #[error("The hvm_start_info structure extends past the end of guest memory")]
     StartInfoPastRamEnd,
     #[error("Error writing hvm_start_info to guest memory")]
diff --git a/arch/src/x86_64/mod.rs b/arch/src/x86_64/mod.rs
index dc2718cee..8c0f179f4 100644
--- a/arch/src/x86_64/mod.rs
+++ b/arch/src/x86_64/mod.rs
@@ -1140,8 +1140,9 @@ pub fn generate_ram_ranges(guest_mem: &GuestMemoryMmap) -> super::Result<Vec<Ram
     // Generate the first usable physical memory range before the gap. The e820 map
     // should only report memory above 1MiB.
     let first_ram_range = {
-        let (first_region_start, first_region_end) =
-            ram_regions.first().ok_or(super::Error::MemmapTableSetup)?;
+        let (first_region_start, first_region_end) = ram_regions
+            .first()
+            .ok_or(super::Error::MemmapTableGeneration)?;
         let high_ram_start = layout::HIGH_RAM_START.raw_value();
         let mem_32bit_reserved_start = layout::MEM_32BIT_RESERVED_START.raw_value();
 
@@ -1154,7 +1155,7 @@ pub fn generate_ram_ranges(guest_mem: &GuestMemoryMmap) -> super::Result<Vec<Ram
                 high_ram_start: 0x{high_ram_start:08x}, mem_32bit_reserved_start: 0x{mem_32bit_reserved_start:08x}"
             );
 
-            return Err(super::Error::MemmapTableSetup);
+            return Err(super::Error::MemmapTableGeneration);
         }
 
         info!(
@@ -1265,7 +1266,7 @@ fn configure_pvh(
     for memmap_entry in memmap {
         guest_mem
             .write_obj(memmap_entry, memmap_start_addr)
-            .map_err(|_| super::Error::MemmapTableSetup)?;
+            .map_err(super::Error::MemmapTableSetup)?;
         memmap_start_addr =
             memmap_start_addr.unchecked_add(mem::size_of::<hvm_memmap_table_entry>() as u64);
     }