linux-kernel - Re: [PATCH v2 3/3] ftrace: add tracepoint for itimer

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: <alpine.LFD.2.00.0907062117590.19480@localhost.localdomain>
Date:	Mon, 6 Jul 2009 21:37:14 +0200 (CEST)
From:	Thomas Gleixner <tglx@...utronix.de>
To:	Mathieu Desnoyers <mathieu.desnoyers@...ymtl.ca>
cc:	Xiao Guangrong <xiaoguangrong@...fujitsu.com>,
	Ingo Molnar <mingo@...e.hu>,
	Steven Rostedt <rostedt@...dmis.org>,
	Frederic Weisbecker <fweisbec@...il.com>,
	Zhaolei <zhaolei@...fujitsu.com>, kosaki.motohiro@...fujitsu.com,
	LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v2 3/3] ftrace: add tracepoint for itimer

On Mon, 6 Jul 2009, Mathieu Desnoyers wrote:
> * Xiao Guangrong (xiaoguangrong@...fujitsu.com) wrote:
> > +
> > +TRACE_EVENT(itimer_state,
> > +
> 
> Hrm, I'd reserve the use of the *_state namespace for something
> different if possible.

See below.

> In LTTng, we have the ability to list a set of data structures at trace
> start (we call this state dump). In this case, we could add a list of
> all active timers at trace start. This is useful if the trace started
> after timer inits.

Can you please spare us the gory details of LTTng. We already know
that it can do everything plus brewing coffee, so no need to advertise
it in every tracing related mail.

> So having "_state" without much meaning here might eventually be
> confusing.
> 
> "_action" or "_call" (like IPCs) might be more appropriate ?

Err, where is this an action or a call ? Care to look at the call site
of this tracepoint. It's at the end of do_setitimer() and records the
_state_ of the itimer _after_ the syscall was handled.

> Mathieu

Can you please remove the hundred lines you are not commenting on next
time ?
 
> > +	TP_PROTO(int which, struct itimerval *value, cputime_t expires),

Thanks,

	tglx
--
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/