[Qemu-devel] [PULL 0/2] migration queue

[Qemu-devel] [PULL 0/2] migration queue

[Juan Quintela]

Hi

Anthony, this fixes two errors on the migration code, could you pull?

Thanks, Juan.

The following changes since commit 1c97e303d4ea80a2691334b0febe87a50660f99d:

  Merge remote-tracking branch 'afaerber/qom-cpu' into staging (2012-12-10 08:35:15 -0600)

are available in the git repository at:


  git://repo.or.cz/qemu/quintela.git migration.next

for you to fetch changes up to 77db8657048f233edf21e1a9ebdc30a367fbdc36:

  migration: Fix madvise breakage if host and guest have different page sizes (2012-12-11 12:45:56 +0100)

----------------------------------------------------------------
David Gibson (2):
      Fix off-by-1 error in RAM migration code
      migration: Fix madvise breakage if host and guest have different page sizes

 arch_init.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

[Qemu-devel] [PATCH 1/2] Fix off-by-1 error in RAM migration code

[Juan Quintela]

From: David Gibson <david@gibson.dropbear.id.au>

The code for migrating (or savevm-ing) memory pages starts off by creating
a dirty bitmap and filling it with 1s.  Except, actually, because bit
addresses are 0-based it fills every bit except bit 0 with 1s and puts an
extra 1 beyond the end of the bitmap, potentially corrupting unrelated
memory.  Oops.  This patch fixes it.

Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Signed-off-by: Juan Quintela <quintela@redhat.com>
---
 arch_init.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch_init.c b/arch_init.c
index e6effe8..b75a4c5 100644
--- a/arch_init.c
+++ b/arch_init.c
@@ -568,7 +568,7 @@ static int ram_save_setup(QEMUFile *f, void *opaque)
     int64_t ram_pages = last_ram_offset() >> TARGET_PAGE_BITS;

     migration_bitmap = bitmap_new(ram_pages);
-    bitmap_set(migration_bitmap, 1, ram_pages);
+    bitmap_set(migration_bitmap, 0, ram_pages);
     migration_dirty_pages = ram_pages;

     bytes_transferred = 0;
-- 
1.7.11.7

[Qemu-devel] [PATCH 2/2] migration: Fix madvise breakage if host and guest have different page sizes

[Juan Quintela]

From: David Gibson <david@gibson.dropbear.id.au>

madvise(DONTNEED) will throw away the contents of the whole page at the
given address, even if the given length is less than the page size.  One
can argue about whether that's the correct behaviour, but that's what it's
done for a long time in Linux at least.

That means that the madvise() in ram_load(), on a setup where
TARGET_PAGE_SIZE is smaller than the host page size, can throw away data
in guest pages adjacent to the one it's actually processing right now,
leading to guest memory corruption on an incoming migration.

This patch therefore, disables the madvise() if the host page size is
larger than TARGET_PAGE_SIZE.  This means we don't get the benefits of that
madvise() in this case, but a more complete fix is more difficult to
accomplish.  This at least fixes the guest memory corruption.

Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Reported-by: Alexey Kardashevskiy <aik@ozlabs.ru>
Signed-off-by: Juan Quintela <quintela@redhat.com>
---
 arch_init.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/arch_init.c b/arch_init.c
index b75a4c5..83dcc53 100644
--- a/arch_init.c
+++ b/arch_init.c
@@ -840,7 +840,8 @@ static int ram_load(QEMUFile *f, void *opaque, int version_id)
             memset(host, ch, TARGET_PAGE_SIZE);
 #ifndef _WIN32
             if (ch == 0 &&
-                (!kvm_enabled() || kvm_has_sync_mmu())) {
+                (!kvm_enabled() || kvm_has_sync_mmu()) &&
+                getpagesize() <= TARGET_PAGE_SIZE) {
                 qemu_madvise(host, TARGET_PAGE_SIZE, QEMU_MADV_DONTNEED);
             }
 #endif
-- 
1.7.11.7

Re: [Qemu-devel] [PULL 0/2] migration queue

[Anthony Liguori]

Thanks, applied.

Regards,

Anthony Liguori

Re: [Qemu-devel] [PULL 0/2] migration queue

[Anthony Liguori]

Anthony Liguori <aliguori@us.ibm.com> writes:

> Thanks, applied.

Please disregard.  That's a bug in my script.  Sorry for the noise.

Regards,

Anthony Liguori

>
> Regards,
>
> Anthony Liguori

