Uh oh!
There was an error while loading. Please reload this page.
check-kmod-load-unload: use TOPDIR, don't rely on PATH - #808
Conversation
marc-hb
commented
Nov 22, 2021
https://sof-ci.01.org/softestpr/PR808/build924/devicetest/?model=APL_UP2_NOCODEC&testcase=check-suspend-resume-with-capture-5 is 5.6-rc1 regression thesofproject/linux#3283 The alsabat failure is also unrelated, this PR changes only one, different test. |
TOPDIR can be re-used to find other test files. Signed-off-by: Marc Herbert <marc.herbert@intel.com>
This was apparently the only test in run-all-tests.sh that required a PATH change. Signed-off-by: Marc Herbert <marc.herbert@intel.com>
This is the place where this "sudo performance" issue is the most visible Signed-off-by: Marc Herbert <marc.herbert@intel.com>
Big regressions in https://sof-ci.01.org/softestpr/PR808/build927/devicetest/ unrelated to this, see thesofproject/linux#3292 for more details. EDIT: fixed by revert thesofproject/linux#3294 |
marc-hb
commented
Nov 24, 2021
SOFCI TEST |
greg-intel
commented
Nov 24, 2021
I'm not sure of the specific intent, but all the changes more or less make sense to me. I don't see any obvious issues, assuming the following isn't related: |
marc-hb
commented
Nov 24, 2021
https://sof-ci.01.org/softestpr/PR808/build929/devicetest/?model=CML_RVP_SDW&testcase=check-suspend-resume-with-playback-5 is "preemptible" BUG thesofproject/linux#3283
Everything else is green. |
Last question, @marc-hb, Did the GitHub actions test these script changes? |
greg-intel
left a comment
There was a problem hiding this comment.
Changes look fairly straightforward.
marc-hb
commented
Dec 2, 2021
Minor, zero functional change up for 10 days in review : merging. |
This was apparently the only test in run-all-tests.sh that required a PATH change.