This is an automated email from Gerrit.

Antonio Borneo ([email protected]) just uploaded a new patch set to 
Gerrit, which you can find at http://openocd.zylin.com/5234

-- gerrit

commit c16ea6ce020f89fcb59b830b3ace8d65b7152dad
Author: Antonio Borneo <[email protected]>
Date:   Fri Jun 14 15:59:17 2019 +0200

    armv8: check the core state to pass the correct arch to gdb
    
    Commit 3799eded6761 ("target/aarch64: add support for
    multi-architecture gdb") passes the constant string "aarch64" as
    architecture to gdb. This is not working if the core is running
    in 32 bits mode; gdb reports:
        Truncated register 8 in remote 'g' packet
    then closes the connection with OpenOCD.
    
    Make the architecture string dependant from the current state of
    the core.
    
    Change-Id: I16e1614ea02ba29bf87f450b3dfe25c83c9a3612
    Signed-off-by: Antonio Borneo <[email protected]>

diff --git a/src/target/armv8.c b/src/target/armv8.c
index e736937..88b9320 100644
--- a/src/target/armv8.c
+++ b/src/target/armv8.c
@@ -1753,7 +1753,8 @@ const struct command_registration 
armv8_command_handlers[] = {
 
 const char *armv8_get_gdb_arch(struct target *target)
 {
-       return "aarch64";
+       struct arm *arm = target_to_arm(target);
+       return arm->core_state == ARM_STATE_AARCH64 ? "aarch64" : "arm";
 }
 
 int armv8_get_gdb_reg_list(struct target *target,

-- 


_______________________________________________
OpenOCD-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openocd-devel

Reply via email to