ARC: nSIM: west: launch cores in direct order for simulation run

We've reversed core launch order to workaround issue of
ARConnect initialization interfere with secondary cores
startup (we don't want to workaround it in runtime as it's
only possible in case of debug session).

However it bring us new issues with the simulation run:
 - mismatch arcnum (core ID) with ARConnect ID
 - mismatch arcnum (core ID) with CPU name in nSIM instruction traces

To avoid these issues let's use direct core order for simulation
runs.

Signed-off-by: Evgeniy Paltsev <PaltsevEvgeniy@gmail.com>
Signed-off-by: Eugeniy Paltsev <Eugeniy.Paltsev@synopsys.com>

Author: evgeniy-paltsev

scripts/west_commands/runners/mdb.py

@@ -34,6 +34,16 @@ def is_flash_cmd_need_exit_immediately(mdb_runner):
     else:
         return True
 
+def smp_core_order(mdb_runner, id):
+    if is_simulation_run(mdb_runner):
+        # for simulation targets we start cores in direct order (core 0 first, core 1 second, etc...)
+        # otherwise we face mismatch arcnum (code ID) with ARConnect ID and core ID in instruction traces
+        return id
+    else:
+        # for HW targets we want to start the primary core last, to avoid ARConnect initialization interfere
+        # with secondary cores startup - so we reverse start order
+        return mdb_runner.cores - 1 - id
+
 def mdb_do_run(mdb_runner, command):
     commander = "mdb64"
 
@@ -81,7 +91,7 @@ def mdb_do_run(mdb_runner, command):
             if i > 0: mdb_sub_cmd += ['-prop=download=2']
             mdb_sub_cmd += mdb_basic_options + mdb_target + [mdb_runner.elf_name]
             mdb_runner.check_call(mdb_sub_cmd, cwd=mdb_runner.build_dir)
-            mdb_multifiles += ('core{}'.format(mdb_runner.cores-1-i) if i == 0 else ',core{}'.format(mdb_runner.cores-1-i))
+            mdb_multifiles += ('core{}' if i == 0 else ',core{}').format(smp_core_order(mdb_runner, i))
 
         # to enable multi-core aware mode for use with the MetaWare debugger,
         # need to set the NSIM_MULTICORE environment variable to a non-zero value

scripts/west_commands/tests/test_mdb.py

@@ -55,7 +55,7 @@
               '-prop=download=2', '-nooptions', '-nogoifmain',
               '-toggle=include_local_symbols=1',
               '-nsim', TEST_BOARD_NSIM_ARGS, RC_KERNEL_ELF]
-TEST_NSIM_CORES_LAUNCH = [TEST_DRIVER_CMD, '-multifiles=core1,core0',
+TEST_NSIM_CORES_LAUNCH = [TEST_DRIVER_CMD, '-multifiles=core0,core1',
               '-run', '-cl']
 
 # mdb-hw