linux-kernel - Re: [PATCH 0/2 v2] Syscalls tracing

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20090315152532.GC5105@nowhere>
Date:	Sun, 15 Mar 2009 16:25:33 +0100
From:	Frederic Weisbecker <fweisbec@...il.com>
To:	Ingo Molnar <mingo@...e.hu>
Cc:	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
	Peter Zijlstra <a.p.zijlstra@...llo.nl>,
	Steven Rostedt <rostedt@...dmis.org>, tglx@...utronix.de,
	Jason Baron <jbaron@...hat.com>,
	"Frank Ch. Eigler" <fche@...hat.com>,
	Mathieu Desnoyers <mathieu.desnoyers@...ymtl.ca>,
	KOSAKI Motohiro <kosaki.motohiro@...fujitsu.com>,
	Lai Jiangshan <laijs@...fujitsu.com>,
	Jiaying Zhang <jiayingz@...gle.com>,
	Michael Rubin <mrubin@...gle.com>,
	Martin Bligh <mbligh@...gle.com>,
	Michael Davidson <md@...gle.com>
Subject: Re: [PATCH 0/2 v2] Syscalls tracing

On Fri, Mar 13, 2009 at 05:17:49PM +0100, Ingo Molnar wrote:
> 
> * Ingo Molnar <mingo@...e.hu> wrote:
> 
> > 
> > * Frederic Weisbecker <fweisbec@...il.com> wrote:
> > 
> > > On Fri, Mar 13, 2009 at 03:42:10PM +0100, Frederic Weisbecker wrote:
> > > > tracing/syscalls: core infrastructure to trace syscalls
> > > > 
> > > > This new iteration addresses a good part of the previous reviews.
> > > 
> > > 
> > > Ah I just discovered that you applied the previous version 
> > > today. But the v2 is not a delta :-s
> > > 
> > > I can rebase them but not until Sunday.
> > 
> > No problem, i'll deltify them and will have a look.
> 
> Ok, i did the deltas, tidied them up and put them into 
> tip:tracing/syscalls.
> 
> Nice stuff! Here's some sample output:
> 
> aldebaran:/debug/tracing> head trace
> # tracer: syscall
> #
> #           TASK-PID    CPU#    TIMESTAMP  FUNCTION
> #              | |       |          |         |
>            <...>-4405  [003]   188.452934: sys_dup2(oldfd: a, newfd: 1) 
>            <...>-4405  [003]   188.452939: sys_dup2 -> 0x1
>            <...>-4405  [003]   188.452940: sys_fcntl(fd: a, cmd: 1, arg: 0) 
>            <...>-4405  [003]   188.452941: sys_fcntl -> 0x1
>            <...>-4405  [003]   188.452942: sys_close(fd: a) 
>            <...>-4405  [003]   188.452943: sys_close -> 0x0
> 
> A suggestion:
> 
>  - Would be nice for all the registered syscalls to show up
>    under /debug/events/syscalls/, one directory per syscall,
>    with an 'enable' and a 'format' file as well.


Nice idea.
It is somehow planned since I want to let the user export the traces
through their binary values (so they will need the format).

 
> And a bugreport:
> 
>  - when using function_graph (after having used the syscall 
>    tracer) i dont see graph traces anymore - only the syscall 
>    trace entries:


Ah exact, it seems I forgot to reset the buffer while switching the tracer.


> # tracer: function_graph
> #
> # CPU  DURATION                  FUNCTION CALLS
> # |     |   |                     |   |   |   |
> ##### CPU 9 buffer started ####
>  at-spi-registry-3063  [009]   322.915058: sys_read -> 0x5a8
>  at-spi-registry-3063  [009]   322.915059: sys_write(fd: 6, buf: 632840, count: 5a8) 
>  at-spi-registry-3063  [009]   322.915062: sys_write -> 0x5a8
>  at-spi-registry-3063  [009]   322.915062: sys_read(fd: 4, buf: 632840, count: 40000) 
> 
> That's not intended, right?


Indeed it's not. I'm cooking the fix.

Thanks.


> 	Ingo

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/