[...]

>> +
>> +    if (rte_pmu_init() < 0)
>> +            return TEST_FAILED;
>
>Can we return TEST_SKIPPED here?
>On aarch64, this feature requires kernel version >= 5.17. CI setups doesn't 
>meet this requirement
>will start to report failure when running fast_tests.
>

Okay. I think that's good enough for CI. 

>> +
>> +    event = rte_pmu_add_event(name);
>> +    while (tries--)
>> +            val += rte_pmu_read(event);
>> +
>> +    rte_pmu_fini();
>> +
>> +    return val ? TEST_SUCCESS : TEST_FAILED; }
>> +
>> +static struct unit_test_suite pmu_tests = {
>> +    .suite_name = "pmu autotest",
>> +    .setup = NULL,
>> +    .teardown = NULL,
>> +    .unit_test_cases = {
>> +            TEST_CASE(test_pmu_read),
>> +            TEST_CASES_END()
>> +    }
>> +};
>> +
><snip>

Reply via email to