Changeset 39744 in vbox for trunk/src/VBox/Runtime/r0drv/freebsd/memobj-r0drv-freebsd.c

Timestamp:

Jan 10, 2012 6:15:04 PM (13 years ago)

Author:

vboxsync

Message:

rtr0memobj: Status code adjustments.

File:

• trunk/src/VBox/Runtime/r0drv/freebsd/memobj-r0drv-freebsd.c (modified) (19 diffs)

trunk/src/VBox/Runtime/r0drv/freebsd/memobj-r0drv-freebsd.c

@@ -45 +45 @@
 #include "internal/memobj.h"
 
+
 /*******************************************************************************
 *   Structures and Typedefs                                                    *
@@ -62 +63 @@
 MALLOC_DEFINE(M_IPRTMOBJ, "iprtmobj", "IPRT - R0MemObj");
 
+
 /**
  * Gets the virtual memory map the specified object is mapped into.
@@ -112 +114 @@
         case RTR0MEMOBJTYPE_LOW:
         case RTR0MEMOBJTYPE_CONT:
-        {
             rc = vm_map_remove(kernel_map,
                                 (vm_offset_t)pMemFreeBSD->Core.pv,
@@ -118 +119 @@
             AssertMsg(rc == KERN_SUCCESS, ("%#x", rc));
             break;
-        }
 
         case RTR0MEMOBJTYPE_LOCK:
@@ -223 +223 @@
 static int rtR0MemObjFreeBSDPhysAllocHelper(vm_object_t pObject, u_long cPages,
                                             vm_paddr_t VmPhysAddrHigh, u_long uAlignment,
-                                            bool fContiguous, bool fWire)
+                                            bool fContiguous, bool fWire, int rcNoMem)
 {
     if (fContiguous)
@@ -230 +230 @@
                                                    uAlignment, fWire) != NULL)
             return VINF_SUCCESS;
-        else
-            return VERR_NO_MEMORY;
+        return rcNoMem;
     }
 
@@ -252 +251 @@
             }
             VM_OBJECT_UNLOCK(pObject);
-            return VERR_NO_MEMORY;
+            return rcNoMem;
         }
     }
@@ -259 +258 @@
 
 static int rtR0MemObjFreeBSDAllocHelper(PRTR0MEMOBJFREEBSD pMemFreeBSD, bool fExecutable,
-                                        vm_paddr_t VmPhysAddrHigh, bool fContiguous)
+                                        vm_paddr_t VmPhysAddrHigh, bool fContiguous, int rcNoMem)
 {
     int rc;
@@ -275 +274 @@
     {
         rc = rtR0MemObjFreeBSDPhysAllocHelper(pMemFreeBSD->pObject, cPages,
-                                              VmPhysAddrHigh, PAGE_SIZE, fContiguous, false);
-        if (RT_SUCCESS(rc)) {
+                                              VmPhysAddrHigh, PAGE_SIZE, fContiguous,
+                                              false, rcNoMem);
+        if (RT_SUCCESS(rc))
+        {
             vm_map_wire(kernel_map, MapAddress, MapAddress + pMemFreeBSD->Core.cb,
                         VM_MAP_WIRE_SYSTEM | VM_MAP_WIRE_NOHOLES);
@@ -287 +288 @@
         vm_map_remove(kernel_map, MapAddress, MapAddress + pMemFreeBSD->Core.cb);
     }
-    rc = VERR_NO_MEMORY; /** @todo fix translation (borrow from darwin) */
+    rc = rcNoMem; /** @todo fix translation (borrow from darwin) */
 
     vm_object_deallocate(pMemFreeBSD->pObject);
@@ -300 +301 @@
         return VERR_NO_MEMORY;
 
-    int rc = rtR0MemObjFreeBSDAllocHelper(pMemFreeBSD, fExecutable, ~(vm_paddr_t)0, false);
+    int rc = rtR0MemObjFreeBSDAllocHelper(pMemFreeBSD, fExecutable, ~(vm_paddr_t)0, false, VERR_NO_MEMORY);
     if (RT_FAILURE(rc))
     {
@@ -319 +320 @@
         return VERR_NO_MEMORY;
 
-    int rc = rtR0MemObjFreeBSDAllocHelper(pMemFreeBSD, fExecutable, _4G, false);
+    int rc = rtR0MemObjFreeBSDAllocHelper(pMemFreeBSD, fExecutable, _4G, false, VERR_NO_LOW_MEMORY);
     if (RT_FAILURE(rc))
     {
@@ -338 +339 @@
         return VERR_NO_MEMORY;
 
-    int rc = rtR0MemObjFreeBSDAllocHelper(pMemFreeBSD, fExecutable, _4G, true);
+    int rc = rtR0MemObjFreeBSDAllocHelper(pMemFreeBSD, fExecutable, _4G, true, VERR_NO_CONT_MEMORY);
     if (RT_FAILURE(rc))
     {
@@ -354 +355 @@
                                            size_t cb,
                                            RTHCPHYS PhysHighest, size_t uAlignment,
-                                           bool fContiguous)
+                                           bool fContiguous, int rcNoMem)
 {
     uint32_t   cPages = atop(cb);
@@ -373 +374 @@
 
     int rc = rtR0MemObjFreeBSDPhysAllocHelper(pMemFreeBSD->pObject, cPages, VmPhysAddrHigh,
-                                              uAlignment, fContiguous, true);
-
-    if (RT_FAILURE(rc)) {
-        vm_object_deallocate(pMemFreeBSD->pObject);
-        rtR0MemObjDelete(&pMemFreeBSD->Core);
-    }
-    else
+                                              uAlignment, fContiguous, true, rcNoMem);
+    if (RT_SUCCESS(rc))
     {
         if (fContiguous)
@@ -392 +388 @@
         *ppMem = &pMemFreeBSD->Core;
     }
+    else
+    {
+        vm_object_deallocate(pMemFreeBSD->pObject);
+        rtR0MemObjDelete(&pMemFreeBSD->Core);
+    }
 
     return rc;
@@ -399 +400 @@
 DECLHIDDEN(int) rtR0MemObjNativeAllocPhys(PPRTR0MEMOBJINTERNAL ppMem, size_t cb, RTHCPHYS PhysHighest, size_t uAlignment)
 {
-    return rtR0MemObjFreeBSDAllocPhysPages(ppMem, RTR0MEMOBJTYPE_PHYS, cb, PhysHighest, uAlignment, true);
+    return rtR0MemObjFreeBSDAllocPhysPages(ppMem, RTR0MEMOBJTYPE_PHYS, cb, PhysHighest, uAlignment, true, VERR_NO_MEMORY);
 }
 
@@ -405 +406 @@
 DECLHIDDEN(int) rtR0MemObjNativeAllocPhysNC(PPRTR0MEMOBJINTERNAL ppMem, size_t cb, RTHCPHYS PhysHighest)
 {
-    return rtR0MemObjFreeBSDAllocPhysPages(ppMem, RTR0MEMOBJTYPE_PHYS_NC, cb, PhysHighest, PAGE_SIZE, false);
+    return rtR0MemObjFreeBSDAllocPhysPages(ppMem, RTR0MEMOBJTYPE_PHYS_NC, cb, PhysHighest, PAGE_SIZE, false, VERR_NO_PHYS_MEMORY);
 }
 
@@ -497 +498 @@
      * The pvFixed address range must be within the VM space when specified.
      */
-    if (pvFixed != (void *)-1
+    if (   pvFixed != (void *)-1
         && (    (vm_offset_t)pvFixed      < vm_map_min(pMap)
             ||  (vm_offset_t)pvFixed + cb > vm_map_max(pMap)))