-
-
Notifications
You must be signed in to change notification settings - Fork 3.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
WLED Crashes and Reboots #3609
Comments
Sorry, cannot reproduce. |
🤔 the common part of this problem description is performing UI actions (changing color, change global brightness) on 8266. Depending on how the UI sliders were used ("dragged" or "tapping") this could create a load of WS messages. We have a known issue on 8266 when many UI events are received in short time, and memory is low. Maybe these problems are related: #3443, #3458, #3382, #3492 |
I just find it strange how downgrading has fixed it, updating to latest beta release same issue occurs. |
I can confirm the problem. This problem only occurs in 0.14.1-b1, it is not a problem with 0.14.0. |
Pleas check available heap prior to crash and report. |
I was able to reproduce, though it is sporadic and is not consistently reproducible.
If you do not need websockets and/or MQTT please compile ESP8266 version without websockets and MQTT to free some RAM. |
Forgot to mention I'm using a D1 Mini also as I see someone mentioned and managed to reproduce. Thanks to all looking into this. |
it happened to me with an ESP32 too. I will try to give more information in a few days. |
I'm seeing this as mentioned in #3613, this is on a brand new d1 mini esp32, mounted in a DigUno with a 5V 6A PSU. |
If you want a speedy resolution, get a debug build and post crash dump something similar to above. |
Hi @blazoncek |
I ran in to an issue just like this yesterday and tracked it back to the Segment backup copy in deserializeSegment incorrectly free'ing the original Segment's FX data, resulting in a use-after-free that corrupted the heap -- ie. exactly the issue @blazoncek just posted a fix for with 5ebc345. I was going to send a PR today but it looks like it's already been taken care of. I can confirm that that patch fixed it for me. |
@willmmiles you seem capable. care to help? |
Sure, what do you need? As far as I can tell, the patch you've written does the trick for fixing the heap crashes following UI config updates. |
On d1mini, 50led ws2801, power on. |
Nothing in particular but we'd need people that understand the code. There are plenty of TODOs in the code. |
Same problem.
|
@zigomatichub @Ucsus please read above. Fix has been committed. |
hi thx |
This has been fixed in latest master source, so debug not needed, debug firmware is the same, just used to log data so can find the issue more easily. You need to download the latest source file and compile the firmware yourself. If not able to create your own, downgrade to Master 0.14.0 release until 0.14.1 Master is released. :) |
Get the B2, it looks like it's fixed |
Got the B2 installed and let it run the whole night with candle multi and its still "burning", nice |
I think the joy was premature and too great, but unfortunately I still have the problem, now it takes longer until it occurs, but after a certain time it restarts and then lights up in standard orange. |
I'm also observing occasional reboots on my ESP8266 setup. I think it's a different issue, though. I've been trying to pin it down for about a week -- it's definitely not related to the FX or transition logic, nor is it a heap exhaustion issue (I've enabled the allocator instrumentation to be sure) -- though applying heap pressure does seem to make it more likely to occur, which leads me to believe we might be looking for another use-after-free somewhere in the network layers. |
Thanks @willmmiles for troubleshooting. ATM I am afraid all of the bells and whistles we added to 0.14 may be a bit too much (paired with newer ESP core needed for 0.14) for poor ESP8266. If possible switch to ESP32 or use WLED 0.13.3 for the time being. |
I'm beginning to suspect it's an internal bug in the newer ESP core. Still investigating; these "hard wdt" crashes don't have much to say on the console, can take a long time to reproduce, and I'm still learning how to elicit more useful debugging information. |
@willmmiles We can see heap corruption (#3641 ) on some ESP8266 which happens somewhere in TCP code. I'd be glad if your expertise can help. |
What happened?
Changing colours can cause WLED to crash and reboot (Lost connection to device warning).
To Reproduce Bug
Crashed on me when using Bouncing Balls effect and selecting a new colour.
Crashed when Aura was selected and tried to increase brightness in the effect settings slider.
Meteor Smooth Crashed when changed colour
Expected Behavior
Not to crash and reboot and effect colour changes
Install Method
Self-Compiled
What version of WLED?
0.14.1.b1
Which microcontroller/board are you seeing the problem on?
ESP8266
Relevant log/trace output
No response
Anything else?
Downgraded to Gold Release of 14.0 and everything working as expected.
Says I self compiled which I didn't unless that means also downloading the file from here and installing, wasn't edited in anyway nor did I install it via the web installer.
Code of Conduct
The text was updated successfully, but these errors were encountered: