From: Erwan Velu <erwanaliasr1@gmail.com>
Date: Wed, 8 Sep 2021 20:43:39 +0000 (+0200)
Subject: engines/sg: Removing useless variable assignment
X-Git-Tag: fio-3.29~104^2~3
X-Git-Url: https://git.kernel.dk/?a=commitdiff_plain;h=691df596922c1e74f74e769269254e0a091ffdc2;p=fio.git

engines/sg: Removing useless variable assignment

ret is set to -1 but the break statement will not use this value.

So let's remove this useless assignment which could be confusing.

Signed-off-by: Erwan Velu <erwanaliasr1@gmail.com>
---

diff --git a/engines/sg.c b/engines/sg.c
index 592c4488..1c019384 100644
--- a/engines/sg.c
+++ b/engines/sg.c
@@ -471,10 +471,9 @@ static enum fio_q_status fio_sgio_rw_doio(struct thread_data *td,
 			if (__io_u == io_u)
 				break;
 
-			if (io_u_sync_complete(td, __io_u)) {
-				ret = -1;
+			if (io_u_sync_complete(td, __io_u))
 				break;
-			}
+
 		} while (1);
 
 		return FIO_Q_COMPLETED;