On Fri, Mar 11, 2022 at 4:27 PM HAGIO KAZUHITO(萩尾 一仁) <k-hagio-ab@nec.com> wrote:
-----Original Message-----
>  v6 --> v7:
>          1.) Simplify the arm64_get_struct_page_max_shift().
>        2.) Add "struct_page_size" dump info in  arm64_dump_machdep_table().
>          3.) Tested it again.

Normally the members of machspec are printed after "macspec:" so
with the following change,

Acked-by: Kazuhito Hagio <k-hagio-ab@nec.com>

diff --git a/arm64.c b/arm64.c
index ac8d9e0c24df..2dcef46dc496 100644
--- a/arm64.c
+++ b/arm64.c
@@ -947,7 +947,6 @@ arm64_dump_machdep_table(ulong arg)
        fprintf(fp, "           pageshift: %d\n", machdep->pageshift);
        fprintf(fp, "            pagemask: %lx\n", (ulong)machdep->pagemask);
        fprintf(fp, "          pageoffset: %lx\n", machdep->pageoffset);
-       fprintf(fp, "    struct_page_size: %ld\n", ms->struct_page_size);
        fprintf(fp, "           stacksize: %ld\n", machdep->stacksize);
        fprintf(fp, "                  hz: %d\n", machdep->hz);
        fprintf(fp, "                 mhz: %ld\n", machdep->mhz);
@@ -1034,6 +1033,7 @@ arm64_dump_machdep_table(ulong arg)
        }

        fprintf(fp, "            machspec: %lx\n", (ulong)ms);
+       fprintf(fp, "      struct_page_size: %ld\n", ms->struct_page_size);
        fprintf(fp, "               VA_BITS: %ld\n", ms->VA_BITS);
        fprintf(fp, "  CONFIG_ARM64_VA_BITS: %ld\n", ms->CONFIG_ARM64_VA_BITS);
        fprintf(fp, "              VA_START: ");

Lianbo, could you please apply with this?


Yes. Good understanding, Kazu.
 
Thanks,
Kazu