From: "Dr. David Alan Gilbert (git)" <dgilbert@redhat.com>
To: qemu-devel@nongnu.org, mst@redhat.com, imammedo@redhat.com
Cc: maxime.coquelin@redhat.com
Subject: [Qemu-devel] [PATCH v7 0/7] Rework vhost memory region updates
Date: Fri, 19 Jan 2018 10:39:17 +0000 [thread overview]
Message-ID: <20180119103924.332-1-dgilbert@redhat.com> (raw)
From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
Hi,
This patch set reworks the way the vhost code handles changes in
physical address space layout that came from a discussion with Igor.
Its intention is to simplify a lot of the update code,
and to make it easier for the postcopy+shared code to
do the hugepage alignments that are needed.
Instead of inserting/removing each section during the add/del
callbacks of the listener, we start afresh and build a list
from the add and nop callbacks, then at the end compare the list
we've built with the exisiting list.
v7
Avoid calling memcmp(NULL,NULL,0) in region list compare
v6
Tidy ups from Igor
The biggest change is moving the 'Move log_dirty check' to be
the last patch in the set.
Dr. David Alan Gilbert (7):
vhost: Build temporary section list and deref after commit
vhost: Simplify ring verification checks
vhost: Merge sections added to temporary list
vhost: Regenerate region list from changed sections list
vhost: Clean out old vhost_set_memory and friends
vhost: Merge and delete unused callbacks
vhost: Move log_dirty check
hw/virtio/trace-events | 6 +
hw/virtio/vhost.c | 497 ++++++++++++++++------------------------------
include/hw/virtio/vhost.h | 5 +-
3 files changed, 180 insertions(+), 328 deletions(-)
--
2.14.3
next reply other threads:[~2018-01-19 10:39 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-01-19 10:39 Dr. David Alan Gilbert (git) [this message]
2018-01-19 10:39 ` [Qemu-devel] [PATCH v7 1/7] vhost: Build temporary section list and deref after commit Dr. David Alan Gilbert (git)
2018-01-19 10:39 ` [Qemu-devel] [PATCH v7 2/7] vhost: Simplify ring verification checks Dr. David Alan Gilbert (git)
2018-01-19 10:39 ` [Qemu-devel] [PATCH v7 3/7] vhost: Merge sections added to temporary list Dr. David Alan Gilbert (git)
2018-01-19 10:39 ` [Qemu-devel] [PATCH v7 4/7] vhost: Regenerate region list from changed sections list Dr. David Alan Gilbert (git)
2018-01-19 14:42 ` Igor Mammedov
2018-01-19 10:39 ` [Qemu-devel] [PATCH v7 5/7] vhost: Clean out old vhost_set_memory and friends Dr. David Alan Gilbert (git)
2018-01-19 10:39 ` [Qemu-devel] [PATCH v7 6/7] vhost: Merge and delete unused callbacks Dr. David Alan Gilbert (git)
2018-01-19 10:39 ` [Qemu-devel] [PATCH v7 7/7] vhost: Move log_dirty check Dr. David Alan Gilbert (git)
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=20180119103924.332-1-dgilbert@redhat.com \
--to=dgilbert@redhat.com \
--cc=imammedo@redhat.com \
--cc=maxime.coquelin@redhat.com \
--cc=mst@redhat.com \
--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).