[Qemu-devel] [PULL 0/2] migration queue

[Dr. David Alan Gilbert (git)]

From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>

The following changes since commit 47d3b60858d90ac8a0cc3a72af7f95c96781125a:

  Merge remote-tracking branch 'remotes/riscv/tags/riscv-qemu-2.12-important-fixes' into staging (2018-03-28 22:13:38 +0100)

are available in the Git repository at:

  git://github.com/dagrh/qemu.git tags/pull-migration-20180329a

for you to fetch changes up to 0746a92612276aee69e66dfe6782b0f882d221d5:

  migration: Don't activate block devices if using -S (2018-03-29 14:53:16 +0100)

----------------------------------------------------------------
Migration pull (small fixes)

A pair of two small fixes for 2.12.

----------------------------------------------------------------
Dr. David Alan Gilbert (1):
      migration: Don't activate block devices if using -S

Marc-André Lureau (1):
      migration: fix pfd leak

 migration/migration.c    | 22 +++++++++++++++-------
 migration/postcopy-ram.c |  1 +
 2 files changed, 16 insertions(+), 7 deletions(-)

Re: [Qemu-devel] [PULL 0/2] migration queue

[Peter Maydell]

On 29 March 2018 at 14:57, Dr. David Alan Gilbert (git)
<dgilbert@redhat.com> wrote:
> From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
>
> The following changes since commit 47d3b60858d90ac8a0cc3a72af7f95c96781125a:
>
>   Merge remote-tracking branch 'remotes/riscv/tags/riscv-qemu-2.12-important-fixes' into staging (2018-03-28 22:13:38 +0100)
>
> are available in the Git repository at:
>
>   git://github.com/dagrh/qemu.git tags/pull-migration-20180329a
>
> for you to fetch changes up to 0746a92612276aee69e66dfe6782b0f882d221d5:
>
>   migration: Don't activate block devices if using -S (2018-03-29 14:53:16 +0100)
>
> ----------------------------------------------------------------
> Migration pull (small fixes)
>
> A pair of two small fixes for 2.12.
>
> ----------------------------------------------------------------
> Dr. David Alan Gilbert (1):
>       migration: Don't activate block devices if using -S
>
> Marc-André Lureau (1):
>       migration: fix pfd leak
>
>  migration/migration.c    | 22 +++++++++++++++-------
>  migration/postcopy-ram.c |  1 +
>  2 files changed, 16 insertions(+), 7 deletions(-)
>

Applied, thanks.

-- PMM

[Qemu-devel] [PULL 0/2] migration queue

[Dr. David Alan Gilbert (git)]

From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>

The following changes since commit 47c1cc30e440860aa695358f7c2dd0b9d7b53d16:

  Update version for v3.1.0-rc2 release (2018-11-20 18:10:26 +0000)

are available in the Git repository at:

  git://github.com/dagrh/qemu.git tags/pull-migration-20181121a

for you to fetch changes up to 7e934f5b27eee1b0d77be3dcf1fd18d4ccc602cc:

  migration/migration.c: Add COLO dependency checks (2018-11-21 11:38:12 +0000)

----------------------------------------------------------------
Migration fixes 2018-11-21

Notably the fix for building with --disable-replication

----------------------------------------------------------------
Zhang Chen (2):
      migration/colo.c: Fix compilation issue when disable replication
      migration/migration.c: Add COLO dependency checks

 migration/colo.c      | 28 +++++++++++++++++++++++++++-
 migration/migration.c |  9 +++++++++
 2 files changed, 36 insertions(+), 1 deletion(-)

Re: [Qemu-devel] [PULL 0/2] migration queue

[Peter Maydell]

On 21 November 2018 at 13:04, Dr. David Alan Gilbert (git)
<dgilbert@redhat.com> wrote:
> From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
>
> The following changes since commit 47c1cc30e440860aa695358f7c2dd0b9d7b53d16:
>
>   Update version for v3.1.0-rc2 release (2018-11-20 18:10:26 +0000)
>
> are available in the Git repository at:
>
>   git://github.com/dagrh/qemu.git tags/pull-migration-20181121a
>
> for you to fetch changes up to 7e934f5b27eee1b0d77be3dcf1fd18d4ccc602cc:
>
>   migration/migration.c: Add COLO dependency checks (2018-11-21 11:38:12 +0000)
>
> ----------------------------------------------------------------
> Migration fixes 2018-11-21
>
> Notably the fix for building with --disable-replication
>

Applied, thanks.

-- PMM

