<div dir="ltr"><div>Hey if i am taking rt_sigretrun,nanosleep as negative testcases i am getting sigusr and sigalarm respectively, i should not get them right?  Also sched.test is also failing for them. Well i have included fstat and open as negatives.<br><br></div><div class="gmail_extra">The way these test is made which expect the last output to be <br>'+++ exited with 0 +++'  -> this is causing the test failure for some tests from the suite.Well should i work to make this test better or there is something else which i am not taking into consideration?<br><br></div><div class="gmail_extra">Also if i am thinking of making this test better by not depending on the making a manual ending like:<br>echo '+++ exited with 0 +++' > "$EXP" <br><br></div><div class="gmail_extra">And the patch is attached.<br></div><div class="gmail_extra"><br></div><div class="gmail_extra"><div class="gmail_quote">On Sat, Mar 18, 2017 at 6:44 AM, Dmitry V. Levin <span dir="ltr"><<a href="mailto:ldv@altlinux.org" target="_blank">ldv@altlinux.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Thu, Mar 16, 2017 at 04:41:22PM +0530, Rishi Bhatt wrote:<br>
[...]<br>
> diff --git a/sysent.h b/sysent.h<br>
> index f4eaa930..7e95ab3a 100644<br>
> --- a/sysent.h<br>
> +++ b/sysent.h<br>
> @@ -22,5 +22,5 @@ typedef struct sysent {<br>
>  #define TRACE_INDIRECT_SUBCALL       02000   /* Syscall is an indirect socket/ipc subcall. */<br>
>  #define COMPAT_SYSCALL_TYPES 04000   /* A compat syscall that uses compat types. */<br>
>  #define TRACE_SCHED          010000  /* Trace scheduler-related syscalls. */<br>
> -<br>
> +#define TRACE_CLOCK          020000  /*Trace clock-related syscalls. */<br>
>  #endif /* !STRACE_SYSENT_H */<br>
<br>
Please do not remove the empty line between #define's and #endif.<br>
Please follow the style used in the file you are patching,<br>
in particular, the style of comments.<br>
<br>
> diff --git a/tests/clock.test b/tests/clock.test<br>
> new file mode 100755<br>
> index 00000000..8b2750c8<br>
> --- /dev/null<br>
> +++ b/tests/clock.test<br>
> @@ -0,0 +1,48 @@<br>
> +#!/bin/sh<br>
> +# Check -e trace=%clock option.<br>
> +<br>
> +try_run_prog()<br>
> +{<br>
> +     "$@" > /dev/null || {<br>
> +             rc=$?<br>
> +             if [ $rc -eq 77 ]; then<br>
> +                     return 1<br>
> +             else<br>
> +                     fail_ "$* failed with code $rc"<br>
> +             fi<br>
> +     }<br>
> +}<br>
> +<br>
> +. "${srcdir=.}/init.sh"<br>
<br>
Eugene has moved try_run_prog() from sched.test to init.sh,<br>
please drop it from clock.test, too.<br>
<br>
> +<br>
> +#<br>
> +# Tracing Clock_* related syscalls.<br>
<br>
These are clock_* related syscalls, not Clock_*.<br>
<br>
> +#<br>
> +while read w i<br>
> +do<br>
> +     try_run_prog ./$i > /dev/null<br>
<br>
The idiom is<br>
        try_run_prog "./$i" || continue<br>
<br>
> +     run_strace -a$w -e trace=%clock ./$i > "$EXP"<br>
> +     match_diff "$LOG" "$EXP"<br>
> +done << EOF<br>
> +40 clock_nanosleep<br>
> +37 clock_adjtime<br>
> +36 clock_xettime<br>
> +EOF<br>
> +<br>
> +echo '+++ exited with 0 +++' > "$EXP"<br>
> +<br>
> +#<br>
> +# Tracing NON_CLOCK related syscalls<br>
<br>
What is NON_CLOCK related?<br>
I understood what you are trying to say here just because I knew in<br>
advance.<br>
<br>
[...]<br>
> +done << EOF<br>
> +11 fchdir<br>
> +27 futex<br>
> +10 fsync<br>
> +EOF<br>
<br>
Could you add a few more of these to the list, please?<br>
<span class="gmail-HOEnZb"><font color="#888888"><br>
<br>
--<br>
ldv<br>
</font></span><br>------------------------------<wbr>------------------------------<wbr>------------------<br>
Check out the vibrant tech community on one of the world's most<br>
engaging tech sites, Slashdot.org! <a href="http://sdm.link/slashdot" rel="noreferrer" target="_blank">http://sdm.link/slashdot</a><br>______________________________<wbr>_________________<br>
Strace-devel mailing list<br>
<a href="mailto:Strace-devel@lists.sourceforge.net">Strace-devel@lists.<wbr>sourceforge.net</a><br>
<a href="https://lists.sourceforge.net/lists/listinfo/strace-devel" rel="noreferrer" target="_blank">https://lists.sourceforge.net/<wbr>lists/listinfo/strace-devel</a><br>
<br></blockquote></div><br></div></div>