Commit 972f7cdd authored by Luke Kenneth Casson Leighton's avatar Luke Kenneth Casson Leighton
Browse files

clear out DEC in core.cur_state.dec due to spurious interrupt.

this is slightly complicated.  the STATE regfile contains pc, msr, svstate,
dec, and tb, being a reflection of CoreState.  reading from STATE regfile
is on a one-clock delay. the DEC/TB FSM needs to decrement DEC and increment
TB, by reading from the STATE regfile and then writing a new value.

of course, the SPR pipeline has to get a word in edgeways as well.

but...

the complication comes in that it is the PowerDecoder2 which receives
a *cached* copy of DEC, and this cached copy is what has (up until now)
been out-of-date with what is in the STATE regfile.

the hack-job-solution is to zero-out the cached copy when the SPR pipeline
writes a new value to DEC.  the DEC/TB FSM will then rewrite a correct
value into it.

given that PowerDecoder2 only uses the MSB of DEC (and the EE bit of MSR)
to determine whether to fire an interrupt, this should be perfectly fine.
parent e19eeb30
......@@ -56,7 +56,8 @@ class SPRMainStage(PipeModBase):
#### MTSPR ####
with m.Case(MicrOp.OP_MTSPR):
with m.Switch(spr):
# State SPRs first
# State SPRs first, note that this triggers a regfile write
# which is monitored right the way down in TestIssuerBase.
with m.Case(SPR.DEC, SPR.TB):
comb += state1_o.data.eq(a_i)
comb += state1_o.ok.eq(1)
......
......@@ -28,6 +28,10 @@ class SPRInputData(FUBaseData):
# convenience
self.a = self.ra
# note that state1 gets a corresponding "state1" write port created
# by core.py which is "monitored" by TestIssuerBase (hack-job, sigh).
# when writes are spotted then the DEC/TB FSM resets and re-reads
# DEC/TB.
class SPROutputData(FUBaseData):
regspec = [('INT', 'o', '0:63'), # RT
......
......@@ -319,7 +319,7 @@ class TestIssuerBase(Elaboratable):
# hack method of keeping an eye on whether branch/trap set the PC
self.state_nia = self.core.regs.rf['state'].w_ports['nia']
self.state_nia.wen.name = 'state_nia_wen'
# and whether SPR pipeline sets DEC or TB
# and whether SPR pipeline sets DEC or TB (fu/spr/main_stage.py)
self.state_spr = self.core.regs.rf['state'].w_ports['state1']
# pulse to synchronize the simulator at instruction end
......@@ -573,6 +573,7 @@ class TestIssuerBase(Elaboratable):
state_r_dectb = state_rf.r_ports['issue'] # DEC/TB
state_w_dectb = state_rf.w_ports['issue'] # DEC/TB
with m.FSM() as fsm:
# initiates read of current DEC
......@@ -1487,6 +1488,7 @@ class TestIssuerInternal(TestIssuerBase):
sync = m.d.sync
dbg = self.dbg
pdecode2 = self.pdecode2
cur_state = self.cur_state
# temporaries
core_busy_o = core.n.o_data.busy_o # core is busy
......@@ -1512,18 +1514,26 @@ class TestIssuerInternal(TestIssuerBase):
# instruction started: must wait till it finishes
with m.State("INSN_ACTIVE"):
# note changes to MSR, PC and SVSTATE, and DEC/TB
# these last two are done together, and passed to the
# DEC/TB FSM
# note changes to MSR, PC and SVSTATE
with m.If(self.state_nia.wen & (1 << StateRegs.SVSTATE)):
sync += self.sv_changed.eq(1)
with m.If(self.state_nia.wen & (1 << StateRegs.MSR)):
sync += self.msr_changed.eq(1)
with m.If(self.state_nia.wen & (1 << StateRegs.PC)):
sync += self.pc_changed.eq(1)
with m.If((self.state_spr.wen &
((1 << StateRegs.DEC) | (1 << StateRegs.TB))).bool()):
# and note changes to DEC/TB, to be passed to DEC/TB FSM
with m.If(self.state_spr.wen & (1 << StateRegs.TB)):
comb += self.pause_dec_tb.eq(1)
# but also zero-out the cur_state DEC so that, on
# the next instruction, if it is "enable interrupt"
# the delay between the DEC/TB FSM reading and updating
# cur_state.dec doesn't trigger a spurious interrupt.
# the DEC/TB FSM will read the regfile and update to
# the correct value, so having cur_state.dec set to zero
# for a while is no big deal.
with m.If(self.state_spr.wen & (1 << StateRegs.DEC)):
comb += self.pause_dec_tb.eq(1)
sync += cur_state.dec.eq(0) # only needs top bit clear
with m.If(~core_busy_o): # instruction done!
comb += exec_pc_o_valid.eq(1)
with m.If(exec_pc_i_ready):
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment