[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PULL 09/11] migration: file URI
From: |
Fabiano Rosas |
Subject: |
Re: [PULL 09/11] migration: file URI |
Date: |
Wed, 04 Oct 2023 11:04:38 -0300 |
Juan Quintela <quintela@redhat.com> writes:
> 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.
Do you want me to do resend? It's no big deal.
- [PULL 04/11] migration: Add co-maintainers for migration, (continued)
- [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