]> git.openfabrics.org - ~shefty/rdma-win.git/commitdiff
[MTHCA] bugfix: data corruption while registering MR
authorleonidk <leonidk@ad392aa1-c5ef-ae45-8dd8-e69d62a5ef86>
Wed, 20 Sep 2006 11:55:57 +0000 (11:55 +0000)
committerleonidk <leonidk@ad392aa1-c5ef-ae45-8dd8-e69d62a5ef86>
Wed, 20 Sep 2006 11:55:57 +0000 (11:55 +0000)
  If a buffer to be registered overlaps a buffer, already registered, a race can happen between HCA, writing to the previously registered buffer and the probing functions (MmProbeAndLockPages, MmSecureVirtualMemory), used in the algorithm of memory registration.
  To prevent the race we maintain reference counters for the physical pages, being registered, and register every physical page FOR THE WRITE ACCESS only once.

git-svn-id: svn://openib.tc.cornell.edu/gen1@501 ad392aa1-c5ef-ae45-8dd8-e69d62a5ef86

trunk/hw/mthca/kernel/SOURCES
trunk/hw/mthca/kernel/hca_driver.c
trunk/hw/mthca/kernel/mt_memory.c
trunk/hw/mthca/kernel/mt_memory.h
trunk/hw/mthca/kernel/mt_pa_cash.c [new file with mode: 0644]
trunk/hw/mthca/kernel/mt_pa_cash.h [new file with mode: 0644]
trunk/hw/mthca/kernel/mthca_provider.c

index 4c7f63ecedff053c23de6cd4dcb9952a0498ea7d..ff1e0dae4b8e329ff799f5a8dd045262a26c29ed 100644 (file)
@@ -14,6 +14,7 @@ SOURCES= \
        mthca_log.mc            \\r
        mthca_log.rc            \\r
        hca.rc                  \\r
+       mthca_log.c             \\r
                                        \\r
        ..\hca_utils.c          \\r
        ..\mt_utils.c           \\r
@@ -37,6 +38,7 @@ SOURCES= \
        mt_ud_header.c  \\r
        mt_uverbs.c             \\r
        mt_verbs.c              \\r
+       mt_pa_cash.c            \\r
                                        \\r
        mthca_allocator.c       \\r
        mthca_av.c              \\r
@@ -44,7 +46,6 @@ SOURCES= \
        mthca_cmd.c             \\r
        mthca_cq.c              \\r
        mthca_eq.c              \\r
-       mthca_log.c             \\r
        mthca_mad.c             \\r
        mthca_main.c            \\r
        mthca_mcg.c             \\r
index df6ad81d6d38ba9e798a8153286e33113a52a69b..38586cd3ed8eaf0bd5f3dbd272afe7b8bd5647ed 100644 (file)
@@ -53,7 +53,7 @@
 #pragma warning( pop )\r
 #include <iba/ib_ci_ifc.h>\r
 #include "mthca/mthca_vc.h"\r
-\r
+#include "mt_pa_cash.h"\r
 /* from \inc\platform\evntrace.h\r
 #define TRACE_LEVEL_NONE        0   // Tracing is not on\r
 #define TRACE_LEVEL_FATAL       1   // Abnormal exit or termination\r
@@ -196,6 +196,15 @@ DriverEntry(
        }\r
 //     cl_memclr( mlnx_hca_array, MLNX_MAX_HCA * sizeof(ci_interface_t) );\r
 \r
+       /* init pa cash */\r
+       status = pa_cash_init();\r
+       if (status) \r
+       {\r
+               HCA_PRINT( TRACE_LEVEL_ERROR ,HCA_DBG_INIT ,\r
+                       ("pa_cash_init failed.\n"));\r
+               return status;\r
+       }\r
+\r
        /*leo:  init function table */\r
        hca_init_vfptr();\r
        \r
