<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Mar 31, 2017 at 5:58 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"><div class="gmail-m_6480072862466908899HOEnZb"><div class="gmail-m_6480072862466908899h5">On Tue, Mar 28, 2017 at 05:17:49AM +0300, Dmitry V. Levin wrote:<br>
> On Tue, Mar 28, 2017 at 03:42:04AM +0530, Rishi Bhatt wrote:<br>
> > From 294db67743eb8509383cf808877470<wbr>16271dd32a Mon Sep 17 00:00:00 2001<br>
> > From: Rishi Bhatt <<a href="mailto:bhatt.rishi062@gmail.com" target="_blank">bhatt.rishi062@gmail.com</a>><br>
> > Date: Tue, 28 Mar 2017 03:31:24 +0530<br>
> > Subject: [PATCH] Implement -e trace=%clock option<br>
> ><br>
> > linux/*/syscallent.h part is modified automatically by:<br>
> ><br>
> >     git grep -Fl 'SEN(clock_' linux/ | xargs sed -i \<br>
> >         's/0\(,[[:space:]]*SEN(clock_<wbr>\)/TCL\1/'<br>
> ><br>
> >     * sysent.h (TRACE_CLOCK): New definition.<br>
> >     * syscall.c: Alias TCL to TRACE_CLOCK around syscallent.h inclusion.<br>
> >     * linux/32/syscallent.h: Add TCL flag for clock_* sycalls.<br>
> >     * linux/64/syscallent.h: Likewise.<br>
> >     * linux/alpha/syscallent.h: Likewise.<br>
> >     * linux/arm/syscallent.h: Likewise.<br>
> >     * linux/avr32/syscallent.h: Likewise.<br>
> >     * linux/bfin/syscallent.h: Likewise.<br>
> >     * linux/crisv10/syscallent.h: Likewise.<br>
> >     * linux/hppa/syscallent.h: Likewise.<br>
> >     * linux/i386/syscallent.h: Likewise.<br>
> >     * linux/ia64/syscallent.h: Likewise.<br>
> >     * linux/m68k/syscallent.h: Likewise.<br>
> >     * linux/microblaze/syscallent.h: Likewise.<br>
> >     * linux/mips/syscallent-n32.h: Likewise.<br>
> >     * linux/mips/syscallent-n64.h: Likewise.<br>
> >     * linux/mips/syscallent-o32.h: Likewise.<br>
> >     * linux/powerpc/syscallent.h: Likewise.<br>
> >     * linux/powerpc64/syscallent.h: Likewise.<br>
> >     * linux/s390/syscallent.h: Likewise.<br>
> >     * linux/s390x/syscallent.h: Likewise.<br>
> >     * linux/sh/syscallent.h: Likewise.<br>
> >     * linux/sh64/syscallent.h: Likewise.<br>
> >     * linux/sparc/syscallent.h: Likewise.<br>
> >     * linux/sparc64/syscallent.h: Likewise.<br>
> >     * linux/x32/syscallent.h: Likewise.<br>
> >     * linux/x86_64/syscallent.h: Likewise.<br>
> >     * linux/xtensa/syscallent.h: Likewise.<br>
> >     * qualify.c (lookup_class): Add TRACE_CLOCK for "%clock".<br>
> >     * tests/ksysent.c: Define TCL to 0.<br>
> >     * tests/nsyscalls.c: Likewise.<br>
> >     * tests/clock.test: New test.<br>
> >     * tests/Makefile.am (DECODER_TESTS): Add it.<br>
> >     * strace.1 (.SS Filtering): Add information about %clock syscall class.<br>
><br>
> Please don't indent it this way.<br>
> Besides that, the patch looks OK.<br>
<br>
</div></div>One more little thing I forgot to add: it's a user visible change so<br>
it needs a NEWS entry similar to trace=%sched.<br></blockquote><div><br></div><div>Should i write it below <br></div><div>"* Added -e trace=%sched option for tracing sched_* syscalls."<br></div><div>->here<br>...<br>...<br></div><div>or in the order in which they are added? <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-m_6480072862466908899HOEnZb"><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" target="_blank">Strace-devel@lists.sourceforge<wbr>.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>