From: Juan Quintela <quintela@redhat.com> To: qemu-devel@nongnu.org Cc: Laurent Vivier <lvivier@redhat.com>, Richard Henderson <rth@twiddle.net>, Paolo Bonzini <pbonzini@redhat.com>, kvm@vger.kernel.org, "Dr. David Alan Gilbert" <dgilbert@redhat.com>, Juan Quintela <quintela@redhat.com>, Thomas Huth <thuth@redhat.com>, Peng Tao <tao.peng@linux.alibaba.com>, Yury Kotov <yury-kotov@yandex-team.ru>, Jiangshan Lai <laijs@hyper.sh>, Xu Wang <xu@hyper.sh> Subject: [PULL 19/19] migration: allow private destination ram with x-ignore-shared Date: Fri, 12 Jul 2019 16:32:07 +0200 [thread overview] Message-ID: <20190712143207.4214-20-quintela@redhat.com> (raw) In-Reply-To: <20190712143207.4214-1-quintela@redhat.com> From: Peng Tao <tao.peng@linux.alibaba.com> By removing the share ram check, qemu is able to migrate to private destination ram when x-ignore-shared capability is on. Then we can create multiple destination VMs based on the same source VM. This changes the x-ignore-shared migration capability to work similar to Lai's original bypass-shared-memory work(https://lists.gnu.org/archive/html/qemu-devel/2018-04/msg00003.html) which enables kata containers (https://katacontainers.io) to implement the VM templating feature. An example usage in kata containers(https://katacontainers.io): 1. Start the source VM: qemu-system-x86 -m 2G \ -object memory-backend-file,id=mem0,size=2G,share=on,mem-path=/tmpfs/template-memory \ -numa node,memdev=mem0 2. Stop the template VM, set migration x-ignore-shared capability, migrate "exec:cat>/tmpfs/state", quit it 3. Start target VM: qemu-system-x86 -m 2G \ -object memory-backend-file,id=mem0,size=2G,share=off,mem-path=/tmpfs/template-memory \ -numa node,memdev=mem0 \ -incoming defer 4. connect to target VM qmp, set migration x-ignore-shared capability, migrate_incoming "exec:cat /tmpfs/state" 5. create more target VMs repeating 3 and 4 Cc: Dr. David Alan Gilbert <dgilbert@redhat.com> Cc: Yury Kotov <yury-kotov@yandex-team.ru> Cc: Jiangshan Lai <laijs@hyper.sh> Cc: Xu Wang <xu@hyper.sh> Signed-off-by: Peng Tao <tao.peng@linux.alibaba.com> Reviewed-by: Juan Quintela <quintela@redhat.com> Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com> Message-Id: <1560494113-1141-1-git-send-email-tao.peng@linux.alibaba.com> Signed-off-by: Juan Quintela <quintela@redhat.com> --- migration/ram.c | 7 ------- 1 file changed, 7 deletions(-) diff --git a/migration/ram.c b/migration/ram.c index 8a6ad61d3d..8622b4dc49 100644 --- a/migration/ram.c +++ b/migration/ram.c @@ -3426,7 +3426,6 @@ static int ram_save_setup(QEMUFile *f, void *opaque) } if (migrate_ignore_shared()) { qemu_put_be64(f, block->mr->addr); - qemu_put_byte(f, ramblock_is_ignored(block) ? 1 : 0); } } @@ -4393,12 +4392,6 @@ static int ram_load(QEMUFile *f, void *opaque, int version_id) } if (migrate_ignore_shared()) { hwaddr addr = qemu_get_be64(f); - bool ignored = qemu_get_byte(f); - if (ignored != ramblock_is_ignored(block)) { - error_report("RAM block %s should %s be migrated", - id, ignored ? "" : "not"); - ret = -EINVAL; - } if (ramblock_is_ignored(block) && block->mr->addr != addr) { error_report("Mismatched GPAs for block %s " -- 2.21.0
WARNING: multiple messages have this Message-ID (diff)
From: Juan Quintela <quintela@redhat.com> To: qemu-devel@nongnu.org Cc: Laurent Vivier <lvivier@redhat.com>, Thomas Huth <thuth@redhat.com>, kvm@vger.kernel.org, Juan Quintela <quintela@redhat.com>, "Dr. David Alan Gilbert" <dgilbert@redhat.com>, Yury Kotov <yury-kotov@yandex-team.ru>, Xu Wang <xu@hyper.sh>, Jiangshan Lai <laijs@hyper.sh>, Paolo Bonzini <pbonzini@redhat.com>, Richard Henderson <rth@twiddle.net> Subject: [Qemu-devel] [PULL 19/19] migration: allow private destination ram with x-ignore-shared Date: Fri, 12 Jul 2019 16:32:07 +0200 [thread overview] Message-ID: <20190712143207.4214-20-quintela@redhat.com> (raw) In-Reply-To: <20190712143207.4214-1-quintela@redhat.com> From: Peng Tao <tao.peng@linux.alibaba.com> By removing the share ram check, qemu is able to migrate to private destination ram when x-ignore-shared capability is on. Then we can create multiple destination VMs based on the same source VM. This changes the x-ignore-shared migration capability to work similar to Lai's original bypass-shared-memory work(https://lists.gnu.org/archive/html/qemu-devel/2018-04/msg00003.html) which enables kata containers (https://katacontainers.io) to implement the VM templating feature. An example usage in kata containers(https://katacontainers.io): 1. Start the source VM: qemu-system-x86 -m 2G \ -object memory-backend-file,id=mem0,size=2G,share=on,mem-path=/tmpfs/template-memory \ -numa node,memdev=mem0 2. Stop the template VM, set migration x-ignore-shared capability, migrate "exec:cat>/tmpfs/state", quit it 3. Start target VM: qemu-system-x86 -m 2G \ -object memory-backend-file,id=mem0,size=2G,share=off,mem-path=/tmpfs/template-memory \ -numa node,memdev=mem0 \ -incoming defer 4. connect to target VM qmp, set migration x-ignore-shared capability, migrate_incoming "exec:cat /tmpfs/state" 5. create more target VMs repeating 3 and 4 Cc: Dr. David Alan Gilbert <dgilbert@redhat.com> Cc: Yury Kotov <yury-kotov@yandex-team.ru> Cc: Jiangshan Lai <laijs@hyper.sh> Cc: Xu Wang <xu@hyper.sh> Signed-off-by: Peng Tao <tao.peng@linux.alibaba.com> Reviewed-by: Juan Quintela <quintela@redhat.com> Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com> Message-Id: <1560494113-1141-1-git-send-email-tao.peng@linux.alibaba.com> Signed-off-by: Juan Quintela <quintela@redhat.com> --- migration/ram.c | 7 ------- 1 file changed, 7 deletions(-) diff --git a/migration/ram.c b/migration/ram.c index 8a6ad61d3d..8622b4dc49 100644 --- a/migration/ram.c +++ b/migration/ram.c @@ -3426,7 +3426,6 @@ static int ram_save_setup(QEMUFile *f, void *opaque) } if (migrate_ignore_shared()) { qemu_put_be64(f, block->mr->addr); - qemu_put_byte(f, ramblock_is_ignored(block) ? 1 : 0); } } @@ -4393,12 +4392,6 @@ static int ram_load(QEMUFile *f, void *opaque, int version_id) } if (migrate_ignore_shared()) { hwaddr addr = qemu_get_be64(f); - bool ignored = qemu_get_byte(f); - if (ignored != ramblock_is_ignored(block)) { - error_report("RAM block %s should %s be migrated", - id, ignored ? "" : "not"); - ret = -EINVAL; - } if (ramblock_is_ignored(block) && block->mr->addr != addr) { error_report("Mismatched GPAs for block %s " -- 2.21.0
next prev parent reply other threads:[~2019-07-12 14:33 UTC|newest] Thread overview: 59+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-07-12 14:31 [PULL 00/19] Migration patches Juan Quintela 2019-07-12 14:31 ` [Qemu-devel] " Juan Quintela 2019-07-12 14:31 ` [PULL 01/19] migration: fix multifd_recv event typo Juan Quintela 2019-07-12 14:31 ` [Qemu-devel] " Juan Quintela 2019-07-12 14:31 ` [PULL 02/19] migration-test: rename parameter to parameter_int Juan Quintela 2019-07-12 14:31 ` [Qemu-devel] " Juan Quintela 2019-07-12 14:31 ` [PULL 03/19] migration-test: Add migration multifd test Juan Quintela 2019-07-12 14:31 ` [Qemu-devel] " Juan Quintela 2019-07-12 14:31 ` [PULL 04/19] migration/multifd: call multifd_send_sync_main when sending RAM_SAVE_FLAG_EOS Juan Quintela 2019-07-12 14:31 ` [Qemu-devel] " Juan Quintela 2019-07-12 14:31 ` [PULL 05/19] migration/xbzrle: update cache and current_data in one place Juan Quintela 2019-07-12 14:31 ` [Qemu-devel] " Juan Quintela 2019-07-12 14:31 ` [PULL 06/19] cutils: remove one unnecessary pointer operation Juan Quintela 2019-07-12 14:31 ` [Qemu-devel] " Juan Quintela 2019-07-12 14:31 ` [PULL 07/19] migration/multifd: sync packet_num after all thread are done Juan Quintela 2019-07-12 14:31 ` [Qemu-devel] " Juan Quintela 2019-07-12 14:31 ` [PULL 08/19] migration/ram.c: reset complete_round when we gets a queued page Juan Quintela 2019-07-12 14:31 ` [Qemu-devel] " Juan Quintela 2019-07-12 14:31 ` [PULL 09/19] migration: No need to take rcu during sync_dirty_bitmap Juan Quintela 2019-07-12 14:31 ` [Qemu-devel] " Juan Quintela 2019-07-12 14:31 ` [PULL 10/19] memory: Don't set migration bitmap when without migration Juan Quintela 2019-07-12 14:31 ` [Qemu-devel] " Juan Quintela 2019-07-12 14:31 ` [PULL 11/19] bitmap: Add bitmap_copy_with_{src|dst}_offset() Juan Quintela 2019-07-12 14:31 ` [Qemu-devel] " Juan Quintela 2019-07-12 14:32 ` [PULL 12/19] memory: Pass mr into snapshot_and_clear_dirty Juan Quintela 2019-07-12 14:32 ` [Qemu-devel] " Juan Quintela 2019-07-12 14:32 ` [PULL 13/19] memory: Introduce memory listener hook log_clear() Juan Quintela 2019-07-12 14:32 ` [Qemu-devel] " Juan Quintela 2019-07-12 14:32 ` [PULL 14/19] kvm: Update comments for sync_dirty_bitmap Juan Quintela 2019-07-12 14:32 ` [Qemu-devel] " Juan Quintela 2019-07-12 14:32 ` [PULL 15/19] kvm: Persistent per kvmslot dirty bitmap Juan Quintela 2019-07-12 14:32 ` [Qemu-devel] " Juan Quintela 2019-07-12 14:32 ` [PULL 16/19] kvm: Introduce slots lock for memory listener Juan Quintela 2019-07-12 14:32 ` [Qemu-devel] " Juan Quintela 2019-07-12 14:32 ` [PULL 17/19] kvm: Support KVM_CLEAR_DIRTY_LOG Juan Quintela 2019-07-12 14:32 ` [Qemu-devel] " Juan Quintela 2019-07-12 14:32 ` [PULL 18/19] migration: Split log_clear() into smaller chunks Juan Quintela 2019-07-12 14:32 ` [Qemu-devel] " Juan Quintela 2019-07-12 14:32 ` Juan Quintela [this message] 2019-07-12 14:32 ` [Qemu-devel] [PULL 19/19] migration: allow private destination ram with x-ignore-shared Juan Quintela 2019-07-12 16:33 ` [Qemu-devel] [PULL 00/19] Migration patches Peter Maydell 2019-07-12 16:33 ` Peter Maydell 2019-07-12 17:54 ` Dr. David Alan Gilbert 2019-07-12 17:54 ` Dr. David Alan Gilbert 2019-07-15 11:16 ` Peter Maydell 2019-07-15 11:16 ` Peter Maydell 2019-07-15 13:44 ` Juan Quintela 2019-07-15 13:44 ` [Qemu-devel] " Juan Quintela 2019-07-15 13:48 ` Peter Maydell 2019-07-15 13:48 ` [Qemu-devel] " Peter Maydell 2019-07-15 14:10 ` Juan Quintela 2019-07-15 14:10 ` [Qemu-devel] " Juan Quintela 2019-07-15 14:15 ` Peter Maydell 2019-07-15 14:15 ` [Qemu-devel] " Peter Maydell 2019-07-15 14:04 ` Daniel P. Berrangé 2019-07-15 14:04 ` Daniel P. Berrangé 2019-07-15 14:17 ` Peter Maydell 2019-07-15 14:17 ` Peter Maydell -- strict thread matches above, loose matches on Subject: below -- 2019-07-11 10:43 Juan Quintela 2019-07-11 10:44 ` [PULL 19/19] migration: allow private destination ram with x-ignore-shared 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=20190712143207.4214-20-quintela@redhat.com \ --to=quintela@redhat.com \ --cc=dgilbert@redhat.com \ --cc=kvm@vger.kernel.org \ --cc=laijs@hyper.sh \ --cc=lvivier@redhat.com \ --cc=pbonzini@redhat.com \ --cc=qemu-devel@nongnu.org \ --cc=rth@twiddle.net \ --cc=tao.peng@linux.alibaba.com \ --cc=thuth@redhat.com \ --cc=xu@hyper.sh \ --cc=yury-kotov@yandex-team.ru \ /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: linkBe 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.