Skip to content

Commit 0c8abcd

Browse files
committed
[cov] Select some FPGA tests for CI setup validation
This commit adds the `coverage` tag to the `fpga_params` of several smoke and functional tests. This allows these tests to be automatically selected by the CI pipeline to validate the coverage collection setup on FPGA targets. Change-Id: I86d8e28ee885cf683d961dd5e1eaf44a100e8ad2 Signed-off-by: Yi-Hsuan Deng <yhdeng@google.com>
1 parent a103b45 commit 0c8abcd

6 files changed

Lines changed: 7 additions & 0 deletions

File tree

sw/device/lib/crypto/drivers/BUILD

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -179,6 +179,7 @@ opentitan_test(
179179
name = "entropy_test",
180180
srcs = ["entropy_test.c"],
181181
exec_env = EARLGREY_TEST_ENVS,
182+
fpga = fpga_params(tags = ["coverage"]),
182183
verilator = verilator_params(
183184
timeout = "long",
184185
tags = ["broken"], # TODO #16672 test broken by icache

sw/device/silicon_creator/lib/BUILD

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -77,6 +77,7 @@ opentitan_test(
7777
"//hw/top_earlgrey:fpga_cw340_rom_with_fake_keys": None,
7878
},
7979
),
80+
fpga = fpga_params(tags = ["coverage"]),
8081
verilator = verilator_params(
8182
timeout = "eternal",
8283
tags = ["flaky"],

sw/device/silicon_creator/lib/cert/BUILD

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ load(
88
)
99
load(
1010
"//rules/opentitan:defs.bzl",
11+
"fpga_params",
1112
"opentitan_test",
1213
)
1314

@@ -215,6 +216,7 @@ opentitan_test(
215216
"//hw/top_earlgrey:fpga_cw310_rom_ext": None,
216217
"//hw/top_earlgrey:sim_qemu_rom_ext": None,
217218
},
219+
fpga = fpga_params(tags = ["coverage"]),
218220
deps = [
219221
":dice_cwt",
220222
"//hw/top_earlgrey/ip_autogen/flash_ctrl:flash_ctrl_c_regs",

sw/device/silicon_creator/rom/e2e/clkmgr/BUILD

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,7 @@ JITTER_TESTCASES = [
5252
"name": "enabled",
5353
"fpga": fpga_params(
5454
otp = ":otp_img_jitter_enabled",
55+
tags = ["coverage"],
5556
),
5657
"defines": ["TEST_JITTER_ENABLED"],
5758
},

sw/device/silicon_creator/rom_ext/e2e/boot_svc/BUILD

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -286,6 +286,7 @@ opentitan_test(
286286
assemble = "{rom_ext}@{rom_ext_slot_a} {firmware}@{owner_slot_a} {firmware}@{owner_slot_b}",
287287
exit_failure = "BFV:.*|PASS|FAIL",
288288
exit_success = "FinalBootLog: 3:AAA\r\n",
289+
tags = ["coverage"],
289290
),
290291
linker_script = "//sw/device/lib/testing/test_framework:ottf_ld_silicon_owner_slot_virtual",
291292
qemu = qemu_params(

sw/device/tests/BUILD

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4576,6 +4576,7 @@ opentitan_test(
45764576
"//hw/top_earlgrey:silicon_creator": None,
45774577
},
45784578
),
4579+
fpga = fpga_params(tags = ["coverage"]),
45794580
deps = [
45804581
"//hw/top_earlgrey/sw/autogen:top_earlgrey",
45814582
"//sw/device/lib/arch:device",

0 commit comments

Comments
 (0)