Replies: 6 comments 3 replies
-
So, a couple of questions :
I've not heard of this sort of problem before, but there's always a first time. |
Beta Was this translation helpful? Give feedback.
-
Je suis désolé, je ne parle pas non plus français.J'utilise ChatGPT pour traduire ce passage, j'espère que vous pourrez comprendre. |
Beta Was this translation helpful? Give feedback.
-
Coincidentally I ran into the same SD card full situation today by triggering a screenshot which finally led to an EM (without restoring control), destroyed radio.yaml, an empty radio_error.yml and a non booting radio. Deleting some logs and restoring radio.yml from a backup got the radio working again. I think we urgently need a warning page as early as in 2.8.5 similar to no failsafe or switch warnings at startup to inform the user if the SD card usage exceeds a certain safety margin (how about 50MB ) discouraging the user to fly until there is enough spare SD storage capacity. A better but properly tested solution that doesn't end in a totally unrecoverable situation should follow. |
Beta Was this translation helpful? Give feedback.
-
Exactly what happened to my radio, replaced sd card with larger capacity 32gb and restored with back up radio now working. Lost a model due to this easily avoidable problem. Taranis running open tx gave pending warning of sd card full. This would have avoided a lot of pain,expense and time
|
Beta Was this translation helpful? Give feedback.
-
@pfeerick SD Card full is a situation that leads into a catastrophic event because the startup after the EM runs into the same problem. Maybe there are more scenarios (hopefully not) but I have tried to address the SD card full scenario. I have added warnings similar to throttle warning at startup (opentx.cpp/checkAll()) as a quick measure to at least make users aware of a present or upcoming major problem:
Of course both are warnings and can be ignored but ignoring them has the potential for a catastrophic event. I think the warnings are mandatory but not enough. EdgeTX should be hardened against low SD storage events, i.e. defining an absolute minimum SD card free memory threshold, monitoring SD card free memory and taking appropriate action once the threshold is violated. Like shutting down logging, shutting down yaml writing, no more screenshots etc. Please advise if starting with warnings is the right thing to do and if I you want me to proceed with a PR. |
Beta Was this translation helpful? Give feedback.
-
Hi AllYes absolutely agree, 50mb sounds about right as a safe buffer to trigger a warning as well👍RegardsDouglas MooreSent from my iPhoneOn 28 Jun 2023, at 08:15, Peter Feerick ***@***.***> wrote:
The warnings as suggested sound perfect in the first instance. The first one (not mounted) may not be needed, as there is already a No SD card warning (unless that doesn't trigger if say the SD card was corrupt / wrong FS format).
Yes, logging and screenshots should be disabled/blocked at say 50MB of free space remaining, nothing else should need to be stopped as they are the consumers.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: ***@***.***>
|
Beta Was this translation helpful? Give feedback.
-
@tipiouk wrote:
Beta Was this translation helpful? Give feedback.
All reactions