This is an automated email from Gerrit.

"Tim Newsome <t...@sifive.com>" just uploaded a new patch set to Gerrit, which 
you can find at https://review.openocd.org/c/openocd/+/6979

-- gerrit

commit 17187148035acd9954d8e11a875f7e5d9644e9cb
Author: Tim Newsome <t...@sifive.com>
Date:   Mon May 16 10:22:32 2022 -0700

    Give each SMP group a unique number.
    
    This helps e.g. if there are 8 cores, and cores 0--3 are in one SMP
    group while 4--7 are in another group. (And there are 2 gdb instances
    connected, one debugging the first group, and one the second.)
    
    Signed-off-by: Tim Newsome <t...@sifive.com>
    Change-Id: I7b6c9382eadf964529105eaf0411a42d48768668

diff --git a/src/target/target.c b/src/target/target.c
index d2dff111a5..03a669c9ba 100644
--- a/src/target/target.c
+++ b/src/target/target.c
@@ -6441,6 +6441,7 @@ static int jim_target_smp(Jim_Interp *interp, int argc, 
Jim_Obj *const *argv)
        int i;
        const char *targetname;
        int retval, len;
+       static int smp_group = 1;
        struct target *target = NULL;
        struct target_list *head, *new;
 
@@ -6472,9 +6473,10 @@ static int jim_target_smp(Jim_Interp *interp, int argc, 
Jim_Obj *const *argv)
        /*  now parse the list of cpu and put the target in smp mode*/
        foreach_smp_target(head, lh) {
                target = head->target;
-               target->smp = 1;
+               target->smp = smp_group;
                target->smp_targets = lh;
        }
+       smp_group++;
 
        if (target && target->rtos)
                retval = rtos_smp_init(head->target);
diff --git a/src/target/target.h b/src/target/target.h
index 1f1a354207..45ef962694 100644
--- a/src/target/target.h
+++ b/src/target/target.h
@@ -200,7 +200,7 @@ struct target {
        bool rtos_auto_detect;                          /* A flag that 
indicates that the RTOS has been specified as "auto"
                                                                                
 * and must be detected when symbols are offered */
        struct backoff_timer backoff;
-       int smp;                                                        /* add 
some target attributes for smp support */
+       int smp;                                                        /* 
Unique non-zero number for each SMP group */
        struct list_head *smp_targets;          /* list all targets in this smp 
group/cluster
                                                                                
 * The head of the list is shared between the
                                                                                
 * cluster, thus here there is a pointer */

-- 

Reply via email to