[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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





On 07/16/2015 02:15 AM, Andrew Cooper wrote:
On 15/07/15 10:18, Yang Hongyang wrote:
From: Wen Congyang <wency@xxxxxxxxxxxxxx>

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@xxxxxxxxxxxxxx>
Signed-off-by: Yang Hongyang <yanghy@xxxxxxxxxxxxxx>
CC: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
---
  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

the existing restore_results seems unchanged, still unsigned long?
see tools/libxl/libxl_save_helper.c helper_stub_restore_results


~Andrew
.


--
Thanks,
Yang.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.