Skip to content

ipc: add error trace with size for page descriptors alloc - #2487

Merged
lgirdwood merged 1 commit into
thesofproject:masterfrom
xiulipan:pr/trace_page_descriptors
Mar 5, 2020
Merged

lgirdwood merged 1 commit into
thesofproject:masterfrom
xiulipan:pr/trace_page_descriptors

Conversation

@xiulipan

@xiulipan xiulipan commented Mar 5, 2020

Copy link
Copy Markdown
Contributor

Add error trace with buffer size for heap to help debug
the page descriptor issue.

Signed-off-by: Pan Xiuli xiuli.pan@linux.intel.com

Add error trace with buffer size for heap to help debug
the page descriptor issue.

Signed-off-by: Pan Xiuli <xiuli.pan@linux.intel.com>
Comment thread src/ipc/ipc-host-ptable.c
if (!elem_array->elems)
if (!elem_array->elems) {
trace_ipc_error("ipc_parse_page_descriptors() error: "
"There is no heap free with this block size: %d",

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Split string?

@paulstelian97 paulstelian97 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Actually getting a specific trace whenever we have an error is always a good thing. Other than the Checkpatch error from the split string this looks good.

@lgirdwood
lgirdwood merged commit b23ca03 into thesofproject:master Mar 5, 2020
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