Hello community, here is the log from the commit of package rdma-core for openSUSE:Factory checked in at 2018-01-06 18:46:21 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/rdma-core (Old) and /work/SRC/openSUSE:Factory/.rdma-core.new (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "rdma-core" Sat Jan 6 18:46:21 2018 rev:16 rq:561307 version:16 Changes: -------- --- /work/SRC/openSUSE:Factory/rdma-core/rdma-core.changes 2017-12-23 12:18:12.355329377 +0100 +++ /work/SRC/openSUSE:Factory/.rdma-core.new/rdma-core.changes 2018-01-06 18:46:22.255132944 +0100 @@ -1,0 +2,6 @@ +Wed Jan 3 09:17:10 UTC 2018 - [email protected] + +- Add srp_daemon-Don-t-create-async_ev_thread-if-only-run-once.patch + to fix bad exit code of ibsrpdm (bsc#1074390) + +------------------------------------------------------------------- New: ---- srp_daemon-Don-t-create-async_ev_thread-if-only-run-once.patch ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ rdma-core.spec ++++++ --- /var/tmp/diff_new_pack.1ysiSe/_old 2018-01-06 18:46:22.971099490 +0100 +++ /var/tmp/diff_new_pack.1ysiSe/_new 2018-01-06 18:46:22.975099303 +0100 @@ -1,7 +1,7 @@ # # spec file for package rdma-core # -# Copyright (c) 2017 SUSE LINUX GmbH, Nuernberg, Germany. +# Copyright (c) 2018 SUSE LINUX GmbH, Nuernberg, Germany. # # All modifications and additions to the file contributed by third parties # remain the property of their copyright owners, unless otherwise agreed @@ -52,6 +52,7 @@ Source1: baselibs.conf Patch1: srp_daemon-handle-SM-lid-change.patch Patch2: srp_daemon-fix-CQ-handling.patch +Patch3: srp_daemon-Don-t-create-async_ev_thread-if-only-run-once.patch BuildRequires: binutils BuildRequires: cmake >= 2.8.11 BuildRequires: gcc @@ -329,6 +330,7 @@ %setup -q -n %{name}-%{version}%{git_ver} %patch1 %patch2 +%patch3 %build ++++++ srp_daemon-Don-t-create-async_ev_thread-if-only-run-once.patch ++++++ commit b1a51eeee28c14dbba332cf59a0e85a182374ed6 Author: Honggang Li <[email protected]> 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 <[email protected]> Signed-off-by: Honggang Li <[email protected]> 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,
