[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 for 1.6 0/8] Guest memory allocation fixes &
From: |
Laszlo Ersek |
Subject: |
Re: [Qemu-devel] [PATCH v3 for 1.6 0/8] Guest memory allocation fixes & cleanup |
Date: |
Wed, 31 Jul 2013 17:35:33 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130621 Thunderbird/17.0.7 |
On 07/31/13 15:11, Markus Armbruster wrote:
> All I wanted to do is exit(1) instead of abort() on guest memory
> allocation failure [07/08]. But that lead me into a minor #ifdef bog,
> and here's what I brought back. Enjoy!
>
> Testing:
> * Christian Borntraeger reports v1 works fine under LPAR (new S390
> KVM, i.e. generic allocation) and as second guest under z/VM (old
> S390 KVM, i.e. legacy S390 allocation). Thanks for testing, and for
> catching a stupid mistake. v2 differs from v1 only in code that
> isn't reachable on S390.
>
> Changes since v2:
> * Straightforward rebase, only 4/8 conflicted
> Changes since v1:
> * 5/8: Fix assertion in qemu_ram_remap() (Paolo)
> * All other patches unchanged except for Acked-by in commit messages
> Changes since RFC:
> * 1-3+8/8 unchanged except for commit message tweaks
> * 4+6/8 rewritten to address Paolo's review
> * 5/8 rewritten: don't fix dead code, just assert it's dead
> * 7/8 fix mistakes caught by Richard Henderson and Peter Maydell
>
> Markus Armbruster (8):
> exec: Fix Xen RAM allocation with unusual options
> exec: Clean up fall back when -mem-path allocation fails
> exec: Reduce ifdeffery around -mem-path
> exec: Simplify the guest physical memory allocation hook
> exec: Drop incorrect & dead S390 code in qemu_ram_remap()
> exec: Clean up unnecessary S390 ifdeffery
> exec: Don't abort when we can't allocate guest memory
> pc_sysfw: Fix ISA BIOS init for ridiculously big flash
>
> exec.c | 120
> ++++++++++++++++++++++++++----------------------
> hw/block/pc_sysfw.c | 5 +-
> include/exec/cpu-all.h | 2 -
> include/exec/exec-all.h | 2 +
> include/sysemu/kvm.h | 5 --
> kvm-all.c | 13 ------
> target-s390x/kvm.c | 23 +++-------
> util/oslib-posix.c | 4 +-
> util/oslib-win32.c | 5 +-
> 9 files changed, 77 insertions(+), 102 deletions(-)
>
Acked-by: Laszlo Ersek <address@hidden>
- [Qemu-devel] [PATCH v3 for 1.6 0/8] Guest memory allocation fixes & cleanup, Markus Armbruster, 2013/07/31
- [Qemu-devel] [PATCH v3 for 1.6 1/8] exec: Fix Xen RAM allocation with unusual options, Markus Armbruster, 2013/07/31
- [Qemu-devel] [PATCH v3 for 1.6 5/8] exec: Drop incorrect & dead S390 code in qemu_ram_remap(), Markus Armbruster, 2013/07/31
- [Qemu-devel] [PATCH v3 for 1.6 6/8] exec: Clean up unnecessary S390 ifdeffery, Markus Armbruster, 2013/07/31
- [Qemu-devel] [PATCH v3 for 1.6 2/8] exec: Clean up fall back when -mem-path allocation fails, Markus Armbruster, 2013/07/31
- [Qemu-devel] [PATCH v3 for 1.6 4/8] exec: Simplify the guest physical memory allocation hook, Markus Armbruster, 2013/07/31
- [Qemu-devel] [PATCH v3 for 1.6 7/8] exec: Don't abort when we can't allocate guest memory, Markus Armbruster, 2013/07/31
- [Qemu-devel] [PATCH v3 for 1.6 8/8] pc_sysfw: Fix ISA BIOS init for ridiculously big flash, Markus Armbruster, 2013/07/31
- Re: [Qemu-devel] [PATCH v3 for 1.6 0/8] Guest memory allocation fixes & cleanup,
Laszlo Ersek <=
- [Qemu-devel] [PATCH v3 for 1.6 3/8] exec: Reduce ifdeffery around -mem-path, Markus Armbruster, 2013/07/31