From: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Date: Mon, 14 Mar 2022 17:31:24 +0000 (-0400)
Subject: Document expected ISO8601 time formats in ABI header
X-Git-Tag: v2.13.3~1
X-Git-Url: https://git.lttng.org./?a=commitdiff_plain;h=a8d1e4ee2344fc4c532aba558665eff50e99213e;p=lttng-modules.git

Document expected ISO8601 time formats in ABI header

Document the expected ISO8601 time formats in the ABI header to justify
the choice of string maximum length.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Change-Id: I4dedde83b5fb81c376245338773ea63677401a09
---

diff --git a/include/lttng/abi.h b/include/lttng/abi.h
index 40317e86..02bcefae 100644
--- a/include/lttng/abi.h
+++ b/include/lttng/abi.h
@@ -22,6 +22,13 @@
 
 #define LTTNG_KERNEL_ABI_SYM_NAME_LEN		256
 #define LTTNG_KERNEL_ABI_SESSION_NAME_LEN	256
+
+/*
+ * The expected iso8601 time formats are either:
+ *
+ * - YYYYmmddTHHMMSS+HHMM           (20 characters + \0)
+ * - YYYY-mm-ddTHH:MM:SS+HH:MM      (25 characters + \0)
+ */
 #define LTTNG_KERNEL_ABI_SESSION_CREATION_TIME_ISO8601_LEN	26
 
 enum lttng_kernel_abi_instrumentation {