Fix: move wait loop increment before first conditional block

[urcu.git] / urcu-qsbr.c

diff --git a/urcu-qsbr.c b/urcu-qsbr.c
index cd3beff7337da0225a3c55f36b19e9499ec052e9..71e7a39e6f73795026e054bf8f0620772ec9a70f 100644 (file)
--- a/urcu-qsbr.c
+++ b/urcu-qsbr.c
@@ -130,6 +130,8 @@ static void wait_for_readers(struct cds_list_head *input_readers,
         * current rcu_gp.ctr value.
         */
        for (;;) {
         * current rcu_gp.ctr value.
         */
        for (;;) {

+               if (wait_loops < RCU_QS_ACTIVE_ATTEMPTS)
+                       wait_loops++;

                if (wait_loops >= RCU_QS_ACTIVE_ATTEMPTS) {
                        uatomic_set(&rcu_gp.futex, -1);
                        /*
                if (wait_loops >= RCU_QS_ACTIVE_ATTEMPTS) {
                        uatomic_set(&rcu_gp.futex, -1);
                        /*


@@ -142,8 +144,6 @@ static void wait_for_readers(struct cds_list_head *input_readers,
                        }
                        /* Write futex before read reader_gp */
                        cmm_smp_mb();
                        }
                        /* Write futex before read reader_gp */
                        cmm_smp_mb();

-               } else {
-                       wait_loops++;

                }
                cds_list_for_each_entry_safe(index, tmp, input_readers, node) {
                        switch (rcu_reader_state(&index->ctr)) {
                }
                cds_list_for_each_entry_safe(index, tmp, input_readers, node) {
                        switch (rcu_reader_state(&index->ctr)) {