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

[Xen-changelog] Fix mapping of shared ring pages into backend drivers.



# HG changeset patch
# User kaf24@xxxxxxxxxxxxxxxxxxxx
# Node ID f71bb61e0500decbe074239f21cca9be2c6e210e
# Parent  55fc0ecc19c3a0dc8838a3b5f0405170246bd702
Fix mapping of shared ring pages into backend drivers.
This should fix observed problems when creating a domain,
causing Xen to print "Could not find PTE entry for
address ...".

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

diff -r 55fc0ecc19c3 -r f71bb61e0500 linux-2.6-xen-sparse/drivers/xen/Makefile
--- a/linux-2.6-xen-sparse/drivers/xen/Makefile Wed Sep 21 14:25:58 2005
+++ b/linux-2.6-xen-sparse/drivers/xen/Makefile Wed Sep 21 16:36:46 2005
@@ -1,4 +1,5 @@
 
+obj-y  += util.o
 
 obj-y  += console/
 obj-y  += evtchn/
diff -r 55fc0ecc19c3 -r f71bb61e0500 
linux-2.6-xen-sparse/drivers/xen/blkback/interface.c
--- a/linux-2.6-xen-sparse/drivers/xen/blkback/interface.c      Wed Sep 21 
14:25:58 2005
+++ b/linux-2.6-xen-sparse/drivers/xen/blkback/interface.c      Wed Sep 21 
16:36:46 2005
@@ -8,6 +8,7 @@
 
 #include "common.h"
 #include <asm-xen/evtchn.h>
+#include <asm-xen/driver_util.h>
 
 static kmem_cache_t *blkif_cachep;
 
@@ -69,12 +70,12 @@
 
     BUG_ON(blkif->remote_evtchn);
 
-    if ( (vma = get_vm_area(PAGE_SIZE, VM_IOREMAP)) == NULL )
+    if ( (vma = prepare_vm_area(PAGE_SIZE)) == NULL )
        return -ENOMEM;
 
     err = map_frontend_page(blkif, (unsigned long)vma->addr, shared_page);
     if (err) {
-        vfree(vma->addr);
+        vunmap(vma->addr);
        return err;
     }
 
@@ -85,7 +86,7 @@
     err = HYPERVISOR_event_channel_op(&op);
     if (err) {
        unmap_frontend_page(blkif);
-       vfree(vma->addr);
+       vunmap(vma->addr);
        return err;
     }
 
@@ -123,7 +124,7 @@
 
     if (blkif->blk_ring.sring) {
        unmap_frontend_page(blkif);
-       vfree(blkif->blk_ring.sring);
+       vunmap(blkif->blk_ring.sring);
        blkif->blk_ring.sring = NULL;
     }
 
diff -r 55fc0ecc19c3 -r f71bb61e0500 
linux-2.6-xen-sparse/drivers/xen/blktap/interface.c
--- a/linux-2.6-xen-sparse/drivers/xen/blktap/interface.c       Wed Sep 21 
14:25:58 2005
+++ b/linux-2.6-xen-sparse/drivers/xen/blktap/interface.c       Wed Sep 21 
16:36:46 2005
@@ -8,6 +8,7 @@
 
 #include "common.h"
 #include <asm-xen/evtchn.h>
+#include <asm-xen/driver_util.h>
 
 static kmem_cache_t *blkif_cachep;
 
@@ -69,12 +70,12 @@
 
     BUG_ON(blkif->remote_evtchn);
 
-    if ( (vma = get_vm_area(PAGE_SIZE, VM_IOREMAP)) == NULL )
+    if ( (vma = prepare_vm_area(PAGE_SIZE)) == NULL )
        return -ENOMEM;
 
     err = map_frontend_page(blkif, (unsigned long)vma->addr, shared_page);
     if (err) {
-        vfree(vma->addr);
+        vunmap(vma->addr);
        return err;
     }
 
@@ -85,7 +86,7 @@
     err = HYPERVISOR_event_channel_op(&op);
     if (err) {
        unmap_frontend_page(blkif);
-       vfree(vma->addr);
+       vunmap(vma->addr);
        return err;
     }
 
