Skip to content

Commit

Permalink
[test] adjust exec_env of otbn_boot_functest
Browse files Browse the repository at this point in the history
The `otbn_boot_services_functest` must run in the ROM_EXT slot since it
manipulates the keygmr state, which normally is done by the ROM_EXT.

This partially addresses #21706.

Signed-off-by: Tim Trippel <[email protected]>
(cherry picked from commit 698b5a6)
  • Loading branch information
timothytrippel committed May 4, 2024
1 parent 6c45e24 commit 6eb7c74
Showing 1 changed file with 7 additions and 11 deletions.
18 changes: 7 additions & 11 deletions sw/device/silicon_creator/lib/BUILD
Original file line number Diff line number Diff line change
Expand Up @@ -66,14 +66,12 @@ cc_test(
opentitan_test(
name = "boot_data_functest",
srcs = ["boot_data_functest.c"],
broken = cw310_params(tags = ["broken"]),
exec_env = dicts.add(
EARLGREY_TEST_ENVS,
{
# FIXME broken in sival ROM_EXT, remove this line when fixed. See #21706.
"//hw/top_earlgrey:fpga_cw310_sival_rom_ext": "broken",
},
),
exec_env = {
"//hw/top_earlgrey:fpga_cw310_rom_with_fake_keys": None,
"//hw/top_earlgrey:fpga_cw310_sival": None,
"//hw/top_earlgrey:sim_dv": None,
"//hw/top_earlgrey:sim_verilator": None,
},
verilator = verilator_params(
timeout = "eternal",
tags = ["flaky"],
Expand Down Expand Up @@ -491,12 +489,10 @@ cc_library(
opentitan_test(
name = "otbn_boot_services_functest",
srcs = ["otbn_boot_services_functest.c"],
broken = cw310_params(tags = ["broken"]),
exec_env = dicts.add(
EARLGREY_TEST_ENVS,
{
# FIXME broken in sival ROM_EXT, remove this line when fixed. See #21706.
"//hw/top_earlgrey:fpga_cw310_sival_rom_ext": "broken",
"//hw/top_earlgrey:fpga_cw310_sival": None,
},
),
# This target uses OTBN pointers internally, so it cannot work host-side.
Expand Down

0 comments on commit 6eb7c74

Please sign in to comment.