Skip to content

check-suspend-resume: sleep 1s before rtcwake iteration - #651

Merged
fredoh9 merged 1 commit into
thesofproject:mainfrom
aiChaoSONG:workaround_timeout
Apr 19, 2021
Merged

fredoh9 merged 1 commit into
thesofproject:mainfrom
aiChaoSONG:workaround_timeout

Conversation

@aiChaoSONG

Copy link
Copy Markdown

After some kernel update, we start to observe a timeout
issue on several JF devices. As we are not going to fix
the kernel, this workaround is applied.

Accroding to the solution in this patch, the device may
get into sleep before netcat can send out messages, this
maybe related to network issue in the kernel, or network
route is too long between JF and SH, etc. In this patch,
we add a 1-second sleep to wait a stable connection before
running rtcwake.

Workaround: #650

Signed-off-by: Chao Song chao.song@linux.intel.com

After some kernel update, we start to observe a timeout
issue on several JF devices. As we are not going to fix
the kernel, this workaround is applied.

Accroding to the solution in this patch, the device may
get into sleep before netcat can send out messages, this
maybe related to network issue in the kernel, or network
route is too long between JF and SH, etc. In this patch,
we add a 1-second sleep to wait a stable connection before
running rtcwake.

Workaround: thesofproject#650

Signed-off-by: Chao Song <chao.song@linux.intel.com>
@aiChaoSONG
aiChaoSONG requested a review from a team as a code owner April 19, 2021 09:42
@keqiaozhang

Copy link
Copy Markdown
Contributor

Nice workaround, I tested it on several JF devices and it works.

@fredoh9 fredoh9 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Good a finding and workaround. Let's merge this and see

@fredoh9
fredoh9 merged commit ad939ac into thesofproject:main Apr 19, 2021
@marc-hb

marc-hb commented Apr 19, 2021

Copy link
Copy Markdown
Collaborator

Thanks for clearly flagging this as a workaround and adding the URL.

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.

4 participants