misc: fastrpc: reduce munmap error log during DSP process exit - #114
Open
Jianping (Jianping-Li) wants to merge 1 commit into
Open
misc: fastrpc: reduce munmap error log during DSP process exit#114Jianping (Jianping-Li) wants to merge 1 commit into
Jianping (Jianping-Li) wants to merge 1 commit into
Conversation
A munmap request can race with userPD teardown on the DSP: the heap GC thread issues the reverse unmap after the process group has already been released. Depending on how far teardown has progressed, the DSP returns either AEE_ENOSUCHPROCESS (0x80000441), when the process group is already gone, or AEE_EBADSTATE (0x8000040d), when the listener has been torn down and the reverse RPC can no longer be delivered. In both cases the memory is still reclaimed by the DSP process release path, so these are expected conditions during process exit rather than mapping failures. Log them at debug level instead of error level to avoid noise in dmesg. Signed-off-by: Jianping Li <jianping.li@oss.qualcomm.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
A munmap request can race with userPD teardown on the DSP: the heap GC thread issues the reverse unmap after the process group has already been released. Depending on how far teardown has progressed, the DSP returns either AEE_ENOSUCHPROCESS (0x80000441), when the process group is already gone, or AEE_EBADSTATE (0x8000040d), when the listener has been torn down and the reverse RPC can no longer be delivered.
In both cases the memory is still reclaimed by the DSP process release path, so these are expected conditions during process exit rather than mapping failures. Log them at debug level instead of error level to avoid noise in dmesg.