@@ -121,7 +122,7 @@
 
     if (blkif->blk_ring.sring) {
        unmap_frontend_page(blkif);
-       vfree(blkif->blk_ring.sring);
+       vunmap(blkif->blk_ring.sring);
        blkif->blk_ring.sring = NULL;
     }
 
diff -r 55fc0ecc19c3 -r f71bb61e0500 
linux-2.6-xen-sparse/drivers/xen/netback/interface.c
--- a/linux-2.6-xen-sparse/drivers/xen/netback/interface.c      Wed Sep 21 
14:25:58 2005
+++ b/linux-2.6-xen-sparse/drivers/xen/netback/interface.c      Wed Sep 21 
16:36:46 2005
@@ -8,6 +8,7 @@
 
 #include "common.h"
 #include <linux/rtnetlink.h>
+#include <asm-xen/driver_util.h>
 
 static void __netif_up(netif_t *netif)
 {
@@ -198,14 +199,14 @@
        evtchn_op_t op = { .cmd = EVTCHNOP_bind_interdomain };
        int err;
 
-       vma = get_vm_area(2*PAGE_SIZE, VM_IOREMAP);
+       vma = prepare_vm_area(2*PAGE_SIZE);
        if (vma == NULL)
                return -ENOMEM;
 
        err = map_frontend_pages(
                netif, (unsigned long)vma->addr, tx_ring_ref, rx_ring_ref);
        if (err) {
-               vfree(vma->addr);
+               vunmap(vma->addr);
                return err;
        }
 
@@ -216,7 +217,7 @@
        err = HYPERVISOR_event_channel_op(&op);
        if (err) {
                unmap_frontend_pages(netif);
-               vfree(vma->addr);
+               vunmap(vma->addr);
                return err;
        }
 
@@ -261,7 +262,7 @@
 
        if (netif->tx) {
                unmap_frontend_pages(netif);
-               vfree(netif->tx); /* Frees netif->rx as well. */
+               vunmap(netif->tx); /* Frees netif->rx as well. */
        }
 
        free_netdev(netif->dev);
diff -r 55fc0ecc19c3 -r f71bb61e0500 
linux-2.6-xen-sparse/drivers/xen/tpmback/interface.c
--- a/linux-2.6-xen-sparse/drivers/xen/tpmback/interface.c      Wed Sep 21 
14:25:58 2005
+++ b/linux-2.6-xen-sparse/drivers/xen/tpmback/interface.c      Wed Sep 21 
16:36:46 2005
@@ -1,4 +1,4 @@
- 
/******************************************************************************
+ /*****************************************************************************
  * drivers/xen/tpmback/interface.c
  *
  * Vritual TPM interface management.
@@ -13,6 +13,7 @@
 
 #include "common.h"
 #include <asm-xen/balloon.h>
+#include <asm-xen/driver_util.h>
 
 #define VMALLOC_VMADDR(x) ((unsigned long)(x))
 
@@ -122,12 +123,12 @@
 
        BUG_ON(tpmif->remote_evtchn);
 
-       if ((vma = get_vm_area(PAGE_SIZE, VM_IOREMAP)) == NULL)
+       if ((vma = prepare_vm_area(PAGE_SIZE)) == NULL)
                return -ENOMEM;
 
        err = map_frontend_page(tpmif, VMALLOC_VMADDR(vma->addr), shared_page);
        if (err) {
-               vfree(vma->addr);
+               vunmap(vma->addr);
                return err;
        }
 
@@ -138,7 +139,7 @@
        err = HYPERVISOR_event_channel_op(&op);
        if (err) {
                unmap_frontend_page(tpmif);
-               vfree(vma->addr);
+               vunmap(vma->addr);
                return err;
        }
 
@@ -174,7 +175,7 @@
 
        if (tpmif->tx) {
                unmap_frontend_page(tpmif);
-               vfree(tpmif->tx);
+               vunmap(tpmif->tx);
        }
 
        free_tpmif(tpmif);
diff -r 55fc0ecc19c3 -r f71bb61e0500 
linux-2.6-xen-sparse/drivers/xen/usbback/interface.c
--- a/linux-2.6-xen-sparse/drivers/xen/usbback/interface.c      Wed Sep 21 
14:25:58 2005
+++ b/linux-2.6-xen-sparse/drivers/xen/usbback/interface.c      Wed Sep 21 
16:36:46 2005
@@ -7,6 +7,7 @@
  */
 
 #include "common.h"
+#include <asm-xen/driver_util.h>
 
 #define USBIF_HASHSZ 1024
 #define USBIF_HASH(_d) (((int)(_d))&(USBIF_HASHSZ-1))
@@ -33,7 +34,7 @@
      * may be outstanding requests at the device whose asynchronous responses
      * must still be notified to the remote driver.
      */
-    vfree(usbif->usb_ring.sring);
+    vunmap(usbif->usb_ring.sring);
 
     /* Construct the deferred response message. */
     cmsg.type         = CMSG_USBIF_BE;
@@ -154,7 +155,7 @@
         return;
     }
 
-    if ( (vma = get_vm_area(PAGE_SIZE, VM_IOREMAP)) == NULL )
+    if ( (vma = prepare_vm_area(PAGE_SIZE)) == NULL )
     {
         connect->status = USBIF_BE_STATUS_OUT_OF_MEMORY;
         return;
@@ -172,14 +173,14 @@
             connect->status = USBIF_BE_STATUS_MAPPING_ERROR;
         else
             connect->status = USBIF_BE_STATUS_ERROR;
-        vfree(vma->addr);
+        vunmap(vma->addr);
         return;
     }
 
     if ( up->status != DISCONNECTED )
     {
         connect->status = USBIF_BE_STATUS_INTERFACE_CONNECTED;
-        vfree(vma->addr);
+        vunmap(vma->addr);
         return;
     }
 
diff -r 55fc0ecc19c3 -r f71bb61e0500 linux-2.6-xen-sparse/drivers/xen/util.c
--- /dev/null   Wed Sep 21 14:25:58 2005
+++ b/linux-2.6-xen-sparse/drivers/xen/util.c   Wed Sep 21 16:36:46 2005
@@ -0,0 +1,46 @@
+#include <linux/config.h>
+#include <linux/mm.h>
+#include <linux/slab.h>
+#include <linux/vmalloc.h>
+#include <asm/uaccess.h>
+
+static int touch_fn(
+       pte_t *pte, struct page *pte_page, unsigned long addr, void *data)
+{
+       char c;
+       BUG_ON(!__get_user(c, (char *)addr));
+       return 0;
+}
+
+struct vm_struct *prepare_vm_area(unsigned long size)
+{
+       struct vm_struct *area;
+
+       area = get_vm_area(size, VM_IOREMAP);
+       if (area == NULL)
+               return NULL;
+
+       /*
+         * This ensures that page tables are constructed for this region
+         * of kernel virtual address space. Furthermore, by touching each
+         * memory page (in touch_fn()) we ensure that the page tables are
+         * mapped into the current mm as well as init_mm.
+         */
+       if (generic_page_range(&init_mm, (unsigned long)area->addr,
+                              area->size, touch_fn, NULL)) {
+               vunmap(area->addr);
+               return NULL;
+       }
+
+       return area;
+}
+
+/*
+ * Local variables:
+ *  c-file-style: "linux"
+ *  indent-tabs-mode: t
+ *  c-indent-level: 8
+ *  c-basic-offset: 8
+ *  tab-width: 8
+ * End:
+ */
diff -r 55fc0ecc19c3 -r f71bb61e0500 
linux-2.6-xen-sparse/include/asm-xen/driver_util.h
--- /dev/null   Wed Sep 21 14:25:58 2005
+++ b/linux-2.6-xen-sparse/include/asm-xen/driver_util.h        Wed Sep 21 
16:36:46 2005
@@ -0,0 +1,10 @@
+
+#ifndef __ASM_XEN_DRIVER_UTIL_H__
+#define __ASM_XEN_DRIVER_UTIL_H__
+
+#include <linux/config.h>
+#include <linux/vmalloc.h>
+
+extern struct vm_struct *prepare_vm_area(unsigned long size);
+
+#endif /* __ASM_XEN_DRIVER_UTIL_H__ */

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


 


Rackspace

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