Skip to content

Commit

Permalink
[entropy_src/dv] Fix a failing rng_vseq test
Browse files Browse the repository at this point in the history
This commit fixes the rng_vseq test which was failing because the
ht_threshold_scope was set to an invalid mubi value in the config.
Whenever the entropy_src enters an alert state due to the configuration,
it reconfigures the DUT in a safe way to run the test. However, for the
ht_threshold_scope the same value is kept. The issue here is, that when
the initialization fails because of the ht_threshold_scope we still have
to set it to a safe value.

Signed-off-by: Hakim Filali <[email protected]>
  • Loading branch information
Hakim Filali committed Feb 14, 2024
1 parent 8c08da6 commit 664b5e6
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion hw/ip/entropy_src/dv/env/seq_lib/entropy_src_rng_vseq.sv
Original file line number Diff line number Diff line change
Expand Up @@ -768,7 +768,10 @@ class entropy_src_rng_vseq extends entropy_src_base_vseq;
// will likely fail.
altcfg.bypass_window_size = cfg.dut_cfg.bypass_window_size;
altcfg.fips_window_size = cfg.dut_cfg.fips_window_size;
altcfg.ht_threshold_scope = cfg.dut_cfg.ht_threshold_scope;
// If the ht_threshold_scope was set to a invalid value, change it anyways.
altcfg.ht_threshold_scope = mubi4_test_invalid(cfg.dut_cfg.ht_threshold_scope) ?
altcfg.ht_threshold_scope :
cfg.dut_cfg.ht_threshold_scope;
end

// Force the DUT to not absorb the new config (to test write protect), unless
Expand Down

0 comments on commit 664b5e6

Please sign in to comment.