s390/vmem: make variable and function names consistent
authorAlexander Gordeev <agordeev@linux.ibm.com>
Tue, 10 Nov 2020 09:36:23 +0000 (10:36 +0100)
committerHeiko Carstens <hca@linux.ibm.com>
Fri, 20 Nov 2020 18:19:12 +0000 (19:19 +0100)
Rename some variable and functions to better clarify
what they are and what they do.

Signed-off-by: Alexander Gordeev <agordeev@linux.ibm.com>
Signed-off-by: Heiko Carstens <hca@linux.ibm.com>
arch/s390/mm/vmem.c

index 56ab9bb770f3adbaeb5a8996f1ac6699f3949e90..01f3a5f58e6420b2d704dcf7955417e9a2e45787 100644 (file)
@@ -76,20 +76,20 @@ static void vmem_pte_free(unsigned long *table)
 
 /*
  * The unused vmemmap range, which was not yet memset(PAGE_UNUSED) ranges
- * from unused_pmd_start to next PMD_SIZE boundary.
+ * from unused_sub_pmd_start to next PMD_SIZE boundary.
  */
-static unsigned long unused_pmd_start;
+static unsigned long unused_sub_pmd_start;
 
-static void vmemmap_flush_unused_pmd(void)
+static void vmemmap_flush_unused_sub_pmd(void)
 {
-       if (!unused_pmd_start)
+       if (!unused_sub_pmd_start)
                return;
-       memset(__va(unused_pmd_start), PAGE_UNUSED,
-              ALIGN(unused_pmd_start, PMD_SIZE) - unused_pmd_start);
-       unused_pmd_start = 0;
+       memset(__va(unused_sub_pmd_start), PAGE_UNUSED,
+              ALIGN(unused_sub_pmd_start, PMD_SIZE) - unused_sub_pmd_start);
+       unused_sub_pmd_start = 0;
 }
 
-static void __vmemmap_use_sub_pmd(unsigned long start, unsigned long end)
+static void vmemmap_mark_sub_pmd_used(unsigned long start, unsigned long end)
 {
        /*
         * As we expect to add in the same granularity as we remove, it's
@@ -106,24 +106,24 @@ static void vmemmap_use_sub_pmd(unsigned long start, unsigned long end)
         * We only optimize if the new used range directly follows the
         * previously unused range (esp., when populating consecutive sections).
         */
-       if (unused_pmd_start == start) {
-               unused_pmd_start = end;
-               if (likely(IS_ALIGNED(unused_pmd_start, PMD_SIZE)))
-                       unused_pmd_start = 0;
+       if (unused_sub_pmd_start == start) {
+               unused_sub_pmd_start = end;
+               if (likely(IS_ALIGNED(unused_sub_pmd_start, PMD_SIZE)))
+                       unused_sub_pmd_start = 0;
                return;
        }
-       vmemmap_flush_unused_pmd();
-       __vmemmap_use_sub_pmd(start, end);
+       vmemmap_flush_unused_sub_pmd();
+       vmemmap_mark_sub_pmd_used(start, end);
 }
 
 static void vmemmap_use_new_sub_pmd(unsigned long start, unsigned long end)
 {
        void *page = __va(ALIGN_DOWN(start, PMD_SIZE));
 
-       vmemmap_flush_unused_pmd();
+       vmemmap_flush_unused_sub_pmd();
 
        /* Could be our memmap page is filled with PAGE_UNUSED already ... */
-       __vmemmap_use_sub_pmd(start, end);
+       vmemmap_mark_sub_pmd_used(start, end);
 
        /* Mark the unused parts of the new memmap page PAGE_UNUSED. */
        if (!IS_ALIGNED(start, PMD_SIZE))
@@ -134,7 +134,7 @@ static void vmemmap_use_new_sub_pmd(unsigned long start, unsigned long end)
         * unused range in the populated PMD.
         */
        if (!IS_ALIGNED(end, PMD_SIZE))
-               unused_pmd_start = end;
+               unused_sub_pmd_start = end;
 }
 
 /* Returns true if the PMD is completely unused and can be freed. */
@@ -142,7 +142,7 @@ static bool vmemmap_unuse_sub_pmd(unsigned long start, unsigned long end)
 {
        void *page = __va(ALIGN_DOWN(start, PMD_SIZE));
 
-       vmemmap_flush_unused_pmd();
+       vmemmap_flush_unused_sub_pmd();
        memset(__va(start), PAGE_UNUSED, end - start);
        return !memchr_inv(page, PAGE_UNUSED, PMD_SIZE);
 }