If the initialization of the uncore power management fails, mark the test as skipped rather than a failure. This prevents false failures in the case where we are running as a normal user, who doesn't have permissions to edit uncore power states.
Signed-off-by: Bruce Richardson <bruce.richard...@intel.com> --- app/test/test_power_intel_uncore.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/app/test/test_power_intel_uncore.c b/app/test/test_power_intel_uncore.c index 049658627d..973dc7a945 100644 --- a/app/test/test_power_intel_uncore.c +++ b/app/test/test_power_intel_uncore.c @@ -35,7 +35,7 @@ static int check_power_uncore_init(void) "may occur if environment is not configured " "correctly(APCI cpufreq) or operating in another valid " "Power management environment\n", VALID_PKG, VALID_DIE); - return -1; + return TEST_SKIPPED; } /* Unsuccessful Test */ @@ -257,8 +257,11 @@ test_power_intel_uncore(void) } ret = check_power_uncore_init(); - if (ret < 0) + if (ret != 0) { + if (ret == TEST_SKIPPED) + return TEST_SKIPPED; goto fail_all; + } ret = check_power_get_uncore_freq(); if (ret < 0) -- 2.48.1