From: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Date: Wed, 22 Oct 2014 11:55:05 +0000 (-0400)
Subject: Fix: lfstack reversed empty/non-empty return value
X-Git-Url: https://git.lttng.org./?a=commitdiff_plain;h=d23382b83c64023453d212919d3807582da3f50b;p=userspace-rcu.git

Fix: lfstack reversed empty/non-empty return value

The return value of lfstack push operation is logically reversed
compared to the documentation, and compared to wfstack and wfcqueue.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
---

diff --git a/urcu/static/lfstack.h b/urcu/static/lfstack.h
index 9531de5..4799bc4 100644
--- a/urcu/static/lfstack.h
+++ b/urcu/static/lfstack.h
@@ -160,7 +160,7 @@ bool _cds_lfs_push(cds_lfs_stack_ptr_t u_s,
 		if (old_head == head)
 			break;
 	}
-	return ___cds_lfs_empty_head(head);
+	return !___cds_lfs_empty_head(head);
 }
 
 /*