[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v8 02/11] migration: pass MigrationIncomingState
From: |
Peter Xu |
Subject: |
Re: [Qemu-devel] [PATCH v8 02/11] migration: pass MigrationIncomingState* into migration check functions |
Date: |
Fri, 9 Jun 2017 15:14:55 +0800 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
On Fri, Jun 09, 2017 at 09:21:38AM +0300, Alexey Perevalov wrote:
> On 06/09/2017 07:10 AM, Peter Xu wrote:
> >On Wed, Jun 07, 2017 at 12:46:29PM +0300, Alexey Perevalov wrote:
> >>That tiny refactoring is necessary to be able to set
> >>UFFD_FEATURE_THREAD_ID while requesting features, and then
> >>to create downtime context in case when kernel supports it.
> >>
> >>Signed-off-by: Alexey Perevalov <address@hidden>
> >>---
> >> migration/migration.c | 3 ++-
> >> migration/postcopy-ram.c | 10 +++++-----
> >> migration/postcopy-ram.h | 2 +-
> >> migration/savevm.c | 2 +-
> >> 4 files changed, 9 insertions(+), 8 deletions(-)
> >>
> >>diff --git a/migration/migration.c b/migration/migration.c
> >>index 48c94c9..2a77636 100644
> >>--- a/migration/migration.c
> >>+++ b/migration/migration.c
> >>@@ -726,6 +726,7 @@ void
> >>qmp_migrate_set_capabilities(MigrationCapabilityStatusList *params,
> >> Error **errp)
> >> {
> >> MigrationState *s = migrate_get_current();
> >>+ MigrationIncomingState *mis = migration_incoming_get_current();
> >If this patch is only servicing patch 6, I'd prefer in patch 6 we call
> >migration_incoming_get_current() (rather than here), then this patch
> >may be dropped?...
> I planed this patch as preparation, I used to separate refactoring from main
> change, for
> easy merging while rebasing.
> mis - is necessary here to have the same behaviour as before.
Could I ask what's the "same behavior" you mentioned?
I thought this patch is only used by patch 6 when creating the
blocktime struct (but not really a clean-up), no?
--
Peter Xu
- [Qemu-devel] [PATCH v8 08/11] migration: postcopy_place_page factoring out, (continued)
- Message not available
Message not available
Message not available
Message not available
Re: [Qemu-devel] [PATCH v8 07/11] migration: add bitmap for copied page, Alexey Perevalov, 2017/06/07
Re: [Qemu-devel] [PATCH v8 07/11] migration: add bitmap for copied page, Peter Xu, 2017/06/09
Re: [Qemu-devel] [PATCH v8 07/11] migration: add bitmap for copied page, Alexey Perevalov, 2017/06/09
Re: [Qemu-devel] [PATCH v8 07/11] migration: add bitmap for copied page, Dr. David Alan Gilbert, 2017/06/12
Re: [Qemu-devel] [PATCH v8 07/11] migration: add bitmap for copied page, Peter Xu, 2017/06/13
Re: [Qemu-devel] [PATCH v8 07/11] migration: add bitmap for copied page, Alexey Perevalov, 2017/06/13