CDB update condition fix

proteus-core · Feb 7, 2025 · c49f06e · c49f06e
1 parent a804ba0
commit c49f06e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/src/main/scala/riscv/plugins/scheduling/dynamic/LoadManager.scala b/src/main/scala/riscv/plugins/scheduling/dynamic/LoadManager.scala
@@ -89,6 +89,7 @@ class LoadManager(
     when(state === State.EXECUTING && loadStage.arbitration.isDone && !activeFlush) {
       rdbStream.valid := True
       rdbStream.payload.robIndex := storedMessage.robIndex
+      rdbStream.payload.willCdbUpdate := True
       for (register <- retirementRegisters.keys) {
         rdbStream.payload.registerMap.element(register) := loadStage.output(register)
       }

diff --git a/src/main/scala/riscv/plugins/scheduling/dynamic/ReservationStation.scala b/src/main/scala/riscv/plugins/scheduling/dynamic/ReservationStation.scala
@@ -171,7 +171,9 @@ class ReservationStation(
         cdbStream.valid := True
       }
 
-      dispatchStream.payload.willCdbUpdate := cdbStream.valid
+      dispatchStream.payload.willCdbUpdate := cdbStream.valid || pipeline
+        .service[LsuService]
+        .operationOutput(exeStage) === LsuOperationType.LOAD
       dispatchStream.valid := True
 
       // Override the assignment of resultCdbMessage to make sure data can be sent in later cycles