<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div></div><div class="gmail_quote"><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> +#ifndef TST_ARCH_H__<br>
> +#define TST_ARCH_H__<br>
> +<br>
> +enum tst_arch_type {<br>
> +     TST_UNKNOWN,<br>
> +     TST_I386,<br>
<br>
I would still probably name this TST_X86, Linux does not support i386<br>
anymore, the remaining 32bit distributions usually require at least i586<br>
or i686...<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">Sure, If go with that TST_X86, we just need to do modifications based on</div><div class="gmail_default" style="font-size:small">V4 like below, right? What else am I missing here?</div></div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">--- a/include/tst_arch.h<br>+++ b/include/tst_arch.h<br>@@ -7,7 +7,7 @@<br> <br> enum tst_arch_type {<br>        TST_UNKNOWN,<br>-       TST_I386,<br>+       TST_X86,<br>        TST_X86_64,<br>        TST_IA64,<br>        TST_PPC,<br>diff --git a/lib/tst_arch.c b/lib/tst_arch.c<br>index 67a955789..f19802a03 100644<br>--- a/lib/tst_arch.c<br>+++ b/lib/tst_arch.c<br>@@ -13,9 +13,9 @@ const struct tst_arch tst_arch = {<br> #if defined(__x86_64__)<br>         .name = "x86_64",<br>         .type = TST_X86_64,<br>-#elif defined(__i386__)<br>-        .name = "i386",<br>-        .type = TST_I386,<br>+#elif defined(__i386__) || defined(__i586__) || defined(__i686__)<br>+        .name = "x86",<br>+        .type = TST_X86,<br> #elif defined(__ia64__)<br>         .name = "ia64",<br>         .type = TST_IA64,<br>@@ -48,6 +48,8 @@ const struct tst_arch tst_arch = {<br> <br> static const char *const arch_type_list[] = {<br>        "i386",<br>+       "i586",<br>+       "i686",<br>        "x86_64",<br>        "ia64",<br>        "ppc",<br>diff --git a/testcases/kernel/mem/tunable/max_map_count.c b/testcases/kernel/mem/tunable/max_map_count.c<br>index bd5af0ff6..a4c3dbf8e 100644<br>--- a/testcases/kernel/mem/tunable/max_map_count.c<br>+++ b/testcases/kernel/mem/tunable/max_map_count.c<br>@@ -88,7 +88,7 @@ static bool filter_map(const char *line)<br>                return false;<br> <br>        switch (tst_arch.type) {<br>-       case TST_I386:<br>+       case TST_X86:<br>        case TST_X86_64:<br>                /* On x86, there's an old compat vsyscall page */<br>                if (!strcmp(buf, "[vsyscall]"))<br></div></div><div><br></div>-- <br><div dir="ltr"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>