From: Andrew Jones <drjones@redhat.com>
To: qemu-devel@nongnu.org, qemu-arm@nongnu.org
Cc: pbonzini@redhat.com, maz@kernel.org, eric.auger@redhat.com,
peter.maydell@linaro.org
Subject: [PATCH v2 0/2] hw/arm/virt: KVM: Set IPA limit when possible
Date: Wed, 10 Mar 2021 14:52:16 +0100 [thread overview]
Message-ID: <20210310135218.255205-1-drjones@redhat.com> (raw)
This series fixes IPA limit setting for mach-virt KVM guests. The
first patch restores the setting of IPA limits for values greater
than 40 (the default) when necessary. The second patch ensures values
less than 40 may also be used. The default KVM type=0 (which means
an IPA limit of 40) is still used for legacy KVM, since it must be.
I tested this with a KVM that supports KVM_CAP_ARM_VM_IPA_SIZE and
with a KVM that does not. mach-virt's memory map didn't allow me
to test with less than 40 on the KVM_CAP_ARM_VM_IPA_SIZE supporting
host, but a quick VM fd opening test seemed to prove KVM would be
happy with that. Testing was done with a typical Linux guest and also
with kvm-unit-tests.
I caught the bug that the first patch fixes by instrumenting QEMU
to observe which IPA limit was getting selected, and then seeing
that QEMU wasn't actually running mach-virt's kvm_type method at
all!
Thanks,
drew
Andrew Jones (2):
accel: kvm: Fix kvm_type invocation
hw/arm/virt: KVM: The IPA lower bound is 32
accel/kvm/kvm-all.c | 2 ++
hw/arm/virt.c | 23 ++++++++++++++++-------
include/hw/boards.h | 1 +
target/arm/kvm.c | 4 +++-
target/arm/kvm_arm.h | 6 ++++--
5 files changed, 26 insertions(+), 10 deletions(-)
--
2.26.2
next reply other threads:[~2021-03-10 13:53 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-03-10 13:52 Andrew Jones [this message]
2021-03-10 13:52 ` [PATCH v2 1/2] accel: kvm: Fix kvm_type invocation Andrew Jones
2021-03-10 14:19 ` Auger Eric
2021-03-10 13:52 ` [PATCH v2 2/2] hw/arm/virt: KVM: The IPA lower bound is 32 Andrew Jones
2021-03-10 14:58 ` Auger Eric
2021-03-11 9:59 ` Marc Zyngier
2021-03-12 12:47 ` [PATCH v2 0/2] hw/arm/virt: KVM: Set IPA limit when possible Peter Maydell
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=20210310135218.255205-1-drjones@redhat.com \
--to=drjones@redhat.com \
--cc=eric.auger@redhat.com \
--cc=maz@kernel.org \
--cc=pbonzini@redhat.com \
--cc=peter.maydell@linaro.org \
--cc=qemu-arm@nongnu.org \
--cc=qemu-devel@nongnu.org \
/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).