Re: [PATCH] Added -e trace=scheduler option for tracing scheduler related syscalls.

2016-03-29 Thread haris iqbal
On Wed, Mar 16, 2016 at 11:14 PM, Dmitry V. Levin wrote: > On Tue, Mar 15, 2016 at 12:01:47PM +0530, student wrote: >> From: Md Haris Iqbal >> >> --- >> NEWS | 1 + >> defs.h| 1 + >> linux/32/syscallent.h | 24 >

Re: [PATCH] Added -e trace=scheduler option for tracing scheduler related syscalls.

2016-03-29 Thread haris iqbal
On Sat, Mar 19, 2016 at 5:24 AM, Dmitry V. Levin wrote: > On Fri, Mar 18, 2016 at 07:21:24PM +0100, Gabriel Laskar wrote: >> I still feel that we need to have a special character before the >> classes, in order to be sure that it will not collide with any other >> syscall. >> >> Also, 'scheduler'

Re: [PATCH 2/4] tests: add option_u.test

2016-03-29 Thread Dmitry V. Levin
On Tue, Mar 29, 2016 at 07:06:09PM +0800, "Zhou, Wenjian/周文剑" wrote: > On 03/25/2016 06:48 AM, Dmitry V. Levin wrote: > > On Wed, Mar 23, 2016 at 04:56:37PM +0800, Zhou Wenjian wrote: > >> --- /dev/null > >> +++ b/tests/option_u.test > >> @@ -0,0 +1,13 @@ > >> +#!/bin/sh > >> + > >> +# Check option

Re: Regarding enforcing coding standard

2016-03-29 Thread Dmitry V. Levin
Hi, On Tue, Mar 29, 2016 at 05:09:58PM +0530, Jay Joshi wrote: > Hi, > > I just came across style checking script for patches used by linux > kernel: http://lxr.free-electrons.com/source/scripts/checkpatch.pl. > > Some modifications will be required to fit strace's coding > standards(like switch

Re: Regarding enforcing coding standard

2016-03-29 Thread Philippe Ombredanne
On Tue, Mar 29, 2016 at 1:39 PM, Jay Joshi wrote: > Hi, > > I just came across style checking script for patches used by linux > kernel: http://lxr.free-electrons.com/source/scripts/checkpatch.pl. > > Some modifications will be required to fit strace's coding > standards(like switch case indentati

Regarding enforcing coding standard

2016-03-29 Thread Jay Joshi
Hi, I just came across style checking script for patches used by linux kernel: http://lxr.free-electrons.com/source/scripts/checkpatch.pl. Some modifications will be required to fit strace's coding standards(like switch case indentations), but I don't think that it's hard. I think 20 days should

Re: [PATCH 2/4] tests: add option_u.test

2016-03-29 Thread Zhou, Wenjian/周文剑
On 03/25/2016 06:48 AM, Dmitry V. Levin wrote: > On Wed, Mar 23, 2016 at 04:56:37PM +0800, Zhou Wenjian wrote: >> --- /dev/null >> +++ b/tests/option_u.test >> @@ -0,0 +1,13 @@ >> +#!/bin/sh >> + >> +# Check option -u. >> + >> +. "${srcdir=.}/init.sh" >> + >> +user=$(tail -n 1 /etc/passwd | awk -F