Update DPI for tick/generate split

src/main/resources/csrc/SimEmulator.cc

@@ -5,27 +5,27 @@
 #include <stdint.h>
 
 extern "C" void emulator_init_rs(int num_lanes);
-
+extern "C" void emulator_tick_rs(uint8_t *vec_d_ready, uint8_t *vec_d_valid,
+                                 uint8_t *vec_d_is_store, int *vec_d_size);
 extern "C" void emulator_generate_rs(uint8_t *vec_a_ready, uint8_t *vec_a_valid,
                                      long long *vec_a_address,
                                      uint8_t *vec_a_is_store, int *vec_a_size,
-                                    long long *vec_a_data, uint8_t *vec_d_ready,
-                                    uint8_t *vec_d_valid,
-                                    uint8_t *vec_d_is_store, int *vec_d_size,
-                                    uint8_t inflight, uint8_t *finished);
+                                     long long *vec_a_data,
+                                     uint8_t *vec_d_ready, uint8_t inflight,
+                                     uint8_t *finished);
 
-extern "C" void emulator_init(int num_lanes) {
-  emulator_init_rs(num_lanes);
+extern "C" void emulator_init(int num_lanes) { emulator_init_rs(num_lanes); }
+
+extern "C" void emulator_tick(uint8_t *vec_d_ready, uint8_t *vec_d_valid,
+                              uint8_t *vec_d_is_store, int *vec_d_size) {
+  emulator_tick_rs(vec_d_ready, vec_d_valid, vec_d_is_store, vec_d_size);
 }
 
 extern "C" void emulator_generate(uint8_t *vec_a_ready, uint8_t *vec_a_valid,
                                   long long *vec_a_address,
                                   uint8_t *vec_a_is_store, int *vec_a_size,
                                   long long *vec_a_data, uint8_t *vec_d_ready,
-                                 uint8_t *vec_d_valid, uint8_t *vec_d_is_store,
-                                 int *vec_d_size, uint8_t inflight,
-                                 uint8_t *finished) {
+                                  uint8_t inflight, uint8_t *finished) {
   emulator_generate_rs(vec_a_ready, vec_a_valid, vec_a_address, vec_a_is_store,
-                      vec_a_size, vec_a_data, vec_d_ready, vec_d_valid,
-                      vec_d_is_store, vec_d_size, inflight, finished);
+                       vec_a_size, vec_a_data, vec_d_ready, inflight, finished);
 }

src/main/resources/vsrc/SimEmulator.v

@@ -8,6 +8,14 @@ import "DPI-C" function void emulator_init(
 // (1) import "DPI-C" declaration
 // (2) C function declaration
 // (3) DPI function calls inside initial/always blocks
+import "DPI-C" function void emulator_tick
+(
+  output bit     vec_d_ready[`MAX_NUM_LANES],
+  input  bit     vec_d_valid[`MAX_NUM_LANES],
+  input  bit     vec_d_is_store[`MAX_NUM_LANES],
+  input  int     vec_d_size[`MAX_NUM_LANES]
+);
+
 import "DPI-C" function void emulator_generate
 (
   input  bit     vec_a_ready[`MAX_NUM_LANES],
@@ -16,11 +24,7 @@ import "DPI-C" function void emulator_generate
   output bit     vec_a_is_store[`MAX_NUM_LANES],
   output int     vec_a_size[`MAX_NUM_LANES],
   output longint vec_a_data[`MAX_NUM_LANES],
-
   output bit     vec_d_ready[`MAX_NUM_LANES],
-  input  bit     vec_d_valid[`MAX_NUM_LANES],
-  input  bit     vec_d_is_store[`MAX_NUM_LANES],
-  input  int     vec_d_size[`MAX_NUM_LANES],
 
   input  bit     inflight,
   output bit     finished
@@ -88,10 +92,7 @@ module SimEmulator #(parameter NUM_LANES = 4) (
     emulator_init(NUM_LANES);
   end
 
-  // negedge is important here; the DPI logic is essentially functioning as
-  // a combinational logic, so we want to reflect the signal change from DPI
-  // at the *current* cycle, not the next.
-  always @(negedge clock) begin
+  always @(posedge clock) begin
     if (reset) begin
       for (integer tid = 0; tid < NUM_LANES; tid = tid + 1) begin
         __in_a_valid[tid]    = 1'b0;
@@ -110,23 +111,29 @@ module SimEmulator #(parameter NUM_LANES = 4) (
         __in_a_is_store,
         __in_a_size,
         __in_a_data,
-
         __in_d_ready,
-        __out_d_valid,
-        __out_d_is_store,
-        __out_d_size,
 
         __out_inflight,
         __in_finished
       );
-      for (integer tid = 0; tid < NUM_LANES; tid = tid + 1) begin
-        $display("verilog: %04d a_valid[%d]=%d, a_address[%d]=0x%x, d_ready[%d]=%d",
-          $time, tid, __in_a_valid[tid], tid, __in_a_address[tid], tid, __in_d_ready[tid]);
+      // for (integer tid = 0; tid < NUM_LANES; tid = tid + 1) begin
+      //   $display("verilog: %04d a_valid[%d]=%d, a_address[%d]=0x%x, d_ready[%d]=%d",
+      //     $time, tid, __in_a_valid[tid], tid, __in_a_address[tid], tid, __in_d_ready[tid]);
+      // end
+    end
   end
 
-      if (finished) begin
-        $finish;
-      end
+  // negedge is important here; the DPI logic is essentially functioning as
+  // a combinational logic, so we want to reflect the signal change from DPI
+  // at the *current* cycle, not the next.
+  always @(negedge clock) begin
+    if (!reset) begin
+      emulator_tick(
+        __in_d_ready,
+        __out_d_valid,
+        __out_d_is_store,
+        __out_d_size
+      );
     end
   end
 endmodule

src/main/scala/radiance/core/Emulator.scala

@@ -61,6 +61,7 @@ class EmulatorImp(
     val finished = Output(Bool())
   })
   val sim = Module(new SimEmulator(numLanes))
+
   sim.io.clock := clock
   sim.io.reset := reset.asBool