feat(flightcontroller): Make MAVFTP more robust on F4 processors

Author: amilcarlucas

ardupilot_methodic_configurator/backend_flightcontroller.py

@@ -31,7 +31,7 @@
 from ardupilot_methodic_configurator.annotate_params import Par
 from ardupilot_methodic_configurator.argparse_check_range import CheckRange
 from ardupilot_methodic_configurator.backend_flightcontroller_info import BackendFlightcontrollerInfo
-from ardupilot_methodic_configurator.backend_mavftp import MAVFTP
+from ardupilot_methodic_configurator.backend_mavftp import MAVFTP, MAVFTPSettings
 
 # pylint: disable=too-many-lines
 
@@ -596,21 +596,82 @@ def __download_params_via_mavlink(
                 break
         return parameters
 
+    def _create_robust_mavftp_settings(self) -> MAVFTPSettings:
+        """
+        Create robust MAVFTP settings optimized for STM32 F4 controllers under workload.
+
+        Returns:
+            MAVFTPSettings: Configured settings with increased timeouts and reduced burst sizes
+
+        """
+        return MAVFTPSettings(
+            [
+                ("debug", int, 0),
+                ("pkt_loss_tx", int, 0),
+                ("pkt_loss_rx", int, 0),
+                ("max_backlog", int, 3),  # Reduced backlog for better reliability
+                ("burst_read_size", int, 60),  # Smaller burst size for stability
+                ("write_size", int, 60),
+                ("write_qsize", int, 3),
+                ("idle_detection_time", float, 8.0),  # Increased for heavy workload scenarios
+                ("read_retry_time", float, 3.0),  # Increased retry time
+                ("retry_time", float, 2.0),  # Increased retry time
+            ]
+        )
+
     def download_params_via_mavftp(
         self, progress_callback: Union[None, Callable[[int, int], None]] = None
     ) -> tuple[dict[str, float], dict[str, "Par"]]:
         if self.master is None:
             return {}, {}
-        mavftp = MAVFTP(self.master, target_system=self.master.target_system, target_component=self.master.target_component)
+
+        # Create more robust MAVFTP settings for parameter download
+        # STM32 F4 controllers under workload need longer timeouts
+        mavftp_settings = self._create_robust_mavftp_settings()
+
+        mavftp = MAVFTP(
+            self.master,
+            target_system=self.master.target_system,
+            target_component=self.master.target_component,
+            settings=mavftp_settings,
+        )
 
         def get_params_progress_callback(completion: float) -> None:
             if progress_callback is not None and completion is not None:
                 progress_callback(int(completion * 100), 100)
 
         complete_param_filename = "complete.param"
         default_param_filename = "00_default.param"
-        mavftp.cmd_getparams([complete_param_filename, default_param_filename], progress_callback=get_params_progress_callback)
-        ret = mavftp.process_ftp_reply("getparams", timeout=10)
+
+        # Retry logic for increased robustness with STM32 F4 controllers under workload
+        max_retries = 3
+        base_timeout = 15
+
+        for attempt in range(max_retries):
+            try:
+                mavftp.cmd_getparams(
+                    [complete_param_filename, default_param_filename], progress_callback=get_params_progress_callback
+                )
+                # Progressive timeout increase for each retry attempt
+                timeout = base_timeout * (2**attempt)  # 15s, 30s, 60s
+                logging_info(_("Attempt %d/%d: Requesting parameters with %ds timeout"), attempt + 1, max_retries, timeout)
+                ret = mavftp.process_ftp_reply("getparams", timeout=timeout)
+
+                if ret.error_code == 0:
+                    break  # Success, exit retry loop
+
+                if attempt < max_retries - 1:  # Don't sleep after last attempt
+                    sleep_time = 2**attempt  # 1s, 2s, 4s exponential backoff
+                    logging_warning(_("Parameter download attempt %d failed, retrying in %ds..."), attempt + 1, sleep_time)
+                    time_sleep(sleep_time)
+                else:
+                    logging_error(_("All %d parameter download attempts failed"), max_retries)
+
+            except Exception as e:
+                logging_error(_("Exception during parameter download attempt %d: %s"), attempt + 1, str(e))
+                if attempt < max_retries - 1:
+                    sleep_time = 2**attempt
+                    time_sleep(sleep_time)
         pdict = {}
         # add a file sync operation to ensure the file is completely written
         time_sleep(0.3)
@@ -1077,14 +1138,23 @@ def upload_file(
         """Upload a file to the flight controller."""
         if self.master is None:
             return False
-        mavftp = MAVFTP(self.master, target_system=self.master.target_system, target_component=self.master.target_component)
+
+        # Use robust MAVFTP settings for better reliability with STM32 F4 controllers
+        mavftp_settings = self._create_robust_mavftp_settings()
+        mavftp = MAVFTP(
+            self.master,
+            target_system=self.master.target_system,
+            target_component=self.master.target_component,
+            settings=mavftp_settings,
+        )
 
         def put_progress_callback(completion: float) -> None:
             if progress_callback is not None and completion is not None:
                 progress_callback(int(completion * 100), 100)
 
         mavftp.cmd_put([local_filename, remote_filename], progress_callback=put_progress_callback)
-        ret = mavftp.process_ftp_reply("CreateFile", timeout=10)
+        # Increased timeout for better reliability
+        ret = mavftp.process_ftp_reply("CreateFile", timeout=20)
         if ret.error_code != 0:
             ret.display_message()
         return ret.error_code == 0
@@ -1104,7 +1174,14 @@ def download_last_flight_log(
             logging_error(error_msg)
             return False
 
-        mavftp = MAVFTP(self.master, target_system=self.master.target_system, target_component=self.master.target_component)
+        # Use robust MAVFTP settings for better reliability with STM32 F4 controllers
+        mavftp_settings = self._create_robust_mavftp_settings()
+        mavftp = MAVFTP(
+            self.master,
+            target_system=self.master.target_system,
+            target_component=self.master.target_component,
+            settings=mavftp_settings,
+        )
 
         def get_progress_callback(completion: float) -> None:
             if progress_callback is not None and completion is not None:
@@ -1193,7 +1270,7 @@ def _get_log_number_by_scanning(self, mavftp: MAVFTP) -> Union[int, None]:
                 # Test if this log file exists
                 temp_test_file = f"temp_test_{mid}.tmp"
                 mavftp.cmd_get([remote_filename, temp_test_file])
-                ret = mavftp.process_ftp_reply("OpenFileRO", timeout=5)  # Must be > idle_detection_time (3.7s)
+                ret = mavftp.process_ftp_reply("OpenFileRO", timeout=2)  # Short timeout for existence check
 
                 # Clean up the temp file if it was created
                 if os.path.exists(temp_test_file):