Skip to content
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]: T1000e shutdown fail #5817

Open
cracky22 opened this issue Jan 11, 2025 · 3 comments
Open

[Bug]: T1000e shutdown fail #5817

cracky22 opened this issue Jan 11, 2025 · 3 comments
Labels
bug Something isn't working

Comments

@cracky22
Copy link

Category

Other

Hardware

Seeed Card Tracker T1000-E

Firmware Version

v2.5.18.89ebafc

Description

meshtastic-log-2025-01-11T11-27-01.114Z.log

Relevant log output

DEBUG | 12:26:52 192 [Button] Radio wait to sleep, txEmpty=0

DEBUG | 12:26:52 192 [Button] Radio wait to sleep, txEmpty=0

DEBUG | 12:26:52 192 [Button] Radio wait to sleep, txEmpty=0

DEBUG | 12:26:52 192 [Button] Radio wait to sleep, txEmpty=0

DEBUG | 12:26:52 192 [Button] Radio wait to sleep, txEmpty=0

DEBUG | 12:26:52 192 [Button] Radio wait to sleep, txEmpty=0

DEBUG | 12:26:52 193 [Button] Radio wait to sleep, txEmpty=0

DEBUG | 12:26:52 193 [Button] Radio wait to sleep, txEmpty=0

DEBUG | 12:26:52 193 [Button] Radio wait to sleep, txEmpty=0

DEBUG | 12:26:53 193 [Button] Radio wait to sleep, txEmpty=0

DEBUG | 12:26:53 193 [Button] Radio wait to sleep, txEmpty=0

DEBUG | 12:26:53 193 [Button] Radio wait to sleep, txEmpty=0

ERROR | 12:26:53 193 [Button] NOTE! Record critical error 2 at src/sleep.cpp:176

ERROR | 12:26:53 193 [Button] assert failed src/sleep.cpp: 177, void waitEnterSleep(bool), te
@cracky22 cracky22 added the bug Something isn't working label Jan 11, 2025
@cracky22
Copy link
Author

@GUVWAF
Copy link
Member

GUVWAF commented Jan 11, 2025

I believe this is because of this:

firmware/src/sleep.cpp

Lines 171 to 172 in e7802d9

while (!doPreflightSleep()) {
delay(100); // Kinda yucky - wait until radio says say we can shutdown (finished in process sends/receives)

This will not work because during delay(), ArduinoThread will not call the NotifiedWorkerThread of RadioLibInterface and thus it won't reach TRANSMIT_DELAY_COMPLETED, keeping the Tx queue non-empty. We'd need to find a way to fix this.

@cracky22
Copy link
Author

cracky22 commented Jan 11, 2025

log from t1000e with testbuild 2.5.19.6f0ab0f60
meshtastic-log-2025-01-11T14-39-41.525Z.log

@fifieldt fifieldt changed the title [Bug]: shutdown fail [Bug]: T1000e shutdown fail Jan 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants