<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, Jul 22, 2020 at 11:52 PM Erico Nunes <<a href="mailto:ernunes@redhat.com">ernunes@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"><span class="gmail_default" style="font-size:small">...</span><br>
> So maybe we could simply do detect the /sys/kernel/../loackdown file as<br>
> your patch,<br>
> but adding an extra warning print when test failed on older than<br>
> kernel-v5.4.<br>
<br>
I like the idea of the warning. The only thing to consider is that the </blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
warning would also show up on all old kernels that don't even support</blockquote><div><div class="gmail_default" style="font-size:small"></div></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
lockdown and then don't have the file. So would you suggest this message<br>
to be something like a tst_res(TWARN, ...) or TINFO or some other less<br>
noisy way?<br></blockquote><div><div class="gmail_default" style="font-size:small"></div><div class="gmail_default" style="font-size:small">Thanks, but I did not suggest to show the warning in any system without the lockdown file. I mean if test getting FAIL on the no 'lockdown' file system, then could consider throwing a warning as a hint. </div><div class="gmail_default" style="font-size:small">And this could be achieved in the test case but not the library.</div></div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">For indicating the 'lockdown' file exist or no-exist, the 'TINFO' is enough.</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>