[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PULL 00/38] Migration 20220127 patches
From: |
Juan Quintela |
Subject: |
Re: [PULL 00/38] Migration 20220127 patches |
Date: |
Mon, 31 Jan 2022 10:03:41 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux) |
Peter Maydell <peter.maydell@linaro.org> wrote:
> On Thu, 27 Jan 2022 at 15:06, Juan Quintela <quintela@redhat.com> wrote:
>>
>> The following changes since commit 48302d4eb628ff0bea4d7e92cbf6b726410eb4c3:
>>
>> Merge remote-tracking branch
>> 'remotes/dgilbert-gitlab/tags/pull-virtiofs-20220126' into staging
>> (2022-01-26 10:59:50 +0000)
>>
>> are available in the Git repository at:
>>
>> https://gitlab.com/juan.quintela/qemu.git
>> tags/migration-20220127-pull-request
>>
>> for you to fetch changes up to 4e29a822de430e8d0d8bf5c61aaff0561a713d84:
>>
>> migration: Move temp page setup and cleanup into separate functions
>> (2022-01-27 13:37:55 +0100)
>>
>> ----------------------------------------------------------------
>> Migration Pull request
>>
>> Hi
>>
>> This pull request includes every patch ack'd on the list for migration. In
>> no particular order:
>>
>> - ram_transferred_add() - (Edmondson)
>> - vmstate canary (dgilbert)
>> - minimum_version_id_old removal (peter maydell)
>> - postcopy cleanups (peter xu)
>> - more cleanups (Philippe)
>> - yet more cleanups (zheng)
>> - colo and cleanups (zhang)
>
> Hi; this fails because we hit this assert in the vmstate canary check
> on some of the CI jobs, for instance:
> https://gitlab.com/qemu-project/qemu/-/jobs/2024872142
> and also on the NetBSD VM in tests/vm.
I dropped them on my PULL request, but there is something fishy on
aarch64 for migration. It makes exactly zero sense to fail with this
patch. Or something is missing a VMSTATE_END that only happens on
aarch64.
I will take a look.
Later, Juan.
PD. As Dave said, the patch makes sense to drop because we need a new
libslirp before pushing for it.
- [PULL 30/38] migration: Drop return code for disgard ram process, (continued)
- [PULL 30/38] migration: Drop return code for disgard ram process, Juan Quintela, 2022/01/27
- [PULL 31/38] migration: Don't return for postcopy_send_discard_bm_ram(), Juan Quintela, 2022/01/27
- [PULL 32/38] migration: Introduce ram_transferred_add(), Juan Quintela, 2022/01/27
- [PULL 33/38] migration: Tally pre-copy, downtime and post-copy bytes independently, Juan Quintela, 2022/01/27
- [PULL 34/38] migration: No off-by-one for pss->page update in host page size, Juan Quintela, 2022/01/27
- [PULL 35/38] migration: Enable UFFD_FEATURE_THREAD_ID even without blocktime feat, Juan Quintela, 2022/01/27
- [PULL 36/38] migration: Add postcopy_has_request(), Juan Quintela, 2022/01/27
- [PULL 37/38] migration: Simplify unqueue_page(), Juan Quintela, 2022/01/27
- [PULL 38/38] migration: Move temp page setup and cleanup into separate functions, Juan Quintela, 2022/01/27
- Re: [PULL 00/38] Migration 20220127 patches, Peter Maydell, 2022/01/28
- Re: [PULL 00/38] Migration 20220127 patches,
Juan Quintela <=