Toggle support for QStartNoAckMode

daniel5151 · Mar 30, 2023 · 0fc4819 · 0fc4819
1 parent d69da3a
commit 0fc4819
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 1 deletion.
diff --git a/example_no_std/src/gdb.rs b/example_no_std/src/gdb.rs
@@ -22,6 +22,12 @@ impl Target for DummyTarget {
         target::ext::base::BaseOps::MultiThread(self)
     }
 
+    // disable `QStartNoAckMode`
+    #[inline(always)]
+    fn use_no_ack_mode(&self) -> bool {
+        false
+    }
+
     // disable X packet optimization in order to save space
     #[inline(always)]
     fn use_x_upcase_packet(&self) -> bool {

diff --git a/src/stub/core_impl/base.rs b/src/stub/core_impl/base.rs
@@ -103,9 +103,12 @@ impl<T: Target, C: Connection> GdbStubImpl<T, C> {
                 res.write_str(concat!(
                     ";vContSupported+",
                     ";multiprocess+",
-                    ";QStartNoAckMode+",
                 ))?;
 
+                if target.use_no_ack_mode() {
+                    res.write_str(";QStartNoAckMode+",)?;
+                }
+
                 if let Some(resume_ops) = target.base_ops().resume_ops() {
                     let (reverse_cont, reverse_step) = match resume_ops {
                         ResumeOps::MultiThread(ops) => (

diff --git a/src/target/mod.rs b/src/target/mod.rs
@@ -543,6 +543,14 @@ pub trait Target {
         true
     }
 
+    /// Enable/disable `QStartNoAckMode`
+    ///
+    /// By default, this method returns `true`.
+    #[inline(always)]
+    fn use_no_ack_mode(&self) -> bool {
+        true
+    }
+
     /// Whether `gdbstub` should provide a "stub" `resume` implementation on
     /// targets without support for resumption.
     ///
@@ -728,6 +736,7 @@ macro_rules! impl_dyn_target {
             __delegate!(fn guard_rail_single_step_gdb_behavior(&self) -> SingleStepGdbBehavior);
 
             __delegate!(fn use_x_upcase_packet(&self) -> bool);
+            __delegate!(fn use_no_ack_mode(&self) -> bool);
             __delegate!(fn use_resume_stub(&self) -> bool);
             __delegate!(fn use_rle(&self) -> bool);
             __delegate!(fn use_target_description_xml(&self) -> bool);