[Xen-changelog] Clean up xenstore/console shared page initialisation, which is

[Xen patchbot -unstable <patchbot-unstable@xxxxxxxxxxxxxxxxxxx>]

# HG changeset patch
# User kaf24@xxxxxxxxxxxxxxxxxxxx
# Node ID 20bd6f55b813aa145fff73891fcdd9a1c4db9b8c
# Parent  8ee7df2c18d14d254c940ac5bd514a59295f869c
Clean up xenstore/console shared page initialisation, which is
now handled solely by the domain builder.

Signed-off-by: Keir Fraser <keir@xxxxxxxxxxxxx>

diff -r 8ee7df2c18d1 -r 20bd6f55b813 
linux-2.6-xen-sparse/drivers/xen/console/xencons_ring.c
--- a/linux-2.6-xen-sparse/drivers/xen/console/xencons_ring.c   Mon Nov 14 
10:35:50 2005
+++ b/linux-2.6-xen-sparse/drivers/xen/console/xencons_ring.c   Mon Nov 14 
10:36:42 2005
@@ -30,6 +30,12 @@
        return mfn_to_virt(xen_start_info->console_mfn);
 }
 
+static inline void notify_daemon(void)
+{
+       /* Use evtchn: this is called early, before irq is set up. */
+       notify_remote_via_evtchn(xen_start_info->console_evtchn);
+}
+
 int xencons_ring_send(const char *data, unsigned len)
 {
        int sent = 0;
@@ -47,8 +53,7 @@
        wmb();
        intf->out_prod = prod;
 
-       /* Use evtchn: this is called early, before irq is set up. */
-       notify_remote_via_evtchn(xen_start_info->console_evtchn);
+       notify_daemon();
 
        return sent;
 }      
@@ -70,8 +75,10 @@
                                1, regs);
        }
 
-       wmb();
+       mb();
        intf->in_cons = cons;
+
+       notify_daemon();
 
        return IRQ_HANDLED;
 }
@@ -102,6 +109,9 @@
 
        xencons_irq = err;
 
+       /* In case we have in-flight data after save/restore... */
+       notify_daemon();
+
        return 0;
 }
 
diff -r 8ee7df2c18d1 -r 20bd6f55b813 
linux-2.6-xen-sparse/drivers/xen/xenbus/xenbus_comms.c
--- a/linux-2.6-xen-sparse/drivers/xen/xenbus/xenbus_comms.c    Mon Nov 14 
10:35:50 2005
+++ b/linux-2.6-xen-sparse/drivers/xen/xenbus/xenbus_comms.c    Mon Nov 14 
10:36:42 2005
@@ -36,7 +36,7 @@
 #include <asm-xen/xenbus.h>
 #include "xenbus_comms.h"
 
-static int xenbus_irq      = 0;
+static int xenbus_irq;
 
 extern void xenbus_probe(void *); 
 extern int xenstored_ready; 
@@ -51,7 +51,7 @@
 
 static irqreturn_t wake_waiting(int irq, void *unused, struct pt_regs *regs)
 {
-       if(unlikely(xenstored_ready == 0)) {
+       if (unlikely(xenstored_ready == 0)) {
                xenstored_ready = 1; 
                schedule_work(&probe_work); 
        } 
@@ -188,9 +188,6 @@
        }
 
        xenbus_irq = err;
-
-       /* FIXME zero out page -- domain builder should probably do this*/
-       memset(mfn_to_virt(xen_start_info->store_mfn), 0, PAGE_SIZE);
 
        return 0;
 }
diff -r 8ee7df2c18d1 -r 20bd6f55b813 tools/libxc/xc_linux_build.c
--- a/tools/libxc/xc_linux_build.c      Mon Nov 14 10:35:50 2005
+++ b/tools/libxc/xc_linux_build.c      Mon Nov 14 10:36:42 2005
@@ -619,7 +619,9 @@
 
     *store_mfn = page_array[(vstoreinfo_start-dsi.v_start) >> PAGE_SHIFT];
     *console_mfn = page_array[(vconsole_start-dsi.v_start) >> PAGE_SHIFT];
-
+    if ( xc_clear_domain_page(xc_handle, dom, *store_mfn) ||
+         xc_clear_domain_page(xc_handle, dom, *console_mfn) )
+        goto error_out;
 
     start_info = xc_map_foreign_range(
         xc_handle, dom, PAGE_SIZE, PROT_READ|PROT_WRITE,
diff -r 8ee7df2c18d1 -r 20bd6f55b813 tools/libxc/xc_linux_restore.c
--- a/tools/libxc/xc_linux_restore.c    Mon Nov 14 10:35:50 2005
+++ b/tools/libxc/xc_linux_restore.c    Mon Nov 14 10:36:42 2005
@@ -11,8 +11,6 @@
 
 #include "xg_private.h"
 #include "xg_save_restore.h"
-
-
 
 /* max mfn of the whole machine */
 static uint32_t max_mfn; 
diff -r 8ee7df2c18d1 -r 20bd6f55b813 tools/libxc/xc_private.c
--- a/tools/libxc/xc_private.c  Mon Nov 14 10:35:50 2005
+++ b/tools/libxc/xc_private.c  Mon Nov 14 10:36:42 2005
@@ -336,6 +336,19 @@
     return 0;
 }
 
+int xc_clear_domain_page(int xc_handle,
+                         uint32_t domid,
+                         unsigned long dst_pfn)
+{
+    void *vaddr = xc_map_foreign_range(
+        xc_handle, domid, PAGE_SIZE, PROT_WRITE, dst_pfn);
+    if ( vaddr == NULL )
+        return -1;
+    memset(vaddr, 0, PAGE_SIZE);
+    munmap(vaddr, PAGE_SIZE);
+    return 0;
+}
+
 unsigned long xc_get_filesz(int fd)
 {
     uint16_t sig;
diff -r 8ee7df2c18d1 -r 20bd6f55b813 tools/libxc/xc_vmx_build.c
--- a/tools/libxc/xc_vmx_build.c        Mon Nov 14 10:35:50 2005
+++ b/tools/libxc/xc_vmx_build.c        Mon Nov 14 10:36:42 2005
@@ -495,6 +495,9 @@
     }
 
     *store_mfn = page_array[(v_end-2) >> PAGE_SHIFT];
+    if ( xc_clear_domain_page(xc_handle, dom, *store_mfn) )
+        goto error_out;
+
     shared_page_frame = (v_end - PAGE_SIZE) >> PAGE_SHIFT;
 
     if ((e820_page = xc_map_foreign_range(
diff -r 8ee7df2c18d1 -r 20bd6f55b813 tools/libxc/xenctrl.h
--- a/tools/libxc/xenctrl.h     Mon Nov 14 10:35:50 2005
+++ b/tools/libxc/xenctrl.h     Mon Nov 14 10:36:42 2005
@@ -424,6 +424,9 @@
 int xc_copy_to_domain_page(int xc_handle, uint32_t domid,
                           unsigned long dst_pfn, void *src_page);
 
+int xc_clear_domain_page(int xc_handle, uint32_t domid,
+                         unsigned long dst_pfn);
+
 int xc_ia64_copy_to_domain_pages(int xc_handle, uint32_t domid,
         void* src_page, unsigned long dst_pfn, int nr_pages);
 

_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-changelog