From 919b7398627ba9efc053cd8cc48861c772028e9a Mon Sep 17 00:00:00 2001
From: Christopher Faulet <cfaulet@haproxy.com>
Date: Tue, 14 Nov 2017 10:17:48 +0100
Subject: [PATCH] CLEANUP: tasks: Remove useless double test on rq_next

No backport is needed, this is purely 1.8-specific.
---
 src/task.c | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/src/task.c b/src/task.c
index 4f9b049ca..6e7b9be55 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);