qemu-trivial
[Top][All Lists]
Advanced

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

Re: [Qemu-trivial] [Xen-devel] [PATCH for v1.6] pc: Fix initialization o


From: Fabio Fantoni
Subject: Re: [Qemu-trivial] [Xen-devel] [PATCH for v1.6] pc: Fix initialization of the ram_memory variable.
Date: Thu, 29 Aug 2013 10:32:53 +0200
User-agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:17.0) Gecko/20130801 Thunderbird/17.0.8

Il 26/08/2013 17:00, Fabio Fantoni ha scritto:
Il 16/08/2013 16:46, Anthony PERARD ha scritto:
In some cases (Xen), it will not be initialized before to be used. This
leads to segv.

Signed-off-by: Anthony PERARD <address@hidden>

Tested-by: Fabio Fantoni <address@hidden>
This patch have solved the critical regression of all hvm domUs that prevented domUs starting with qemu 1.6.


I not see this patch on git for now, I think is urgent apply this patch or solve the problem in other way because qemu 1.6 is unusable at all for hvm domUs on xen and 1.5.x doesn't works with hvm domUs with ram > 2gb, with qemu 1.6 the regression of > 2 gb ram is solved but without this patch doesn't works at all.
Thanks for any reply and sorry for my bad english.


---

It works with this simple initialization to NULL, but would it be
necessary (or better) to assign a proper value to this variables ?
---
  hw/i386/pc_piix.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/hw/i386/pc_piix.c b/hw/i386/pc_piix.c
index 6e1e654..596d433 100644
--- a/hw/i386/pc_piix.c
+++ b/hw/i386/pc_piix.c
@@ -86,7 +86,7 @@ static void pc_init1(MemoryRegion *system_memory,
      BusState *idebus[MAX_IDE_BUS];
      ISADevice *rtc_state;
      ISADevice *floppy;
-    MemoryRegion *ram_memory;
+    MemoryRegion *ram_memory = NULL;
      MemoryRegion *pci_memory;
      MemoryRegion *rom_memory;
      DeviceState *icc_bridge;





reply via email to

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