[IA64] contig.c: Function find_bootmap_location has 2 bugs.

Firstly, if it's done successfully, it should return -1 instead of 1
because its caller, efi_memmap_walk, will end when find_bootmap_location
returns a value smaller than 0.

Secondly, statement "free_start = PAGE_ALIGN(rsvd_region[i].end)" should
been moved forward. free_start needs to be initialized for every loop
iteration.  Current implementation is buggy where initialization is
skipped if range_end <= range_start.  Skipping initializing will leads
to overlapping bootmap with one of the rsvd_regions and subsequently
kernel hang at boot time.
Singed-off-by: (sic) Zhang Yanmin <yanmin.zhang@intel.com>
Signed-off-by: Yao Jun      <junx.yao@intel.com>
Signed-off-by: Tony Luck    <tony.luck@intel.com>

arch/ia64/mm/contig.c

@@ -116,19 +116,19 @@ find_bootmap_location (unsigned long start, unsigned long end, void *arg)
 		range_start = max(start, free_start);
 		range_end   = min(end, rsvd_region[i].start & PAGE_MASK);
 
+		free_start = PAGE_ALIGN(rsvd_region[i].end);
+
 		if (range_end <= range_start)
 			continue; /* skip over empty range */
 
-	       	if (range_end - range_start >= needed) {
+		if (range_end - range_start >= needed) {
 			bootmap_start = __pa(range_start);
-			return 1;	/* done */
+			return -1;	/* done */
 		}
 
 		/* nothing more available in this segment */
 		if (range_end == end)
 			return 0;
-
-		free_start = PAGE_ALIGN(rsvd_region[i].end);
 	}
 	return 0;
 }