Revert "Introduce LTTNG_UST_LOADED environment variable"

This reverts commit 2315088b93f925c5cccee865ccb8f6bc542e52dc.

We will introduce a solution based on a weak symbol instead, which does
not "leak" to children's exec.

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

diff --git a/doc/man/lttng-ust.3.txt b/doc/man/lttng-ust.3.txt
index a1d4f204c26ff47367902bdd04d2c1f6b355fe46..7a23943daf9eb134f46dba17459a120fcaf3d219 100644 (file)
--- a/doc/man/lttng-ust.3.txt
+++ b/doc/man/lttng-ust.3.txt
@@ -1113,15 +1113,6 @@ are located in a specific directory under `$LTTNG_HOME` (or `$HOME` if
     documentation under
     https://github.com/lttng/lttng-ust/tree/master/doc/examples/getcpu-override[`examples/getcpu-override`].
 
-`LTTNG_UST_LOADED`::
-    Set internally by `liblttng-ust`'s constructor. Can be used by
-    applications to detect if lttng-ust is loaded, even if liblttng-ust
-    is not directly linked by the application. This environment
-    variable stays set across fork and exec, so an application could
-    observe this environment variable being set if a parent process had
-    lttng-ust loaded when issuing fork, even if this application does
-    not have lttng-ust in its address space.
-
 `LTTNG_UST_REGISTER_TIMEOUT`::
     Waiting time for the _registration done_ session daemon command
     before proceeding to execute the main program (milliseconds).

diff --git a/liblttng-ust/lttng-ust-comm.c b/liblttng-ust/lttng-ust-comm.c
index cc9b6b5e1e8706a1b941de1b0d09ea37dddaffd3..6105403a25d032b618df9c6202644ff138a0a034 100644 (file)
--- a/liblttng-ust/lttng-ust-comm.c
+++ b/liblttng-ust/lttng-ust-comm.c
@@ -1547,13 +1547,6 @@ void lttng_ust_malloc_wrapper_init(void)
 {
 }
 
-static
-void init_ust_env(void)
-{
-       if (putenv("LTTNG_UST_LOADED=1"))
-               DBG("Error setting LTTNG_UST_LOADED environment variable");
-}
-
 /*
  * sessiond monitoring thread: monitor presence of global and per-user
  * sessiond by polling the application common named pipe.
@@ -1581,8 +1574,6 @@ void __attribute__((constructor)) lttng_ust_init(void)
        lttng_fixup_procname_tls();
        lttng_fixup_ust_mutex_nest_tls();
 
-       init_ust_env();
-
        /*
         * We want precise control over the order in which we construct
         * our sub-libraries vs starting to receive commands from