Re: [PATCH v5 0/3] Add metric value validation test

From: Namhyung Kim
Date: Wed Jun 21 2023 - 20:09:13 EST


Hello,

On Tue, Jun 20, 2023 at 10:00 AM Weilin Wang <weilin.wang@xxxxxxxxx> wrote:
>
> This is the fifth version of metric value validation tests.
>
> We made the following changes from v4 to v5:
> - Update "()" to "{}" to avoid creating sub shell and successfully skip test on non-Intel
> platform. [Ravi]
>
> v4: https://lore.kernel.org/lkml/20230618172820.751560-1-weilin.wang@xxxxxxxxx/
>
> Weilin Wang (3):
> perf test: Add metric value validation test
> perf test: Add skip list for metrics known would fail
> perf test: Rerun failed metrics with longer workload

Tested-by: Namhyung Kim <namhyung@xxxxxxxxxx>

Thanks,
Namhyung


$ ./perf test -v validation

107: perf metrics value validation :
--- start ---
test child forked, pid 1900992
Launch python validation script ./tests/shell/lib/perf_metric_validation.py
Output will be stored in: /tmp/__perf_test.program.Mm9Rw
Starting perf collection
...
Workload: perf bench futex hash -r 2 -s
Total metrics collected: 200
Non-negative metric count: 200
Total Test Count: 100
Passed Test Count: 100
Test validation finished. Final report:
[
{
"Workload": "perf bench futex hash -r 2 -s",
"Report": {
"Metric Validation Statistics": {
"Total Rule Count": 100,
"Passed Rule Count": 100
},
"Tests in Category": {
"PositiveValueTest": {
"Total Tests": 200,
"Passed Tests": 200,
"Failed Tests": []
},
"RelationshipTest": {
"Total Tests": 5,
"Passed Tests": 5,
"Failed Tests": []
},
"SingleMetricTest": {
"Total Tests": 95,
"Passed Tests": 95,
"Failed Tests": []
}
},
"Errors": []
}
}
]
test child finished with 0
---- end ----
perf metrics value validation: Ok