Re: [PATCH v8 --for 4.6 COLO 15/25] send store mfn and console mfn to xl before resuming secondary vm

[Andrew Cooper <andrew.cooper3@citrix.com>]

On 15/07/15 10:18, Yang Hongyang wrote:
> From: Wen Congyang <wency@cn.fujitsu.com>
>
> We will call libxl__xc_domain_restore_done() to rebuild secondary vm. But
> we need store mfn and console mfn when rebuilding secondary vm. So make
> restore_results a function pointer in callback struct and struct
> {save,restore}_callbacks, and use this callback to send store mfn and
> console mfn to xl.
>
> Signed-off-by: Wen Congyang <wency@cn.fujitsu.com>
> Signed-off-by: Yang Hongyang <yanghy@cn.fujitsu.com>
> CC: Andrew Cooper <andrew.cooper3@citrix.com>
> ---
>  tools/libxc/include/xenguest.h     | 8 ++++++++
>  tools/libxc/xc_sr_restore.c        | 7 +++++--
>  tools/libxl/libxl_colo_restore.c   | 5 -----
>  tools/libxl/libxl_create.c         | 2 ++
>  tools/libxl/libxl_save_msgs_gen.pl | 2 +-
>  5 files changed, 16 insertions(+), 8 deletions(-)
>
> diff --git a/tools/libxc/include/xenguest.h b/tools/libxc/include/xenguest.h
> index 1e7e1bb..d7bdfb5 100644
> --- a/tools/libxc/include/xenguest.h
> +++ b/tools/libxc/include/xenguest.h
> @@ -140,6 +140,14 @@ struct restore_callbacks {
>       */
>      int (*should_checkpoint)(void* data);
>  
> +    /*
> +     * callback to send store mfn and console mfn to xl
> +     * if we want to resume vm before xc_domain_save()
> +     * exits.
> +     */
> +    void (*restore_results)(unsigned long store_mfn, unsigned long console_mfn,
> +                            void *data);

These need to be xen_pfn_t to be usable in arm.  Also, they are gfn's,
not mfn's

A whole lot of terminology in this area is wrong.  The top of
xen/include/xen/mm.h is the authoritative description of terms, starting
from c/s e758ed1

~Andrew