mirror of
https://github.com/xen-project/xen.git
synced 2026-10-28 02:01:24 +01:00
libxc/save: support COLO save
After suspend primary vm, get dirty bitmap on secondary vm, and send pages both dirty on primary/secondary to secondary. Signed-off-by: Yang Hongyang <hongyang.yang@easystack.cn> Signed-off-by: Wen Congyang <wency@cn.fujitsu.com> Signed-off-by: Changlong Xie <xiecl.fnst@cn.fujitsu.com> CC: Andrew Cooper <andrew.cooper3@citrix.com> Acked-by: Ian Jackson <ian.jackson@eu.citrix.com> Signed-off-by: Ian Jackson <ian.jackson@eu.citrix.com>
This commit is contained in:
1 parent
f8c8727d25
commit
73fde4e75a
2 files changed
+95
-2
No files matched your search
@@ -184,6 +184,8 @@ struct xc_sr_context
|
||||
{
|
||||
struct /* Save data. */
|
||||
{
|
||||
int recv_fd;
|
||||
|
||||
struct xc_sr_save_ops ops;
|
||||
struct save_callbacks *callbacks;
|
||||
|
||||
|
||||
@@ -518,6 +518,58 @@ static int send_memory_live(struct xc_sr_context *ctx)
|
||||
return rc;
|
||||
}
|
||||
|
||||
static int colo_merge_secondary_dirty_bitmap(struct xc_sr_context *ctx)
|
||||
{
|
||||
xc_interface *xch = ctx->xch;
|
||||
struct xc_sr_record rec;
|
||||
uint64_t *pfns = NULL;
|
||||
uint64_t pfn;
|
||||
unsigned count, i;
|
||||
int rc;
|
||||
DECLARE_HYPERCALL_BUFFER_SHADOW(unsigned long, dirty_bitmap,
|
||||
&ctx->save.dirty_bitmap_hbuf);
|
||||
|
||||
rc = read_record(ctx, ctx->save.recv_fd, &rec);
|
||||
if ( rc )
|
||||
goto err;
|
||||
|
||||
if ( rec.type != REC_TYPE_CHECKPOINT_DIRTY_PFN_LIST )
|
||||
{
|
||||
PERROR("Expect dirty bitmap record, but received %u", rec.type );
|
||||
rc = -1;
|
||||
goto err;
|
||||
}
|
||||
|
||||
if ( rec.length % sizeof(*pfns) )
|
||||
{
|
||||
PERROR("Invalid dirty pfn list record length %u", rec.length );
|
||||
rc = -1;
|
||||
goto err;
|
||||
}
|
||||
|
||||
count = rec.length / sizeof(*pfns);
|
||||
pfns = rec.data;
|
||||
|
||||
for ( i = 0; i < count; i++ )
|
||||
{
|
||||
pfn = pfns[i];
|
||||
if (pfn > ctx->save.p2m_size)
|
||||
{
|
||||
PERROR("Invalid pfn 0x%" PRIpfn "", (unsigned long)pfn );
|
||||
rc = -1;
|
||||
goto err;
|
||||
}
|
||||
|
||||
set_bit(pfn, dirty_bitmap);
|
||||
}
|
||||
|
||||
rc = 0;
|
||||
|
||||
err:
|
||||
free(rec.data);
|
||||
return rc;
|
||||
}
|
||||
|
||||
/*
|
||||
* Suspend the domain and send dirty memory.
|
||||
* This is the last iteration of the live migration and the
|
||||
@@ -559,6 +611,16 @@ static int suspend_and_send_dirty(struct xc_sr_context *ctx)
|
||||
|
||||
bitmap_or(dirty_bitmap, ctx->save.deferred_pages, ctx->save.p2m_size);
|
||||
|
||||
if ( !ctx->save.live && ctx->save.checkpointed == XC_MIG_STREAM_COLO )
|
||||
{
|
||||
rc = colo_merge_secondary_dirty_bitmap(ctx);
|
||||
if ( rc )
|
||||
{
|
||||
PERROR("Failed to get secondary vm's dirty pages");
|
||||
goto out;
|
||||
}
|
||||
}
|
||||
|
||||
rc = send_dirty_pages(ctx, stats.dirty_count + ctx->save.nr_deferred_pages);
|
||||
if ( rc )
|
||||
goto out;
|
||||
@@ -792,13 +854,39 @@ static int save(struct xc_sr_context *ctx, uint16_t guest_type)
|
||||
if ( rc )
|
||||
goto err;
|
||||
|
||||
if ( ctx->save.checkpointed == XC_MIG_STREAM_COLO )
|
||||
{
|
||||
rc = ctx->save.callbacks->checkpoint(ctx->save.callbacks->data);
|
||||
if ( !rc )
|
||||
{
|
||||
rc = -1;
|
||||
goto err;
|
||||
}
|
||||
}
|
||||
|
||||
rc = ctx->save.callbacks->postcopy(ctx->save.callbacks->data);
|
||||
if ( rc <= 0 )
|
||||
goto err;
|
||||
|
||||
rc = ctx->save.callbacks->checkpoint(ctx->save.callbacks->data);
|
||||
if ( rc <= 0 )
|
||||
if ( ctx->save.checkpointed == XC_MIG_STREAM_COLO )
|
||||
{
|
||||
rc = ctx->save.callbacks->wait_checkpoint(
|
||||
ctx->save.callbacks->data);
|
||||
if ( rc <= 0 )
|
||||
goto err;
|
||||
}
|
||||
else if ( ctx->save.checkpointed == XC_MIG_STREAM_REMUS )
|
||||
{
|
||||
rc = ctx->save.callbacks->checkpoint(ctx->save.callbacks->data);
|
||||
if ( rc <= 0 )
|
||||
goto err;
|
||||
}
|
||||
else
|
||||
{
|
||||
ERROR("Unknown checkpointed stream");
|
||||
rc = -1;
|
||||
goto err;
|
||||
}
|
||||
}
|
||||
} while ( ctx->save.checkpointed != XC_MIG_STREAM_NONE );
|
||||
|
||||
@@ -844,6 +932,7 @@ int xc_domain_save(xc_interface *xch, int io_fd, uint32_t dom,
|
||||
ctx.save.live = !!(flags & XCFLAGS_LIVE);
|
||||
ctx.save.debug = !!(flags & XCFLAGS_DEBUG);
|
||||
ctx.save.checkpointed = stream_type;
|
||||
ctx.save.recv_fd = recv_fd;
|
||||
|
||||
/* If altering migration_stream update this assert too. */
|
||||
assert(stream_type == XC_MIG_STREAM_NONE ||
|
||||
@@ -864,6 +953,8 @@ int xc_domain_save(xc_interface *xch, int io_fd, uint32_t dom,
|
||||
assert(callbacks->switch_qemu_logdirty);
|
||||
if ( ctx.save.checkpointed )
|
||||
assert(callbacks->checkpoint && callbacks->postcopy);
|
||||
if ( ctx.save.checkpointed == XC_MIG_STREAM_COLO )
|
||||
assert(callbacks->wait_checkpoint);
|
||||
|
||||
DPRINTF("fd %d, dom %u, max_iters %u, max_factor %u, flags %u, hvm %d",
|
||||
io_fd, dom, max_iters, max_factor, flags, hvm);
|
||||
|
||||