Skip to content

RFC Guest memory allocation fixes & cleanup

Markus Armbruster qemu-devel 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 RFC Latest8 patches 14 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/8exec: Fix Xen RAM allocation with unusual optionsexec.c+12 -81 reply
2/8exec: Clean up fall back when -mem-path allocation failsexec.c+2 -51 reply
3/8exec: Reduce ifdeffery around -mem-pathexec.c include/exec/cpu-all.h+10 -290 replies
4/8s390: Simplify the RAM allocation hookexec.c include/sysemu/kvm.h kvm-all.c +1 more+11 -282 replies
5/8s390: Make qemu_ram_remap() consistent with allocationexec.c include/sysemu/kvm.h kvm-all.c +1 more+19 -171 reply
6/8exec: Clean up unnecessary S390 ifdefferyexec.c+19 -123 replies
7/8exec: Don't abort when we can't allocate guest memoryexec.c target-s390x/kvm.c util/oslib-posix.c +1 more+17 -135 replies
8/8pc_sysfw: Fix ISA BIOS init for ridiculously big flashhw/block/pc_sysfw.c+1 -40 replies

Reviews trailers per patch

4 of 8 patches have a Reviewed-by or Acked-by.

PatchAndreas FärberStefano StabelliniPaolo Bonzini
1/8A-b
2/8R-b
3/8
4/8R-b
5/8
6/8
7/8A-b
8/8

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 3 reviewers.

Read the v1 thread

Cover letter [Qemu-devel] [PATCH RFC 0/8] Guest memory allocation fixes & cleanup

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!

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
s390: Simplify the RAM allocation hook
s390: Make qemu_ram_remap() consistent with allocation
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                 | 117 ++++++++++++++++++++++++-------------------------
 hw/block/pc_sysfw.c    |   5 +--
 include/exec/cpu-all.h |   2 -
 include/sysemu/kvm.h   |   4 +-
 kvm-all.c              |  16 ++-----
 target-s390x/kvm.c     |  34 +++++++-------
 util/oslib-posix.c     |   4 +-
 util/oslib-win32.c     |   5 +--
 8 files changed, 81 insertions(+), 106 deletions(-)
Signature · 1 line
1.7.11.7