[GSOC 2014][JSON] stacktrace in JSON format

2014-06-03 Thread Masatake YAMATO
Hi, YangMin I have not read the thread about your work, JSON format because I have concentrated on unwind(stacktrace) patch. (The patch is finally merge into the master tree!) Will your work conver the output of stacktrace? If yes, please, read the rest of this mail. Currently the stacktrace

[PATCH v4] Decode paths associated with file descriptors returned by syscalls

2014-06-03 Thread zubin . mithra
From: Zubin Mithra * defs.h (RVAL_FD): New macro. * defs.h (RVAL_MASK, RVAL_STR, RVAL_NONE): Macro values modified. * desc.c (sys_dup): New function. * desc.c (sys_delete_module): New function. * desc.c (do_dup2, decode_open, sys_creat): Modified to return RVAL_FD. * linux/dummy.h (sys_delete_m

Re: [GSOC 2014][JSON] refactor the output code to support JSON format

2014-06-03 Thread Philippe Ombredanne
On Tue, Jun 3, 2014 at 5:43 PM, yangmin zhu wrote: > I'm not sure whether should I send my patches directly to this mailing > list or just put these patch in my github > and we discuss here? > > you can always find my latest commits here: > https://github.com/zym0017d/strace_GSOC/commits/master Y

Re: [PATCH v3] Decode paths associated with file descriptors returned by syscalls

2014-06-03 Thread Dmitry V. Levin
On Tue, Jun 03, 2014 at 09:08:23PM +0530, [email protected] wrote: [...] > --- a/file.c > +++ b/file.c > @@ -309,6 +309,12 @@ decode_open(struct tcb *tcp, int offset) > int > sys_open(struct tcb *tcp) > { > + decode_open(tcp, 0); > + return RVAL_FD; > +} > + > +int sys_delete_module

Re: [GSOC 2014][JSON] refactor the output code to support JSON format

2014-06-03 Thread yangmin zhu
On Sat, May 31, 2014 at 8:21 AM, Dmitry V. Levin wrote: > > YangMin, please enable pre-commit hook on your side, it will stop > whitespace errors that make all your commits look red: > > $ mv .git/hooks/pre-commit{.sample,} > I had cleared all the white space error in my commit history when made

[PATCH v3] Decode paths associated with file descriptors returned by syscalls

2014-06-03 Thread zubin . mithra
From: Zubin Mithra * defs.h (RVAL_FD): New macro. * defs.h (RVAL_MASK, RVAL_STR, RVAL_NONE): Macro values modified. * desc.c (sys_dup): New function. * desc.c (sys_delete_module): New function. * desc.c (do_dup2, sys_open): Modified to return RVAL_FD. * linux/dummy.h (sys_delete_module, sys_dup):

Re: [PATCH v2] Decode paths associated with file descriptors returned by syscalls

2014-06-03 Thread Dmitry V. Levin
On Tue, Jun 03, 2014 at 08:21:06PM +0530, Zubin Mithra wrote: > > Same here, lets change decode_open to return RVAL_FD instead. > > > >> +int sys_delete_module(struct tcb *tcp) > >> +{ > >> return decode_open(tcp, 0); > >> } > > > > btw, it's not right for sys_delete_module to use decode_ope

Re: [PATCH v2] Decode paths associated with file descriptors returned by syscalls

2014-06-03 Thread Zubin Mithra
> > Same here, lets change decode_open to return RVAL_FD instead. > >> +int sys_delete_module(struct tcb *tcp) >> +{ >> return decode_open(tcp, 0); >> } > > btw, it's not right for sys_delete_module to use decode_open, > but that's a subject for another patch. > I see -- so do I create a du

Re: [PATCH v2] Decode paths associated with file descriptors returned by syscalls

2014-06-03 Thread Dmitry V. Levin
On Tue, Jun 03, 2014 at 09:02:39AM +0530, [email protected] wrote: > From: Zubin Mithra > > * defs.h (RVAL_FD): New macro. > * defs.h (RVAL_MASK, RVAL_STR, RVAL_NONE): Macro values modified. > * desc.c (sys_dup): New function. > * desc.c (sys_delete_module): New function. > * desc.c (sys_ope

[PATCH v2] v4l2.c: a new (incomplete) decoder for Video4Linux ioctls args

2014-06-03 Thread Philippe De Muyter
* v4l2.c, xlat/v4l2_*.in: new files. * configure.ac: check for availabilty of v4l2 enum constants. * ioctl.c, defs.h, Makefile.am: hook v4l2.o:v4l2_ioctl into strace. * NEWS: spread the news :) Introduce v4l2.c, a decoder for a subset of the v4l2 ioctl's args. This is a combination of - previous w