[SeaBIOS] Re: regression: cannot boot from nvme anymore, disabling 64bit pci io window helps

Gerd Hoffmann posted 1 patch 1 month, 3 weeks ago
Failed in applying to current master (apply log)
src/fw/pciinit.c | 2 ++
1 file changed, 2 insertions(+)
[SeaBIOS] Re: regression: cannot boot from nvme anymore, disabling 64bit pci io window helps
Posted by Gerd Hoffmann 1 month, 3 weeks ago
  Hi,

> After an update some time ago, a VM wit 8GB RAM cannot boot from nvme anymore.

From 7b54087dbd7f201a6cdb2794518c71a84aa0d8a6 Mon Sep 17 00:00:00 2001
From: Gerd Hoffmann <kraxel@redhat.com>
Date: Mon, 15 Jul 2024 10:53:23 +0200
Subject: [PATCH] pciinit: ignore nvme for 64-bit mmio window

Add nvme drives to the list of devices which are not
moved to the 64-bit mmio window.

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
---
 src/fw/pciinit.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/fw/pciinit.c b/src/fw/pciinit.c
index b3e359d7fe62..f6aa1a554853 100644
--- a/src/fw/pciinit.c
+++ b/src/fw/pciinit.c
@@ -764,6 +764,8 @@ static void pci_region_migrate_64bit_entries(struct pci_region *from,
             continue;
         if (entry->dev->class == PCI_CLASS_SERIAL_USB)
             continue;
+        if (entry->dev->class == PCI_CLASS_STORAGE_NVME)
+            continue;
         // Move from source list to destination list.
         hlist_del(&entry->node);
         hlist_add(&entry->node, last);
-- 
2.45.2

_______________________________________________
SeaBIOS mailing list -- seabios@seabios.org
To unsubscribe send an email to seabios-leave@seabios.org
[SeaBIOS] Re: regression: cannot boot from nvme anymore, disabling 64bit pci io window helps
Posted by Marek Marczykowski-Górecki 1 month, 3 weeks ago
On Mon, Jul 15, 2024 at 10:59:12AM +0200, Gerd Hoffmann wrote:
>   Hi,
> 
> > After an update some time ago, a VM wit 8GB RAM cannot boot from nvme anymore.
> 
> From 7b54087dbd7f201a6cdb2794518c71a84aa0d8a6 Mon Sep 17 00:00:00 2001
> From: Gerd Hoffmann <kraxel@redhat.com>
> Date: Mon, 15 Jul 2024 10:53:23 +0200
> Subject: [PATCH] pciinit: ignore nvme for 64-bit mmio window
> 
> Add nvme drives to the list of devices which are not
> moved to the 64-bit mmio window.
> 
> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>

Thanks, I can confirm this helps too.

Tested-By: Marek Marczykowski-Górecki <marmarek@invisiblethingslab.com>

> ---
>  src/fw/pciinit.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/src/fw/pciinit.c b/src/fw/pciinit.c
> index b3e359d7fe62..f6aa1a554853 100644
> --- a/src/fw/pciinit.c
> +++ b/src/fw/pciinit.c
> @@ -764,6 +764,8 @@ static void pci_region_migrate_64bit_entries(struct pci_region *from,
>              continue;
>          if (entry->dev->class == PCI_CLASS_SERIAL_USB)
>              continue;
> +        if (entry->dev->class == PCI_CLASS_STORAGE_NVME)
> +            continue;
>          // Move from source list to destination list.
>          hlist_del(&entry->node);
>          hlist_add(&entry->node, last);
> -- 
> 2.45.2
> 

-- 
Best Regards,
Marek Marczykowski-Górecki
Invisible Things Lab
_______________________________________________
SeaBIOS mailing list -- seabios@seabios.org
To unsubscribe send an email to seabios-leave@seabios.org