From: "Dr. David Alan Gilbert (git)" <dgilbert@redhat.com>
To: qemu-devel@nongnu.org, vkuznets@redhat.com, mst@redhat.com,
jasowang@redhat.com, pbonzini@redhat.com
Subject: [PATCH v3 0/2] Fix hyperv synic on vhost
Date: Thu, 16 Jan 2020 20:24:12 +0000 [thread overview]
Message-ID: <20200116202414.157959-1-dgilbert@redhat.com> (raw)
From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
Hyperv's synic (that we emulate) is a feature that allows the guest
to place some magic (4k) pages of RAM anywhere it likes in GPA.
This confuses vhost's RAM section merging when these pages
land over the top of hugepages.
This v3 takes a different approach to v2 and v1.
It avoids doing the hugepage alignment except for vhost-user:
a) Vhost kernel : doesn't need alignment, it's turned off
synic won't cause a problem.
b) vhost user : Already filters out anything without an fd
synic won't cause a problem.
(Not tried vhost-user yet, it currently seems broken even without this).
This might also cause some other reported problems with vga
pages causing similar issues.
bz: https://bugzilla.redhat.com/show_bug.cgi?id=1779041
Dr. David Alan Gilbert (2):
vhost: Add names to section rounded warning
vhost: Only align sections for vhost-user
hw/virtio/vhost.c | 39 +++++++++++++++++++++------------------
1 file changed, 21 insertions(+), 18 deletions(-)
--
2.24.1
next reply other threads:[~2020-01-16 20:25 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-01-16 20:24 Dr. David Alan Gilbert (git) [this message]
2020-01-16 20:24 ` [PATCH v3 1/2] vhost: Add names to section rounded warning Dr. David Alan Gilbert (git)
2020-01-16 20:24 ` [PATCH v3 2/2] vhost: Only align sections for vhost-user Dr. David Alan Gilbert (git)
2020-01-17 12:52 ` Paolo Bonzini
2020-01-17 13:40 ` Michael S. Tsirkin
2020-01-17 13:58 ` Paolo Bonzini
2020-01-17 14:25 ` Michael S. Tsirkin
2020-01-17 15:06 ` Paolo Bonzini
2020-01-17 15:10 ` Dr. David Alan Gilbert
2020-01-17 0:24 ` [PATCH v3 0/2] Fix hyperv synic on vhost no-reply
2020-01-17 12:47 ` Dr. David Alan Gilbert
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=20200116202414.157959-1-dgilbert@redhat.com \
--to=dgilbert@redhat.com \
--cc=jasowang@redhat.com \
--cc=mst@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=vkuznets@redhat.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).