Skip to content

check-sof-logger: add missing sof-kernel-log-check.sh - #912

Merged
plbossart merged 1 commit into
thesofproject:mainfrom
marc-hb:logger-checks-kernel
May 26, 2022
Merged

check-sof-logger: add missing sof-kernel-log-check.sh#912
plbossart merged 1 commit into
thesofproject:mainfrom
marc-hb:logger-checks-kernel

Conversation

@marc-hb

@marc-hbmarc-hb commented May 26, 2022

Copy link
Copy Markdown
Collaborator

This would have caught kernel crash
https://sof-ci.01.org/linuxpr/PR3645/build166/devicetest/?model=CML_RVP_SDW_ZEPHYR&testcase=check-sof-logger

More details in

This check is especially important considering check-sof-logger unload
and reloads kernel modules.

Signed-off-by: Marc Herbert marc.herbert@intel.com

This would have caught kernel crash
https://sof-ci.01.org/linuxpr/PR3645/build166/devicetest/?model=CML_RVP_SDW_ZEPHYR&testcase=check-sof-logger
This check is especially important considering check-sof-logger unload
and reloads kernel modules.
Signed-off-by: Marc Herbert <marc.herbert@intel.com>
@marc-hb

Copy link
Copy Markdown
CollaboratorAuthor

SOFCI TEST

@marc-hb
marc-hb marked this pull request as ready for review May 26, 2022 04:12
@marc-hb
marc-hb requested a review from a team as a code ownerMay 26, 2022 04:12
@marc-hb

Copy link
Copy Markdown
CollaboratorAuthor

The TIMEOUT in https://sof-ci.01.org/softestpr/PR912/build39/devicetest/?model=BYT_MB_NOCODEC&testcase=check-ipc-flood was in a different test: unrelated.

Everything else was green.

@marc-hb

Copy link
Copy Markdown
CollaboratorAuthor

Refreshed #799 also hardens this test and start looking at firmware ERRORs for the first time ever!

@keqiaozhangkeqiaozhang 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.

LGTM

@plbossart
plbossart merged commit 874f415 into thesofproject:mainMay 26, 2022
@marc-hb
marc-hb deleted the logger-checks-kernel branch May 27, 2022 20:08
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@marc-hb@plbossart@keqiaozhang