<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Nov 4, 2020 at 12:10 AM Cyril Hrubis <<a href="mailto:chrubis@suse.cz">chrubis@suse.cz</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Now each string in the kconfig[] array in tst_test structure is an<br>
boolean expression which is evaluated. All expressions has to be true in<br>
order for the test to continue.<br>
<br>
This also makes the parser for the kernel config a bit more robust as it<br>
was pointed out that there may have been cases where it could be mislead<br>
by hand edited config files.<br>
<br>
+ Update the docs.<br>
<br>
Signed-off-by: Cyril Hrubis <<a href="mailto:chrubis@suse.cz" target="_blank">chrubis@suse.cz</a>><br>
CC: Pengfei Xu <<a href="mailto:pengfei.xu@intel.com" target="_blank">pengfei.xu@intel.com</a>><br></blockquote><div><br></div><div class="gmail_default" style="font-size:small">For series:</div><div class="gmail_default" style="font-size:small">Reviewed-by: Li Wang <<a href="mailto:liwang@redhat.com">liwang@redhat.com</a>></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">
diff --git a/lib/newlib_tests/test_kconfig.c b/lib/newlib_tests/test_kconfig.c<br>
index d9c662fc5..1f659b95a 100644<br>
--- a/lib/newlib_tests/test_kconfig.c<br>
+++ b/lib/newlib_tests/test_kconfig.c<br>
@@ -14,6 +14,8 @@ static const char *kconfigs[] = {<br>
        "CONFIG_MMU",<br>
        "CONFIG_EXT4_FS=m",<br>
        "CONFIG_PGTABLE_LEVELS=4",<br>
+       "CONFIG_MMU & CONFIG_EXT4_FS=m",<br>
+       "CONFIG_EXT4_FS=m | CONFIG_MMU",<br>
        NULL<br>
 };<br></blockquote><div><br></div><div><br></div><div><div class="gmail_default" style="font-size:small">I tried with more complicated configurations for test parsing, most works</div><div class="gmail_default" style="font-size:small">as expected but except the below one with parentheses ():</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small"><span class="gmail_default">    </span>"CONFIG_DEFAULT_HOSTNAME=\"(none)\"",</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">That's because the parentheses are the element of expression and used</div><div class="gmail_default" style="font-size:small">to distinguish tokens, I‘m not sure whether we can change it as a general</div><div class="gmail_default" style="font-size:small">character to fix this problem, if no, we might need to comment on this in </div><div class="gmail_default" style="font-size:small">the documentation.</div></div><div><br></div><div><div class="gmail_default" style="font-size:small">--- a/lib/newlib_tests/test_kconfig.c</div>+++ b/lib/newlib_tests/test_kconfig.c<br>@@ -16,6 +16,11 @@ static const char *kconfigs[] = {<br>        "CONFIG_PGTABLE_LEVELS=4",<br>        "CONFIG_MMU & CONFIG_EXT4_FS=m",<br>        "CONFIG_EXT4_FS=m | CONFIG_MMU",<br>+      <span class="gmail_default" style="font-size:small"></span> "CONFIG_DEFAULT_HOSTNAME=\"(none)\"",<br>+       "CONFIG_LOCALVERSION=\"\" | CONFIG_MODULE_SIG_HASH=\"sha256\"",<br>+       "CONFIG_OUTPUT_FORMAT=\"elf64-x86-64\"",<br>+       "CONFIG_ARCH_DEFCONFIG=\"arch/x86/configs/x86_64_defconfig\"",<br>+       "CONFIG_ILLEGAL_POINTER_VALUE=0xdead000000000000",<br>        NULL<br> };<br><div class="gmail_default" style="font-size:small"><br></div></div></div><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>