Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

config: platforms-chromeos: Add serial delay for some MediaTek platforms #626

Merged
merged 1 commit into from
Jun 20, 2024

Conversation

nfraprado
Copy link
Contributor

Add test_character_delay to the Spherion and Tomato platforms to workaround the fact that they're sometimes unable to process serial input fast enough, resulting in mangled commands and consequently flaky test results, as described in
kernelci/kernelci-project#366.

The right place to do this change would be in the device-type template as described in LAVA's documentation [1]. This overriding in KernelCI is meant only as a temporary workaround to verify whether this fixes the issue. If it does, then we'll do it in LAVA upstream instead.

[1] https://docs.lavasoftware.org/lava/debugging.html#differences-in-input-speeds
Signed-off-by: Nícolas F. R. A. Prado [email protected]

@nuclearcat
Copy link
Member

Is it works fine and improve things? If it is tested, i can merge it.

@laura-nao
Copy link
Contributor

@nfraprado I just caught an instance of input overrun on corsola as well, causing a false regression being reported in the watchdog-reset test: https://lava.collabora.dev/scheduler/job/14402618#L2265 (KernelCI node: https://staging.kernelci.org:9000/viewer?node_id=6670b75ed0b7694b3998a7ea)

It'd be great to have the delay added for corsola too in this PR, so we can confirm that it fixes the watchdog-reset issue. Alternatively, I can open a separate PR to tackle this.

Add test_character_delay to the Spherion, Tomato and Steelix platforms
to workaround the fact that they're sometimes unable to process serial
input fast enough, resulting in mangled commands and consequently flaky
test results, as described in
kernelci/kernelci-project#366.

The right place to do this change would be in the device-type template
as described in LAVA's documentation [1]. This overriding in KernelCI is
meant only as a temporary workaround to verify whether this fixes the
issue. If it does, then we'll do it in LAVA upstream instead.

[1] https://docs.lavasoftware.org/lava/debugging.html#differences-in-input-speeds
Signed-off-by: Nícolas F. R. A. Prado <[email protected]>
@nfraprado nfraprado force-pushed the serial-delay-mt8192-95 branch from b9c94b4 to 71be8ba Compare June 18, 2024 15:11
@nfraprado
Copy link
Contributor Author

@laura-nao good catch! I've added corsola to the PR now.

@nfraprado nfraprado changed the title config: platforms-chromeos: Add serial delay for spherion and tomato config: platforms-chromeos: Add serial delay for some MediaTek platforms Jun 18, 2024
@nuclearcat
Copy link
Member

Merging, as it seems improve results.

@nuclearcat nuclearcat added this pull request to the merge queue Jun 20, 2024
Merged via the queue into kernelci:main with commit cd8361f Jun 20, 2024
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.

3 participants