Skip to content

Commit 0db448a

Browse files
committed
fix(test status): return packed skip results from entry paths
- Status reporting now expects packed RESULT_* values. - Several early-skip paths in test entry functions returned raw TEST_SKIP(0x7), which is not encoded and was printed as STATUS:0x00000007STATUS:0x00000007 in logs. - Fix this by replacing raw return TEST_SKIP; with return RESULT_SKIP(0); in affected entry paths. Signed-off-by: Sujana M <sujana.murali@arm.com> Change-Id: I0974ec8a673f1aef62decd96f6d4e0384e6e534e
1 parent 918d6db commit 0db448a

3 files changed

Lines changed: 3 additions & 3 deletions

File tree

test_pool/drtm/interface010.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ interface010_entry(uint32_t num_pe)
102102

103103
if (status != ACS_STATUS_SKIP) {
104104
if (val_gic_its_configure() != ACS_STATUS_PASS)
105-
return TEST_SKIP;
105+
return RESULT_SKIP(0);
106106
/* execute payload, which will execute relevant functions on current and other PEs */
107107
payload(num_pe);
108108
}

test_pool/drtm/interface011.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ interface011_entry(uint32_t num_pe)
9191

9292
if (status != ACS_STATUS_SKIP) {
9393
if (val_gic_its_configure() != ACS_STATUS_PASS)
94-
return TEST_SKIP;
94+
return RESULT_SKIP(0);
9595
/* execute payload, which will execute relevant functions on current and other PEs */
9696
payload(num_pe);
9797
}

val/src/test_wrappers.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -444,7 +444,7 @@ v_l1wk_02_05_entry(uint32_t num_pe)
444444
{
445445
#ifdef TARGET_LINUX
446446
// Test not applicable for Linux target
447-
return TEST_SKIP;
447+
return RESULT_SKIP(0);
448448
#endif
449449

450450
if (acs_policy_get_el1skiptrap_mask() & EL1SKIPTRAP_CNTPCT) {

0 commit comments

Comments
 (0)