Re: GSOC:Microproject

2018-02-23 Thread Eugene Syromyatnikov
On Fri, Feb 23, 2018 at 8:47 AM, Masatake YAMATO  wrote:
> How about allowing a user to define ones' own class in ~/.strace?

By the way, since you've mentioned that, we've been thinking about
some options for configuration file for quite some time (not only for
custom group generation, but also for color output configuration, lua
invasiveness and other not-yet-merged stuff). Does anyone have any
ideas regarding a good configuration file syntax?

The other question is about prefix for user-configurable groups, it
probably shouldn't conflict with any existing syscall qualification
syntax, but, for example, "%%%" looks too unwieldy.

-- 
Eugene Syromyatnikov
mailto:evg...@gmail.com
xmpp:esyr@jabber.{ru|org}

--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
___
Strace-devel mailing list
Strace-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/strace-devel


Re: GSOC:Microproject

2018-02-23 Thread Masatake YAMATO
How about allowing a user to define ones' own class in ~/.strace?

(cat ~/.strace)

classes:
- name: xattr
  - syscall: setxattr
  - syscall: lsetxattr
  - syscall: fsetxattr
  - syscall: getxattr
  - syscall: listxattr
  - syscall: removexattr
- name: setxattr
  - syscall: setxattr
  - syscall: lsetxattr
  - syscall: fsetxattr

So you can do -e trace=%xattr or -e traec=%setxattr as you want.

However, I think we cannot say thie extension a microproject:-P

Masatake YAMATO



> On Thu, 15 Feb 2018 19:07:59 +0530
> Abhishek Tiwari  wrote:
> 
>> Hi,
>> 
>> I want to work on microproject
>> 
>> extend -e trace=class syntax with new classes
>> 
>> 
>> I have gone through mailing list. Is following grouping ok
>> 
>> {,l,f}setxattr   TRACE_SETXATTR  TFX
>> 
>> 
>> -e trace=%setxattr
> 
> This is a good start, but from what I see, maybe this group is too
> small, maybe all the syscalls that deals with xattr should be a better
> idea, something like -e trace=%xattr.
> 
> 
> -- 
> Gabriel Laskar
> 
> --
> Check out the vibrant tech community on one of the world's most
> engaging tech sites, Slashdot.org! http://sdm.link/slashdot
> ___
> Strace-devel mailing list
> Strace-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/strace-devel

--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
___
Strace-devel mailing list
Strace-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/strace-devel


Re: GSOC:Microproject

2018-02-23 Thread Gabriel Laskar
On Thu, 15 Feb 2018 19:07:59 +0530
Abhishek Tiwari  wrote:

> Hi,
> 
> I want to work on microproject
> 
> extend -e trace=class syntax with new classes
> 
> 
> I have gone through mailing list. Is following grouping ok
> 
> {,l,f}setxattr   TRACE_SETXATTR  TFX
> 
> 
> -e trace=%setxattr

This is a good start, but from what I see, maybe this group is too
small, maybe all the syscalls that deals with xattr should be a better
idea, something like -e trace=%xattr.


-- 
Gabriel Laskar

--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
___
Strace-devel mailing list
Strace-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/strace-devel


Re: GSOC:Microproject

2018-02-22 Thread Eugene Syromiatnikov
On Thu, Feb 15, 2018 at 07:07:59PM +0530, Abhishek Tiwari wrote:
> Hi,
> 
> I want to work on microproject
> 
> extend -e trace=class syntax with new classes
> 
> 
> I have gone through mailing list. Is following grouping ok
> 
> {,l,f}setxattr   TRACE_SETXATTR  TFX
Doesn't -e trace='/[lf]?setxattr' cover precisely this set of syscalls
on any architecture?

> 
> 
> -e trace=%setxattr
> 
> Thank you

--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
___
Strace-devel mailing list
Strace-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/strace-devel


Re: GSOC:Microproject

2018-02-21 Thread Abhishek Tiwari
On Mon, Feb 19, 2018 at 3:43 PM, Gabriel Laskar 
wrote:

> On Thu, 15 Feb 2018 19:07:59 +0530
> Abhishek Tiwari  wrote:
>
> > Hi,
> >
> > I want to work on microproject
> >
> > extend -e trace=class syntax with new classes
> >
> >
> > I have gone through mailing list. Is following grouping ok
> >
> > {,l,f}setxattr   TRACE_SETXATTR  TFX
> >
> >
> > -e trace=%setxattr
>
> This is a good start, but from what I see, maybe this group is too
> small, maybe all the syscalls that deals with xattr should be a better
> idea, something like -e trace=%xattr.
>
>
> --
> Gabriel Laskar
>

I have grouped all xattr syscalls and following is attached patch for the
same. Please review.


-- 
Abhishek Tiwari
M.Tech (CS),
IIT Kharagpur.
From d35b8770dc1e14bdc020dd4d5825e352c55b5af3 Mon Sep 17 00:00:00 2001
From: Abhishek Tiwari 
Date: Wed, 21 Feb 2018 23:46:00 +0530
Subject: [PATCH] Implement -e trace=%xattr option

