From db126b3ee1e0a0b6aa95787f9ca4b1f4d12c271c Mon Sep 17 00:00:00 2001
From: Kozlowski Mateusz <mateusz.kozlowski@intel.com>
Date: Wed, 18 Dec 2019 06:41:14 -0500
Subject: [PATCH] lib/app: Change signal handler to use
 spdk_thread_send_critical_msg

Fixes deadlock when closing application with SIGTERM during call of
spdk_thread_send_msg.

Fixes #1105

Signed-off-by: Kozlowski Mateusz <mateusz.kozlowski@intel.com>
Change-Id: If191550f9681a30674fc9c0704289fcdb70b33cb
Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/478291
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
Reviewed-by: Jim Harris <james.r.harris@intel.com>
Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
---
 lib/event/app.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/event/app.c b/lib/event/app.c
index ac05626c2d..404b881761 100644
--- a/lib/event/app.c
+++ b/lib/event/app.c
@@ -235,7 +235,7 @@ app_start_shutdown(void *ctx)
 void
 spdk_app_start_shutdown(void)
 {
-	spdk_thread_send_msg(g_app_thread, app_start_shutdown, NULL);
+	spdk_thread_send_critical_msg(g_app_thread, app_start_shutdown);
 }
 
 static void