[Xen-devel] [PATCH RFC v1 72/74] libxl: pvshim: Introduce pvhshim_extra

[Wei Liu <wei.liu2@xxxxxxxxxx>]

From: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>

And move the debugging options from the default config into a doc
comment in libxl_types.idl.

Signed-off-by: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
---
 tools/libxl/libxl.h          | 2 +-
 tools/libxl/libxl_create.c   | 5 ++++-
 tools/libxl/libxl_internal.h | 2 +-
 tools/libxl/libxl_types.idl  | 1 +
 4 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/tools/libxl/libxl.h b/tools/libxl/libxl.h
index 81dfcc80ad..7e40155079 100644
--- a/tools/libxl/libxl.h
+++ b/tools/libxl/libxl.h
@@ -1105,7 +1105,7 @@ void libxl_mac_copy(libxl_ctx *ctx, libxl_mac *dst, const 
libxl_mac *src);
  * LIBXL_HAVE_PV_SHIM
  *
  * If this is defined, libxl_domain_build_info's pv type information
- * contains members pvhshim, pvhshim_path, pvhshim_cmdline.
+ * contains members pvhshim, pvhshim_path, pvhshim_cmdline, pvhshim_extra.
  */
 #define LIBXL_HAVE_PV_SHIM 1
 
diff --git a/tools/libxl/libxl_create.c b/tools/libxl/libxl_create.c
index cd98522b9b..94b2c90c5e 100644
--- a/tools/libxl/libxl_create.c
+++ b/tools/libxl/libxl_create.c
@@ -519,7 +519,10 @@ int libxl__domain_build(libxl__gc *gc,
             if (ret) goto out;
 
             state->shim_path = info->u.pv.pvhshim_path;
-            state->shim_cmdline = info->u.pv.pvhshim_cmdline;
+            state->shim_cmdline = GCSPRINTF("%s%s%s",
+                                            info->u.pv.pvhshim_cmdline,
+                                            info->u.pv.pvhshim_extra ? " " : 
"",
+                                            info->u.pv.pvhshim_extra ? 
info->u.pv.pvhshim_extra : "");
             ret = libxl__build_hvm(gc, domid,
                                    d_config, &shim_info,
                                    state);
diff --git a/tools/libxl/libxl_internal.h b/tools/libxl/libxl_internal.h
index 2897e7c3bb..040d9c6b30 100644
--- a/tools/libxl/libxl_internal.h
+++ b/tools/libxl/libxl_internal.h
@@ -119,7 +119,7 @@
 #define DOMID_XS_PATH "domid"
 #define INVALID_DOMID ~0
 #define PVSHIM_BASENAME "xen-shim"
-#define PVSHIM_CMDLINE "pv-shim console=xen,pv sched=null loglvl=all 
guest_loglvl=all apic_verbosity=debug e820-verbose"
+#define PVSHIM_CMDLINE "pv-shim console=xen,pv sched=null"
 
 /* Size macros. */
 #define __AC(X,Y)   (X##Y)
diff --git a/tools/libxl/libxl_types.idl b/tools/libxl/libxl_types.idl
index a6ebea0178..7ff807acf3 100644
--- a/tools/libxl/libxl_types.idl
+++ b/tools/libxl/libxl_types.idl
@@ -594,6 +594,7 @@ libxl_domain_build_info = Struct("domain_build_info",[
                                       ("pvhshim", libxl_defbool),
                                       ("pvhshim_path", string),
                                       ("pvhshim_cmdline", string),
+                                      ("pvhshim_extra", string), # eg 
"loglvl=all guest_loglvl=all apic_verbosity=debug e820-verbose"
                                       ])),
                  ("pvh", None),
                  ("invalid", None),
-- 
2.11.0


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel