[Top][All Lists]

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

[qemu-s390x] [PATCH v1 0/5] Fix misuses of memory_region_allocate_system

From: Igor Mammedov
Subject: [qemu-s390x] [PATCH v1 0/5] Fix misuses of memory_region_allocate_system_memory()
Date: Mon, 15 Apr 2019 15:27:17 +0200

memory_region_allocate_system_memory() is supposed to be called only once,
but due to lack of the check some boards abused it by  calling it multiple 
So here goes fixes that replace it with  memory_region_init_ram() in most of
the cases.
However s390 case a bit convolted as we have to keep cross version migration
working and need -mem-path to work there as well. To make it happen I hacked
MemoryRegion aliases to work with migration by allocating 'fake' RAMBlock that
aliases original RAM pointer (asking for comments on approach since I have no
idea how migration works really). It is one way to fix s390 while not abusing
memory_region_allocate_system_memory() and making it work with -mem-path
(another a bit more dirty way would be adding machine callback to handle
 -mem-path and overiding it in s390, but migratable aliases approach looks more
cleaner to me API wise and have potential to be reused if we would need to
fix up RAM layout but keep migration stream compatible).

CC: Paolo Bonzini <address@hidden> 
CC: Richard Henderson <address@hidden> 
CC: Helge Deller <address@hidden> 
CC: "Hervé Poussineau" <address@hidden> 
CC: David Gibson <address@hidden> 
CC: Cornelia Huck <address@hidden> 
CC: Halil Pasic <address@hidden> 
CC: Christian Borntraeger <address@hidden> 
CC: David Hildenbrand <address@hidden> 
CC: Artyom Tarasenko <address@hidden> 
CC: Mark Cave-Ayland <address@hidden> 
CC: address@hidden
CC: address@hidden

Igor Mammedov (5):
  sparc64: use memory_region_allocate_system_memory() only for '-m'
    specified RAM
  ppc: rs6000_mc: drop usage of memory_region_allocate_system_memory()
  hppa: drop usage of memory_region_allocate_system_memory() for ROM
  memory: make MemoryRegion alias migratable
  s390: do not call memory_region_allocate_system_memory() multiple

 exec.c                     |  7 ++++---
 hw/hppa/machine.c          |  5 ++---
 hw/ppc/rs6000_mc.c         | 15 ++++++++++-----
 hw/s390x/s390-virtio-ccw.c | 20 +++++++++++++++-----
 hw/sparc64/niagara.c       | 25 +++++++++++++------------
 memory.c                   |  5 +++++
 6 files changed, 49 insertions(+), 28 deletions(-)


reply via email to

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