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:41:19 +0200 [thread overview]
Message-ID: <87k278nz5c.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.
I guess that the problem for them is that the bitmap size was wrong
after memory resizing. But I don't know if this has ever worked.
> 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.
prev parent reply other threads:[~2017-03-29 10:41 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
2017-03-29 10:41 ` Juan Quintela [this message]
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=87k278nz5c.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.