<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, Jul 4, 2019 at 4:08 PM Jan Stancek <<a href="mailto:jstancek@redhat.com">jstancek@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"><br>
I'd rephrase it to:<br>
<br>
   If a crash (e.g. triggered by signal SIGSEGV) is expected in testing, you<br>
   can avoid creation of core files by calling tst_no_corefile() function.<br>
   This takes effect for process (and its children) which invoked it, unless<br>
   they subsequently modify RLIMIT_CORE.<br>
<br>
   Note that LTP library will reap any processes that test didn't reap itself,<br>
   and report any non-zero exit code as failure.<br></blockquote><div><br></div><div class="gmail_default" style="font-size:small">This looks better, I missed the effect to its children before.</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">
> +static inline void tst_no_corefile(int verbose)<br>
> +{<br>
> +       struct rlimit r;<br>
> +<br>
> +       r.rlim_cur = 1;<br>
> +       r.rlim_max = 1;<br>
> +       SAFE_SETRLIMIT(RLIMIT_CORE, &r);<br>
<br>
SAFE_SETRLIMIT is fine if needs_root = 1. But if test runs as unprivileged user<br>
and RLIMIT_CORE is already 0, unprivileged user won't be able to increase it,<br>
so we get TBROK here.<br>
</blockquote></div><div class="gmail_default" style="font-size:small"></div><div class="gmail_default" style="font-size:small">You are right. The SAFE_SETRLIMIT is not good for that achievement. I will make use of the original type and do error handle with TWARN slightly. And then I think we probably need to move it out form tst_safe_mcro.h.</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>