ibex icon indicating copy to clipboard operation
ibex copied to clipboard

Store instructions do not always raise RV32E exceptions

Open mndstrmr opened this issue 1 year ago • 3 comments

Store instructions will be decoded with alu_op_b_mux_sel_o = OP_B_IMM: https://github.com/lowRISC/ibex/blob/1449ed5ea8f3d03127f7c3624e635e132830367d/rtl/ibex_decoder.sv#L778-L788

When RV32E = 1 an illegal instruction exception should be raised if any of the registers used by a given instruction are outside of the 0-15 range. In the case of the store instruction this should apply to both the storage address and the stored value.

The existing logic in ibex will not enforce this since alu_op_b_mux_sel_o = OP_B_IMM: https://github.com/lowRISC/ibex/blob/1449ed5ea8f3d03127f7c3624e635e132830367d/rtl/ibex_decoder.sv#L179-L183

Therefore in RV32E mode a store instruction may attempt to store a value from registers 16-31, which is not well defined since the register file will not have those entries.

B-TYPE instructions, which have the same format, are likely unaffected since in their first cycle they do set alu_op_{a,b}_mux_sel_o = OP_{A,B}_REG_{A,B}.

mndstrmr avatar Jun 26 '24 14:06 mndstrmr

Can you give me an example of an instruction that triggers this? I think the code that you quoted in the second snippet is trying to say "Don't read from a high register on A or B side, and don't write to a high register".

Is the problem that the RV32E spec doesn't allow immediate stores when the value is big? If so, I didn't know that was part of the spec. Can you give a reference?

rswarbrick avatar Jun 27 '24 11:06 rswarbrick

The issue here is that for store instruction, both rs1 and rs2 are used. So if, say, we have sd x31, 0(x10), then it should be illegal instruction.

But since decoder sets alu_op_b_mux_sel_o to OP_B_IMM, the illegal_reg_rv32e check will pass because it doesn't check if rs2 is used, merely if the ALU uses rs2.

nbdd0121 avatar Jun 27 '24 12:06 nbdd0121

Oh, I see. Thanks for the explanation.

rswarbrick avatar Jun 28 '24 08:06 rswarbrick