make the idom "always call realloc_memory_ranges when filling a range entry"

kexec was core dumping after using 5 exclude_range pairs when only 3
were allocated.

also delcare realloc_memory_ranges to take void.

Signed-off-by: Milton Miller <milt...@bga.com>
---
v2:  compare variable j not i in a spot noticed by  Michael Ellerman

Index: kexec-tools/kexec/arch/ppc64/kexec-ppc64.c
===================================================================
--- kexec-tools.orig/kexec/arch/ppc64/kexec-ppc64.c     2009-01-08 
01:21:43.000000000 -0600
+++ kexec-tools/kexec/arch/ppc64/kexec-ppc64.c  2009-01-08 01:22:33.000000000 
-0600
@@ -96,7 +96,7 @@ err1:
 
 }
 
-static int realloc_memory_ranges()
+static int realloc_memory_ranges(void)
 {
        size_t memory_range_len;
 
@@ -469,6 +469,8 @@ static int get_devtree_details(unsigned 
                                exclude_range[i].start = initrd_start;
                                exclude_range[i].end = initrd_end;
                                i++;
+                               if (i >= max_memory_ranges)
+                                       realloc_memory_ranges();
                        }
                } /* chosen */
 
@@ -581,6 +583,8 @@ static int get_devtree_details(unsigned 
                        exclude_range[i].start = tce_base;
                        exclude_range[i].end = tce_base + tce_size;
                        i++;
+                       if (i >= max_memory_ranges)
+                               realloc_memory_ranges();
                        if (kexec_flags & KEXEC_ON_CRASH)
                                add_usable_mem_rgns(tce_base, tce_size);
                        closedir(cdir);
@@ -634,6 +638,8 @@ int setup_memory_ranges(unsigned long ke
                                memory_range[j].end = exclude_range[i].start - 
1;
                                memory_range[j].type = RANGE_RAM;
                                j++;
+                               if (j >= max_memory_ranges)
+                                       realloc_memory_ranges();
                        }
                } /* i == 0 */
                /* If the last exclude range does not end at memory_max, include
@@ -646,6 +652,8 @@ int setup_memory_ranges(unsigned long ke
                                memory_range[j].end = memory_max;
                                memory_range[j].type = RANGE_RAM;
                                j++;
+                               if (j >= max_memory_ranges)
+                                       realloc_memory_ranges();
                                /* Limit the end to rmo_top */
                                if (memory_range[j-1].start >= rmo_top) {
                                        j--;
@@ -666,6 +674,8 @@ int setup_memory_ranges(unsigned long ke
                memory_range[j].end = exclude_range[i+1].start - 1;
                memory_range[j].type = RANGE_RAM;
                j++;
+               if (j >= max_memory_ranges)
+                       realloc_memory_ranges();
                /* Limit range to rmo_top */
                if (memory_range[j-1].start >= rmo_top) {
                        j--;
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev

Reply via email to