Solved a small bug that let FlowNodes update while they are in a clos…

…ed state. (#283)
atlarge-research · Dec 12, 2024 · f55aaed · f55aaed
1 parent 8bbc3de
commit f55aaed
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/...mulator-compute/src/main/java/org/opendc/simulator/compute/workload/SimTraceWorkload.java b/...mulator-compute/src/main/java/org/opendc/simulator/compute/workload/SimTraceWorkload.java
@@ -245,6 +245,9 @@ public void addSupplierEdge(FlowEdge supplierEdge) {
      */
     @Override
     public void removeSupplierEdge(FlowEdge supplierEdge) {
+        if (this.machineEdge == null) {
+            return;
+        }
         this.stopWorkload();
     }
 }
diff --git a/...lator/opendc-simulator-flow/src/main/java/org/opendc/simulator/engine/graph/FlowNode.java b/...lator/opendc-simulator-flow/src/main/java/org/opendc/simulator/engine/graph/FlowNode.java
@@ -157,6 +157,11 @@ public void invalidate() {
      * Update the state of the stage.
      */
     public void update(long now) {
+        if (this.nodeState == NodeState.CLOSED) {
+            this.deadline = Long.MAX_VALUE;
+            return;
+        }
+
         this.nodeState = NodeState.UPDATING;
 
         long newDeadline = this.deadline;