linux/*/syscallent*.h part is modified automatically by:

sed -i 's/TF\(,[[:space:]]*SEN.*xattr\)/TF|TFX\1/' \
linux/*/syscallent*.h

sed -i 's/TD\(,[[:space:]]*SEN.*xattr\)/TD|TFX\1/' \
linux/*/syscallent*.h

* NEWS: Mention this change
* linux/32/syscallent.h: Add TFX flag.
* linux/64/syscallent.h: Likewise.
* linux/alpha/syscallent.h: Likewise.
* linux/arm/syscallent.h: Likewise.
* linux/avr32/syscallent.h: Likewise.
* linux/bfin/syscallent.h: Likewise.
* linux/hppa/syscallent.h: Likewise.
* linux/i386/syscallent.h: Likewise.
* linux/ia64/syscallent.h: Likewise.
* linux/m68k/syscallent.h: Likewise.
* linux/microblaze/syscallent.h: Likewise.
* linux/mips/syscallent-n32.h: Likewise.
* linux/mips/syscallent-n64.h: Likewise.
* linux/mips/syscallent-o32.h: Likewise.
* linux/powerpc/syscallent.h: Likewise.
* linux/powerpc64/syscallent.h: Likewise.
* linux/s390/syscallent.h: Likewise.
* linux/s390x/syscallent.h: Likewise.
* linux/sh/syscallent.h: Likewise.
* linux/sh64/syscallent.h: Likewise.
* linux/sparc/syscallent.h: Likewise.
* linux/sparc64/syscallent.h: Likewise.
* linux/x32/syscallent.h: Likewise.
* linux/x86_64/syscallent.h: Likewise.
* linux/xtensa/syscallent.h: Likewise.
* strace.1.in (.SS Filtering): Add information about %xattr syscall class.
* sysent.h (TRACE_XATTR): New macro.
* sysent_shorthand_defs.h: Add TFX shorthand for TRACE_XATTR
* tests/Makefile.am (DECODER_TESTS): Add it.
* tests/trace_xattr.in: New test.
---
 NEWS  |  2 ++
 linux/32/syscallent.h | 24 
 linux/64/syscallent.h | 24 
 linux/alpha/syscallent.h  | 24 
 linux/arm/syscallent.h| 24 
 linux/avr32/syscallent.h  | 24 
 linux/bfin/syscallent.h   | 24 
 linux/hppa/syscallent.h   | 24 
 linux/i386/syscallent.h   | 24 
 linux/ia64/syscallent.h   | 24 
 linux/m68k/syscallent.h   | 24 
 linux/microblaze/syscallent.h | 24 
 linux/mips/syscallent-n32.h   | 24 
 linux/mips/syscallent-n64.h   | 24 
 linux/mips/syscallent-o32.h   | 24 
 linux/powerpc/syscallent.h| 24 
 linux/powerpc64/syscallent.h  | 24 
 linux/s390/syscallent.h   | 24 
 linux/s390x/syscallent.h  | 24 
 linux/sh/syscallent.h | 24 
 linux/sh64/syscallent.h   | 24 
 linux/sparc/syscallent.h  | 24 
 linux/sparc64/syscallent.h| 24 
 linux/x32/syscallent.h| 24 
 linux/x86_64/syscallent.h | 24 
 linux/xtensa/syscallent.h | 24 
 strace.1.in   |  3 +++
 sysent.h  |  1 +
 sysent_shorthand_defs.h   |  2 ++
 tests/Makefile.am |  1 +
 tests/trace_xattr.in  | 12 
 31 files changed, 321 insertions(+), 300 deletions(-)
 create mode 100644 tests/trace_xattr.in

diff --git a/NEWS b/NEWS
index 0e74e044..2b82d534 100644
--- a/NEWS
+++ b/NEWS
@@ -217,6 +217,8 @@ Noteworthy changes in release 4.17 (2017-05-24)
 syscalls.
   * Added -e trace=%%statfs option for tracing syscalls used for querying file
 system statistics information (statfs-like, fstatfs-like and ustat).
+  * Added -e trace=%xattr option for tracing syscalls used for modifying file
+attributes (xattr-like).
   * Added -e trace=/regex option for filtering syscalls using regular
 expressions.
   * Added support for prepending of question mark before syscall qualification
diff --git a/linux/32/syscallent.h b/linux/32/syscallent.h
index b53bf88f..44167c08 100644
--- a/linux/32/syscallent.h
++

Re: GSOC:Microproject

2018-02-18 Thread Abhishek Tiwari
please do reply

On Thu, Feb 15, 2018 at 7:07 PM, Abhishek Tiwari <
erabhishektiwar...@gmail.com> wrote:

> Hi,
>
> I want to work on microproject
>
> extend -e trace=class syntax with new classes
>
>
> I have gone through mailing list. Is following grouping ok
>
> {,l,f}setxattr   TRACE_SETXATTR  TFX
>
>
> -e trace=%setxattr
>
> Thank you
> --
> Abhishek Tiwari
> M.Tech (CS),
> IIT Kharagpur.
>



-- 
Abhishek Tiwari
M.Tech (CS),
IIT Kharagpur.
--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot___
Strace-devel mailing list
Strace-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/strace-devel