File srp_daemon-Don-t-create-async_ev_thread-if-only... of Package rdma-core

Overview Repositories Revisions Requests Users Attributes Meta

File srp_daemon-Don-t-create-async_ev_thread-if-only-run-once.patch of Package rdma-core (Revision 4c6ce202f0e223cd0f953535eee3fd76)

Currently displaying revision 4c6ce202f0e223cd0f953535eee3fd76 , Show latest

commit b1a51eeee28c14dbba332cf59a0e85a182374ed6
Author: Honggang Li <honli@redhat.com>
Date:   Wed Dec 20 03:09:58 2017 +0800

srp_daemon: Don't create async_ev_thread if only run once
    
    fd3005f0cd34 moves the signal handler setup from ibsrpdm path. So,
    default signal handler will be used when the main pthread send signal
    SIGINT to pthread async_ev_thread. ibsrpdm will exit with non-zero
    exit code as default signal handler killed it. ibsrpdm should return
    with exit code zero, if no error emerged.
    
    We should not create async_ev_thread for ibsrpdm.
    
    Fixes: fd3005f0cd34 ("srp_daemon: Move the setup of the wakeup_pipe after openlog")
    Reviewed-by: Bart Van Assche <bart.vanassche@wdc.com>
    Signed-off-by: Honggang Li <honli@redhat.com>

diff --git srp_daemon/srp_daemon.c srp_daemon/srp_daemon.c
index 36df5c3bfe79..a7e7807774c5 100644
--- srp_daemon/srp_daemon.c
+++ srp_daemon/srp_daemon.c
@@ -1945,12 +1945,12 @@ static struct resources *alloc_res(void)
 				     run_thread_get_trap_notices, &res->res);
 		if (ret)
 			goto err;
-	}
 
-	ret = pthread_create(&res->res.async_ev_thread, NULL,
-			     run_thread_listen_to_events, &res->res);
-	if (ret)
-		goto err;
+		ret = pthread_create(&res->res.async_ev_thread, NULL,
+				     run_thread_listen_to_events, &res->res);
+		if (ret)
+			goto err;
+	}
 
 	if (config->retry_timeout && !config->once) {
 		ret = pthread_create(&res->res.reconnect_thread, NULL,

Places

File srp_daemon-Don-t-create-async_ev_thread-if-only-run-once.patch of Package rdma-core (Revision 4c6ce202f0e223cd0f953535eee3fd76)

Places