Re: GSOC: Introduction

2017-03-15 Thread Abhishek Tiwari
On Wed, Mar 15, 2017 at 11:14 PM, Abhishek Tiwari wrote: > On Tue, Mar 14, 2017 at 8:44 AM, Abhishek Tiwari > wrote: >> On Tue, Mar 14, 2017 at 2:00 AM, Dmitry V. Levin wrote: >>> On Tue, Mar 14, 2017 at 12:40:46AM +0530, Abhishek Tiwari wrote: > Per-file summary is not in GNU change log fo

Re: [PATCH v2] Implemented -e trace=%clock option for clock_* related syscalls

2017-03-15 Thread Dmitry V. Levin
On Thu, Mar 16, 2017 at 12:17:08AM +0530, Rishi Bhatt wrote: > linux/*/syscallent.h part is modified automatically by: > > git grep -Fl 'SEN(clock_' linux/ | xargs sed -i \ > 's/0\(,[[:space:]]*SEN(clock_\)/TC\1/' I think TC is too short, let's use 3-letter abbreviations for new class

Re: Gsoc Introduction.

2017-03-15 Thread Dmitry V. Levin
On Wed, Mar 15, 2017 at 03:22:36PM +0700, Nikolay Marchuk wrote: > Hello, > I have implemented NS_* ioctls. Should I use common ioctlent interface to > print them with additional argument or should I print them without it? Why do you think printing additional arguments makes any sense? > From f03

Re: [PATCH v2] Decode RUSAGE_THREAD

2017-03-15 Thread Dmitry V. Levin
On Thu, Mar 16, 2017 at 01:01:40AM +0300, Victor Krapivensky wrote: > * xlat/usagewho.in: Add values for existing values, add RUSAGE_THREAD. > * tests/getrusage.c: Test decoding of RUSAGE_THREAD. > --- > tests/getrusage.c | 31 +-- > xlat/usagewho.in | 7 --- > 2

Re: [PATCH v5] Implement decoding of statx syscall

2017-03-15 Thread Dmitry V. Levin
On Wed, Mar 15, 2017 at 02:57:02PM +0300, Victor Krapivensky wrote: > * Makefile.am (strace_SOURCES): Add statx.c and stat.h. There is a shorter form for this: * statx.c: New file. * statx.h: Likewise. * Makefile.am (strace_SOURCES): Add them. > * linux/i386/syscallent.h: Add statx entry. > * li

Q: statfs related syscalls (was: Re: GSOC: Introduction)

2017-03-15 Thread Dmitry V. Levin
On Wed, Mar 15, 2017 at 11:14:29PM +0530, Abhishek Tiwari wrote: [...] > Is it fine to group statfs+ statfs64+ fstatfs + fstatfs64 + ustat as > %statfs or it should be (statfs+statfs64 + ustat) and > (fstatfs+ftsatfs64) i.e. two different classes ? Well, I don't have a ready answer to this questi

[PATCH v2] Decode RUSAGE_THREAD

2017-03-15 Thread Victor Krapivensky
* xlat/usagewho.in: Add values for existing values, add RUSAGE_THREAD. * tests/getrusage.c: Test decoding of RUSAGE_THREAD. --- tests/getrusage.c | 31 +-- xlat/usagewho.in | 7 --- 2 files changed, 29 insertions(+), 9 deletions(-) diff --git a/tests/getrusage.c

[PATCH v2-0003] Implemented -e trace=%clock option

2017-03-15 Thread Rishi Bhatt
Hey, i am unable to do git send-email,there is some issue. so i am attaching the patch here i have used git am -3 dont know why git am is failing. From e9b229032356ea85edd256269acdaaf71b74827e Mon Sep 17 00:00:00 2001 From: Rishi Bhatt Date: Wed, 15 Mar 2017 19:06:03 +0530 Subject: [PATCH v2] I

Re: GSOC: Introduction

2017-03-15 Thread Abhishek Tiwari
On Tue, Mar 14, 2017 at 8:44 AM, Abhishek Tiwari wrote: > On Tue, Mar 14, 2017 at 2:00 AM, Dmitry V. Levin wrote: >> On Tue, Mar 14, 2017 at 12:40:46AM +0530, Abhishek Tiwari wrote: >>> > Per-file summary is not in GNU change log format, please refer to >>> > https://www.gnu.org/prep/standards/ht

Re: [PATCH] Decode RUSAGE_THREAD

2017-03-15 Thread Eugene Syromyatnikov
Please do not answer in an existing thread and start a new thread for a new separate patch. On Wed, Mar 15, 2017 at 3:25 PM, Victor Krapivensky wrote: > * xlat/usagewho.in: Add RUSAGE_THREAD. > * tests/getrusage.c: Test decoding of RUSAGE_THREAD. > --- > tests/getrusage.c | 21 +++---

[PATCH] Decode RUSAGE_THREAD

2017-03-15 Thread Victor Krapivensky
* xlat/usagewho.in: Add RUSAGE_THREAD. * tests/getrusage.c: Test decoding of RUSAGE_THREAD. --- tests/getrusage.c | 21 +++-- xlat/usagewho.in | 1 + 2 files changed, 16 insertions(+), 6 deletions(-) diff --git a/tests/getrusage.c b/tests/getrusage.c index 8b76eff..ba1a7ef 10064

Re: Implemented -e trace=%clock option

2017-03-15 Thread Dmitry V. Levin
On Wed, Mar 15, 2017 at 05:53:37PM +0530, Rishi Bhatt wrote: > Sorry i have sent a wrong one. > This is the right one. Not really. ;) > Well i have changed permission of clock.test to 100755 so i guess we should > not get a permission denied error now and also those whitespace are also > eliminat

[PATCH v5] Implement decoding of statx syscall

2017-03-15 Thread Victor Krapivensky
* Makefile.am (strace_SOURCES): Add statx.c and stat.h. * linux/i386/syscallent.h: Add statx entry. * linux/x32/syscallent.h: Likewise. * linux/x86_64/syscallent.h: Likewise. * pathtrace.c (pathtrace_match): Handle SEN_statx. * statx.c: New file. * statx.h: Likewise. * tests/.gitignore: Add statx.

Re: Gsoc Introduction.

2017-03-15 Thread Nikolay Marchuk
Hello, I have implemented NS_* ioctls. Should I use common ioctlent interface to print them with additional argument or should I print them without it? On Tue, Mar 14, 2017 at 9:21 AM, Dmitry V. Levin wrote: > Hi, > > On Tue, Mar 14, 2017 at 08:35:45AM +0700, Николай Марчук wrote: > > Hello, > >