This is an automated email from Gerrit.

"Daniel Goehring <dgoeh...@os.amperecomputing.com>" just uploaded a new patch 
set to Gerrit, which you can find at https://review.openocd.org/c/openocd/+/8800

-- gerrit

commit 6ad7cb362088de96cb5861df926da4e599b4052f
Author: Daniel Goehring <dgoeh...@os.amperecomputing.com>
Date:   Wed Mar 5 17:48:01 2025 -0500

    target/target: fix RTOS thread awareness support
    
    This prior patch replaces "LOG_xxx()" with "LOG_TARGET_xxx()"
    to indicate which target the message belongs to.
    
    commit 7f2db80ebc16 ("rtos/hwthread: Use LOG_TARGET_xxx()")
    
    To support this change for hardware thread awareness,
    the target command name needs to be established before
    calling the "target_configure()" routine.
    
    Change-Id: I0dc70c23b84e983a2ee694fb5b9d01758f5c84a3
    Signed-off-by: Daniel Goehring <dgoeh...@os.amperecomputing.com>

diff --git a/src/target/target.c b/src/target/target.c
index 9c5a33d3d2..73fa3d35de 100644
--- a/src/target/target.c
+++ b/src/target/target.c
@@ -5823,6 +5823,16 @@ static int target_create(struct jim_getopt_info *goi)
        target->gdb_port_override = NULL;
        target->gdb_max_connections = 1;
 
+       cp = Jim_GetString(new_cmd, NULL);
+       target->cmd_name = strdup(cp);
+       if (!target->cmd_name) {
+               LOG_ERROR("Out of memory");
+               free(target->trace_info);
+               free(target->type);
+               free(target);
+               return JIM_ERR;
+       }
+
        /* Do the rest as "configure" options */
        goi->is_configure = true;
        e = target_configure(goi, target);
@@ -5859,19 +5869,6 @@ static int target_create(struct jim_getopt_info *goi)
                target->endianness = TARGET_LITTLE_ENDIAN;
        }
 
-       cp = Jim_GetString(new_cmd, NULL);
-       target->cmd_name = strdup(cp);
-       if (!target->cmd_name) {
-               LOG_ERROR("Out of memory");
-               rtos_destroy(target);
-               free(target->gdb_port_override);
-               free(target->trace_info);
-               free(target->type);
-               free(target->private_config);
-               free(target);
-               return JIM_ERR;
-       }
-
        if (target->type->target_create) {
                e = (*(target->type->target_create))(target, goi->interp);
                if (e != ERROR_OK) {

-- 

Reply via email to