s390/vmalloc: fix VMALLOC_START calculation
authorMikhail Zaslonko <zaslonko@linux.ibm.com>
Wed, 10 Oct 2018 11:44:45 +0000 (13:44 +0200)
committerMartin Schwidefsky <schwidefsky@de.ibm.com>
Thu, 11 Oct 2018 15:02:38 +0000 (17:02 +0200)
With the introduction of the module area on top of the vmalloc area, the
calculation of VMALLOC_START in setup_memory_end() function hasn't been
adjusted. As a result we got vmalloc area 2 Gb (MODULES_LEN) smaller than
it should be and the preceding vmemmap area got extra memory instead.
The patch fixes this calculation error although there were no visible
negative effects.
Apart from that, change 'tmp' variable to 'vmemmap' in memory_end
calculation for better readability.

Reviewed-by: Heiko Carstens <heiko.carstens@de.ibm.com>
Signed-off-by: Mikhail Zaslonko <zaslonko@linux.ibm.com>
Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
arch/s390/kernel/setup.c

index 4b2039f3e2f4c3dfd7fdd197febaf7a87aee5286..a2e952b662487453b8baedefc53402eb04d70a83 100644 (file)
@@ -551,7 +551,7 @@ static void __init setup_memory_end(void)
        MODULES_END = vmax;
        MODULES_VADDR = MODULES_END - MODULES_LEN;
        VMALLOC_END = MODULES_VADDR;
-       VMALLOC_START = vmax - vmalloc_size;
+       VMALLOC_START = VMALLOC_END - vmalloc_size;
 
        /* Split remaining virtual space between 1:1 mapping & vmemmap array */
        tmp = VMALLOC_START / (PAGE_SIZE + sizeof(struct page));
@@ -563,7 +563,7 @@ static void __init setup_memory_end(void)
        vmemmap = (struct page *) tmp;
 
        /* Take care that memory_end is set and <= vmemmap */
-       memory_end = min(memory_end ?: max_physmem_end, tmp);
+       memory_end = min(memory_end ?: max_physmem_end, (unsigned long)vmemmap);
 #ifdef CONFIG_KASAN
        /* fit in kasan shadow memory region between 1:1 and vmemmap */
        memory_end = min(memory_end, KASAN_SHADOW_START);