Skip to content

Commit

Permalink
[dd, aon_timer cov_closure] CCOV exclusion file for aon_timer
Browse files Browse the repository at this point in the history
All the exclusions in this file are unreachable
In addition, rtl exclusions added to config file

Signed-off-by: Antonio Martinez Zambrana <[email protected]>
  • Loading branch information
antmarzam committed Jan 29, 2025
1 parent 22168a3 commit a468271
Show file tree
Hide file tree
Showing 2 changed files with 103 additions and 1 deletion.
3 changes: 2 additions & 1 deletion hw/ip/aon_timer/dv/aon_timer_sim_cfg.hjson
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,8 @@
reseed: 50

// Add specific exclusion files.
vcs_cov_excl_files: ["{proj_root}/hw/ip/aon_timer/dv/cov/aon_timer_unr_excl.el"]
vcs_cov_excl_files: ["{proj_root}/hw/ip/aon_timer/dv/cov/aon_timer_unr_excl.el",
"{proj_root}/hw/ip/aon_timer/dv/cov/aon_timer_unr_manually_excluded_excl.el"]

// Default UVM test and seq class name.
uvm_test: aon_timer_base_test
Expand Down
101 changes: 101 additions & 0 deletions hw/ip/aon_timer/dv/cov/aon_timer_unr_manually_excluded_excl.el
Original file line number Diff line number Diff line change
@@ -0,0 +1,101 @@
// Copyright lowRISC contributors (OpenTitan project).
// Licensed under the Apache License, Version 2.0, see LICENSE for details.
// SPDX-License-Identifier: Apache-2.0
//
//==================================================
// This file contains the Excluded objects
// Generated By User: antonio
// Format Version: 2
// Date: Wed Dec 18 17:15:12 2024
// ExclMode: default
//==================================================
CHECKSUM: "1706182284 132761700"
INSTANCE: tb.dut.u_reg.u_reg_if.u_rsp_intg_gen
ANNOTATION: "tl_i.d_user is hardcoded to 0x0 "
Block 1 "461445014" "assign rsp_intg = tl_i.d_user.rsp_intg;"
ANNOTATION: "tl_i.d_user is hardcoded to 0x0"
Block 2 "2643129081" "assign data_intg = tl_i.d_user.data_intg;"
CHECKSUM: "2815520955 1214062232"
INSTANCE: tb.dut.u_reg.u_wkup_count_hi_cdc.u_arb
ANNOTATION: "Default statement can't be hit unless some wrong value is forced in the `state_d` signal"
Block 43 "3846199569" "gen_wr_req.state_d = StIdle;"
CHECKSUM: "2815520955 1214062232"
INSTANCE: tb.dut.u_reg.u_wkup_count_lo_cdc.u_arb
ANNOTATION: "Default statement can't be hit unless some wrong value is forced in the `state_d` signal"
Block 43 "3846199569" "gen_wr_req.state_d = StIdle;"
CHECKSUM: "2815520955 1214062232"
INSTANCE: tb.dut.u_reg.u_wdog_count_cdc.u_arb
ANNOTATION: "Default statement can't be hit unless some wrong value is forced in the `state_d` signal"
Block 43 "3846199569" "gen_wr_req.state_d = StIdle;"
CHECKSUM: "2815520955 1214062232"
INSTANCE: tb.dut.u_reg.u_wkup_cause_cdc.u_arb
ANNOTATION: "Default statement can't be hit unless some wrong value is forced in the `state_d` signal"
Block 43 "3846199569" "gen_wr_req.state_d = StIdle;"
CHECKSUM: "2815520955 4051034043"
INSTANCE: tb.dut.u_reg.u_wkup_count_hi_cdc.u_arb
ANNOTATION: "Default statement can't be hit unless some wrong value is forced in the `state_d` signal"
Branch 4 "3547459906" "gen_wr_req.state_q" (6) "gen_wr_req.state_q default,-,-,-,-"
CHECKSUM: "2815520955 4051034043"
INSTANCE: tb.dut.u_reg.u_wkup_count_lo_cdc.u_arb
ANNOTATION: "Default statement can't be hit unless some wrong value is forced in the `state_d` signal"
Branch 4 "3547459906" "gen_wr_req.state_q" (6) "gen_wr_req.state_q default,-,-,-,-"
CHECKSUM: "2815520955 4051034043"
INSTANCE: tb.dut.u_reg.u_wdog_count_cdc.u_arb
ANNOTATION: "Default statement can't be hit unless some wrong value is forced in the `state_d` signal"
Branch 4 "3547459906" "gen_wr_req.state_q" (6) "gen_wr_req.state_q default,-,-,-,-"
CHECKSUM: "2815520955 4051034043"
INSTANCE: tb.dut.u_reg.u_wkup_cause_cdc.u_arb
ANNOTATION: "Default statement can't be hit unless some wrong value is forced in the `state_d` signal"
Branch 4 "3547459906" "gen_wr_req.state_q" (6) "gen_wr_req.state_q default,-,-,-,-"
CHECKSUM: "2815520955 4109606122"
INSTANCE: tb.dut.u_reg.u_wkup_count_hi_cdc.u_arb
ANNOTATION: "It can't be hit because dst_update_ack won't be set prior to dst_update_req"
Condition 3 "3080886878" "(gen_wr_req.dst_update_req && gen_wr_req.dst_update_ack) 1 -1" (1 "01")
CHECKSUM: "2815520955 4109606122"
INSTANCE: tb.dut.u_reg.u_wkup_count_lo_cdc.u_arb
ANNOTATION: "It can't be hit because dst_update_ack won't be set prior to dst_update_req"
Condition 3 "3080886878" "(gen_wr_req.dst_update_req && gen_wr_req.dst_update_ack) 1 -1" (1 "01")
ANNOTATION: "It can't be hit because the case where dst_req=1 and dst_lat_d=1 is covered in the branch above."
Condition 5 "593451913" "(((!gen_wr_req.dst_req)) && gen_wr_req.dst_lat_d) 1 -1" (1 "01")
CHECKSUM: "2815520955 4109606122"
INSTANCE: tb.dut.u_reg.u_wdog_count_cdc.u_arb
ANNOTATION: "It can't be hit because dst_update_ack won't be set prior to dst_update_req"
Condition 3 "3080886878" "(gen_wr_req.dst_update_req && gen_wr_req.dst_update_ack) 1 -1" (1 "01")
ANNOTATION: "It can't be hit because the case where dst_req=1 and dst_lat_d=1 is covered in the branch above."
Condition 5 "593451913" "(((!gen_wr_req.dst_req)) && gen_wr_req.dst_lat_d) 1 -1" (1 "01")
CHECKSUM: "2815520955 4109606122"
INSTANCE: tb.dut.u_reg.u_wkup_cause_cdc.u_arb
ANNOTATION: "It can't be hit because dst_update_ack won't be set prior to dst_update_req"
Condition 3 "3080886878" "(gen_wr_req.dst_update_req && gen_wr_req.dst_update_ack) 1 -1" (1 "01")
ANNOTATION: "It can't be hit because the case where dst_req=1 and dst_lat_d=1 is covered in the branch above."
Condition 5 "593451913" "(((!gen_wr_req.dst_req)) && gen_wr_req.dst_lat_d) 1 -1" (1 "01")
CHECKSUM: "3643792208 1147758610"
INSTANCE: tb.dut.u_reg.u_wkup_count_hi_cdc.u_arb.gen_wr_req.u_dst_update_sync
ANNOTATION: "src_ack becomes one 1-cycle after dst_ack goes high. Since dst_ack_i is wired to dst_req_o, it means src_ack_o can't be set until after src_req_i is set."
Condition 2 "700807773" "(dst_req_o & dst_ack_i) 1 -1" (2 "10")
ANNOTATION: "src_ack becomes one 1-cycle after dst_ack goes high. Since dst_ack_i is wired to dst_req_o, it means src_ack_o can't be set until after src_req_i is set."
Condition 2 "700807773" "(dst_req_o & dst_ack_i) 1 -1" (1 "01")
ANNOTATION: "Output and input are connected to the same signal, hence the missing coverage is not reachable"
Condition 1 "1414883863" "(src_req_i & src_ack_o) 1 -1" (1 "01")
CHECKSUM: "3643792208 1147758610"
INSTANCE: tb.dut.u_reg.u_wkup_count_lo_cdc.u_arb.gen_wr_req.u_dst_update_sync
ANNOTATION: "src_ack becomes one 1-cycle after dst_ack goes high. Since dst_ack_i is wired to dst_req_o, it means src_ack_o can't be set until after src_req_i is set."
Condition 2 "700807773" "(dst_req_o & dst_ack_i) 1 -1" (2 "10")
ANNOTATION: "src_ack becomes one 1-cycle after dst_ack goes high. Since dst_ack_i is wired to dst_req_o, it means src_ack_o can't be set until after src_req_i is set."
Condition 2 "700807773" "(dst_req_o & dst_ack_i) 1 -1" (1 "01")
ANNOTATION: "Output and input are connected to the same signal, hence the missing coverage is not reachable"
Condition 1 "1414883863" "(src_req_i & src_ack_o) 1 -1" (1 "01")
CHECKSUM: "3643792208 1147758610"
INSTANCE: tb.dut.u_reg.u_wdog_count_cdc.u_arb.gen_wr_req.u_dst_update_sync
ANNOTATION: "src_ack becomes one 1-cycle after dst_ack goes high. Since dst_ack_i is wired to dst_req_o, it means src_ack_o can't be set until after src_req_i is set."
Condition 2 "700807773" "(dst_req_o & dst_ack_i) 1 -1" (2 "10")
ANNOTATION: "src_ack becomes one 1-cycle after dst_ack goes high. Since dst_ack_i is wired to dst_req_o, it means src_ack_o can't be set until after src_req_i is set."
Condition 2 "700807773" "(dst_req_o & dst_ack_i) 1 -1" (1 "01")
ANNOTATION: "Output and input are connected to the same signal, hence the missing coverage is not reachable"
Condition 1 "1414883863" "(src_req_i & src_ack_o) 1 -1" (1 "01")
CHECKSUM: "3643792208 1147758610"
INSTANCE: tb.dut.u_reg.u_wkup_cause_cdc.u_arb.gen_wr_req.u_dst_update_sync
ANNOTATION: "src_ack becomes one 1-cycle after dst_ack goes high. Since dst_ack_i is wired to dst_req_o, it means src_ack_o can't be set until after src_req_i is set."
Condition 1 "1414883863" "(src_req_i & src_ack_o) 1 -1"
ANNOTATION: "Output and input are connected to the same signal, hence the missing coverage is not reachable"
Condition 2 "700807773" "(dst_req_o & dst_ack_i) 1 -1"

0 comments on commit a468271

Please sign in to comment.