All of lore.kernel.org
 help / color / mirror / Atom feed
From: Juan Quintela <quintela@redhat.com>
To: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
Cc: lizhijian@cn.fujitsu.com, wency@cn.fujitsu.com, qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [RFC 0/2] Disable hotplug during migration
Date: Wed, 29 Mar 2017 12:40:36 +0200	[thread overview]
Message-ID: <87lgronz6j.fsf@secure.mitica> (raw)
In-Reply-To: <20170328180321.GH5740@work-vm> (David Alan Gilbert's message of "Tue, 28 Mar 2017 19:03:21 +0100")

"Dr. David Alan Gilbert" <dgilbert@redhat.com> wrote:
> * Juan Quintela (quintela@redhat.com) wrote:
>> Hi
>> 
>> This series disable hotplug/unplug during migration.  Thank to Markus
>> for explaining where I had to put the checks.  Why?  Because during
>> migration we will fail if there are changes.  For instance, in
>> postcopy, if we add a memory region, we would failing.  Same for other
>> devices if they are not setup exactly the same on destination.
>> 
>> Iidea would be to disable it, andthen enable for the thing that we know that work.
>> 
>> This series are on top of my previous RAMState v2 serie.
>> 
>> Commets, please?
>
> So I think this is probably a good idea, but we should ask Li Zhijian who added
> migration_bitmap_extend if the reason for adding it was because they
> needed the hot add to work.
> cc'd.
>
> IMHO we really need a 'configuration change mutex' that you can take
> to stop any changes in the VM hardware, and that would probably reduce
> a lot of the places that hold the BQL for just stopping changes.

That was the previous patch.  With that one, you can't add/remove
hardware during migration.

Later, Juan.

  reply	other threads:[~2017-03-29 10:40 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-03-23 20:50 [Qemu-devel] [RFC 0/2] Disable hotplug during migration Juan Quintela
2017-03-23 20:50 ` [Qemu-devel] [PATCH 1/2] migration: Disable hotplug/unplug " Juan Quintela
2017-03-24  9:48   ` Thomas Huth
2017-03-24  9:50     ` Juan Quintela
2017-03-24 14:32   ` Igor Mammedov
2017-03-28 10:48     ` Juan Quintela
2017-03-23 20:50 ` [Qemu-devel] [PATCH 2/2] ram: remove migration_bitmap_extend() Juan Quintela
2017-03-28 18:03 ` [Qemu-devel] [RFC 0/2] Disable hotplug during migration Dr. David Alan Gilbert
2017-03-29 10:40   ` Juan Quintela [this message]
2017-03-29 10:41   ` Juan Quintela

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=87lgronz6j.fsf@secure.mitica \
    --to=quintela@redhat.com \
    --cc=dgilbert@redhat.com \
    --cc=lizhijian@cn.fujitsu.com \
    --cc=qemu-devel@nongnu.org \
    --cc=wency@cn.fujitsu.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.