Attachment 146739 Details for Bug 222467 – Enable shadow before memory reservation for HVM

[patch] Enable shadow before memory reservation for HVM

p (text/plain), 3.44 KB, created by Herbert Xu on 2007-01-27 05:17:32 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Herbert Xu

Created: 2007-01-27 05:17:32 UTC

Size: 3.44 KB

patch

obsolete

>diff -ur xen-3.0.3_0-src.orig/tools/libxc/xc_hvm_build.c xen-3.0.3_0-src/tools/libxc/xc_hvm_build.c
>--- xen-3.0.3_0-src.orig/tools/libxc/xc_hvm_build.c	2006-10-15 22:22:03.000000000 +1000
>+++ xen-3.0.3_0-src/tools/libxc/xc_hvm_build.c	2007-01-27 15:46:43.000000000 +1100
>@@ -441,18 +441,6 @@
>         goto error_out;
>     }
> 
>-    /* HVM domains must be put into shadow mode at the start of day */
>-    if ( xc_shadow_control(xc_handle, domid, XEN_DOMCTL_SHADOW_OP_ENABLE,
>-                           NULL, 0, NULL, 
>-                           XEN_DOMCTL_SHADOW_ENABLE_REFCOUNT  |
>-                           XEN_DOMCTL_SHADOW_ENABLE_TRANSLATE |
>-                           XEN_DOMCTL_SHADOW_ENABLE_EXTERNAL, 
>-                           NULL) )
>-    {
>-        PERROR("Could not enable shadow paging for domain.\n");
>-        goto error_out;
>-    }        
>-
>     memset(ctxt, 0, sizeof(*ctxt));
> 
>     ctxt->flags = VGCF_HVM_GUEST;
>diff -ur xen-3.0.3_0-src.orig/tools/python/xen/lowlevel/xc/xc.c xen-3.0.3_0-src/tools/python/xen/lowlevel/xc/xc.c
>--- xen-3.0.3_0-src.orig/tools/python/xen/lowlevel/xc/xc.c	2007-01-23 22:51:50.000000000 +1100
>+++ xen-3.0.3_0-src/tools/python/xen/lowlevel/xc/xc.c	2007-01-27 15:40:48.000000000 +1100
>@@ -647,6 +647,31 @@
>     return Py_BuildValue("i", mbarg);
> }
> 
>+static PyObject *pyxc_shadow_enable(PyObject *self,
>+                                    PyObject *args,
>+                                    PyObject *kwds)
>+{
>+    XcObject *xc = (XcObject *)self;
>+
>+    uint32_t dom;
>+
>+    static char *kwd_list[] = { "dom", NULL };
>+
>+    if ( !PyArg_ParseTupleAndKeywords(args, kwds, "i", kwd_list, &dom) )
>+        return NULL;
>+
>+    if ( xc_shadow_control(xc->xc_handle, dom, XEN_DOMCTL_SHADOW_OP_ENABLE,
>+                           NULL, 0, NULL, 
>+                           XEN_DOMCTL_SHADOW_ENABLE_REFCOUNT  |
>+                           XEN_DOMCTL_SHADOW_ENABLE_TRANSLATE |
>+                           XEN_DOMCTL_SHADOW_ENABLE_EXTERNAL, 
>+                           NULL) )
>+        return PyErr_SetFromErrno(xc_error);
>+    
>+    Py_INCREF(zero);
>+    return zero;
>+}
>+
> static PyObject *pyxc_sched_credit_domain_set(XcObject *self,
>                                               PyObject *args,
>                                               PyObject *kwds)
>@@ -1089,6 +1114,13 @@
>       " mb [int, -1]: MB of shadow memory this domain should have.\n\n"
>       "Returns: [int] MB of shadow memory in use by this domain.\n" },
> 
>+    { "shadow_enable", 
>+      (PyCFunction)pyxc_shadow_enable, 
>+      METH_VARARGS | METH_KEYWORDS, "\n"
>+      "Enable shadow pagetable\n"
>+      " dom [int]:   Identifier of domain.\n\n"
>+      "Returns: [int] 0 on success; -1 on error.\n" },
>+
>     { "domain_setmaxmem", 
>       (PyCFunction)pyxc_domain_setmaxmem, 
>       METH_VARARGS, "\n"
>diff -ur xen-3.0.3_0-src.orig/tools/python/xen/xend/XendDomainInfo.py xen-3.0.3_0-src/tools/python/xen/xend/XendDomainInfo.py
>--- xen-3.0.3_0-src.orig/tools/python/xen/xend/XendDomainInfo.py	2007-01-23 22:51:50.000000000 +1100
>+++ xen-3.0.3_0-src/tools/python/xen/xend/XendDomainInfo.py	2007-01-27 15:56:10.000000000 +1100
>@@ -1349,6 +1349,8 @@
>             shadow_cur = xc.shadow_mem_control(self.domid, shadow / 1024)
>             self.info['shadow_memory'] = shadow_cur
> 
>+            xc.shadow_enable(self.domid);
>+
>             # initial memory reservation
>             xc.domain_memory_increase_reservation(self.domid, reservation, 0,
>                                                   0)

Actions: View | Diff

Attachments on bug 222467: 146739 | 146761