From 27c9d6eceb15b8a4e400fd2b053fb84d0db10da4 Mon Sep 17 00:00:00 2001
From: Paul Elder <paul.elder@ideasonboard.com>
Date: Wed, 28 Oct 2020 19:10:27 +0900
Subject: libcamera: request: Add tracepoints

Add and use tracepoints in Request. Requests are core to libcamera
operation, thus detecting delays in their processing is important, and
serves as a good usage example of tracepoints.

Signed-off-by: Paul Elder <paul.elder@ideasonboard.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
---
 .../libcamera/internal/tracepoints/buffer_enums.tp |  9 +++
 include/libcamera/internal/tracepoints/meson.build |  8 +++
 include/libcamera/internal/tracepoints/pipeline.tp | 25 ++++++++
 include/libcamera/internal/tracepoints/request.tp  | 68 ++++++++++++++++++++++
 .../internal/tracepoints/request_enums.tp          |  9 +++
 5 files changed, 119 insertions(+)
 create mode 100644 include/libcamera/internal/tracepoints/buffer_enums.tp
 create mode 100644 include/libcamera/internal/tracepoints/pipeline.tp
 create mode 100644 include/libcamera/internal/tracepoints/request.tp
 create mode 100644 include/libcamera/internal/tracepoints/request_enums.tp

(limited to 'include/libcamera')

diff --git a/include/libcamera/internal/tracepoints/buffer_enums.tp b/include/libcamera/internal/tracepoints/buffer_enums.tp
new file mode 100644
index 00000000..fdbea69a
--- /dev/null
+++ b/include/libcamera/internal/tracepoints/buffer_enums.tp
@@ -0,0 +1,9 @@
+TRACEPOINT_ENUM(
+	libcamera,
+	buffer_status,
+	TP_ENUM_VALUES(
+		ctf_enum_value("FrameSuccess", 0)
+		ctf_enum_value("FrameError", 1)
+		ctf_enum_value("FrameCancelled", 2)
+	)
+)
diff --git a/include/libcamera/internal/tracepoints/meson.build b/include/libcamera/internal/tracepoints/meson.build
index 2dd6733e..d9b2fca5 100644
--- a/include/libcamera/internal/tracepoints/meson.build
+++ b/include/libcamera/internal/tracepoints/meson.build
@@ -1,4 +1,12 @@
 # SPDX-License-Identifier: CC0-1.0
 
+# enum files must go first
 tracepoint_files = files([
+    'buffer_enums.tp',
+    'request_enums.tp',
+])
+
+tracepoint_files += files([
+    'pipeline.tp',
+    'request.tp',
 ])
diff --git a/include/libcamera/internal/tracepoints/pipeline.tp b/include/libcamera/internal/tracepoints/pipeline.tp
new file mode 100644
index 00000000..c0a94635
--- /dev/null
+++ b/include/libcamera/internal/tracepoints/pipeline.tp
@@ -0,0 +1,25 @@
+TRACEPOINT_EVENT(
+	libcamera,
+	ipa_call_begin,
+	TP_ARGS(
+		const char *, pipe,
+		const char *, func
+	),
+	TP_FIELDS(
+		ctf_string(pipeline_name, pipe)
+		ctf_string(function_name, func)
+	)
+)
+
+TRACEPOINT_EVENT(
+	libcamera,
+	ipa_call_end,
+	TP_ARGS(
+		const char *, pipe,
+		const char *, func
+	),
+	TP_FIELDS(
+		ctf_string(pipeline_name, pipe)
+		ctf_string(function_name, func)
+	)
+)
diff --git a/include/libcamera/internal/tracepoints/request.tp b/include/libcamera/internal/tracepoints/request.tp
new file mode 100644
index 00000000..dd28a714
--- /dev/null
+++ b/include/libcamera/internal/tracepoints/request.tp
@@ -0,0 +1,68 @@
+#include <libcamera/buffer.h>
+#include <libcamera/request.h>
+
+TRACEPOINT_EVENT_CLASS(
+	libcamera,
+	request,
+	TP_ARGS(
+		libcamera::Request *, req
+	),
+	TP_FIELDS(
+		ctf_integer_hex(uintptr_t, request, reinterpret_cast<uintptr_t>(req))
+		ctf_integer(uint64_t, cookie, req->cookie())
+		ctf_enum(libcamera, request_status, uint32_t, status, req->status())
+	)
+)
+
+TRACEPOINT_EVENT_INSTANCE(
+	libcamera,
+	request,
+	request_construct,
+	TP_ARGS(
+		libcamera::Request *, req
+	)
+)
+
+TRACEPOINT_EVENT_INSTANCE(
+	libcamera,
+	request,
+	request_destroy,
+	TP_ARGS(
+		libcamera::Request *, req
+	)
+)
+
+TRACEPOINT_EVENT_INSTANCE(
+	libcamera,
+	request,
+	request_reuse,
+	TP_ARGS(
+		libcamera::Request *, req
+	)
+)
+
+TRACEPOINT_EVENT_INSTANCE(
+	libcamera,
+	request,
+	request_complete,
+	TP_ARGS(
+		libcamera::Request *, req
+	)
+)
+
+
+TRACEPOINT_EVENT(
+	libcamera,
+	request_complete_buffer,
+	TP_ARGS(
+		libcamera::Request *, req,
+		libcamera::FrameBuffer *, buf
+	),
+	TP_FIELDS(
+		ctf_integer_hex(uintptr_t, request, reinterpret_cast<uintptr_t>(req))
+		ctf_integer(uint64_t, cookie, req->cookie())
+		ctf_integer(int, status, req->status())
+		ctf_integer_hex(uintptr_t, buffer, reinterpret_cast<uintptr_t>(buf))
+		ctf_enum(libcamera, buffer_status, uint32_t, buf_status, buf->metadata().status)
+	)
+)
diff --git a/include/libcamera/internal/tracepoints/request_enums.tp b/include/libcamera/internal/tracepoints/request_enums.tp
new file mode 100644
index 00000000..371f6544
--- /dev/null
+++ b/include/libcamera/internal/tracepoints/request_enums.tp
@@ -0,0 +1,9 @@
+TRACEPOINT_ENUM(
+	libcamera,
+	request_status,
+	TP_ENUM_VALUES(
+		ctf_enum_value("RequestPending", 0)
+		ctf_enum_value("RequestComplete", 1)
+		ctf_enum_value("RequestCancelled", 2)
+	)
+)
-- 
cgit v1.2.1