From 0df522b99d98618d9ff5f95f622dbd79267ae728 Mon Sep 17 00:00:00 2001 From: Luke Kenneth Casson Leighton Date: Sun, 6 Sep 2020 12:56:48 +0100 Subject: [PATCH] add a DEC/TB FSM to TestIssuer this operates on alternative cycles, because it reads/writes from the Fast Regfile directly --- src/soc/simple/issuer.py | 53 +++++++++++++++++++++++++++++++++++++--- 1 file changed, 50 insertions(+), 3 deletions(-) diff --git a/src/soc/simple/issuer.py b/src/soc/simple/issuer.py index ff525e98..745f4a3d 100644 --- a/src/soc/simple/issuer.py +++ b/src/soc/simple/issuer.py @@ -25,7 +25,7 @@ from soc.decoder.power_decoder import create_pdecode from soc.decoder.power_decoder2 import PowerDecode2 from soc.decoder.decode2execute1 import Data from soc.experiment.testmem import TestMemory # test only for instructions -from soc.regfile.regfiles import StateRegs +from soc.regfile.regfiles import StateRegs, FastRegs from soc.simple.core import NonProductionCore from soc.config.test.test_loadstore import TestMemPspec from soc.config.ifetch import ConfigFetchUnit @@ -81,7 +81,7 @@ class TestIssuer(Elaboratable): self.busy_o = Signal(reset_less=True) self.memerr_o = Signal(reset_less=True) - # FAST regfile read /write ports for PC and MSR + # FAST regfile read /write ports for PC, MSR, DEC/TB staterf = self.core.regs.rf['state'] self.state_r_pc = staterf.r_ports['cia'] # PC rd self.state_w_pc = staterf.w_ports['d_wr1'] # PC wr @@ -264,7 +264,6 @@ class TestIssuer(Elaboratable): comb += core_ivalid_i.eq(1) # instruction is valid comb += core_issue_i.eq(1) # and issued - m.next = "INSN_ACTIVE" # move to "wait completion" # instruction started: must wait till it finishes @@ -320,6 +319,54 @@ class TestIssuer(Elaboratable): comb += d_xer.data.eq(self.xer_r.data_o) comb += d_xer.ack.eq(1) + # DEC and TB inc/dec FSM + self.tb_dec_fsm(m) + + return m + + def tb_dec_fsm(self, m): + """tb_dec_fsm + + this is a FSM for updating either dec or tb. it runs alternately + DEC, TB, DEC, TB. note that SPR pipeline could have written a new + value to DEC, however the regfile has "passthrough" on it so this + *should* be ok. + """ + + comb, sync = m.d.comb, m.d.sync + fast_rf = self.core.regs.rf['fast'] + fast_r_dectb = fast_rf.r_ports['issue'] # DEC/TB + fast_w_dectb = fast_rf.w_ports['issue'] # DEC/TB + + with m.FSM() as fsm: + + # initiates read of current DEC + with m.State("DEC_READ"): + comb += fast_r_dectb.ren.eq(1<