Skip to content

check-sof-logger.sh: cat correct error file on mtrace failure - #1086

Merged
fredoh9 merged 1 commit into
thesofproject:mainfrom
marc-hb:fix-mtrace-check-logger
Jul 18, 2023
Merged

check-sof-logger.sh: cat correct error file on mtrace failure#1086
fredoh9 merged 1 commit into
thesofproject:mainfrom
marc-hb:fix-mtrace-check-logger

Conversation

@marc-hb

Copy link
Copy Markdown
Collaborator

Fixes commit 1431303 ("check-sof-logger: add support for mtrace
log with IPC4")

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

Fixes commit 1431303 ("check-sof-logger: add support for mtrace
log with IPC4")
Signed-off-by: Marc Herbert <marc.herbert@intel.com>
@marc-hb
marc-hbforce-pushed the fix-mtrace-check-logger branch from ad1e0b2 to 6a2277aCompareJuly 18, 2023 09:14
@marc-hb

marc-hb commented Jul 18, 2023

Copy link
Copy Markdown
CollaboratorAuthor

https://sof-ci.01.org/softestpr/PR1086/build617/devicetest/index.html shows the fix in action thanks to temporary test hack 6a2277a

@marc-hb
marc-hb requested a review from keqiaozhangJuly 18, 2023 09:30
@marc-hb
marc-hbforce-pushed the fix-mtrace-check-logger branch from 6a2277a to 989e264CompareJuly 18, 2023 09:31
@marc-hb
marc-hb marked this pull request as ready for review July 18, 2023 09:32
@marc-hb
marc-hb requested a review from a team as a code ownerJuly 18, 2023 09:32
@marc-hb
marc-hb requested a review from ujfalusiJuly 18, 2023 09:38
@marc-hb

marc-hb commented Jul 18, 2023

Copy link
Copy Markdown
CollaboratorAuthor

@marc-hb
marc-hb requested a review from aiChaoSONGJuly 18, 2023 12:58

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

Right, IPC4 (mtrace) is logging to etrace files...

@fredoh9
fredoh9 merged commit bd0f9cd into thesofproject:mainJul 18, 2023
@marc-hb
marc-hb deleted the fix-mtrace-check-logger branch July 19, 2023 05:58
Sign up for freeto 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

@marc-hb@ujfalusi@fredoh9