<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 Thu, Sep 19, 2019 at 6:02 PM 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">Hi!<br>
> 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>
There is a reason for generating TBROK, we do not want the test to be<br>
skipped silently in this case. If kernel is tainted something went wrong<br>
anyways and we are looking only for a specific flags.<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">That's true. The TBROK can highlight the tainted kernel as a good reminder to the tester.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">But in a real testing situation, LTP sometimes not being run in the first, after we go here cve-2017-17053, the kernel very probably has already been tainted and reported by other tests. So this new TBROK will drive tester to double-check if something wrong as new. </div></div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Imagine that, if there are many test cases invoke a tainted checking(e.g. TST_TAINT_W) in LTP, then all of them will be skipped and report TBORK on such tainted-kernel, that seems not to be friendly to the tester to review the result.</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>