src_delays=[0, 2, 1], dest_delays=[0])
yield from op.issue([9, 2, 0], MicrOp.OP_NOP, [9],
src_delays=[2, 1, 0], dest_delays=[2])
+ # test all combinations of masked input ports
+ yield from op.issue([5, 2, 0], MicrOp.OP_NOP, [0],
+ rdmaskn=[1, 0, 0],
+ src_delays=[0, 2, 1], dest_delays=[0])
+ yield from op.issue([9, 2, 0], MicrOp.OP_NOP, [9],
+ rdmaskn=[0, 1, 0],
+ src_delays=[2, 1, 0], dest_delays=[2])
+ yield from op.issue([5, 2, 0], MicrOp.OP_NOP, [5],
+ rdmaskn=[0, 0, 1],
+ src_delays=[2, 1, 0], dest_delays=[2])
+ yield from op.issue([9, 2, 0], MicrOp.OP_NOP, [9],
+ rdmaskn=[0, 1, 1],
+ src_delays=[2, 1, 0], dest_delays=[2])
+ yield from op.issue([9, 2, 0], MicrOp.OP_NOP, [0],
+ rdmaskn=[1, 1, 0],
+ src_delays=[2, 1, 0], dest_delays=[2])
+ yield from op.issue([9, 2, 0], MicrOp.OP_NOP, [0],
+ rdmaskn=[1, 1, 1],
+ src_delays=[2, 1, 0], dest_delays=[2])
class OpSim:
self.op_count = 0
self.zero_a_count = 0
self.imm_ok_count = 0
+ self.rdmaskn_count = [0] * len(dut.src_i)
self.dut = dut
# create one operand producer for each input port
self.producers = list()
for i in range(len(dut.dest)):
self.consumers.append(ResultConsumer(sim, dut, i))
def issue(self, src_i, op, expected, src_delays, dest_delays,
- inv_a=0, imm=0, imm_ok=0, zero_a=0):
+ inv_a=0, imm=0, imm_ok=0, zero_a=0, rdmaskn=None):
"""Executes the issue operation"""
dut = self.dut
producers = self.producers
consumers = self.consumers
+ if rdmaskn is None:
+ rdmaskn = [0] * len(src_i)
yield dut.issue_i.eq(0)
yield
# forward data and delays to the producers and consumers
yield dut.oper_i.imm_data.ok.eq(imm_ok)
if hasattr(dut.oper_i, "zero_a"):
yield dut.oper_i.zero_a.eq(zero_a)
+ if hasattr(dut, "rdmaskn"):
+ rdmaskn_bits = 0
+ for i in range(len(rdmaskn)):
+ rdmaskn_bits |= rdmaskn[i] << i
+ yield dut.rdmaskn.eq(rdmaskn_bits)
yield dut.issue_i.eq(1)
yield
yield dut.issue_i.eq(0)
+ # deactivate decoder inputs along with issue_i, so we can be sure they
+ # were latched at the correct cycle
+ # note: rdmaskn is not latched, and must be held as long as
+ # busy_o is active
+ # todo: is the above restriction on rdmaskn intentional?
+ # todo: shouldn't it be latched by issue_i, like the others?
+ yield self.dut.oper_i.insn_type.eq(0)
+ if hasattr(dut.oper_i, "invert_in"):
+ yield self.dut.oper_i.invert_in.eq(0)
+ if hasattr(dut.oper_i, "imm_data"):
+ yield self.dut.oper_i.imm_data.data.eq(0)
+ yield self.dut.oper_i.imm_data.ok.eq(0)
+ if hasattr(dut.oper_i, "zero_a"):
+ yield self.dut.oper_i.zero_a.eq(0)
# wait for busy to be negated
yield Settle()
while (yield dut.busy_o):
yield
yield Settle()
+ # now, deactivate rdmaskn
+ if hasattr(dut, "rdmaskn"):
+ yield dut.rdmaskn.eq(0)
# update the operation count
self.op_count = (self.op_count + 1) & 255
- # On zero_a and imm_ok executions, the producer counters will fall
- # behind. But, by summing the following counts, the invariant is
+ # On zero_a, imm_ok and rdmaskn executions, the producer counters will
+ # fall behind. But, by summing the following counts, the invariant is
# preserved.
- if zero_a:
+ if zero_a and not rdmaskn[0]:
self.zero_a_count = self.zero_a_count + 1
- if imm_ok:
+ if imm_ok and not rdmaskn[1]:
self.imm_ok_count = self.imm_ok_count + 1
+ for i in range(len(rdmaskn)):
+ if rdmaskn[i]:
+ self.rdmaskn_count[i] = self.rdmaskn_count[i] + 1
# check that producers and consumers have the same count
# this assures that no data was left unused or was lost
# first, check special cases (zero_a and imm_ok)
- assert (yield producers[0].count) + self.zero_a_count == self.op_count
- assert (yield producers[1].count) + self.imm_ok_count == self.op_count
+ port_a_cnt = \
+ (yield producers[0].count) \
+ + self.zero_a_count \
+ + self.rdmaskn_count[0]
+ port_b_cnt = \
+ (yield producers[1].count) \
+ + self.imm_ok_count \
+ + self.rdmaskn_count[1]
+ assert port_a_cnt == self.op_count
+ assert port_b_cnt == self.op_count
# then, check the rest (if any)
for i in range(2, len(producers)):
- assert (yield producers[i].count) == self.op_count
+ port_cnt = (yield producers[i].count) + self.rdmaskn_count[i]
+ assert port_cnt == self.op_count
+ # check write counter
for i in range(len(consumers)):
assert (yield consumers[i].count) == self.op_count
# 5 - 2 = 3
yield from op.issue([5, 2], MicrOp.OP_NOP, [3],
src_delays=[0, 1], dest_delays=[2])
+ # test all combinations of masked input ports
+ # 5 + 0 (masked) = 5
+ yield from op.issue([5, 2], MicrOp.OP_ADD, [5],
+ rdmaskn=[0, 1],
+ src_delays=[2, 1], dest_delays=[0])
+ # 0 (masked) + 2 = 2
+ yield from op.issue([5, 2], MicrOp.OP_ADD, [2],
+ rdmaskn=[1, 0],
+ src_delays=[1, 2], dest_delays=[1])
+ # 0 (masked) + 0 (masked) = 0
+ yield from op.issue([5, 2], MicrOp.OP_ADD, [0],
+ rdmaskn=[1, 1],
+ src_delays=[1, 2], dest_delays=[1])
def test_compunit_fsm():
- top = "top.cu" if is_engine_pysim() else "cu"
style = {
'in': {'color': 'orange'},
'out': {'color': 'yellow'},
'src2_i[7:0]']),
('result port', 'out', [
'cu_wr__rel_o', 'cu_wr__go_i', 'dest1_o[7:0]']),
- ('alu', {'module': top+'.alu'}, [
+ ('alu', {'submodule': 'alu'}, [
('prev port', 'in', [
'op__sdir', 'p_data_i[7:0]', 'p_shift_i[7:0]',
- 'p_valid_i', 'p_ready_o']),
+ ({'submodule': 'p'},
+ ['p_valid_i', 'p_ready_o'])]),
('next port', 'out', [
- 'n_data_o[7:0]', 'n_valid_o', 'n_ready_i']),
- ]),
+ 'n_data_o[7:0]',
+ ({'submodule': 'n'},
+ ['n_valid_o', 'n_ready_i'])])]),
('debug', {'module': 'top'},
['src1_count[7:0]', 'src2_count[7:0]', 'dest1_count[7:0]'])]
"test_compunit_fsm1.gtkw",
"test_compunit_fsm1.vcd",
traces, style,
- module=top
+ module='top.cu'
)
m = Module()
alu = Shifter(8)
sim.run()
-class CompUnitParallelTest:
- def __init__(self, dut):
- self.dut = dut
-
- # Operation cycle should not take longer than this:
- self.MAX_BUSY_WAIT = 50
-
- # Minimum duration in which issue_i will be kept inactive,
- # during which busy_o must remain low.
- self.MIN_BUSY_LOW = 5
-
- # Number of cycles to stall until the assertion of go.
- # One value, for each port. Can be zero, for no delay.
- self.RD_GO_DELAY = [0, 3]
-
- # store common data for the input operation of the processes
- # input operation:
- self.op = 0
- self.inv_a = self.zero_a = 0
- self.imm = self.imm_ok = 0
- self.imm_control = (0, 0)
- self.rdmaskn = (0, 0)
- # input data:
- self.operands = (0, 0)
-
- # Indicates completion of the sub-processes
- self.rd_complete = [False, False]
-
- def driver(self):
- print("Begin parallel test.")
- yield from self.operation(5, 2, MicrOp.OP_ADD)
-
- def operation(self, a, b, op, inv_a=0, imm=0, imm_ok=0, zero_a=0,
- rdmaskn=(0, 0)):
- # store data for the operation
- self.operands = (a, b)
- self.op = op
- self.inv_a = inv_a
- self.imm = imm
- self.imm_ok = imm_ok
- self.zero_a = zero_a
- self.imm_control = (zero_a, imm_ok)
- self.rdmaskn = rdmaskn
-
- # Initialize completion flags
- self.rd_complete = [False, False]
-
- # trigger operation cycle
- yield from self.issue()
-
- # check that the sub-processes completed, before the busy_o cycle ended
- for completion in self.rd_complete:
- assert completion
-
- def issue(self):
- # issue_i starts inactive
- yield self.dut.issue_i.eq(0)
-
- for n in range(self.MIN_BUSY_LOW):
- yield
- # busy_o must remain inactive. It cannot rise on its own.
- busy_o = yield self.dut.busy_o
- assert not busy_o
-
- # activate issue_i to begin the operation cycle
- yield self.dut.issue_i.eq(1)
-
- # at the same time, present the operation
- yield self.dut.oper_i.insn_type.eq(self.op)
- yield self.dut.oper_i.invert_in.eq(self.inv_a)
- yield self.dut.oper_i.imm_data.data.eq(self.imm)
- yield self.dut.oper_i.imm_data.ok.eq(self.imm_ok)
- yield self.dut.oper_i.zero_a.eq(self.zero_a)
- rdmaskn = self.rdmaskn[0] | (self.rdmaskn[1] << 1)
- yield self.dut.rdmaskn.eq(rdmaskn)
-
- # give one cycle for the CompUnit to latch the data
- yield
-
- # busy_o must keep being low in this cycle, because issue_i was
- # low on the previous cycle.
- # It cannot rise on its own.
- # Also, busy_o and issue_i must never be active at the same time, ever.
- busy_o = yield self.dut.busy_o
- assert not busy_o
-
- # Lower issue_i
- yield self.dut.issue_i.eq(0)
-
- # deactivate inputs along with issue_i, so we can be sure the data
- # was latched at the correct cycle
- # note: rdmaskn must be held, while busy_o is active
- # TODO: deactivate rdmaskn when the busy_o cycle ends
- yield self.dut.oper_i.insn_type.eq(0)
- yield self.dut.oper_i.invert_in.eq(0)
- yield self.dut.oper_i.imm_data.data.eq(0)
- yield self.dut.oper_i.imm_data.ok.eq(0)
- yield self.dut.oper_i.zero_a.eq(0)
- yield
-
- # wait for busy_o to lower
- # timeout after self.MAX_BUSY_WAIT cycles
- for n in range(self.MAX_BUSY_WAIT):
- # sample busy_o in the current cycle
- busy_o = yield self.dut.busy_o
- if not busy_o:
- # operation cycle ends when busy_o becomes inactive
- break
- yield
-
- # if busy_o is still active, a timeout has occurred
- # TODO: Uncomment this, once the test is complete:
- # assert not busy_o
-
- if busy_o:
- print("If you are reading this, "
- "it's because the above test failed, as expected,\n"
- "with a timeout. It must pass, once the test is complete.")
- return
-
- print("If you are reading this, "
- "it's because the above test unexpectedly passed.")
-
- def rd(self, rd_idx):
- # wait for issue_i to rise
- while True:
- issue_i = yield self.dut.issue_i
- if issue_i:
- break
- # issue_i has not risen yet, so rd must keep low
- rel = yield self.dut.rd.rel_o[rd_idx]
- assert not rel
- yield
-
- # we do not want rd to rise on an immediate operand
- # if it is immediate, exit the process
- # likewise, if the read mask is active
- # TODO: don't exit the process, monitor rd instead to ensure it
- # doesn't rise on its own
- if self.rdmaskn[rd_idx] or self.imm_control[rd_idx]:
- self.rd_complete[rd_idx] = True
- return
-
- # issue_i has risen. rel must rise on the next cycle
- rel = yield self.dut.rd.rel_o[rd_idx]
- assert not rel
-
- # stall for additional cycles. Check that rel doesn't fall on its own
- for n in range(self.RD_GO_DELAY[rd_idx]):
- yield
- rel = yield self.dut.rd.rel_o[rd_idx]
- assert rel
-
- # Before asserting "go", make sure "rel" has risen.
- # The use of Settle allows "go" to be set combinatorially,
- # rising on the same cycle as "rel".
- yield Settle()
- rel = yield self.dut.rd.rel_o[rd_idx]
- assert rel
-
- # assert go for one cycle, passing along the operand value
- yield self.dut.rd.go_i[rd_idx].eq(1)
- yield self.dut.src_i[rd_idx].eq(self.operands[rd_idx])
- # check that the operand was sent to the alu
- # TODO: Properly check the alu protocol
- yield Settle()
- alu_input = yield self.dut.get_in(rd_idx)
- assert alu_input == self.operands[rd_idx]
- yield
-
- # rel must keep high, since go was inactive in the last cycle
- rel = yield self.dut.rd.rel_o[rd_idx]
- assert rel
-
- # finish the go one-clock pulse
- yield self.dut.rd.go_i[rd_idx].eq(0)
- yield self.dut.src_i[rd_idx].eq(0)
- yield
-
- # rel must have gone low in response to go being high
- # on the previous cycle
- rel = yield self.dut.rd.rel_o[rd_idx]
- assert not rel
-
- self.rd_complete[rd_idx] = True
-
- # TODO: check that rel doesn't rise again until the end of the
- # busy_o cycle
-
- def wr(self, wr_idx):
- # monitor self.dut.wr.req[rd_idx] and sets dut.wr.go[idx] for one cycle
- yield
- # TODO: also when dut.wr.go is set, check the output against the
- # self.expected_o and assert. use dut.get_out(wr_idx) to do so.
-
- def run_simulation(self, vcd_name):
- m = Module()
- m.submodules.cu = self.dut
- sim = Simulator(m)
- sim.add_clock(1e-6)
-
- sim.add_sync_process(wrap(self.driver()))
- sim.add_sync_process(wrap(self.rd(0)))
- sim.add_sync_process(wrap(self.rd(1)))
- sim.add_sync_process(wrap(self.wr(0)))
- sim_writer = sim.write_vcd(vcd_name)
- with sim_writer:
- sim.run()
-
-
def test_compunit_regspec2_fsm():
inspec = [('INT', 'data', '0:15'),
('operation port', {'color': 'red'}, [
'cu_issue_i', 'cu_busy_o',
{'comment': 'operation'},
- ('oper_i_None__insn_type', {'display': 'insn_type'})]),
+ ('oper_i_None__insn_type'
+ + ('' if is_engine_pysim() else '[6:0]'),
+ {'display': 'insn_type'})]),
('operand 1 port', 'in', [
('cu_rd__rel_o[2:0]', {'bit': 2}),
('cu_rd__go_i[2:0]', {'bit': 2}),
'src1_i[15:0]']),
('result port', 'out', [
'cu_wr__rel_o', 'cu_wr__go_i', 'dest1_o[15:0]']),
- ('alu', {'module': 'top.cu.alu'}, [
+ ('alu', {'submodule': 'alu'}, [
('prev port', 'in', [
'oper_i_None__insn_type', 'i1[15:0]',
'valid_i', 'ready_o']),
('operation port', {'color': 'red'}, [
'cu_issue_i', 'cu_busy_o',
{'comment': 'operation'},
- ('oper_i_None__insn_type', {'display': 'insn_type'}),
+ ('oper_i_None__insn_type'
+ + ('' if is_engine_pysim() else '[6:0]'),
+ {'display': 'insn_type'}),
('oper_i_None__invert_in', {'display': 'invert_in'}),
('oper_i_None__imm_data__data[63:0]', {'display': 'data[63:0]'}),
('oper_i_None__imm_data__ok', {'display': 'imm_ok'}),
'src2_i[15:0]']),
('result port', 'out', [
'cu_wr__rel_o', 'cu_wr__go_i', 'dest1_o[15:0]']),
- ('alu', {'module': 'top.cu.alu'}, [
+ ('alu', {'submodule': 'alu'}, [
('prev port', 'in', [
'op__insn_type', 'op__invert_in', 'a[15:0]', 'b[15:0]',
'valid_i', 'ready_o']),
with sim_writer:
sim.run()
- test = CompUnitParallelTest(dut)
- test.run_simulation("test_compunit_parallel.vcd")
-
if __name__ == '__main__':
test_compunit()
projects / soc.git / blobdiff