@@ -345,6 +354,7 @@ hca_drv_unload(
 \r
        UNUSED_PARAM( p_driver_obj );\r
 \r
+       pa_cash_release();\r
        ib_uverbs_cleanup();\r
        ib_core_cleanup();\r
        cl_free( g_param_path.Buffer );\r
index a91b5f0862b84e1d2d51a385a4bcfbdb3d0b78ec..755ea961351bbd56c6b047e534224ff7fba96cc0 100644 (file)
@@ -42,6 +42,8 @@
 #include "mt_memory.tmh"
 #endif 
 
+#include "mt_pa_cash.h"
+
 
 /*
 *      Function: map user buffer to kernel and lock it
@@ -343,7 +345,7 @@ static int register_segment(
        IN              u64 size,
        IN              int is_user,
        IN              ib_access_t acc,
-    IN OUT mt_iobuf_t * iobuf_p)
+       OUT mt_iobuf_seg_t **iobuf_seg)
 {
        PMDL mdl_p;
        int rc;
@@ -375,15 +377,15 @@ static int register_segment(
        // make context-dependent things
        if (is_user) {
                ASSERT(KeGetCurrentIrql() < DISPATCH_LEVEL);
-       mode = UserMode;
+               mode = UserMode;
        }
        else {  /* Mapping to kernel virtual address */
                //    MmBuildMdlForNonPagedPool(mdl_p);   // fill MDL ??? - should we do that really ?
-       mode = KernelMode;
+               mode = KernelMode;
        }
 
        __try { /* try */
-       MmProbeAndLockPages( mdl_p, mode, Operation );   /* lock memory */
+               MmProbeAndLockPages( mdl_p, mode, Operation );   /* lock memory */
        } /* try */
                
        __except (EXCEPTION_EXECUTE_HANDLER)    {
@@ -400,7 +402,7 @@ static int register_segment(
        new_iobuf->nr_pages = ADDRESS_AND_SIZE_TO_SPAN_PAGES( va, size );
        new_iobuf->mdl_p = mdl_p;
        new_iobuf->is_user = is_user;
-       InsertTailList( &iobuf_p->seg_que, &new_iobuf->link );
+       *iobuf_seg = new_iobuf;
        return 0;
 
 err_probe:
@@ -411,6 +413,21 @@ err_nomem:
        return rc;
 }
 
+void iobuf_init(
+       IN              u64 va,
+       IN              u64 size,
+       IN              int is_user,
+       IN OUT  mt_iobuf_t *iobuf_p)
+{
+       iobuf_p->va = va;
+       iobuf_p->size= size;
+       iobuf_p->is_user = is_user;
+       InitializeListHead( &iobuf_p->seg_que );
+       iobuf_p->seg_num = 0;
+       iobuf_p->nr_pages = 0;
+       iobuf_p->is_cashed = 0;
+}
+
 int iobuf_register(
        IN              u64 va,
        IN              u64 size,
@@ -423,6 +440,7 @@ int iobuf_register(
        u64 seg_size;   // current segment size
        u64 rdc;                        // remain data counter - what is rest to lock
        u64 delta;                              // he size of the last not full page of the first segment
+       mt_iobuf_seg_t * new_iobuf;
        unsigned page_size = PAGE_SIZE;
 
 // 32 - for any case  
@@ -431,27 +449,27 @@ int iobuf_register(
 
        ASSERT(KeGetCurrentIrql() <= DISPATCH_LEVEL);
 
-       // init IOBUF object
-       InitializeListHead( &iobuf_p->seg_que );
-       iobuf_p->seg_num = 0;
-
-       // Round the seg_va down to a page boundary so that we always get a seg_size
-       // that is an integral number of pages.
-       delta = va & (PAGE_SIZE - 1);
-       seg_va = va - delta;
-       // Since we rounded down the seg_va, we need to round up the rdc and size.
-       seg_size = rdc = size + delta;
+       // we'll try to register all at once.
+       seg_va = va;
+       seg_size = rdc = size;
                
        // allocate segments
        while (rdc > 0) {
                // map a segment
-               rc = register_segment(seg_va, seg_size, is_user, acc, iobuf_p );
+               rc = register_segment(seg_va, seg_size, is_user, acc, &new_iobuf );
 
                // success - move to another segment
                if (!rc) {
                        rdc -= seg_size;
                        seg_va += seg_size;
+                       InsertTailList( &iobuf_p->seg_que, &new_iobuf->link );
                        iobuf_p->seg_num++;
+                       // round the segment size to the next page boundary 
+                       delta = (seg_va + seg_size) & (page_size - 1);
+                       if (delta) {
+                               seg_size -= delta;
+                               seg_size += page_size;
+                       }
                        if (seg_size > rdc)
                                seg_size = rdc;
                        continue;
@@ -464,14 +482,14 @@ int iobuf_register(
                                break;
                        // lessen the size
                        seg_size >>= 1;
-                       // round the segment size to the page boundary (only for the first segment)
-                       if (iobuf_p->seg_num == 0) {
-                               delta = (seg_va + seg_size) & (page_size - 1);
+                       // round the segment size to the next page boundary 
+                       delta = (seg_va + seg_size) & (page_size - 1);
+                       if (delta) {
                                seg_size -= delta;
                                seg_size += page_size;
-                               if (seg_size > rdc)
-                                       seg_size = rdc;
                        }
+                       if (seg_size > rdc)
+                               seg_size = rdc;
                        continue;
                }
 
@@ -482,18 +500,157 @@ int iobuf_register(
        // SUCCESS
        if (rc) 
                iobuf_deregister( iobuf_p );
-       else     {
-               // fill IOBUF object
-               iobuf_p->va = va;
-               iobuf_p->size= size;
-               iobuf_p->nr_pages = ADDRESS_AND_SIZE_TO_SPAN_PAGES( va, size );
-               iobuf_p->is_user = is_user;
-       }
+       else     
+               iobuf_p->nr_pages += ADDRESS_AND_SIZE_TO_SPAN_PAGES( va, size );
 
        return rc;
 }
 
 
+static void __iobuf_copy(
+       IN OUT  mt_iobuf_t *dst_iobuf_p,
+       IN              mt_iobuf_t *src_iobuf_p
+       )
+{
+       int i;
+       mt_iobuf_seg_t *iobuf_seg_p;
+       
+       *dst_iobuf_p = *src_iobuf_p;
+       InitializeListHead( &dst_iobuf_p->seg_que );
+       for (i=0; i<src_iobuf_p->seg_num; ++i) {
+               iobuf_seg_p = (mt_iobuf_seg_t *)(PVOID)RemoveHeadList( &src_iobuf_p->seg_que );
+               InsertTailList( &dst_iobuf_p->seg_que, &iobuf_seg_p->link );
+       }
+}
+
+/* if the buffer to be registered overlaps a buffer, already registered, 
+       a race can happen between HCA, writing to the previously registered
+       buffer and the probing functions (MmProbeAndLockPages, MmSecureVirtualMemory),
+       used in the algorithm of memory registration.
+       To prevent the race we maintain reference counters for the physical pages, being registered, 
+       and register every physical page FOR THE WRITE ACCESS only once.*/
+
+int iobuf_register_with_cash(
+       IN              u64 vaddr,
+       IN              u64 size,
+       IN              int is_user,
+       IN OUT  ib_access_t *acc_p,
+       IN OUT  mt_iobuf_t *iobuf_p)
+{
+       int rc, pa_in;
+       mt_iobuf_t sec_iobuf;
+       int i, page_in , page_out, page_in_total;
+       int nr_pages;
+       char *subregion_start, *va;
+       u64 subregion_size;
+       u64 rdc;                                        // remain data counter - what is rest to lock
+       u64 delta;                              // he size of the last not full page of the first segment
+       ib_access_t acc;
+
+       down(&g_pa_mutex);
+
+       // register memory for read access to bring pages into the memory
+       rc = iobuf_register( vaddr, size, is_user, 0, iobuf_p);
+
+       // on error or read access - exit
+       if (rc || !(*acc_p & IB_AC_LOCAL_WRITE))
+               goto exit;
+
+       // re-register buffer with the correct access rights
+       iobuf_init( (u64)vaddr, size, is_user, &sec_iobuf );
+       nr_pages = ADDRESS_AND_SIZE_TO_SPAN_PAGES( vaddr, size );
+       subregion_start = va = (char*)(ULONG_PTR)vaddr;
+       rdc = size;
+       pa_in = page_in = page_in_total = page_out = 0;
+
+       for (i=0; i<nr_pages; ++i, va+=PAGE_SIZE) {
+               // check whether a phys page is to be registered
+               PHYSICAL_ADDRESS pa = MmGetPhysicalAddress(va);
+               pa_in = pa_is_registered(pa.QuadPart);
+               if (pa_in) {
+                       ++page_in;
+                       ++page_in_total;
+               }
+               else
+                       ++page_out;
+
+               // check whether we get at the end of a subregion with the same rights wrt cash
+               if (page_in && page_out) {
+                       // prepare to registration of the subregion
+                       if (pa_in) {            // SUBREGION WITH WRITE ACCESS
+                               acc = IB_AC_LOCAL_WRITE;
+                               subregion_size = (u64)page_out * PAGE_SIZE;
+                               page_out = 0;
+                       }
+                       else {          // SUBREGION WITH READ ACCESS
+                               acc = 0;
+                               subregion_size = (u64)page_in * PAGE_SIZE;
+                               page_in = 0;
+                       }
+                       
+                       // round the subregion size to the page boundary 
+                       delta = (u64)(subregion_start + subregion_size) & (PAGE_SIZE - 1);
+                       subregion_size -= delta;
+                       if (subregion_size > rdc)
+                               subregion_size = rdc;
+
+                       // register the subregion
+                       rc = iobuf_register( (u64)subregion_start, subregion_size, is_user, acc, &sec_iobuf);
+                       if (rc)
+                               goto cleanup;
+
+                       // prepare to the next loop
+                       rdc -= subregion_size;
+                       subregion_start +=subregion_size;
+               }
+       }
+
+       // prepare to registration of the subregion
+       if (pa_in) {            // SUBREGION WITH READ ACCESS
+               acc = 0;
+               subregion_size = (u64)page_in * PAGE_SIZE;
+       }
+       else {          // SUBREGION WITH WRITE ACCESS
+               acc = IB_AC_LOCAL_WRITE;
+               subregion_size = (u64)page_out * PAGE_SIZE;
+       }
+       
+       // round the subregion size to the page boundary 
+       delta = (u64)(subregion_start + subregion_size) & (PAGE_SIZE - 1);
+       subregion_size -= delta;
+       if (subregion_size > rdc)
+               subregion_size = rdc;
+       
+       // register the subregion
+       rc = iobuf_register( (u64)subregion_start, subregion_size, is_user, acc, &sec_iobuf);
+       if (rc)
+               goto cleanup;
+
+       // cash phys pages
+       rc = pa_register(iobuf_p);
+       if (rc)
+               goto err_pa_reg;
+
+       // replace the iobuf
+       iobuf_deregister( iobuf_p );
+       sec_iobuf.is_cashed = TRUE;
+       __iobuf_copy( iobuf_p, &sec_iobuf );
+       
+       // buffer is a part of also registered buffer - change the rights 
+       if (page_in_total)
+               *acc_p = MTHCA_ACCESS_REMOTE_READ;
+
+       goto exit;
+       
+err_pa_reg:    
+       iobuf_deregister( &sec_iobuf );
+cleanup:
+       iobuf_deregister( iobuf_p );
+exit:  
+       up(&g_pa_mutex);
+       return rc;
+}
+
 static void deregister_segment(mt_iobuf_seg_t * iobuf_seg_p)
 {
        MmUnlockPages( iobuf_seg_p->mdl_p );    // unlock the buffer 
@@ -516,6 +673,17 @@ void iobuf_deregister(mt_iobuf_t *iobuf_p)
        ASSERT(iobuf_p->seg_num == 0);
 }
 
+void iobuf_deregister_with_cash(mt_iobuf_t *iobuf_p)
+{
+       ASSERT(KeGetCurrentIrql() < DISPATCH_LEVEL);
+
+       down(&g_pa_mutex);
+       if (iobuf_p->is_cashed)
+               pa_deregister(iobuf_p);
+        iobuf_deregister(iobuf_p);
+       up(&g_pa_mutex);
+}
+
 void iobuf_iter_init(
        IN              mt_iobuf_t *iobuf_p, 
        IN OUT  mt_iobuf_iter_t *iterator_p)
index 0c24911fc2e126db0b9b914c0628b7435b398629..98b0d545af333a67df1f2cd3420309d77f6aec8f 100644 (file)
@@ -258,6 +258,7 @@ typedef struct _mt_iobuf {
        u32 nr_pages;
        int is_user;
        int seg_num;
+       int is_cashed;
 } mt_iobuf_t;
 
 /* iterator for getting segments of tpt */
@@ -266,7 +267,22 @@ typedef struct _mt_iobuf_iter {
        unsigned int pfn_ix; /* index from where to take the next translation */
 } mt_iobuf_iter_t;
 
-void iobuf_deregister(mt_iobuf_t *iobuf_p);
+void iobuf_deregister_with_cash(IN mt_iobuf_t *iobuf_p);
+
+void iobuf_deregister(IN mt_iobuf_t *iobuf_p);
+
+void iobuf_init(
+       IN              u64 va,
+       IN              u64 size,
+       IN              int is_user,
+       IN OUT  mt_iobuf_t *iobuf_p);
+
+int iobuf_register_with_cash(
+       IN              u64 vaddr,
+       IN              u64 size,
+       IN              int is_user,
+       IN OUT  ib_access_t *acc_p,
+       IN OUT  mt_iobuf_t *iobuf_p);
 
 int iobuf_register(
        IN              u64 va,
diff --git a/trunk/hw/mthca/kernel/mt_pa_cash.c b/trunk/hw/mthca/kernel/mt_pa_cash.c
new file mode 100644 (file)
index 0000000..0a86cf6
--- /dev/null
@@ -0,0 +1,210 @@
+/*
+ * Copyright (c) 2005 Topspin Communications.  All rights reserved.
+ * Copyright (c) 2005 Mellanox Technologies Ltd.  All rights reserved.
+ *
+ * This software is available to you under a choice of one of two
+ * licenses.  You may choose to be licensed under the terms of the GNU
+ * General Public License (GPL) Version 2, available from the file
+ * COPYING in the main directory of this source tree, or the
+ * OpenIB.org BSD license below:
+ *
+ *     Redistribution and use in source and binary forms, with or
+ *     without modification, are permitted provided that the following
+ *     conditions are met:
+ *
+ *      - Redistributions of source code must retain the above
+ *        copyright notice, this list of conditions and the following
+ *        disclaimer.
+ *
+ *      - Redistributions in binary form must reproduce the above
+ *        copyright notice, this list of conditions and the following
+ *        disclaimer in the documentation and/or other materials
+ *        provided with the distribution.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+ * EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+ * MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+ * NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS
+ * BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN
+ * ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
+ * CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+ * SOFTWARE.
+ *
+ * $Id: mlnx_uvp_cq.c 1611 2006-08-20 14:48:55Z sleybo $
+ */
+
+#include "mt_pa_cash.h"
+
+#ifdef _WIN64
+#define MAX_PAGES_SUPPORTED    (64 * 1024 * 1024)
+#else
+#define MAX_PAGES_SUPPORTED    (16 * 1024 * 1024)
+#endif
+
+typedef struct {
+       int     ref_cnt;
+} pa_table_entry_t;
+
+#define PA_TABLE_ENTRY_SIZE    sizeof(pa_table_entry_t)
+#define PA_TABLE_ENTRY_NUM     (PAGE_SIZE / PA_TABLE_ENTRY_SIZE)
+#define PA_TABLE_SIZE                  (PA_TABLE_ENTRY_SIZE * PA_TABLE_ENTRY_NUM)
+
+#define PA_DIR_ENTRY_SIZE              sizeof(void*)
+#define PA_DIR_ENTRY_NUM               (MAX_PAGES_SUPPORTED /PA_TABLE_ENTRY_NUM)
+#define PA_DIR_SIZE                            (PA_DIR_ENTRY_SIZE * PA_DIR_ENTRY_NUM)
+
+struct pa_cash_s {
+       pa_table_entry_t **pa_dir;
+       uint32_t max_nr_pages;
+} g_cash = { NULL, 0 };
+
+KMUTEX g_pa_mutex;
+u64 g_pa[1024];
+
+int pa_cash_init()
+{
+       void *pa_dir;
+       pa_dir = kzalloc(PA_DIR_SIZE, GFP_KERNEL);
+
+       if (!pa_dir)
+               return -ENOMEM;
+       g_cash.pa_dir = pa_dir;
+       g_cash.max_nr_pages = PA_TABLE_ENTRY_NUM * PA_DIR_ENTRY_NUM;
+       KeInitializeMutex(&g_pa_mutex, 0);
+       return 0;
+}
+
+void pa_cash_release()
+{
+       int i;
+       /* free cash tables */
+       for (i=0; i<PA_DIR_ENTRY_NUM; ++i)
+               if (g_cash.pa_dir[i])
+                       kfree(g_cash.pa_dir[i]);
+                       
+       if (g_cash.pa_dir)
+               kfree(g_cash.pa_dir);
+}
+
+int pa_is_registered(uint64_t pa)
+{
+       uint32_t ix = (uint32_t)(pa >> PAGE_SHIFT);
+       pa_table_entry_t *pa_te;
+
+       /* or pa is incorrect or memory that big is not supported */
+       if (ix > g_cash.max_nr_pages) {
+               ASSERT(FALSE);
+               return -EFAULT;
+       }
+               
+       pa_te =  g_cash.pa_dir[ix / PA_TABLE_ENTRY_NUM];
+
+       /* no this page_table */
+       if (!pa_te)
+               return 0;
+
+       return pa_te[ix % PA_TABLE_ENTRY_NUM].ref_cnt;
+}
+
+int __add_pa(uint64_t pa)
+{
+       uint32_t ix = (uint32_t)(pa >> PAGE_SHIFT);
+       pa_table_entry_t *pa_te;
+
+       /* or pa is incorrect or memory that big is not supported */
+       if (ix > g_cash.max_nr_pages) {
+               ASSERT(FALSE);
+               return -EFAULT;
+       }
+               
+       pa_te =  g_cash.pa_dir[ix / PA_TABLE_ENTRY_NUM];
+
+       /* no this page_table - add a new one */
+       if (!pa_te) {
+               pa_te = kzalloc(PA_DIR_SIZE, GFP_KERNEL);
+               if (!pa_te) 
+                       return -ENOMEM;
+               g_cash.pa_dir[ix / PA_TABLE_ENTRY_NUM] = pa_te;
+       }
+
+       /* register page address */
+       ++pa_te[ix % PA_TABLE_ENTRY_NUM].ref_cnt;
+       return 0;
+}
+
+int __rmv_pa(uint64_t pa)
+{
+       uint32_t ix = (uint32_t)(pa >> PAGE_SHIFT);
+       pa_table_entry_t *pa_te;
+
+       /* or pa is incorrect or memory that big is not supported */
+       if (ix > g_cash.max_nr_pages) {
+               ASSERT(FALSE);
+               return -EFAULT;
+       }
+               
+       pa_te =  g_cash.pa_dir[ix / PA_TABLE_ENTRY_NUM];
+
+       /* no this page_table - error*/
+       if (!pa_te)  {
+               ASSERT(FALSE);
+               return -EFAULT;
+       }
+
+       /* deregister page address */
+       --pa_te[ix % PA_TABLE_ENTRY_NUM].ref_cnt;
+       ASSERT(pa_te[ix % PA_TABLE_ENTRY_NUM].ref_cnt >= 0);
+       return 0;
+}
+
+int pa_register(mt_iobuf_t *iobuf_p)
+{
+       int i,j,n;
+       mt_iobuf_iter_t iobuf_iter;
+
+       iobuf_iter_init( iobuf_p, &iobuf_iter );
+       n = 0;
+       for (;;) {
+               i = iobuf_get_tpt_seg( iobuf_p, &iobuf_iter, 
+                       sizeof(g_pa) / sizeof (u64), g_pa );
+               if (!i)
+                       break;
+               for (j=0; j<i; ++j, ++n)
+                       if (__add_pa(g_pa[j]))
+                               goto cleanup;
+       }
+       return 0;
+
+cleanup:       
+       iobuf_iter_init( iobuf_p, &iobuf_iter );
+       for (;;) {
+               i = iobuf_get_tpt_seg( iobuf_p, &iobuf_iter, 
+                       sizeof(g_pa) / sizeof (u64), g_pa );
+               if (!i)
+                       break;
+               for (j=0; j<i; ++j, --n) {
+                       __rmv_pa(g_pa[j]);
+                       if (n<=0)
+                               goto done;
+               }
+       }
+done:  
+       return -ENOMEM;
+}
+
+void pa_deregister(mt_iobuf_t *iobuf_p)
+{
+       int i,j;
+       mt_iobuf_iter_t iobuf_iter;
+
+       iobuf_iter_init( iobuf_p, &iobuf_iter );
+       for (;;) {
+               i = iobuf_get_tpt_seg( iobuf_p, &iobuf_iter, 
+                       sizeof(g_pa) / sizeof (u64), g_pa );
+               if (!i)
+                       break;
+               for (j=0; j<i; ++j)
+                       __rmv_pa(g_pa[j]);
+       }
+}
+
diff --git a/trunk/hw/mthca/kernel/mt_pa_cash.h b/trunk/hw/mthca/kernel/mt_pa_cash.h
new file mode 100644 (file)
index 0000000..ca2c043
--- /dev/null
@@ -0,0 +1,49 @@
+/*
+ * Copyright (c) 2005 Topspin Communications.  All rights reserved.
+ * Copyright (c) 2005 Mellanox Technologies Ltd.  All rights reserved.
+ *
+ * This software is available to you under a choice of one of two
+ * licenses.  You may choose to be licensed under the terms of the GNU
+ * General Public License (GPL) Version 2, available from the file
+ * COPYING in the main directory of this source tree, or the
+ * OpenIB.org BSD license below:
+ *
+ *     Redistribution and use in source and binary forms, with or
+ *     without modification, are permitted provided that the following
+ *     conditions are met:
+ *
+ *      - Redistributions of source code must retain the above
+ *        copyright notice, this list of conditions and the following
+ *        disclaimer.
+ *
+ *      - Redistributions in binary form must reproduce the above
+ *        copyright notice, this list of conditions and the following
+ *        disclaimer in the documentation and/or other materials
+ *        provided with the distribution.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+ * EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+ * MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+ * NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS
+ * BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN
+ * ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
+ * CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+ * SOFTWARE.
+ *
+ * $Id: mlnx_uvp_cq.c 1611 2006-08-20 14:48:55Z sleybo $
+ */
+
+#include "mthca_dev.h"
+
+extern KMUTEX g_pa_mutex;
+int pa_cash_init();
+void pa_cash_release();
+int pa_is_registerable(
+       IN      void* __ptr64   vaddr, 
+       IN      uint64_t length, 
+       IN OUT  mthca_qp_access_t *acc);
+
+int pa_is_registered(uint64_t pa);
+int pa_register(mt_iobuf_t *iobuf_p);
+void pa_deregister(mt_iobuf_t *iobuf_p);
+
index f4bd0c205175fb31855af9c02f0b66bf3d0f8c52..a919c2f7f9ceb77e3b2441c9e4839f1cf532e952 100644 (file)
@@ -40,6 +40,7 @@
 
 #include "mx_abi.h"
 #include "mthca_dev.h"
+#include "mt_pa_cash.h"
 
 #if defined(EVENT_TRACING)
 #ifdef offsetof
@@ -997,6 +998,7 @@ struct ib_mr *mthca_reg_virt_mr(struct ib_pd *pd,
        uint32_t i, n;
        mt_iobuf_t *iobuf_p;
        mt_iobuf_iter_t iobuf_iter;
+       ib_access_t ib_acc;
 
        /*
         * Be friendly to WRITE_MTT command and leave two 
@@ -1006,7 +1008,6 @@ struct ib_mr *mthca_reg_virt_mr(struct ib_pd *pd,
 
        HCA_ENTER(HCA_DBG_MEMORY);
 
-
        mr = kzalloc(sizeof *mr, GFP_KERNEL);
        if (!mr) {
                err = -ENOMEM;
@@ -1023,8 +1024,10 @@ struct ib_mr *mthca_reg_virt_mr(struct ib_pd *pd,
 
        // try register the buffer
        iobuf_p = &mr->iobuf;
-       err =  iobuf_register( (u64)vaddr, length, um_call, 
-               (acc & ~MTHCA_ACCESS_REMOTE_READ) ? IB_AC_LOCAL_WRITE : 0, iobuf_p );
+       iobuf_init( (u64)vaddr, length, um_call, iobuf_p);
+       ib_acc = (acc & ~MTHCA_ACCESS_REMOTE_READ) ? IB_AC_LOCAL_WRITE : 0;
+       err =  iobuf_register_with_cash( (u64)vaddr, length, um_call, 
+               &ib_acc, iobuf_p );
        if (err)
                goto err_reg_mem;
        mr->iobuf_used = TRUE;
@@ -1076,7 +1079,7 @@ struct ib_mr *mthca_reg_virt_mr(struct ib_pd *pd,
                goto done;
        __try {
                mr->secure_handle = MmSecureVirtualMemory ( vaddr, (SIZE_T)length,
-                       (acc & ~MTHCA_ACCESS_REMOTE_READ) ? PAGE_READWRITE : PAGE_READONLY );
+                       (ib_acc & IB_AC_LOCAL_WRITE) ? PAGE_READWRITE : PAGE_READONLY );
                if (mr->secure_handle == NULL) {
                        err = -EFAULT;
                        goto err_secure;
@@ -1090,6 +1093,7 @@ struct ib_mr *mthca_reg_virt_mr(struct ib_pd *pd,
                err = -EFAULT;
                goto err_secure;
        }
+
 done:  
        free_page((void*) pages);
 
@@ -1117,11 +1121,11 @@ int mthca_dereg_mr(struct ib_mr *mr)
        struct mthca_mr *mmr = to_mmr(mr);
        struct mthca_dev* dev = to_mdev(mr->device);
 
-       if (mr->pd->ucontext)
+       if (mmr->secure_handle)
                MmUnsecureVirtualMemory ( mmr->secure_handle );
        mthca_free_mr(dev, mmr);
        if (mmr->iobuf_used)
-               iobuf_deregister(&mmr->iobuf);
+               iobuf_deregister_with_cash(&mmr->iobuf);
        kfree(mmr);
        return 0;
 }