From: Yang Hongyang <yanghy@cn.fujitsu.com>
To: Ian Jackson <Ian.Jackson@eu.citrix.com>
Cc: wei.liu2@citrix.com, ian.campbell@citrix.com,
wency@cn.fujitsu.com, andrew.cooper3@citrix.com,
yunhong.jiang@intel.com, eddie.dong@intel.com,
xen-devel@lists.xen.org, guijianfeng@cn.fujitsu.com,
rshriram@cs.ubc.ca
Subject: Re: [PATCH v2 COLOPre 09/13] tools/libxl: Update libxl_save_msgs_gen.pl to support return data from xl to xc
Date: Tue, 16 Jun 2015 22:19:03 +0800 [thread overview]
Message-ID: <55803057.1060801@cn.fujitsu.com> (raw)
In-Reply-To: <21888.754.417540.230463@mariner.uk.xensource.com>
Hi Ian,
On 06/16/2015 07:05 PM, Ian Jackson wrote:
> Yang Hongyang writes ("[Xen-devel] [PATCH v2 COLOPre 09/13] tools/libxl: Update libxl_save_msgs_gen.pl to support return data from xl to xc"):
>> From: Wen Congyang <wency@cn.fujitsu.com>
>>
>> Currently, all callbacks return an integer value or void. We cannot
>> return some data to xc via callback. Update libxl_save_msgs_gen.pl
>> to support this case.
>
> Thanks for this. I would have some comments on the details, but first
> I want to properly understand your use case. So while I'm the author
> and maintainer of this save helper, I won't review this in detail just
> yet. I'm following the thread about what this is for...
We need to send secondary's dirty page pfn back to primary. Primary will
then send pages that are both dirtied on primary/secondary to secondary.
in this way the secondary's memory will be consistent with primary.
As we disscussed in [PATCH v2 COLOPre 04/13] tools/libxc: export xc_bitops.h
If we move this operation to libxc layer, this patch could be dropped.
>
> Thanks,
> Ian.
> .
>
--
Thanks,
Yang.
next prev parent reply other threads:[~2015-06-16 14:19 UTC|newest]
Thread overview: 106+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-06-08 3:43 [PATCH v2 COLOPre 00/13] Prerequisite patches for COLO Yang Hongyang
2015-06-08 3:43 ` [PATCH v2 COLOPre 01/13] libxc/restore: fix error handle of process_record Yang Hongyang
2015-06-08 9:24 ` Andrew Cooper
2015-06-08 9:37 ` Yang Hongyang
2015-06-08 9:39 ` Andrew Cooper
2015-06-10 14:55 ` Ian Campbell
2015-06-11 2:10 ` Yang Hongyang
2015-06-08 3:43 ` [PATCH v2 COLOPre 02/13] tools/libxc: support to resume uncooperative HVM guests Yang Hongyang
2015-06-10 15:18 ` Ian Campbell
2015-06-11 2:42 ` Wen Congyang
2015-06-11 8:44 ` Ian Campbell
2015-06-11 8:56 ` Wen Congyang
2015-06-11 9:41 ` Ian Campbell
2015-06-08 3:43 ` [PATCH v2 COLOPre 03/13] libxc/restore: zero ioreq page only one time Yang Hongyang
2015-06-08 9:46 ` Andrew Cooper
2015-06-08 9:49 ` Andrew Cooper
2015-06-08 9:58 ` Yang Hongyang
2015-06-08 10:15 ` Andrew Cooper
2015-06-09 0:59 ` Yang Hongyang
2015-06-09 7:30 ` Andrew Cooper
2015-06-10 5:26 ` Yang Hongyang
2015-06-10 7:44 ` Andrew Cooper
2015-06-10 9:06 ` Wen Congyang
2015-06-10 10:08 ` Andrew Cooper
2015-06-10 10:35 ` Paul Durrant
2015-06-10 10:40 ` Paul Durrant
2015-06-10 10:54 ` Wen Congyang
2015-06-10 10:58 ` Paul Durrant
2015-06-10 11:37 ` Wen Congyang
2015-06-10 11:47 ` Paul Durrant
2015-06-11 1:13 ` Wen Congyang
2015-06-11 8:32 ` Paul Durrant
2015-06-11 8:48 ` Wen Congyang
2015-06-11 10:20 ` Paul Durrant
2015-06-11 11:14 ` Wen Congyang
2015-06-11 12:54 ` Yang Hongyang
2015-06-12 3:39 ` Yang Hongyang
2015-06-11 12:58 ` Yang Hongyang
2015-06-11 13:25 ` Paul Durrant
2015-06-12 3:22 ` Wen Congyang
2015-06-12 7:41 ` Paul Durrant
2015-06-12 10:26 ` Wen Congyang
2015-06-12 10:54 ` Paul Durrant
2015-06-12 11:09 ` Wen Congyang
2015-06-12 11:48 ` Paul Durrant
2015-06-12 15:04 ` Wen Congyang
2015-06-12 15:31 ` Paul Durrant
2015-06-13 5:58 ` Wen Congyang
2015-06-08 3:43 ` [PATCH v2 COLOPre 04/13] tools/libxc: export xc_bitops.h Yang Hongyang
2015-06-08 10:04 ` Yang Hongyang
2015-06-10 15:20 ` Ian Campbell
2015-06-11 2:07 ` Yang Hongyang
2015-06-11 8:41 ` Ian Campbell
2015-06-11 10:45 ` Andrew Cooper
2015-06-11 10:55 ` Ian Campbell
2015-06-15 1:50 ` Yang Hongyang
2015-06-08 3:43 ` [PATCH v2 COLOPre 05/13] tools/libxl: introduce a new API libxl__domain_restore() to load qemu state Yang Hongyang
2015-06-10 15:35 ` Ian Campbell
2015-06-11 2:09 ` Yang Hongyang
2015-06-11 8:43 ` Ian Campbell
2015-06-11 8:55 ` Yang Hongyang
2015-06-11 9:41 ` Ian Campbell
2015-06-08 3:43 ` [PATCH v2 COLOPre 06/13] tools/libxl: Introduce a new internal API libxl__domain_unpause() Yang Hongyang
2015-06-10 15:37 ` Ian Campbell
2015-06-11 2:21 ` Yang Hongyang
2015-06-11 8:43 ` Ian Campbell
2015-06-11 9:09 ` Wen Congyang
2015-06-11 9:42 ` Ian Campbell
2015-06-11 9:48 ` Wen Congyang
2015-06-12 11:23 ` Ian Jackson
2015-06-08 3:43 ` [PATCH v2 COLOPre 07/13] tools/libxl: Update libxl__domain_unpause() to support qemu-xen Yang Hongyang
2015-06-12 12:33 ` Wei Liu
2015-06-15 1:29 ` Yang Hongyang
2015-06-15 16:22 ` Wei Liu
2015-06-17 9:02 ` Yang Hongyang
2015-06-08 3:43 ` [PATCH v2 COLOPre 08/13] tools/libxl: introduce libxl__domain_common_switch_qemu_logdirty() Yang Hongyang
2015-06-16 10:45 ` Ian Campbell
2015-06-08 3:43 ` [PATCH v2 COLOPre 09/13] tools/libxl: Update libxl_save_msgs_gen.pl to support return data from xl to xc Yang Hongyang
2015-06-16 10:49 ` Ian Campbell
2015-06-16 10:54 ` Wen Congyang
2015-06-16 10:56 ` Ian Jackson
2015-06-16 11:01 ` Ian Jackson
2015-06-16 11:05 ` Ian Jackson
2015-06-16 14:19 ` Yang Hongyang [this message]
2015-06-08 3:43 ` [PATCH v2 COLOPre 10/13] tools/libxl: Add back channel to allow migration target send data back Yang Hongyang
2015-06-12 12:54 ` Wei Liu
2015-06-12 15:04 ` Ian Jackson
2015-06-15 1:38 ` Yang Hongyang
2015-06-16 10:52 ` Ian Campbell
2015-06-16 10:58 ` Ian Jackson
2015-06-15 1:33 ` Yang Hongyang
2015-06-08 3:43 ` [PATCH v2 COLOPre 11/13] tools/libxl: rename remus device to checkpoint device Yang Hongyang
2015-06-12 13:30 ` Wei Liu
2015-06-12 13:35 ` Wei Liu
2015-06-12 14:57 ` Ian Jackson
2015-06-15 1:45 ` Yang Hongyang
2015-06-15 16:24 ` Wei Liu
2015-06-16 10:53 ` Ian Campbell
2015-06-25 5:00 ` Yang Hongyang
2015-06-25 9:09 ` Wei Liu
2015-06-25 9:16 ` Yang Hongyang
2015-06-08 3:43 ` [PATCH v2 COLOPre 12/13] tools/libxl: adjust the indentation Yang Hongyang
2015-06-16 10:53 ` Ian Campbell
2015-06-08 3:43 ` [PATCH v2 COLOPre 13/13] tools/libxl: don't touch remus in checkpoint_device Yang Hongyang
2015-06-12 13:28 ` Wei Liu
2015-06-15 1:46 ` Yang Hongyang
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=55803057.1060801@cn.fujitsu.com \
--to=yanghy@cn.fujitsu.com \
--cc=Ian.Jackson@eu.citrix.com \
--cc=andrew.cooper3@citrix.com \
--cc=eddie.dong@intel.com \
--cc=guijianfeng@cn.fujitsu.com \
--cc=ian.campbell@citrix.com \
--cc=rshriram@cs.ubc.ca \
--cc=wei.liu2@citrix.com \
--cc=wency@cn.fujitsu.com \
--cc=xen-devel@lists.xen.org \
--cc=yunhong.jiang@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.