From: Jens Axboe <jens.axboe@oracle.com>
Date: Wed, 18 Apr 2007 10:25:41 +0000 (+0200)
Subject: Revert "We do need to send a SIGQUIT to a process/thread, even if it is running"
X-Git-Tag: fio-1.16~6
X-Git-Url: https://git.kernel.dk/?p=fio.git;a=commitdiff_plain;h=7a93ab190e5fea78edb21affeb0189a9d61a2cd8;hp=90a87d4ba3acb26cb54e9b15adf9f39aea02818b

Revert "We do need to send a SIGQUIT to a process/thread, even if it is running"

This reverts commit ef5fffa591e583de26c6ce14f9448104d6dfbe79.

The problem with the commit is that if we SIGQUIT a thread, it
gets exited without updating runtime/etc info. And then we
don't get a new status display on exit.

Signed-off-by: Jens Axboe <jens.axboe@oracle.com>
---

diff --git a/fio.c b/fio.c
index 565b3d22..d8c14333 100644
--- a/fio.c
+++ b/fio.c
@@ -70,7 +70,10 @@ static void terminate_threads(int group_id)
 
 	for_each_td(td, i) {
 		if (group_id == TERMINATE_ALL || groupid == td->groupid) {
-			if (td->runstate <= TD_RUNNING)
+			/*
+			 * if the thread is running, just let it exit
+			 */
+			if (td->runstate < TD_RUNNING)
 				kill(td->pid, SIGQUIT);
 			td->terminate = 1;
 			td->o.start_delay = 0;