qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
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

             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).