From: Don Slutz <dslutz@verizon.com>
To: xen-devel@lists.xensource.com, qemu-devel@nongnu.org,
Stefano Stabellini <stefano.stabellini@eu.citrix.com>
Cc: "Igor Mammedov" <imammedo@redhat.com>,
"Don Slutz" <dslutz@verizon.com>,
"Andreas Färber" <afaerber@suse.de>,
"Anthony Liguori" <aliguori@amazon.com>,
"Michael S. Tsirkin" <mst@redhat.com>
Subject: [PATCH v5 0/3] Add max-ram-below-4g (was Add pci_hole_min_size machine option)
Date: Fri, 6 Jun 2014 13:52:03 -0400 [thread overview]
Message-ID: <1402077126-17799-1-git-send-email-dslutz@verizon.com> (raw)
Changes v4 to v5:
Re-work based on:
https://github.com/imammedo/qemu/commits/memory-hotplug-v11
And so it now depends on this patch set.
Stefano Stabellini:
#3 "xen-hvm: Pass is_default to xen_hvm_init"
Acked-by
Minor change of pmc to pcms.
Changes v3 to v4:
Split out #2 "GlobalProperty: Display warning about unused -global"
rebase on e00fcfe (origin/master)
rename xen-all to xen-hvm
Adjust #1 "xen-hvm: Fix xen_hvm_init() to adjust pc memory layout"
Switch Acked-by & Signed-off-by
rebase on master
Rework #3 "xen-hvm: Pass is_default to xen_hvm_init":
To pass is_default instead of max_ram_below_4g.
Also did not add "Acked-by: Stefano Stabellini" since code changed a lot.
Andreas Färber:
all: Remove dot at end of subject
#3 "xen-hvm: Pass is_default to xen_hvm_init"
Adjust comment formatting.
Andreas Färber, Paolo Bonzini, Marcel Apfelbaum:
rework to use "opts per machine"
Drop old #3, new #2.
Changes v2 to v3:
Stefano Stabellini:
Acked-by #1 "xen-all: Fix xen_hvm_init() to adjust pc memory"
Adjust for code readability #4 "xen-all: Pass max_ram_below_4g to xen_hvm_init."
Set max_ram_below_4g always and use it to calculate above_4g_mem_size,
below_4g_mem_size.
Changes v1 to v2:
Michael S. Tsirkin:
Rename option.
Only add it to machine types that support it.
Split into 4 parts.
1/4 -- xen-all: Fix xen_hvm_init() to adjust pc memory layout
This looks to be a possible bug that has yet to be found.
below_4g_mem_size and above_4g_mem_size are stored in PcGuestInfo
(pc_guest_info_init) which are currently not "correct". This and
4/4 change the same lines.
2/4 -- GlobalProperty: Display warning about unused -global
My testing showed that setting a global property on an object
that is not used is not reported at all. This is added to help
when the new global is set but not used. The negative not_used
was picked so that all static objects are assumed to be used
even when they are not.
3/4 -- pc & q35: Add new object pc-memory-layout
The objects that it might make sense to add this property to all
get created too late. So add a new object just to hold this
property. Name it so that it is expected that only pc (and q35)
machine types support it.
4/4 -- xen-all: Pass max_ram_below_4g to xen_hvm_init
Seprate the xen only part of the change. Currectly based on patch 1/4
Don Slutz (3):
xen-hvm: Fix xen_hvm_init() to adjust pc memory layout
pc & q35: Add new machine opt max-ram-below-4g
xen-hvm: Pass is_default to xen_hvm_init
hw/i386/pc.c | 38 ++++++++++++++++++++++++++++++++++++++
hw/i386/pc_piix.c | 41 ++++++++++++++++++++++++++---------------
hw/i386/pc_q35.c | 39 +++++++++++++++++++++++++--------------
include/hw/i386/pc.h | 3 +++
include/hw/xen/xen.h | 3 ++-
vl.c | 4 ++++
xen-hvm-stub.c | 3 ++-
xen-hvm.c | 46 +++++++++++++++++++++++++++-------------------
8 files changed, 127 insertions(+), 50 deletions(-)
--
1.8.4
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
next reply other threads:[~2014-06-06 17:52 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-06-06 17:52 Don Slutz [this message]
2014-06-06 17:52 ` [PATCH v5 1/3] xen-hvm: Fix xen_hvm_init() to adjust pc memory layout Don Slutz
2014-06-06 17:52 ` [PATCH v5 2/3] pc & q35: Add new machine opt max-ram-below-4g Don Slutz
2014-06-08 15:40 ` Michael S. Tsirkin
2014-06-09 14:20 ` Don Slutz
2014-06-09 14:38 ` Michael S. Tsirkin
2014-06-09 15:10 ` [Qemu-devel] " Marcel Apfelbaum
2014-06-09 15:37 ` Igor Mammedov
2014-06-09 17:33 ` Marcel Apfelbaum
2014-06-09 20:03 ` Don Slutz
2014-06-09 19:13 ` Don Slutz
2014-06-10 7:36 ` Gerd Hoffmann
2014-06-17 17:51 ` Don Slutz
2014-06-18 9:52 ` Gerd Hoffmann
2014-06-17 18:22 ` Michael S. Tsirkin
2014-06-17 18:44 ` Don Slutz
2014-06-17 19:43 ` Michael S. Tsirkin
2014-06-17 20:05 ` Michael S. Tsirkin
2014-06-17 20:08 ` [Xen-devel] " Konrad Rzeszutek Wilk
2014-06-17 20:17 ` Michael S. Tsirkin
2014-06-18 13:28 ` Slutz, Donald Christopher
2014-06-17 20:35 ` [Xen-devel] " Pasi Kärkkäinen
2014-06-06 17:52 ` [PATCH v5 3/3] xen-hvm: Pass is_default to xen_hvm_init Don Slutz
2014-06-08 15:24 ` Michael S. Tsirkin
2014-06-09 14:25 ` Don Slutz
2014-06-09 14:39 ` Michael S. Tsirkin
2014-06-08 15:42 ` Michael S. Tsirkin
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1402077126-17799-1-git-send-email-dslutz@verizon.com \
--to=dslutz@verizon.com \
--cc=afaerber@suse.de \
--cc=aliguori@amazon.com \
--cc=imammedo@redhat.com \
--cc=mst@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=stefano.stabellini@eu.citrix.com \
--cc=xen-devel@lists.xensource.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).