Reduce lagSpikeMillis to 200. Fix lag spike profiling never ending

MinimallyCorrect · Feb 14, 2016 · 3723fda · 3723fda
1 parent e4bab6e
commit 3723fda
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/src/main/java/nallar/tickprofiler/minecraft/profiling/LagSpikeProfiler.java b/src/main/java/nallar/tickprofiler/minecraft/profiling/LagSpikeProfiler.java
@@ -17,8 +17,8 @@
 
 public class LagSpikeProfiler {
     private static final StackTraceElement[] EMPTY_STACK_TRACE = new StackTraceElement[0];
-    private static final int deadLockMillis = 400;
-    private static final long deadLockNanoSeconds = TimeUnit.SECONDS.toNanos(deadLockMillis);
+    private static final int lagSpikeMillis = 200;
+    private static final long lagSpikeNanoSeconds = TimeUnit.SECONDS.toNanos(lagSpikeMillis);
     private static boolean inProgress;
     private static volatile long lastTickTime = 0;
     private final ICommandSender commandSender;
@@ -189,14 +189,17 @@ private static String toString(ThreadInfo threadInfo, boolean name) {
     }
 
     private void start() {
-        final int sleepTime = Math.max(1000, (deadLockMillis * 1000) / 6);
+        final int sleepTime = Math.max(1000, (lagSpikeMillis * 1000) / 6);
         Thread detectorThread = new Thread(new Runnable() {
             @Override
             public void run() {
                 trySleep(60000);
                 while (!stopping && checkForLagSpikes()) {
                     trySleep(sleepTime);
                 }
+                synchronized (LagSpikeProfiler.class) {
+                    inProgress = false;
+                }
             }
         });
         detectorThread.setName("Lag Spike Detector");
@@ -214,7 +217,7 @@ boolean checkForLagSpikes() {
             return false;
 
         long deadTime = time - lastTickTime;
-        if (deadTime < deadLockNanoSeconds) {
+        if (deadTime < lagSpikeNanoSeconds) {
             detected = false;
             return true;
         }