lowRISC / opentitan

OpenTitan: Open source silicon root of trust
https://www.opentitan.org
Apache License 2.0
2.55k stars 761 forks source link

[test-triage] i2c_csr_bit_bash assertion failure `TxWriteStableBeforeHandshake_A` #23815

Closed luismarques closed 2 months ago

luismarques commented 3 months ago

Hierarchy of regression failure

Block level

Failure Description

UVM_INFO @ 0 ps: reporter [RNTST] Running test i2c_base_test...
UVM_INFO @   5141628 ps: (dv_base_vseq.sv:182) uvm_test_top.env.virtual_sequencer [uvm_test_top.env.virtual_sequencer.i2c_common_vseq] Running csr bit_bash vseq iteration 1/2.
UVM_INFO @ 397210254 ps: (dv_base_vseq.sv:182) uvm_test_top.env.virtual_sequencer [uvm_test_top.env.virtual_sequencer.i2c_common_vseq] Running csr bit_bash vseq iteration 2/2.
"../src/lowrisc_ip_i2c_0.1/rtl/i2c_fifos.sv", 316: tb.dut.i2c_core.u_fifos.TxWriteStableBeforeHandshake_A: started at 669944951ps failed at 670011618ps
    Offending '($stable(tx_fifo_wvalid_i) && $stable(tx_fifo_wdata_i))'
UVM_ERROR @ 670011618 ps: (i2c_fifos.sv:316) [ASSERT FAILED] TxWriteStableBeforeHandshake_A
UVM_INFO @ 670011618 ps: (uvm_report_catcher.svh:705) [UVM/REPORT/CATCHER]

I've checked a lot of commits and this error has existed for over a year (though sometimes you need other seeds to reproduce).

Steps to Reproduce

Tests with similar or related failures

No response

a-will commented 3 months ago

It's a CSR test exclusion that needs to happen. The bit bash test fills the FIFO and causes an overflow, leading to the assertion firing.

hcallahan-lowrisc commented 2 months ago

Fixed by #23865 Duplicate of #22853