Skip to content

Conversation

@PLangowski
Copy link
Contributor

@PLangowski PLangowski commented Sep 22, 2025

Add parameter which sets a threshold for the average latency value. If avg_ms exceeds this value, the test will fail.

@PLangowski PLangowski marked this pull request as draft September 22, 2025 08:45
@sofci
Copy link
Collaborator

sofci commented Sep 22, 2025

Can one of the admins verify this patch?

reply test this please to run this test once

@marc-hb
Copy link
Collaborator

marc-hb commented Sep 22, 2025

test this please

@PLangowski PLangowski force-pushed the latency-threshold branch 4 times, most recently from 7a96d74 to 3d72f8e Compare September 29, 2025 12:00
@marc-hb marc-hb removed request for a team, golowanow, lgirdwood and marc-hb September 29, 2025 16:46
@PLangowski PLangowski marked this pull request as ready for review September 30, 2025 07:16
@PLangowski PLangowski marked this pull request as draft September 30, 2025 08:35
@PLangowski PLangowski force-pushed the latency-threshold branch 2 times, most recently from 13def7a to fbfa14f Compare September 30, 2025 09:31
@PLangowski PLangowski marked this pull request as ready for review September 30, 2025 09:31
@PLangowski
Copy link
Contributor Author

Tested - works

@PLangowski PLangowski changed the title Latency threshold latency-metrics.sh: Add latency threshold parameter Oct 6, 2025
Comment on lines 91 to 92
OPT_NAME['l']='latency_threshold' OPT_DESC['l']='latency threshold in ms'
OPT_HAS_ARG['l']=1 OPT_VAL['l']=85
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Align all opts in lines starting from top.

The parameter sets a threshold for the average latency value. If
`avg_ms` exceeds this value, the test will fail.

Signed-off-by: Pawel Langowski <pawelx.langowski@intel.com>
@majunkier majunkier merged commit 2af5ed8 into thesofproject:main Oct 6, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants