[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PULL 09/11] migration: file URI
From: |
Juan Quintela |
Subject: |
Re: [PULL 09/11] migration: file URI |
Date: |
Wed, 04 Oct 2023 15:47:06 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.3 (gnu/linux) |
Fabiano Rosas <farosas@suse.de> wrote:
> Juan Quintela <quintela@redhat.com> writes:
>
>> From: Steve Sistare <steven.sistare@oracle.com>
>>
>> Extend the migration URI to support file:<filename>. This can be used for
>> any migration scenario that does not require a reverse path. It can be
>> used as an alternative to 'exec:cat > file' in minimized containers that
>> do not contain /bin/sh, and it is easier to use than the fd:<fdname> URI.
>> It can be used in HMP commands, and as a qemu command-line parameter.
>>
>> For best performance, guest ram should be shared and x-ignore-shared
>> should be true, so guest pages are not written to the file, in which case
>> the guest may remain running. If ram is not so configured, then the user
>> is advised to stop the guest first. Otherwise, a busy guest may re-dirty
>> the same page, causing it to be appended to the file multiple times,
>> and the file may grow unboundedly. That issue is being addressed in the
>> "fixed-ram" patch series.
>>
>> Signed-off-by: Steve Sistare <steven.sistare@oracle.com>
>> Tested-by: Michael Galaxy <mgalaxy@akamai.com>
>> Reviewed-by: Michael Galaxy <mgalaxy@akamai.com>
>> Reviewed-by: Fabiano Rosas <farosas@suse.de>
>> Reviewed-by: Peter Xu <peterx@redhat.com>
>> Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
>> Reviewed-by: Juan Quintela <quintela@redhat.com>
>> Signed-off-by: Juan Quintela <quintela@redhat.com>
>> Message-ID: <1694182931-61390-2-git-send-email-steven.sistare@oracle.com>
>
> Juan, FYI we have tests for this feature already reviewed and ready to
> merge:
>
> [PATCH v5 0/6] migration: Test the new "file:" migration
> https://lore.kernel.org/r/20230712190742.22294-1-farosas@suse.de
I searched for URI and didn't found it.
Will got it on next PULL request.
Thanks, Juan.
- [PULL 05/11] migration/rdma: zore out head.repeat to make the error more clear, (continued)
- [PULL 05/11] migration/rdma: zore out head.repeat to make the error more clear, Juan Quintela, 2023/10/04
- [PULL 04/11] migration: Add co-maintainers for migration, Juan Quintela, 2023/10/04
- [PULL 02/11] migration: Update error description outside migration.c, Juan Quintela, 2023/10/04
- [PULL 01/11] migration/vmstate: Introduce vmstate_save_state_with_err, Juan Quintela, 2023/10/04
- [PULL 11/11] migration: Unify and trace vmstate field_exists() checks, Juan Quintela, 2023/10/04
- [PULL 03/11] MAINTAINERS: Add entry for rdma migration, Juan Quintela, 2023/10/04
- [PULL 06/11] i386/a-b-bootblock: factor test memory addresses out into constants, Juan Quintela, 2023/10/04
- [PULL 08/11] s390x/a-b-bios: zero the first byte of each page on start, Juan Quintela, 2023/10/04
- [PULL 09/11] migration: file URI, Juan Quintela, 2023/10/04
- [PULL 07/11] i386/a-b-bootblock: zero the first byte of each page on start, Juan Quintela, 2023/10/04
- [PULL 10/11] migration: file URI offset, Juan Quintela, 2023/10/04
- Re: [PULL 00/11] Migration 20231004 patches, Stefan Hajnoczi, 2023/10/05