linux-kernel - Re: [RFC patch 14/32] trace event power remove semicolons

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: <BANLkTikdj8Wak1kS9L5KPpyGmbXwE4dxkA@mail.gmail.com>
Date:	Tue, 3 May 2011 14:59:37 +0200
From:	"Pihet-XID, Jean" <j-pihet@...com>
To:	Mathieu Desnoyers <mathieu.desnoyers@...icios.com>
Cc:	LKML <linux-kernel@...r.kernel.org>,
	Steven Rostedt <rostedt@...dmis.org>,
	Ingo Molnar <mingo@...e.hu>,
	Thomas Gleixner <tglx@...utronix.de>,
	Frederic Weisbecker <fweisbec@...il.com>,
	Arjan van de Ven <arjan@...ux.intel.com>,
	Thomas Renninger <trenn@...e.de>,
	Len Brown <len.brown@...el.com>
Subject: Re: [RFC patch 14/32] trace event power remove semicolons

On Mon, May 2, 2011 at 11:11 PM, Mathieu Desnoyers
<mathieu.desnoyers@...icios.com> wrote:
> Part of the gradual TRACE_EVENT() semicolon removal. Enables creation of array
> of events, thus saving space for trace event probes.
>
> Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@...icios.com>
> CC: Steven Rostedt <rostedt@...dmis.org>
> CC: Frederic Weisbecker <fweisbec@...il.com>
> CC: Ingo Molnar <mingo@...e.hu>
> CC: Thomas Gleixner <tglx@...utronix.de>
> CC: Jean Pihet <j-pihet@...com>
> CC: Arjan van de Ven <arjan@...ux.intel.com>
> CC: Thomas Renninger <trenn@...e.de>
> CC: Len Brown <len.brown@...el.com>
> ---
>  include/trace/events/power.h |   28 ++++++++++++++--------------
>  1 file changed, 14 insertions(+), 14 deletions(-)

Acked-by: j-pihet@...com

>
> Index: linux-2.6-lttng/include/trace/events/power.h
> ===================================================================
> --- linux-2.6-lttng.orig/include/trace/events/power.h
> +++ linux-2.6-lttng/include/trace/events/power.h
> @@ -25,14 +25,14 @@ DECLARE_EVENT_CLASS(cpu,
>
>        TP_printk("state=%lu cpu_id=%lu", (unsigned long)__entry->state,
>                  (unsigned long)__entry->cpu_id)
> -);
> +)
>
>  DEFINE_EVENT(cpu, cpu_idle,
>
>        TP_PROTO(unsigned int state, unsigned int cpu_id),
>
>        TP_ARGS(state, cpu_id)
> -);
> +)
>
>  /* This file can get included multiple times, TRACE_HEADER_MULTI_READ at top */
>  #ifndef _PWR_EVENT_AVOID_DOUBLE_DEFINING
> @@ -46,7 +46,7 @@ DEFINE_EVENT(cpu, cpu_frequency,
>        TP_PROTO(unsigned int frequency, unsigned int cpu_id),
>
>        TP_ARGS(frequency, cpu_id)
> -);
> +)
>
>  TRACE_EVENT(machine_suspend,
>
> @@ -63,7 +63,7 @@ TRACE_EVENT(machine_suspend,
>        ),
>
>        TP_printk("state=%lu", (unsigned long)__entry->state)
> -);
> +)
>
>  /* This code will be removed after deprecation time exceeded (2.6.41) */
>  #ifdef CONFIG_EVENT_POWER_TRACING_DEPRECATED
> @@ -92,21 +92,21 @@ DECLARE_EVENT_CLASS(power,
>
>        TP_printk("type=%lu state=%lu cpu_id=%lu", (unsigned long)__entry->type,
>                (unsigned long)__entry->state, (unsigned long)__entry->cpu_id)
> -);
> +)
>
>  DEFINE_EVENT(power, power_start,
>
>        TP_PROTO(unsigned int type, unsigned int state, unsigned int cpu_id),
>
>        TP_ARGS(type, state, cpu_id)
> -);
> +)
>
>  DEFINE_EVENT(power, power_frequency,
>
>        TP_PROTO(unsigned int type, unsigned int state, unsigned int cpu_id),
>
>        TP_ARGS(type, state, cpu_id)
> -);
> +)
>
>  TRACE_EVENT(power_end,
>
> @@ -124,7 +124,7 @@ TRACE_EVENT(power_end,
>
>        TP_printk("cpu_id=%lu", (unsigned long)__entry->cpu_id)
>
> -);
> +)
>
>  /* Deprecated dummy functions must be protected against multi-declartion */
>  #ifndef _PWR_EVENT_AVOID_DOUBLE_DEFINING_DEPRECATED
> @@ -180,28 +180,28 @@ DECLARE_EVENT_CLASS(clock,
>
>        TP_printk("%s state=%lu cpu_id=%lu", __get_str(name),
>                (unsigned long)__entry->state, (unsigned long)__entry->cpu_id)
> -);
> +)
>
>  DEFINE_EVENT(clock, clock_enable,
>
>        TP_PROTO(const char *name, unsigned int state, unsigned int cpu_id),
>
>        TP_ARGS(name, state, cpu_id)
> -);
> +)
>
>  DEFINE_EVENT(clock, clock_disable,
>
>        TP_PROTO(const char *name, unsigned int state, unsigned int cpu_id),
>
>        TP_ARGS(name, state, cpu_id)
> -);
> +)
>
>  DEFINE_EVENT(clock, clock_set_rate,
>
>        TP_PROTO(const char *name, unsigned int state, unsigned int cpu_id),
>
>        TP_ARGS(name, state, cpu_id)
> -);
> +)
>
>  /*
>  * The power domain events are used for power domains transitions
> @@ -226,14 +226,14 @@ DECLARE_EVENT_CLASS(power_domain,
>
>        TP_printk("%s state=%lu cpu_id=%lu", __get_str(name),
>                (unsigned long)__entry->state, (unsigned long)__entry->cpu_id)
> -);
> +)
>
>  DEFINE_EVENT(power_domain, power_domain_target,
>
>        TP_PROTO(const char *name, unsigned int state, unsigned int cpu_id),
>
>        TP_ARGS(name, state, cpu_id)
> -);
> +)
>  #endif /* _TRACE_POWER_H */
>
>  /* This part must be outside protection */
>
>
--
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/