-
Notifications
You must be signed in to change notification settings - Fork 7.5k
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
Possible compiler bug (IDFGH-6049) #5783
Comments
@olicooper Could you please attach the ELF file of the program with/without the workaround commit? (i am not sure if the ELF file you have attached has the workaround or not) |
That firmware was built on before the patch was applied so it should exhibit the problem I describe. The workaround includes a function called |
So you have all the information here is a new elf file for Arduino v1.0.6 and v2.0.0 and associated logs. The builds are based on commit 60fbacf2acfa1341cf9cec9d9d86aec16b042eb6 Arduino v2.0.0Build log
Debug log
Elf file: Arduino v1.0.6Build log
Debug log
Elf file: firmware-v106-202110201123.zip |
Hello @olicooper, adding this to roadmap and we will investigate this more. |
Any chance you can validate this again on v2.0.3-RC1? thanks! |
@olicooper Any chance you have tested this? Thanks |
My test environment uses PlatformIO, so I have validated against Arduino v2.0.2 as this is the latest release available. Is there something specific in v2.0.3-RC1 that will address this issue? The kernel panic does not seem to be occurring anymore (using a 'Lolin 32 Lite' MCU) but I get an assert issue now. The assert can be found here Build output:
Debug output:
Compiling for Arduino v1.0.6 it works fine and I get the following debug output:
I don't have the 'Heltec Wireless Stick' to test with anymore, so I have used a 'Lolin 32 Lite' with an external RFM95W lora module instead. Without the exact hardware I cannot be sure that the kernel panic issue has been resolved but the kernel panic occurs on the 'Lolin 32 Lite' when using the original arduino-upstream platform so I assume it is fixed now? This is the lorawan-test repo that I am using. |
Issue added to our Roadmap to be tested. Thanks for providing more info. |
Hi @olicooper, I'm having trouble running the project under Platform IO (I never used it before). |
Closing as expired |
Environment
main
branch v4.4-dev-2313-gc69f0ec32 /mem-optimized
branch v4.4-dev-3401-gb86fe0c66c(esp-2021r1) 8.4.0
(maybe platformio version number??) ORcrosstool-ng-1.22.0-97-gc752ad5
Problem Description
I recently opened an issue with the
MCCI Arduino LoRaWAN
library here: mcci-catena/arduino-lorawan#181After some debugging it has become apparent that there seems to be a compiler issue rather than an issue with the library.
I am unsure of the details as @terrillmoore is the library maintainer and understands the issue better than me (see this comment).
Expected Behavior
Program should run as normal (as it did in Arduino v1.0.6).
Actual Behavior
There are no build or link errors, but at runtime kernel panic occurs with a LoadStoreError.
Steps to reproduce
I created a repo for testing here: https://github.com/olicooper/arduino-v2-esp32-platformio/tree/60fbacf2acfa1341cf9cec9d9d86aec16b042eb6
It is a PlatformIO project that runs Arduino v2.0.0 - be sure to use commit
60fbacf2acfa1341cf9cec9d9d86aec16b042eb6
as the latest commit has a workaround for the compiler issue.This is the fix that was applied to the library: olicooper/arduino-lorawan@8e834e7 based on this comment: mcci-catena/arduino-lorawan#181 (comment)
All the details can be found in the original ticket.
Other items if possible
build
folder: firmware-202110181418.zipThe text was updated successfully, but these errors were encountered: