Skip to content

Commit

Permalink
Improve the way we disable audit tests
Browse files Browse the repository at this point in the history
This diff should likely be revisited, but for now check that we're
targeting a CPU that supports CHERI rather than that we're building
optional CHERI programs.

With this change, MK_CHERI only controls optional programs that require
CHERI and aren't likely to be upstreamed in a CHERI merge.
  • Loading branch information
brooksdavis committed Jul 21, 2023
1 parent b0e9ca0 commit de3f22d
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion tests/sys/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ TESTS_SUBDIRS+= vmm
# during cleanup. They start and stop a new auditd for every test CASE which
# takes a really long time on QEMU. In the latest jenkins run these tests took
# 9.5 hours to run! Skip them until we can run them at a reasonable speed.
.if ${MK_AUDIT} != "no" && ${MK_CHERI} == "no"
.if ${MK_AUDIT} != "no" && ${MACHINE_CPU:Mcheri}
_audit= audit
.endif

Expand Down

0 comments on commit de3f22d

Please sign in to comment.