-
Notifications
You must be signed in to change notification settings - Fork 349
sof-logger: fail when an invalid address is found in the DMA trace #5340
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
Conversation
Let's test DMA corruption bug thesofproject#5120 for real. Signed-off-by: Marc Herbert <marc.herbert@intel.com>
|
As just explained in #5329 (comment), the corruption left _ZEPHYR tests. Now there's only something "milder" affecting CML... EDIT: filed as #5345 |
|
SOFCI TEST |
|
@marc-hb still draft ? |
|
Yes still a draft, let's focus first on issues that cause the logs to be entirely empty (#5410 etc.) PS: please don't look at drafts! They have a grey icon and can be ignored with a simple |
|
Can one of the admins verify this patch? |
|
@dbaluta , @paulstelian97 would like to take over this? Intel doesn't use the sof-logger anymore in the main branch. |
|
I'll consider taking over this patch, but I also want to get some understanding on what logging tools Intel will use from now on, as well as in how said tooling can be ported so we can use it. We'll only use sof-logger until the better tools are confirmed working for us (right now the state is that we haven't gotten them to work just yet, but sof-logger also has reliability issues when we test our Zephyr builds). Once the weekend passes I'll bring it up in an internal meeting with Daniel and see what we should do. |
|
@paulstelian97 NXP should be migrating to Zephyr tooling for logs @thesofproject/amd @thesofproject/mediatek fyi this is up to you guys on how/if you want to handle ? |
|
We should migrate to the Zephyr tools but that's something to be done further on in the future. For the moment we still have Xtos builds and sof-logger. I'll take over helping out with the logger until we stop using it. |
Let's test DMA corruption bug #5120 for real.
Signed-off-by: Marc Herbert marc.herbert@intel.com