From 12a838c6d4b9232e495285edee76636f87cf32e5 Mon Sep 17 00:00:00 2001 From: Paul Fertser Date: Wed, 17 May 2017 11:03:19 +0300 Subject: [PATCH] server: gdb: always print target information in multi-target setups When multiple targets are declared, it's not always obvious which target the connection was made for, this can lead to very confusing errors. Reported by zjason on IRC. Change-Id: I52906320394e89cb6cfe82054a3f94b27c999689 Signed-off-by: Paul Fertser Reviewed-on: http://openocd.zylin.com/4135 Tested-by: jenkins --- src/server/gdb_server.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/server/gdb_server.c b/src/server/gdb_server.c index 626179c3..fc0868b0 100644 --- a/src/server/gdb_server.c +++ b/src/server/gdb_server.c @@ -991,7 +991,9 @@ static int gdb_new_connection(struct connection *connection) } gdb_actual_connections++; - LOG_DEBUG("New GDB Connection: %d, Target %s, state: %s", + log_printf_lf(all_targets->next != NULL ? LOG_LVL_INFO : LOG_LVL_DEBUG, + __FILE__, __LINE__, __func__, + "New GDB Connection: %d, Target %s, state: %s", gdb_actual_connections, target_name(target), target_state_name(target)); -- 2.39.5