From: Christopher Faulet <cfaulet@haproxy.com>
Date: Tue, 14 Nov 2017 09:17:48 +0000 (+0100)
Subject: CLEANUP: tasks: Remove useless double test on rq_next
X-Git-Tag: v1.8-rc4~23
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=919b7398627ba9efc053cd8cc48861c772028e9a;p=thirdparty%2Fhaproxy.git

CLEANUP: tasks: Remove useless double test on rq_next

No backport is needed, this is purely 1.8-specific.
---

diff --git a/src/task.c b/src/task.c
index 4f9b049ca5..6e7b9be552 100644
--- a/src/task.c
+++ b/src/task.c
@@ -268,11 +268,9 @@ void process_runnable_tasks()
 				 * the last half. Let's loop back to the beginning
 				 * of the tree now.
 				 */
-				if (unlikely(!rq_next)) {
-					rq_next = eb32sc_first(&rqueue, tid_bit);
-					if (!rq_next)
-						break;
-				}
+				rq_next = eb32sc_first(&rqueue, tid_bit);
+				if (!rq_next)
+					break;
 			}
 
 			t = eb32sc_entry(rq_next, struct task, rq);