[lttng-dev] [PATCH] Fix: set thread in daemon mode

[Mathieu Desnoyers]

merged, thanks!

Mathieu

----- Original Message -----
> From: "David Goulet" <dgoulet at efficios.com>
> To: lttng-dev at lists.lttng.org
> Cc: "mathieu desnoyers" <mathieu.desnoyers at efficios.com>, "David Goulet" <dgoulet at efficios.com>
> Sent: Thursday, June 26, 2014 3:07:54 PM
> Subject: [PATCH] Fix: set thread in daemon mode
> 
> This way, when the JVM stops the main thread, the agent's thread will be
> automatically shutdown thus making the .dispose() call optional.
> 
> Signed-off-by: David Goulet <dgoulet at efficios.com>
> ---
>  liblttng-ust-jul/org/lttng/ust/jul/LTTngAgent.java | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/liblttng-ust-jul/org/lttng/ust/jul/LTTngAgent.java
> b/liblttng-ust-jul/org/lttng/ust/jul/LTTngAgent.java
> index 2485be2..6717cd1 100644
> --- a/liblttng-ust-jul/org/lttng/ust/jul/LTTngAgent.java
> +++ b/liblttng-ust-jul/org/lttng/ust/jul/LTTngAgent.java
> @@ -128,6 +128,7 @@ public class LTTngAgent {
>  		this.lttngThreadUser = new LTTngThread(this.sessiondAddr,
>  				this.lttngHandlerUser, this.registerSem);
>  		this.sessiondThUser = new Thread(lttngThreadUser);
> +		this.sessiondThUser.setDaemon(true);
>  		this.sessiondThUser.start();
>  		/* Wait for registration done of per-user sessiond */
>  		nr_acquires++;
> @@ -136,6 +137,7 @@ public class LTTngAgent {
>  		this.lttngThreadRoot = new LTTngThread(this.sessiondAddr,
>  				this.lttngHandlerRoot, this.registerSem);
>  		this.sessiondThRoot = new Thread(lttngThreadRoot);
> +		this.sessiondThRoot.setDaemon(true);
>  		this.sessiondThRoot.start();
>  		/* Wait for registration done of system-wide sessiond */
>  		nr_acquires++;
> --
> 2.0.0
> 
> 

-- 
Mathieu Desnoyers
EfficiOS Inc.
http://www.efficios.com