diff --git a/gdb/arm-tdep.h b/gdb/arm-tdep.h index 23dd40ea8beb1b00289a4cd4e65647399d351580..c56ed5dc6508a9c0994eebe1c90e60ede2df892f 100644 --- a/gdb/arm-tdep.h +++ b/gdb/arm-tdep.h @@ -259,6 +259,7 @@ ULONGEST arm_get_next_pcs_read_memory_unsigned_integer (CORE_ADDR memaddr, CORE_ADDR arm_get_next_pcs_addr_bits_remove (struct arm_get_next_pcs *self, CORE_ADDR val); +int check_section_name (struct obj_section *); int arm_get_next_pcs_is_thumb (struct arm_get_next_pcs *self); diff --git a/gdb/arm-tdep.c b/gdb/arm-tdep.c index d244707210628ab045f677c0cbad3d8b0c6d6269..b51663031b45bda1df61ed6caca37deb531a5cd7 100644 --- a/gdb/arm-tdep.c +++ b/gdb/arm-tdep.c @@ -8212,6 +8212,53 @@ arm_get_longjmp_target (struct frame_info *frame, CORE_ADDR *pc) return 1; } +/* A call to cmse secure entry function "foo" at "a" is modified by + GNU ld as "b". + a) bl xxxx + + + xxxx: + + b) bl yyyy <__acle_se_foo> + + section .gnu.sgstubs: + + yyyy: sg // secure gateway + b.w xxxx <__acle_se_foo> // original_branch_dest + + <__acle_se_foo> + xxxx: + + When the control at "b", the pc contains "yyyy" (sg address) which is a + trampoline and does not exist in source code. This function returns + the target pc "xxxx". */ + +static CORE_ADDR +arm_skip_sg_jump_to_bw (CORE_ADDR pc, const char *name, struct objfile *objfile) +{ + struct bound_minimal_symbol minsym; + char *target_name; + int target_len = strlen (name) + strlen ("__acle_se_") + 1; + target_name = (char *) alloca (target_len); + snprintf (target_name, target_len, "%s%s", "__acle_se_",name); + minsym = lookup_minimal_symbol (target_name, NULL, objfile); + if (minsym.minsym != NULL) + return BMSYMBOL_VALUE_ADDRESS (minsym); + return 0; +} + +/* Return 1 when pc holds an address that belongs to ".gnu.sgstubs" + section. */ +int +check_section_name (struct obj_section *sec) +{ + if (sec != NULL && sec->the_bfd_section != NULL + && sec->the_bfd_section->name != NULL + && !strcmp (sec->the_bfd_section->name,".gnu.sgstubs")) + return 1; + return 0; +} + /* Recognize GCC and GNU ld's trampolines. If we are in a trampoline, return the target PC. Otherwise return 0. */ @@ -8221,6 +8268,7 @@ arm_skip_stub (struct frame_info *frame, CORE_ADDR pc) const char *name; int namelen; CORE_ADDR start_addr; + struct obj_section *section; /* Find the starting address and name of the function containing the PC. */ if (find_pc_partial_function (pc, &name, &start_addr, NULL) == 0) @@ -8290,6 +8338,10 @@ arm_skip_stub (struct frame_info *frame, CORE_ADDR pc) return 0; } + section = find_pc_section (pc); + /* checks whether target pc belows to ".gnu.sgstubs" section. */ + if (check_section_name (section)) + return arm_skip_sg_jump_to_bw (pc, name, section->objfile); return 0; /* not a stub */ } diff --git a/gdb/testsuite/gdb.base/arm-main-cmse.c b/gdb/testsuite/gdb.base/arm-main-cmse.c new file mode 100644 index 0000000000000000000000000000000000000000..bde55075fdc21bc781a5dd6ac8221a550faca4c5 --- /dev/null +++ b/gdb/testsuite/gdb.base/arm-main-cmse.c @@ -0,0 +1,29 @@ +#include +extern void func(); +void __acle_se_func () +{ + printf("__acle_se_func\n"); +} + +asm ("\t.section .gnu.sgstubs,\"ax\",%progbits\n" + "\t.global func\n" + "\t.type func, %function\n" + "func:\n" + "\tnop @sg\n" + "\tb __acle_se_func @b.w"); + +void fun1 () +{ + printf("In fun1\n"); +} + +int main (void) +{ + while(1) + { + func(); + fun1(); + __acle_se_func(); + } + return 0; +} diff --git a/gdb/testsuite/gdb.base/arm-main-cmse.exp b/gdb/testsuite/gdb.base/arm-main-cmse.exp new file mode 100644 index 0000000000000000000000000000000000000000..5c9abf43908f505f13151f3753f81db44b9eb6b3 --- /dev/null +++ b/gdb/testsuite/gdb.base/arm-main-cmse.exp @@ -0,0 +1,56 @@ +# Copyright 2019 Free Software Foundation, Inc. + +# This program is free software; you can redistribute it and/or modify +# it under the terms of the GNU General Public License as published by +# the Free Software Foundation; either version 3 of the License, or +# (at your option) any later version. +# +# This program is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. +# +# You should have received a copy of the GNU General Public License +# along with this program. If not, see . + +# This file is part of the gdb testsuite. + +if { ![istarget "arm*-*-*"]} { + return 1 +} + +standard_testfile arm-main-cmse.c +set exefile [standard_output_file ${testfile}.out] +set opts [list debug "additional_flags=-g -dA"] + +if { [gdb_compile ${srcdir}/${subdir}/${testfile}.c ${exefile} executable $opts] != ""} { + return -1 +} + +clean_restart ${exefile} +if ![runto_main] { + return -1 +} + +set test "branch to func from main" +gdb_test "si" "0x.*" "$test" + +set test "next instruction in func" +gdb_test "ni" "0x.*" "$test" + +set test "branch to __acle_se_func from func" +gdb_test "ni" "__acle_se_func .*" "$test" + +set test "next in __acle_se_func function" +gdb_test "next" "5 .*" "$test" + +set test "next in __acle_se_func function outputs __acle_se_func" +gdb_test "next" "__acle_se_func.*" "$test" + +gdb_test "continue" "Continuing.*" "continue to func() in main" + +set test "control branches to __acle_se_func from main via func" +gdb_test "step" "__acle_se_fun.*" "$test" + +set test "next in __acle_se_func function via func" +gdb_test "next" "__acle_se_func.*" "$test"