ipc: gdb: initialise GDB after IPC completion

To avoid timed out IPC on the host side, enter the GDB stub only
after replying to the IPC.

Signed-off-by: Guennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>

Author: lyakh

src/include/sof/ipc/common.h

@@ -243,4 +243,7 @@ void ipc_msg_reply(struct sof_ipc_reply *reply);
  */
 void ipc_complete_cmd(struct ipc *ipc);
 
+/* GDB stub: should enter GDB after completing the IPC processing */
+extern bool ipc_enter_gdb;
+
 #endif /* __SOF_DRIVERS_IPC_H__ */

src/ipc/ipc-common.c

@@ -9,6 +9,7 @@
 #include <sof/audio/component_ext.h>
 #include <sof/audio/pipeline.h>
 #include <sof/common.h>
+#include <sof/debug/gdb/gdb.h>
 #include <rtos/idc.h>
 #include <rtos/symbol.h>
 #include <sof/ipc/topology.h>
@@ -352,6 +353,8 @@ void ipc_complete_cmd(struct ipc *ipc)
 	ipc_platform_complete_cmd(ipc);
 }
 
+bool ipc_enter_gdb;
+
 static void ipc_complete_task(void *data)
 {
 	struct ipc *ipc = data;
@@ -361,6 +364,13 @@ static void ipc_complete_task(void *data)
 	ipc->task_mask &= ~IPC_TASK_INLINE;
 	ipc_complete_cmd(ipc);
 	k_spin_unlock(&ipc->lock, key);
+#if CONFIG_GDBSTUB
+	if (ipc_enter_gdb) {
+		ipc_enter_gdb = false;
+		k_msleep(5);
+		gdb_init();
+	}
+#endif
 }
 
 static enum task_state ipc_do_cmd(void *data)

src/ipc/ipc3/handler.c

@@ -973,7 +973,7 @@ static int ipc_glb_gdb_debug(uint32_t header)
 	(void) header;
 
 #if CONFIG_GDBSTUB
-	gdb_init();
+	ipc_enter_gdb = true;
 	return 0;
 // TODO: remove old GDB stub?
 #elif CONFIG_GDB_DEBUG