Skip to content

Commit 3c47792

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 78b1f5e commit 3c47792

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
@@ -78,6 +78,7 @@ opentitan_test(
7878
"//hw/top_earlgrey:fpga_cw340_rom_with_fake_keys": None,
7979
},
8080
),
81+
fpga = fpga_params(tags = ["coverage"]),
8182
verilator = verilator_params(
8283
timeout = "eternal",
8384
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
@@ -269,6 +269,7 @@ opentitan_test(
269269
assemble = "{rom_ext}@{rom_ext_slot_a} {firmware}@{owner_slot_a} {firmware}@{owner_slot_b}",
270270
exit_failure = "BFV:.*|PASS|FAIL",
271271
exit_success = "FinalBootLog: 3:AAA\r\n",
272+
tags = ["coverage"],
272273
),
273274
linker_script = "//sw/device/lib/testing/test_framework:ottf_ld_silicon_owner_slot_virtual",
274275
qemu = qemu_params(

sw/device/tests/BUILD

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4512,6 +4512,7 @@ opentitan_test(
45124512
"//hw/top_earlgrey:silicon_creator": None,
45134513
},
45144514
),
4515+
fpga = fpga_params(tags = ["coverage"]),
45154516
deps = [
45164517
"//hw/top_earlgrey/sw/autogen:top_earlgrey",
45174518
"//sw/device/lib/arch:device",

0 commit comments

Comments
 (0)