<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, Sep 18, 2019 at 1:35 PM Li Wang <<a href="mailto:liwang@redhat.com">liwang@redhat.com</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">As the tst_taint_init comments described, If the tainted-flags are already set<br>
by the kernel, there is no reason to continue and TCONF is generated. But in<br>
the function achieve, it uses TBROK.<br>
<br>
  cmdline="cve-2017-17053"<br>
  tst_test.c:1096: INFO: Timeout per run is 0h 10m 00s<br>
  tst_taint.c:88: BROK: Kernel is already tainted: 536871424<br>
<br>
Signed-off-by: Li Wang <<a href="mailto:liwang@redhat.com" target="_blank">liwang@redhat.com</a>><br>
Cc: Chang Yin <<a href="mailto:cyin@redhat.com" target="_blank">cyin@redhat.com</a>><br>
Cc: Michael Moese <<a href="mailto:mmoese@suse.de" target="_blank">mmoese@suse.de</a>><br>
---<br>
<br>
Notes:<br>
    Hi Cyril & Michael,<br>
<br>
    I'm not sure if that's a good choice to skip whole test if the kernel already<br>
    tainted, so maybe we could also perform it but not do a strict tainted-flags<br>
    check after testing? The reason I think is it probably have chance to find<br>
    new issue even with a tainted kernel.<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">Something maybe change like below, which way is better?</div><div class="gmail_default" style="font-size:small"><br></div>--- a/lib/tst_taint.c<br>+++ b/lib/tst_taint.c<br>@@ -7,6 +7,7 @@<br> #define TAINT_FILE "/proc/sys/kernel/tainted"<br> <br> static unsigned int taint_mask = -1;<br>+static unsigned int taint_check = 1;<br> <br> static unsigned int tst_taint_read(void)<br> {<br>@@ -84,8 +85,10 @@ void tst_taint_init(unsigned int mask)<br>        taint_mask = mask;<br> <br>        taint = tst_taint_read();<br>-       if ((taint & mask) != 0)<br>-               tst_brk(TCONF, "Kernel is already tainted: %u", taint);<br>+       if ((taint & mask) != 0) {<br>+               tst_res(TINFO, "Kernel is already tainted: %u", taint);<br>+               taint_check = 0;<br>+       }<br> }<br> <br> <br>@@ -98,5 +101,5 @@ unsigned int tst_taint_check(void)<br> <br>        taint = tst_taint_read();<br> <br>-       return (taint & taint_mask);<br>+       return (taint & taint_mask & taint_check);<br> }<br><div class="gmail_default" style="font-size:small"></div></div></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>