Conversation
Srikanth Muppandam (smuppand)
left a comment
There was a problem hiding this comment.
only_hap_example_failed() now has no callers and still documents the SM8850 exception being removed. Remove the function and obsolete comment in this PR.
|
Hi Srikanth Muppandam (@smuppand), With the upcoming tag to be released for fastrpc recipe : we are enabling gpdsp test support as-well. |
2adfe81 to
d8a040c
Compare
There was a problem hiding this comment.
With these new changes, you have to fix the following code as well.
Runner/utils/lib_fastrpc.sh:151 — artifact discovery validates generic directories, not FastRPC artifacts
-
Issue: FASTRPC_RESOLVED_LIB_SYS_DIR becomes /usr/lib whenever that directory exists. Likewise, the runner checks FASTRPC_RESOLVED_SKEL_BASE,
although the base can exist without either v75 or v68. FASTRPC_RESOLVED_LIB_TEST_DIR is discovered but never required. -
Recommended fix: Resolve and validate the actual required library files, require a non-empty FASTRPC_RESOLVED_SKEL_PATH, and validate the required
test-library artifacts rather than generic directories.
Runner/suites/Multimedia/CDSP/fastrpc_test/fastrpc_test.yaml:11 — YAML still documents removed GPDSP filtering
- Issue: The description says GPDSP domains are skipped on QCS9075/QCS8275/QCS8300/QCS9100, while the PR removes that blacklist.
- Recommended fix: Describe runtime domain and endpoint discovery instead.
Runner/suites/Multimedia/CDSP/fastrpc_test/run.sh:188 — remove unused SOC_MACHINE
- Recommended fix: Remove the assignment.
d8a040c to
283b5cf
Compare
283b5cf to
7dac287
Compare
7dac287 to
f34a642
Compare
Srikanth Muppandam (smuppand)
left a comment
There was a problem hiding this comment.
anankulk Please address the pending comments too
f34a642 to
4528956
Compare
4528956 to
a14fa9c
Compare
a14fa9c to
39a5eeb
Compare
|
anankulk sign-off is missing from the recent patches. |
39a5eeb to
e58bf32
Compare
e58bf32 to
f54abca
Compare
…dation Remove hardcoded SoC-based domain filtering and per-SoC PD support maps. Use fastrpc-healthcheck as the primary capability source for domain online state and signed/unsigned PD support. Fall back to remoteproc/endpoint discovery for domains and a conservative protocol map for PD support when healthcheck is unavailable. Enables previously blacklisted platforms and derives PD support from live hardware capability. Add --healthcheck-timeout, enforce finite --timeout (default 120s), log domain_source and pd_source in [FASTRPC-POLICY], and discover DSP skeletons from actual .so presence rather than hardcoded ABI directories. Signed-off-by: Anand Kulkarni <anankulk@qti.qualcomm.com>
f54abca to
1d79009
Compare
|
anankulk Restore executable mode on run.sh. |
Summary
Remove hardcoded SoC-based domain filtering and per-SoC PD support maps.
Previously blacklisted platforms (e.g. Glymur CRD) were fully skipped
regardless of actual hardware capability. This PR replaces that approach
with runtime discovery driven by
fastrpc-healthcheck.Changes
fastrpc-healthcheckas the primary capability source for domainonline state and signed/unsigned PD support per domain
/dev/fastrpc-<domain>[-secure]endpoint presence for domain discovery when healthcheck is unavailable
(ADSP/MDSP/SDSP: signed only; CDSP/CDSP1/GDSP0/GDSP1: signed + unsigned)
domain_sourceandpd_sourcefields to[FASTRPC-POLICY]log line--healthcheck-timeoutparameter (default 15s)--timeoutwith explicit default of 120s[FASTRPC-MATRIX]log line reporting invocation count and timeout budget.sopresence, not hardcoded ABI namesValidation