[Qemu-devel] [PULL 0/2] migration queue

[Dr. David Alan Gilbert (git)]

From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>

The following changes since commit 10546e09e174e0bb185b66a4c397aa845efcd36e:

  Merge remote-tracking branch 'remotes/palmer/tags/riscv-for-master-4.0-rc3-v2' into staging (2019-04-05 04:50:30 +0100)

are available in the Git repository at:

  git://github.com/dagrh/qemu.git tags/pull-migration-20190405a

for you to fetch changes up to d013283a46e1f197594e76ed320f867067ab80b3:

  migration: Fix migrate_set_parameter (2019-04-05 15:32:13 +0100)

----------------------------------------------------------------
Migration fixes pull for 4.0

A couple of fixes for crashes in colo and
migration parameters.

----------------------------------------------------------------
Juan Quintela (1):
      migration: Fix migrate_set_parameter

Zhang Chen (1):
      migration/ram.c: Fix codes conflict about bitmap_mutex

 hmp.c           | 6 ++++--
 migration/ram.c | 2 ++
 2 files changed, 6 insertions(+), 2 deletions(-)

[Qemu-devel] [PULL 0/2] migration queue

[Dr. David Alan Gilbert (git)]

From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>

The following changes since commit 10546e09e174e0bb185b66a4c397aa845efcd36e:

  Merge remote-tracking branch 'remotes/palmer/tags/riscv-for-master-4.0-rc3-v2' into staging (2019-04-05 04:50:30 +0100)

are available in the Git repository at:

  git://github.com/dagrh/qemu.git tags/pull-migration-20190405a

for you to fetch changes up to d013283a46e1f197594e76ed320f867067ab80b3:

  migration: Fix migrate_set_parameter (2019-04-05 15:32:13 +0100)

----------------------------------------------------------------
Migration fixes pull for 4.0

A couple of fixes for crashes in colo and
migration parameters.

----------------------------------------------------------------
Juan Quintela (1):
      migration: Fix migrate_set_parameter

Zhang Chen (1):
      migration/ram.c: Fix codes conflict about bitmap_mutex

 hmp.c           | 6 ++++--
 migration/ram.c | 2 ++
 2 files changed, 6 insertions(+), 2 deletions(-)

Re: [Qemu-devel] [PULL 0/2] migration queue

[Peter Maydell]

On Fri, 5 Apr 2019 at 16:51, Dr. David Alan Gilbert (git)
<dgilbert@redhat.com> wrote:
>
> From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
>
> The following changes since commit 10546e09e174e0bb185b66a4c397aa845efcd36e:
>
>   Merge remote-tracking branch 'remotes/palmer/tags/riscv-for-master-4.0-rc3-v2' into staging (2019-04-05 04:50:30 +0100)
>
> are available in the Git repository at:
>
>   git://github.com/dagrh/qemu.git tags/pull-migration-20190405a
>
> for you to fetch changes up to d013283a46e1f197594e76ed320f867067ab80b3:
>
>   migration: Fix migrate_set_parameter (2019-04-05 15:32:13 +0100)
>
> ----------------------------------------------------------------
> Migration fixes pull for 4.0
>
> A couple of fixes for crashes in colo and
> migration parameters.
>

Applied, thanks.

Please update the changelog at https://wiki.qemu.org/ChangeLog/4.0
for any user-visible changes.

-- PMM

Re: [Qemu-devel] [PULL 0/2] migration queue

[Peter Maydell]

On Fri, 5 Apr 2019 at 16:51, Dr. David Alan Gilbert (git)
<dgilbert@redhat.com> wrote:
>
> From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
>
> The following changes since commit 10546e09e174e0bb185b66a4c397aa845efcd36e:
>
>   Merge remote-tracking branch 'remotes/palmer/tags/riscv-for-master-4.0-rc3-v2' into staging (2019-04-05 04:50:30 +0100)
>
> are available in the Git repository at:
>
>   git://github.com/dagrh/qemu.git tags/pull-migration-20190405a
>
> for you to fetch changes up to d013283a46e1f197594e76ed320f867067ab80b3:
>
>   migration: Fix migrate_set_parameter (2019-04-05 15:32:13 +0100)
>
> ----------------------------------------------------------------
> Migration fixes pull for 4.0
>
> A couple of fixes for crashes in colo and
> migration parameters.
>

Applied, thanks.

Please update the changelog at https://wiki.qemu.org/ChangeLog/4.0
for any user-visible changes.

-- PMM