qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] isapc: give system address space when pci is di


From: Hervé Poussineau
Subject: Re: [Qemu-devel] [PATCH] isapc: give system address space when pci is disabled
Date: Tue, 20 Sep 2011 23:25:01 +0200
User-agent: Thunderbird 2.0.0.23 (Windows/20090812)

Jan Kiszka a écrit :
On 2011-09-19 20:49, Hervé Poussineau wrote:
Jan Kiszka a écrit :
On 2011-09-18 18:04, Hervé Poussineau wrote:
Signed-off-by: Hervé Poussineau <address@hidden>
---
 hw/pc_piix.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/hw/pc_piix.c b/hw/pc_piix.c
index da6fa55..c0b8a3a 100644
--- a/hw/pc_piix.c
+++ b/hw/pc_piix.c
@@ -121,7 +121,7 @@ static void pc_init1(MemoryRegion *system_memory,
         pc_memory_init(system_memory,
                        kernel_filename, kernel_cmdline, initrd_filename,
                        below_4g_mem_size, above_4g_mem_size,
-                       pci_memory, &ram_memory);
+                       pci_enabled ? pci_memory : system_memory, &ram_memory);
     }
if (!xen_enabled()) {
Makes sense. I guess we should rename 'pci_memory' inside pc_memory_init
to reflect this - 'rom_memory'?

But -M isapc still only gives me a black screen. Are there further fixes
required?

Yes, you also need to rollback pc-bios/bios.bin to previous version (ie 8b06c62ae48b67b320f7420dcd4854c5559e1532)

Did you already try to bisect that probable regression in SeaBIOS?


Yes, I regtested the problem to this SeaBIOS commit:

01a5c8813b2e709809c07c5d7fab9d1c3ddb4989 is the first bad commit
commit 01a5c8813b2e709809c07c5d7fab9d1c3ddb4989
Author: Gerd Hoffmann <address@hidden>
Date:   Mon Jul 11 09:20:29 2011 +0200

    pci: activate two-pass pci initialization code

    This patch actually enables the two-pass pci initialization and
    deactivates the old pci initialization bits.

    Signed-off-by: Gerd Hoffmann <address@hidden>

Binary is current qemu + this patch
Command line is: qemu-system-i386 -M isapc -vga std

Hervé



reply via email to

[Prev in Thread] Current Thread [Next in Thread]