x86/kaslr: Drop redundant cur_entry from __process_mem_region()
authorArvind Sankar <nivedita@alum.mit.edu>
Tue, 28 Jul 2020 22:57:07 +0000 (18:57 -0400)
committerIngo Molnar <mingo@kernel.org>
Fri, 31 Jul 2020 09:08:17 +0000 (11:08 +0200)
cur_entry is only used as cur_entry.start + cur_entry.size, which is
always equal to end.

Signed-off-by: Arvind Sankar <nivedita@alum.mit.edu>
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Link: https://lore.kernel.org/r/20200728225722.67457-7-nivedita@alum.mit.edu
arch/x86/boot/compressed/kaslr.c

index 848346f..f2454ee 100644 (file)
@@ -624,7 +624,6 @@ static void __process_mem_region(struct mem_vector *entry,
 {
        struct mem_vector region, overlap;
        unsigned long start_orig, end;
-       struct mem_vector cur_entry;
 
        /* Ignore entries entirely below our minimum. */
        if (entry->start + entry->size < minimum)
@@ -634,11 +633,9 @@ static void __process_mem_region(struct mem_vector *entry,
        end = min(entry->size + entry->start, mem_limit);
        if (entry->start >= end)
                return;
-       cur_entry.start = entry->start;
-       cur_entry.size = end - entry->start;
 
-       region.start = cur_entry.start;
-       region.size = cur_entry.size;
+       region.start = entry->start;
+       region.size = end - entry->start;
 
        /* Give up if slot area array is full. */
        while (slot_area_index < MAX_SLOT_AREA) {
@@ -652,7 +649,7 @@ static void __process_mem_region(struct mem_vector *entry,
                region.start = ALIGN(region.start, CONFIG_PHYSICAL_ALIGN);
 
                /* Did we raise the address above the passed in memory entry? */
-               if (region.start > cur_entry.start + cur_entry.size)
+               if (region.start > end)
                        return;
 
                /* Reduce size by any delta from the original address. */