diff mbox series

[v5,10/30] tools/lib/traceevent: Man page for tep_register_trace_clock()

Message ID 20190412133811.15878-11-tstoyanov@vmware.com (mailing list archive)
State Rejected
Headers show
Series Libtraceevent MAN pages | expand

Commit Message

Tzvetomir Stoyanov April 12, 2019, 1:37 p.m. UTC
Create man page for tep_register_trace_clock() libtraceevent API.

Signed-off-by: Tzvetomir Stoyanov <tstoyanov@vmware.com>
---
 .../Documentation/libtraceevent-reg_clock.txt | 77 +++++++++++++++++++
 1 file changed, 77 insertions(+)
 create mode 100644 tools/lib/traceevent/Documentation/libtraceevent-reg_clock.txt

Comments

Steven Rostedt April 26, 2019, 2:47 a.m. UTC | #1
On Fri, 12 Apr 2019 16:37:51 +0300
Tzvetomir Stoyanov <tstoyanov@vmware.com> wrote:

> Create man page for tep_register_trace_clock() libtraceevent API.
> 
> Signed-off-by: Tzvetomir Stoyanov <tstoyanov@vmware.com>
> ---
>  .../Documentation/libtraceevent-reg_clock.txt | 77 +++++++++++++++++++
>  1 file changed, 77 insertions(+)
>  create mode 100644 tools/lib/traceevent/Documentation/libtraceevent-reg_clock.txt
> 
> diff --git a/tools/lib/traceevent/Documentation/libtraceevent-reg_clock.txt b/tools/lib/traceevent/Documentation/libtraceevent-reg_clock.txt
> new file mode 100644
> index 000000000000..8bf6c5348fcf
> --- /dev/null
> +++ b/tools/lib/traceevent/Documentation/libtraceevent-reg_clock.txt
> @@ -0,0 +1,77 @@
> +libtraceevent(3)
> +================
> +
> +NAME
> +----
> +tep_register_trace_clock - Registers the clock, used for the timestamp of
> +the events.
> +
> +SYNOPSIS
> +--------
> +[verse]
> +--
> +*#include <event-parse.h>*
> +
> +int *tep_register_trace_clock*(struct tep_handle pass:[*]_tep_, const char pass:[*]_trace_clock_);
> +--
> +
> +DESCRIPTION
> +-----------
> +The _tep_register_trace_clock()_ function registers the clock, used for
> +the timestamps of the events. When an event is recorded, a timestamp is attached
> +to it. This timestamp comes from a specified clock. Supported clocks can be seen
> +in tracefs/trace_clock, the selected one is in square brackets.
> +The _tep_ argument is the trace event parser context. The _trace_clock_ argument
> +is the name of the clock, the string is copied internally.
> +
> +RETURN VALUE
> +------------
> +The _tep_register_trace_clock()_ function returns 0 in case of success.
> +If there is not enough memory to copy the _trace_clock_ string -1 is returned
> +and errno is set to ENOMEM.
>

Hmm, this is more code I think we should remove from libtraceevent, and
have the tools just set the tep flag to print in usecs or not. The
library shouldn't be setting policy.

The tools (perf, trace-cmd, etc) should just implement their own
tep_register_trace_clock() (not calling it that), and set the USEC flag
depending on what it wants.

-- Steve
diff mbox series

Patch

diff --git a/tools/lib/traceevent/Documentation/libtraceevent-reg_clock.txt b/tools/lib/traceevent/Documentation/libtraceevent-reg_clock.txt
new file mode 100644
index 000000000000..8bf6c5348fcf
--- /dev/null
+++ b/tools/lib/traceevent/Documentation/libtraceevent-reg_clock.txt
@@ -0,0 +1,77 @@ 
+libtraceevent(3)
+================
+
+NAME
+----
+tep_register_trace_clock - Registers the clock, used for the timestamp of
+the events.
+
+SYNOPSIS
+--------
+[verse]
+--
+*#include <event-parse.h>*
+
+int *tep_register_trace_clock*(struct tep_handle pass:[*]_tep_, const char pass:[*]_trace_clock_);
+--
+
+DESCRIPTION
+-----------
+The _tep_register_trace_clock()_ function registers the clock, used for
+the timestamps of the events. When an event is recorded, a timestamp is attached
+to it. This timestamp comes from a specified clock. Supported clocks can be seen
+in tracefs/trace_clock, the selected one is in square brackets.
+The _tep_ argument is the trace event parser context. The _trace_clock_ argument
+is the name of the clock, the string is copied internally.
+
+RETURN VALUE
+------------
+The _tep_register_trace_clock()_ function returns 0 in case of success.
+If there is not enough memory to copy the _trace_clock_ string -1 is returned
+and errno is set to ENOMEM.
+
+EXAMPLE
+-------
+[source,c]
+--
+#include <event-parse.h>
+...
+struct tep_handle *tep = tep_alloc();
+...
+	if(tep_register_trace_clock(tep, "local") != 0) {
+		/* Failed to register the clock */
+	}
+--
+
+FILES
+-----
+[verse]
+--
+*event-parse.h*
+	Header file to include in order to have access to the library APIs.
+*-ltraceevent*
+	Linker switch to add when building a program that uses the library.
+--
+
+SEE ALSO
+--------
+_libtraceevent(3)_, _trace-cmd(1)_
+
+AUTHOR
+------
+[verse]
+--
+*Steven Rostedt* <rostedt@goodmis.org>, author of *libtraceevent*.
+*Tzvetomir Stoyanov* <tz.stoyanov@gmail.com>, author of this man page.
+--
+REPORTING BUGS
+--------------
+Report bugs to  <linux-trace-devel@vger.kernel.org>
+
+LICENSE
+-------
+libtraceevent is Free Software licensed under the GNU LGPL 2.1
+
+RESOURCES
+---------
+https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git