-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[BUG] esp32s3-devkit:smp's ostest fails without warning after #14722 #14749
Comments
…tion" This reverts commit f2aeb5e. Because regressions are reported: * apache#14722 (comment) * apache#14722 (comment) * apache#14749
@tmedicci ostest on esp32s3-devkit:smp doesn't work reliably for me, even with the change in question reverted. does it work better for you? |
…tion" This reverts commit f2aeb5e. Because regressions are reported: * #14722 (comment) * #14722 (comment) * #14749
it seems it was because of an unrelated issue. |
Hi @yamt , there were nested issues pending. I reported that |
do you mean you have no idea how reliably it successes? |
i've run the test several times, with and without an equivalent of #14722. |
Did you test with the exact references of nuttx and nuttx-apps repos I reported? |
Description / Steps to reproduce the issue
In addition to the errors reported at #14722 (comment), there is an even critical issue regarding
ostest
for ESP32-S3 (and, eventually, for other archs): it fails without returning any error code after applying this PR.How to Reproduce
Build/flash NuttX
f2aeb5e56ff6ec7ba15958403289e462a8c84ab8
with Apps0c467dc02d1f03f3f9f3defb16f36cb4f53b4c9d
, with:And run
ostest
:The firmware asserted, but the return code of
ostest
is still0
! This assertion occurs only with this firmware version. So, there are two open questions to be investigated: why is this test asserting after this PR and whyostest
returns0
even after assertion.On which OS does this issue occur?
[OS: Linux]
What is the version of your OS?
Manjaro
NuttX Version
f2aeb5e
Issue Architecture
[Arch: xtensa]
Issue Area
[Area: Applications]
Verification
The text was updated successfully, but these errors were encountered: