s390/boot: always align vmalloc area on segment boundary
authorAlexander Gordeev <agordeev@linux.ibm.com>
Sun, 16 Jul 2023 08:56:00 +0000 (10:56 +0200)
committerAlexander Gordeev <agordeev@linux.ibm.com>
Wed, 22 Nov 2023 13:07:28 +0000 (14:07 +0100)
The size of vmalloc area depends from various factors
on boot and could be set to:

1. Default size as determined by VMALLOC_DEFAULT_SIZE macro;
2. One half of the virtual address space not occupied by
   modules and fixed mappings;
3. The size provided by user with vmalloc= kernel command
   line parameter;

In cases [1] and [2] the vmalloc area base address is aligned
on Region3 table type boundary, while in case [3] in might get
aligned on page boundary.

Limit the waste of page tables and always align vmalloc area
size and base address on segment boundary.

Acked-by: Heiko Carstens <hca@linux.ibm.com>
Signed-off-by: Alexander Gordeev <agordeev@linux.ibm.com>
arch/s390/boot/ipl_parm.c
arch/s390/boot/startup.c

index 2ab4872fbee1c9ab6938c2bf973d9590ccea2d06..b24de9aabf7d8a895ef4583e06370e2068395de2 100644 (file)
@@ -274,7 +274,7 @@ void parse_boot_command_line(void)
                        memory_limit = round_down(memparse(val, NULL), PAGE_SIZE);
 
                if (!strcmp(param, "vmalloc") && val) {
-                       vmalloc_size = round_up(memparse(val, NULL), PAGE_SIZE);
+                       vmalloc_size = round_up(memparse(val, NULL), _SEGMENT_SIZE);
                        vmalloc_size_set = 1;
                }
 
index 8104e0e3d188d8e58b3c2824672d84ea780f0d7f..9cc76e631759b17f16f639a082c3deb8c00f09f2 100644 (file)
@@ -255,7 +255,8 @@ static unsigned long setup_kernel_memory_layout(void)
        VMALLOC_END = MODULES_VADDR;
 
        /* allow vmalloc area to occupy up to about 1/2 of the rest virtual space left */
-       vmalloc_size = min(vmalloc_size, round_down(VMALLOC_END / 2, _REGION3_SIZE));
+       vsize = round_down(VMALLOC_END / 2, _SEGMENT_SIZE);
+       vmalloc_size = min(vmalloc_size, vsize);
        VMALLOC_START = VMALLOC_END - vmalloc_size;
 
        /* split remaining virtual space between 1:1 mapping & vmemmap array */