FROMLIST: misc: fastrpc: create duplicate sessions after all CB probing#718
Open
quic-vkatoch wants to merge 1 commit into
Open
FROMLIST: misc: fastrpc: create duplicate sessions after all CB probing#718quic-vkatoch wants to merge 1 commit into
quic-vkatoch wants to merge 1 commit into
Conversation
For ADSP, only a limited number of FastRPC context banks (CBs) are available. Each CB supports a single session, which means only a few processes can run on ADSP simultaneously. If all sessions are consumed by fastrpc daemons, no session remains available when a user application starts, causing the application to fail. To address this limitation, a Device Tree change was used till now: qcom,nsessions = <5>; However, feedback from the upstream community indicated that this change should not be made in the Device Tree. Instead, it was recommended to handle this as a driver-level change. Instead of duplicating sessions inline during fastrpc_cb_probe() using the qcom,nsessions DT property, defer duplication until after of_platform_populate() returns in fastrpc_rpmsg_probe(), at which point all compute-CB child nodes have been probed and the session array is fully populated. For the ADSP domain, append FASTRPC_DUP_SESSIONS (4) copies of the last probed session once of_platform_populate() succeeds. This keeps the per-CB probe path simple and ensures duplicates are always derived from a stable, fully-initialised session state. The qcom,nsessions DT property is no longer consumed by the driver; the binding and DT sources are left unchanged. Link: https://lore.kernel.org/all/20260609-dup-sessions-v1-1-26934abb9fa3@oss.qualcomm.com/ Signed-off-by: Vinayak Katoch <vinayak.katoch@oss.qualcomm.com>
|
Merge Check Failed: CR Not Eligible for Merge CR 4508505 is not eligible for merge. The parent software image for kernel.qli.2.0 is not development complete. Entity: Please ensure the CR passes both CCT (ComponentChangeTasks) and ICT (Integration Change Tasks) validations. |
ekanshibu
approved these changes
Jun 16, 2026
Test Matrix
|
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
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.
Patch: Replacing the qcom,nsessions DT property with a driver-level approach that appends FASTRPC_DUP_SESSIONS copies of the last probed session for the ADSP domain.
Link: https://lore.kernel.org/all/20260609-dup-sessions-v1-1-26934abb9fa3@oss.qualcomm.com/
CRs-Fixed: 4508505