<div dir="ltr"><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">
> diff --git a/include/tst_arch.h b/include/tst_arch.h<br>
> new file mode 100644<br>
> index 000000000..784c3093b<br>
> --- /dev/null<br>
> +++ b/include/tst_arch.h<br>
> @@ -0,0 +1,39 @@<br>
> +/* SPDX-License-Identifier: GPL-2.0-or-later<br>
> + * Copyright (c) 2021 Li Wang <<a href="mailto:liwang@redhat.com" target="_blank">liwang@redhat.com</a>><br>
> + */<br>
> +<br>
> +#ifndef TST_ARCH_H__<br>
> +#define TST_ARCH_H__<br>
> +<br>
> +enum tst_arch_type {<br>
> +     TST_I386,<br>
> +     TST_X86,<br>
<br>
Why do we have both i386 and X86 here? Isn't __i386__ synonymous for<br>
__x86__ does gcc even define __x86__?<br></blockquote><div><br></div><div class="gmail_default" style="font-size:small">My fault, I just copy that from testcase defines and didn't check</div><div class="gmail_default" style="font-size:small">if __x86__ validatable or not.</div><div class="gmail_default" style="font-size:small"></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
I doubt that we care about the differencies between i386, i586 and i686<br>
at all. I would just keep TST_X86 in the list for any 32bit intel<br>
compatible hardware.<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">I prefer to only keep TST_I386 and TST_X86_64 for use. Becuase</div><div class="gmail_default" style="font-size:small">so far I didn't see there is a requirement on i[4|5|6]86 in LTP at all.</div><div class="gmail_default" style="font-size:small">And, we can add that if we really need it in the future.</div><br></div><div class="gmail_default" style="font-size:small">The rest of the comments looks good, thanks.</div></div><div><br></div>-- <br><div dir="ltr"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>