]> pilppa.org Git - linux-2.6-omap-h63xx.git/blobdiff - lib/lmb.c
slab_err: Pass parameters correctly to slab_bug
[linux-2.6-omap-h63xx.git] / lib / lmb.c
index 549fbb3d70cf523f404c9e1ee7fd78158684c77f..896e2832099ea207b36c02af276a0fd2c15516ea 100644 (file)
--- a/lib/lmb.c
+++ b/lib/lmb.c
 #include <linux/bitops.h>
 #include <linux/lmb.h>
 
-#undef DEBUG
-
-#ifdef DEBUG
-#define DBG(fmt...) LMB_DBG(fmt)
-#else
-#define DBG(fmt...)
-#endif
-
 #define LMB_ALLOC_ANYWHERE     0
 
 struct lmb lmb;
@@ -32,32 +24,32 @@ void lmb_dump_all(void)
 #ifdef DEBUG
        unsigned long i;
 
-       DBG("lmb_dump_all:\n");
-       DBG("    memory.cnt               = 0x%lx\n", lmb.memory.cnt);
-       DBG("    memory.size              = 0x%llx\n",
+       pr_debug("lmb_dump_all:\n");
+       pr_debug("    memory.cnt                  = 0x%lx\n", lmb.memory.cnt);
+       pr_debug("    memory.size                 = 0x%llx\n",
            (unsigned long long)lmb.memory.size);
        for (i=0; i < lmb.memory.cnt ;i++) {
-               DBG("    memory.region[0x%x].base       = 0x%llx\n",
+               pr_debug("    memory.region[0x%x].base       = 0x%llx\n",
                    i, (unsigned long long)lmb.memory.region[i].base);
-               DBG("                 .size     = 0x%llx\n",
+               pr_debug("                    .size     = 0x%llx\n",
                    (unsigned long long)lmb.memory.region[i].size);
        }
 
-       DBG("\n    reserved.cnt   = 0x%lx\n", lmb.reserved.cnt);
-       DBG("    reserved.size    = 0x%lx\n", lmb.reserved.size);
+       pr_debug("    reserved.cnt        = 0x%lx\n", lmb.reserved.cnt);
+       pr_debug("    reserved.size       = 0x%lx\n", lmb.reserved.size);
        for (i=0; i < lmb.reserved.cnt ;i++) {
-               DBG("    reserved.region[0x%x].base       = 0x%llx\n",
+               pr_debug("    reserved.region[0x%x].base       = 0x%llx\n",
                    i, (unsigned long long)lmb.reserved.region[i].base);
-               DBG("                 .size     = 0x%llx\n",
+               pr_debug("                    .size     = 0x%llx\n",
                    (unsigned long long)lmb.reserved.region[i].size);
        }
 #endif /* DEBUG */
 }
 
-static unsigned long __init lmb_addrs_overlap(u64 base1,
-               u64 size1, u64 base2, u64 size2)
+static unsigned long __init lmb_addrs_overlap(u64 base1, u64 size1,
+               u64 base2, u64 size2)
 {
-       return ((base1 < (base2+size2)) && (base2 < (base1+size1)));
+       return ((base1 < (base2 + size2)) && (base2 < (base1 + size1)));
 }
 
 static long __init lmb_addrs_adjacent(u64 base1, u64 size1,
@@ -101,7 +93,6 @@ static void __init lmb_coalesce_regions(struct lmb_region *rgn,
        lmb_remove_region(rgn, r2);
 }
 
-/* This routine called with relocation disabled. */
 void __init lmb_init(void)
 {
        /* Create a dummy zero size LMB which will get coalesced away later.
@@ -117,7 +108,6 @@ void __init lmb_init(void)
        lmb.reserved.cnt = 1;
 }
 
-/* This routine may be called with relocation disabled. */
 void __init lmb_analyze(void)
 {
        int i;
@@ -128,7 +118,6 @@ void __init lmb_analyze(void)
                lmb.memory.size += lmb.memory.region[i].size;
 }
 
-/* This routine called with relocation disabled. */
 static long __init lmb_add_region(struct lmb_region *rgn, u64 base, u64 size)
 {
        unsigned long coalesced = 0;
@@ -141,7 +130,7 @@ static long __init lmb_add_region(struct lmb_region *rgn, u64 base, u64 size)
        }
 
        /* First try and coalesce this LMB with another. */
-       for (i=0; i < rgn->cnt; i++) {
+       for (i = 0; i < rgn->cnt; i++) {
                u64 rgnbase = rgn->region[i].base;
                u64 rgnsize = rgn->region[i].size;
 
@@ -149,21 +138,20 @@ static long __init lmb_add_region(struct lmb_region *rgn, u64 base, u64 size)
                        /* Already have this region, so we're done */
                        return 0;
 
-               adjacent = lmb_addrs_adjacent(base,size,rgnbase,rgnsize);
-               if ( adjacent > 0 ) {
+               adjacent = lmb_addrs_adjacent(base, size, rgnbase, rgnsize);
+               if (adjacent > 0) {
                        rgn->region[i].base -= size;
                        rgn->region[i].size += size;
                        coalesced++;
                        break;
-               }
-               else if ( adjacent < 0 ) {
+               } else if (adjacent < 0) {
                        rgn->region[i].size += size;
                        coalesced++;
                        break;
                }
        }
 
-       if ((i < rgn->cnt-1) && lmb_regions_adjacent(rgn, i, i+1) ) {
+       if ((i < rgn->cnt - 1) && lmb_regions_adjacent(rgn, i, i+1)) {
                lmb_coalesce_regions(rgn, i, i+1);
                coalesced++;
        }
@@ -174,7 +162,7 @@ static long __init lmb_add_region(struct lmb_region *rgn, u64 base, u64 size)
                return -1;
 
        /* Couldn't coalesce the LMB, so add it to the sorted table. */
-       for (i = rgn->cnt-1; i >= 0; i--) {
+       for (i = rgn->cnt - 1; i >= 0; i--) {
                if (base < rgn->region[i].base) {
                        rgn->region[i+1].base = rgn->region[i].base;
                        rgn->region[i+1].size = rgn->region[i].size;
@@ -194,10 +182,9 @@ static long __init lmb_add_region(struct lmb_region *rgn, u64 base, u64 size)
        return 0;
 }
 
-/* This routine may be called with relocation disabled. */
 long __init lmb_add(u64 base, u64 size)
 {
-       struct lmb_region *_rgn = &(lmb.memory);
+       struct lmb_region *_rgn = &lmb.memory;
 
        /* On pSeries LPAR systems, the first LMB is our RMO region. */
        if (base == 0)
@@ -209,24 +196,22 @@ long __init lmb_add(u64 base, u64 size)
 
 long __init lmb_reserve(u64 base, u64 size)
 {
-       struct lmb_region *_rgn = &(lmb.reserved);
+       struct lmb_region *_rgn = &lmb.reserved;
 
        BUG_ON(0 == size);
 
        return lmb_add_region(_rgn, base, size);
 }
 
-long __init lmb_overlaps_region(struct lmb_region *rgn, u64 base,
-                               u64 size)
+long __init lmb_overlaps_region(struct lmb_region *rgn, u64 base, u64 size)
 {
        unsigned long i;
 
-       for (i=0; i < rgn->cnt; i++) {
+       for (i = 0; i < rgn->cnt; i++) {
                u64 rgnbase = rgn->region[i].base;
                u64 rgnsize = rgn->region[i].size;
-               if ( lmb_addrs_overlap(base,size,rgnbase,rgnsize) ) {
+               if (lmb_addrs_overlap(base, size, rgnbase, rgnsize))
                        break;
-               }
        }
 
        return (i < rgn->cnt) ? i : -1;
@@ -245,20 +230,23 @@ static u64 lmb_align_up(u64 addr, u64 size)
 static u64 __init lmb_alloc_nid_unreserved(u64 start, u64 end,
                                           u64 size, u64 align)
 {
-       u64 base;
+       u64 base, res_base;
        long j;
 
        base = lmb_align_down((end - size), align);
-       while (start <= base &&
-              ((j = lmb_overlaps_region(&lmb.reserved, base, size)) >= 0))
-               base = lmb_align_down(lmb.reserved.region[j].base - size,
-                                     align);
-
-       if (base != 0 && start <= base) {
-               if (lmb_add_region(&lmb.reserved, base,
-                                  lmb_align_up(size, align)) < 0)
-                       base = ~(u64)0;
-               return base;
+       while (start <= base) {
+               j = lmb_overlaps_region(&lmb.reserved, base, size);
+               if (j < 0) {
+                       /* this area isn't reserved, take it */
+                       if (lmb_add_region(&lmb.reserved, base,
+                                          lmb_align_up(size, align)) < 0)
+                               base = ~(u64)0;
+                       return base;
+               }
+               res_base = lmb.reserved.region[j].base;
+               if (res_base < size)
+                       break;
+               base = lmb_align_down(res_base - size, align);
        }
 
        return ~(u64)0;
@@ -330,17 +318,21 @@ u64 __init __lmb_alloc_base(u64 size, u64 align, u64 max_addr)
 {
        long i, j;
        u64 base = 0;
+       u64 res_base;
 
        BUG_ON(0 == size);
 
        /* On some platforms, make sure we allocate lowmem */
+       /* Note that LMB_REAL_LIMIT may be LMB_ALLOC_ANYWHERE */
        if (max_addr == LMB_ALLOC_ANYWHERE)
                max_addr = LMB_REAL_LIMIT;
 
-       for (i = lmb.memory.cnt-1; i >= 0; i--) {
+       for (i = lmb.memory.cnt - 1; i >= 0; i--) {
                u64 lmbbase = lmb.memory.region[i].base;
                u64 lmbsize = lmb.memory.region[i].size;
 
+               if (lmbsize < size)
+                       continue;
                if (max_addr == LMB_ALLOC_ANYWHERE)
                        base = lmb_align_down(lmbbase + lmbsize - size, align);
                else if (lmbbase < max_addr) {
@@ -349,22 +341,22 @@ u64 __init __lmb_alloc_base(u64 size, u64 align, u64 max_addr)
                } else
                        continue;
 
-               while ((lmbbase <= base) &&
-                      ((j = lmb_overlaps_region(&lmb.reserved, base, size)) >= 0) )
-                       base = lmb_align_down(lmb.reserved.region[j].base - size,
-                                             align);
-
-               if ((base != 0) && (lmbbase <= base))
-                       break;
+               while (base && lmbbase <= base) {
+                       j = lmb_overlaps_region(&lmb.reserved, base, size);
+                       if (j < 0) {
+                               /* this area isn't reserved, take it */
+                               if (lmb_add_region(&lmb.reserved, base,
+                                                  size) < 0)
+                                       return 0;
+                               return base;
+                       }
+                       res_base = lmb.reserved.region[j].base;
+                       if (res_base < size)
+                               break;
+                       base = lmb_align_down(res_base - size, align);
+               }
        }
-
-       if (i < 0)
-               return 0;
-
-       if (lmb_add_region(&lmb.reserved, base, lmb_align_up(size, align)) < 0)
-               return 0;
-
-       return base;
+       return 0;
 }
 
 /* You must call lmb_analyze() before this. */
@@ -387,7 +379,7 @@ void __init lmb_enforce_memory_limit(u64 memory_limit)
        u64 limit;
        struct lmb_property *p;
 
-       if (! memory_limit)
+       if (!memory_limit)
                return;
 
        /* Truncate the lmb regions to satisfy the memory limit. */