415062 SelectorManager wakeup optimisation

Some further cleanup after sbordet review

1 files changed, 2 insertions, 3 deletions

diff --git a/jetty-io/src/main/java/org/eclipse/jetty/io/SelectorManager.java b/jetty-io/src/main/java/org/eclipse/jetty/io/SelectorManager.java
index 6db2e74453..8645e5edc5 100644
--- a/jetty-io/src/main/java/org/eclipse/jetty/io/SelectorManager.java
+++ b/jetty-io/src/main/java/org/eclipse/jetty/io/SelectorManager.java
@@ -366,12 +366,11 @@ public abstract class SelectorManager extends AbstractLifeCycle implements Dumpa
             if (_thread==Thread.currentThread())
             {
                 // If we are already iterating over the changes, just add this change to the list.
-                // No race here because it is this thread that is iterating over the changes.
-                if (_state.compareAndSet(SelectorState.CHANGING,SelectorState.MORE_CHANGES))
+                if (_state.get()==SelectorState.CHANGING)
                     _changes.offer(change);
                 else
                 {
-                    // Otherwise we run the queued changes
+                    // Otherwise we run the queued changes, list should mostly be empty
                     runChanges();
                     // and then directly run the passed change
                     runChange(change);