dp: clear memory domain when freeing#10676
Conversation
lgirdwood
left a comment
lgirdwood
left a comment
There was a problem hiding this comment.
I guess we need the west update too here ?
@lgirdwood no, the currently used Zephyr version already includes it |
lyakh
changed the title
|
zephyrproject-rtos/zephyr#106923 has been merged and the current west.yml already points to a version, containing it. This should be safe now. |
lyakh
requested review from
LaurentiuM1234,
dbaluta and
marcinszkudlinski
as code owners
| /* All partitions removed, the domain can be freed now */ | ||
| pmod->mdom = NULL; | ||
| k_mem_domain_deinit(mdom); | ||
| objpool_free(&dp_mdom_head, mdom); |
There was a problem hiding this comment.
@lyakh can you resolve this, I think we are good then.
There was a problem hiding this comment.
@lgirdwood changed to not call scheduler_dp_domain_free() when domain initialisation fails
|
@lrudyX WCL test device had a problem? |
I have triggered manual rerun. That was DUT problem. |
Recently a new function has been added to Zephyr to safely remove memory domains. Use it when releasing domains in SOF DP. Signed-off-by: Guennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>
|
@lrudyX IIUC "build" cannot run without "codecheck" and here "build" has already failed while "codecheck" is still expected - looks like a networking issue to me? |
Recently a new function has been added to Zephyr to safely remove memory domains. Use it when releasing domains in SOF DP.