From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 1983) id 9AC693858C3A; Tue, 17 Jan 2023 16:47:56 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 9AC693858C3A DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1673974076; bh=4GPTSsca5a+m7iTaCR7WiwlpKdC4D8NrjcKwCjCQp7M=; h=From:To:Subject:Date:From; b=UABq5Li9VHWziBEnnK1AcX4whL5Tw+NBnKaS9ognITAIgwBLTly3J/iewYG3iKcLs Dcdx9Eg9Bn6Xp41lMBbrmu7ExPzdCQy9LjhwAZ35pmyRHIUc5s3ar9WItlnREUNIJG XQ7UWHkb9SxpTTlY0gnpOAO37h+KY5EJr/e+jFtE= Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable From: Carl Love To: gdb-cvs@sourceware.org Subject: [binutils-gdb] PowerPC: fix for gdb.reverse/finish-precsave.exp and gdb.reverse/finish-reverse.exp X-Act-Checkin: binutils-gdb X-Git-Author: Carl Love X-Git-Refname: refs/heads/master X-Git-Oldrev: b22548ddb30bfb167708e82d3bb932461c1b703a X-Git-Newrev: 92e07580db6a5572573d5177ca23933064158f89 Message-Id: <20230117164756.9AC693858C3A@sourceware.org> Date: Tue, 17 Jan 2023 16:47:56 +0000 (GMT) List-Id: https://sourceware.org/git/gitweb.cgi?p=3Dbinutils-gdb.git;h=3D92e07580db6a= 5572573d5177ca23933064158f89 commit 92e07580db6a5572573d5177ca23933064158f89 Author: Carl Love Date: Fri Jan 13 17:59:33 2023 -0500 PowerPC: fix for gdb.reverse/finish-precsave.exp and gdb.reverse/finish= -reverse.exp =20 PR record/29927 - reverse-finish requires two reverse next instructions= to reach previous source line =20 PowerPC uses two entry points called the local entry point (LEP) and the global entry point (GEP). Normally the LEP is used when calling a function. However, if the table of contents (TOC) value in register 2 = is not valid the GEP is called to setup the TOC before execution continues= at the LEP. When executing in reverse, the function finish_backward sets = the break point at the alternate entry point (GEP). However if the forward execution enters via the normal entry point (LEP), the reverse execution never sees the break point at the GEP of the function. Reverse executi= on continues until the next break point is encountered or the end of the recorded log is reached causing gdb to stop at the wrong place. =20 This patch adds a new address to struct execution_control_state to hold= the address of the alternate function start address, known as the GEP on PowerPC. The finish_backwards function is updated. If the stopping po= int is between the two entry points (the LEP and GEP on PowerPC), the stepp= ing range is set to execute back to the alternate entry point (GEP on Power= PC). Otherwise, a breakpoint is inserted at the normal entry point (LEP on PowerPC). =20 Function process_event_stop_test checks uses a stepping range to stop execution in the caller at the first instruction of the source code lin= e. Note, on systems that only support one entry point, the address of the = two entry points are the same. =20 Test finish-reverse-next.exp is updated to include tests for the reverse-finish command when the function is entered via the normal entry point (i.e. the LEP) and the alternate entry point (i.e. the GEP). =20 The patch has been tested on X86 and PowerPC with no regressions. Diff: --- gdb/infcmd.c | 40 ++++++---- gdb/infrun.c | 16 +++- gdb/testsuite/gdb.reverse/finish-reverse-next.c | 39 ++++++++- gdb/testsuite/gdb.reverse/finish-reverse-next.exp | 96 +++++++++++++++++++= +--- 4 files changed, 160 insertions(+), 31 deletions(-) diff --git a/gdb/infcmd.c b/gdb/infcmd.c index 5fcd2abd983..978c07f176d 100644 --- a/gdb/infcmd.c +++ b/gdb/infcmd.c @@ -1720,22 +1720,25 @@ finish_backward (struct finish_command_fsm *sm) sal =3D find_pc_line (func_addr, 0); =20 frame_info_ptr frame =3D get_selected_frame (nullptr); + struct gdbarch *gdbarch =3D get_frame_arch (frame); + CORE_ADDR alt_entry_point =3D sal.pc; + CORE_ADDR entry_point =3D alt_entry_point; =20 - if (sal.pc !=3D pc) + if (gdbarch_skip_entrypoint_p (gdbarch)) { - struct gdbarch *gdbarch =3D get_frame_arch (frame); - - /* Set a step-resume at the function's entry point. Once that's - hit, we'll do one more step backwards. */ - symtab_and_line sr_sal; - sr_sal.pc =3D sal.pc; - sr_sal.pspace =3D get_frame_program_space (frame); - insert_step_resume_breakpoint_at_sal (gdbarch, - sr_sal, null_frame_id); + /* Some architectures, like PowerPC use local and global entry point= s. + There is only one Entry Point (GEP =3D LEP) for other architectures. + The GEP is an alternate entry point. The LEP is the normal entry + point. The value of entry_point was initialized to the alternate + entry point (GEP). It will be adjusted if the normal entry point + (LEP) was used. */ + entry_point =3D gdbarch_skip_entrypoint (gdbarch, entry_point); } - else + + if (alt_entry_point <=3D pc && pc <=3D entry_point) { - /* We are exactly at the function entry point. Note that this + /* We are exactly at the function entry point, or between the entry + point on platforms that have two (like PowerPC). Note that this can only happen at frame #0. =20 When setting a step range, need to call set_step_info @@ -1744,8 +1747,17 @@ finish_backward (struct finish_command_fsm *sm) =20 /* Return using a step range so we will keep stepping back to the first instruction in the source code line. */ - tp->control.step_range_start =3D sal.pc; - tp->control.step_range_end =3D sal.pc; + tp->control.step_range_start =3D alt_entry_point; + tp->control.step_range_end =3D entry_point; + } + else + { + symtab_and_line sr_sal; + /* Set a step-resume at the function's entry point. */ + sr_sal.pc =3D entry_point; + sr_sal.pspace =3D get_frame_program_space (frame); + insert_step_resume_breakpoint_at_sal (gdbarch, + sr_sal, null_frame_id); } proceed ((CORE_ADDR) -1, GDB_SIGNAL_DEFAULT); } diff --git a/gdb/infrun.c b/gdb/infrun.c index 05b150b1b63..9c81521683c 100644 --- a/gdb/infrun.c +++ b/gdb/infrun.c @@ -1868,6 +1868,7 @@ struct execution_control_state =20 struct target_waitstatus ws; int stop_func_filled_in =3D 0; + CORE_ADDR stop_func_alt_start =3D 0; CORE_ADDR stop_func_start =3D 0; CORE_ADDR stop_func_end =3D 0; const char *stop_func_name =3D nullptr; @@ -4663,6 +4664,12 @@ fill_in_stop_func (struct gdbarch *gdbarch, &block); ecs->stop_func_name =3D gsi =3D=3D nullptr ? nullptr : gsi->print_na= me (); =20 + /* PowerPC functions have a Local Entry Point and a Global Entry + Point. There is only one Entry Point (GEP =3D LEP) for other + architectures. Save the alternate entry point address (GEP) for + use later. */ + ecs->stop_func_alt_start =3D ecs->stop_func_start; + /* The call to find_pc_partial_function, above, will set stop_func_start and stop_func_end to the start and end of the range containing the stop pc. If this range @@ -4679,6 +4686,9 @@ fill_in_stop_func (struct gdbarch *gdbarch, +=3D gdbarch_deprecated_function_start_offset (gdbarch); =20 if (gdbarch_skip_entrypoint_p (gdbarch)) + /* The PowerPC architecture uses two entry points. Stop at the + regular entry point (LEP on PowerPC) initially. Will setup a + breakpoint for the alternate entry point (GEP) later. */ ecs->stop_func_start =3D gdbarch_skip_entrypoint (gdbarch, ecs->stop_func_start); } @@ -6754,7 +6764,7 @@ process_event_stop_test (struct execution_control_sta= te *ecs) =20 /* Return using a step range so we will keep stepping back to the first instruction in the source code line. */ - tp->control.step_range_start =3D ecs->stop_func_start; + tp->control.step_range_start =3D ecs->stop_func_alt_start; tp->control.step_range_end =3D ecs->stop_func_start; keep_going (ecs); return; @@ -6891,8 +6901,10 @@ process_event_stop_test (struct execution_control_st= ate *ecs) (unless it's the function entry point, in which case keep going back to the call point). */ CORE_ADDR stop_pc =3D ecs->event_thread->stop_pc (); + if (stop_pc =3D=3D ecs->event_thread->control.step_range_start - && stop_pc !=3D ecs->stop_func_start + && (stop_pc < ecs->stop_func_alt_start + || stop_pc > ecs->stop_func_start) && execution_direction =3D=3D EXEC_REVERSE) end_stepping_range (ecs); else diff --git a/gdb/testsuite/gdb.reverse/finish-reverse-next.c b/gdb/testsuit= e/gdb.reverse/finish-reverse-next.c index f90ecbb93cb..0347906961d 100644 --- a/gdb/testsuite/gdb.reverse/finish-reverse-next.c +++ b/gdb/testsuite/gdb.reverse/finish-reverse-next.c @@ -24,11 +24,37 @@ This test verifies the fix for gdb bugzilla: =20 https://sourceware.org/bugzilla/show_bug.cgi?id=3D29927 -*/ + + PowerPC supports two entry points to a function. The normal entry point + is called the local entry point (LEP). The alternat entry point is cal= led + the global entry point (GEP). The GEP is only used if the table of + contents (TOC) value stored in register r2 needs to be setup prior to + execution starting at the LEP. A function call via a function pointer + will entry via the GEP. A normal function call will enter via the LEP. + + This test has been expanded to include tests to verify the reverse-fini= sh + command works properly if the function is called via the GEP. The orig= inal + test only verified the reverse-finish command for a normal call that us= ed + the LEP. */ =20 int function1 (int a, int b) // FUNCTION1 { + /* The assembly code for this function when compiled for PowerPC is as + follows: + + 0000000010000758 : + 10000758: 02 10 40 3c lis r2,4098 <- GEP + 1000075c: 00 7f 42 38 addi r2,r2,32512 + 10000760: a6 02 08 7c mflr r0 <- LEP + 10000764: 10 00 01 f8 std r0,16(r1) + .... + + When the function is called on PowerPC with function1 (a, b) the call + enters at the Local Entry Point (LEP). When the function is called v= ia + a function pointer, the Global Entry Point (GEP) for function1 is use= d. + The GEP sets up register 2 before reaching the LEP. + */ int ret =3D 0; =20 ret =3D a + b; @@ -39,10 +65,19 @@ int main(int argc, char* argv[]) { int a, b; + int (*funp) (int, int) =3D &function1; + + /* Call function via Local Entry Point (LEP). */ =20 a =3D 1; b =3D 5; =20 - function1 (a, b); // CALL FUNCTION + function1 (a, b); // CALL VIA LEP + + /* Call function via Global Entry Point (GEP). */ + a =3D 10; + b =3D 50; + + funp (a, b); // CALL VIA GEP return 0; } diff --git a/gdb/testsuite/gdb.reverse/finish-reverse-next.exp b/gdb/testsu= ite/gdb.reverse/finish-reverse-next.exp index 63305c109e1..a9c895dfcd4 100644 --- a/gdb/testsuite/gdb.reverse/finish-reverse-next.exp +++ b/gdb/testsuite/gdb.reverse/finish-reverse-next.exp @@ -31,6 +31,16 @@ # This test verifies the fix for gdb bugzilla: # https://sourceware.org/bugzilla/show_bug.cgi?id=3D29927 =20 +# PowerPC supports two entry points to a function. The normal entry point +# is called the local entry point (LEP). The alternat entry point is call= ed +# the global entry point (GEP). A function call via a function pointer +# will entry via the GEP. A normal function call will enter via the LEP. +# +# This test has been expanded to include tests to verify the reverse-finish +# command works properly if the function is called via the GEP. The origi= nal +# test only verified the reverse-finish command for a normal call that used +# the LEP. + if ![supports_reverse] { return } @@ -50,30 +60,30 @@ if [supports_process_record] { } =20 =20 -### TEST 1: reverse finish from the entry point instruction in -### function1. +### TEST 1: reverse finish from the entry point instruction (LEP) in +### function1 when called using the normal entry point (LEP). =20 # Set breakpoint at call to function1 in main. -set bp_FUNCTION [gdb_get_line_number "CALL FUNCTION" $srcfile] -gdb_breakpoint $srcfile:$bp_FUNCTION temporary +set bp_LEP_test [gdb_get_line_number "CALL VIA LEP" $srcfile] +gdb_breakpoint $srcfile:$bp_LEP_test temporary =20 # Continue to break point at function1 call in main. gdb_continue_to_breakpoint \ "stopped at function1 entry point instruction to stepi into function" \ - ".*$srcfile:$bp_FUNCTION\r\n.*" + ".*$srcfile:$bp_LEP_test\r\n.*" =20 # stepi until we see "{" indicating we entered function1 -repeat_cmd_until "stepi" "CALL FUNCTION" "{" "stepi into function1 call" +repeat_cmd_until "stepi" "CALL VIA LEP" "{" "stepi into function1 call" =20 -gdb_test "reverse-finish" ".*function1 \\(a, b\\); // CALL FUNCTION.*" \ - "reverse-finish function1 " +gdb_test "reverse-finish" ".*function1 \\(a, b\\); // CALL VIA LEP.*" \ + "reverse-finish function1 LEP call from LEP " =20 # Check to make sure we stopped at the first instruction in the source code # line. It should only take one reverse next command to get to the previo= us # source line. If GDB stops at the last instruction in the source code l= ine # it will take two reverse next instructions to get to the previous source # line. -gdb_test "reverse-next" ".*b =3D 5;.*" "reverse next at b =3D 5, call from= function" +gdb_test "reverse-next" ".*b =3D 5;.*" "reverse next at b =3D 5, call from= LEP" =20 # Clear the recorded log. gdb_test "record stop" "Process record is stopped.*" \ @@ -84,21 +94,81 @@ gdb_test_no_output "record" "turn on process record for= test2" ### TEST 2: reverse finish from the body of function1. =20 # Set breakpoint at call to function1 in main. -gdb_breakpoint $srcfile:$bp_FUNCTION temporary +gdb_breakpoint $srcfile:$bp_LEP_test temporary =20 # Continue to break point at function1 call in main. gdb_continue_to_breakpoint \ "at function1 entry point instruction to step to body of function" \ - ".*$srcfile:$bp_FUNCTION\r\n.*" + ".*$srcfile:$bp_LEP_test\r\n.*" =20 # do a step instruction to get to the body of the function gdb_test "step" ".*int ret =3D 0;.*" "step test 1" =20 -gdb_test "reverse-finish" ".*function1 \\(a, b\\); // CALL FUNCTION.*" \ - "reverse-finish function1 call from function body" +gdb_test "reverse-finish" ".*function1 \\(a, b\\); // CALL VIA LEP.*" \ + "reverse-finish function1 LEP call from function body" =20 # Check to make sure we stopped at the first instruction in the source code # line. It should only take one reverse next command to get to the previo= us # source line. gdb_test "reverse-next" ".*b =3D 5;.*" \ "reverse next at b =3D 5, from function body" + +# Turn off record to clear logs and turn on again +gdb_test "record stop" "Process record is stopped.*" \ + "turn off process record for test2" +gdb_test_no_output "record" "turn on process record for test3" + + +### TEST 3: reverse finish from the alternate entry point instruction (GEP= ) in +### function1 when called using the alternate entry point (GEP). + +# Set breakpoint at call to funp in main. +set bp_GEP_test [gdb_get_line_number "CALL VIA GEP" $srcfile] +gdb_breakpoint $srcfile:$bp_GEP_test temporary + +# Continue to break point at funp call in main. +gdb_continue_to_breakpoint \ + "stopped at function1 entry point instruction to stepi into funp" \ + ".*$srcfile:$bp_GEP_test\r\n.*" + +# stepi until we see "{" indicating we entered function. +repeat_cmd_until "stepi" "CALL VIA GEP" "{" "stepi into funp call" + +gdb_test "reverse-finish" ".*funp \\(a, b\\);.*" \ + "function1 GEP call call from GEP" + +# Check to make sure we stopped at the first instruction in the source code +# line. It should only take one reverse next command to get to the previo= us +# source line. If GDB stops at the last instruction in the source code li= ne +# it will take two reverse next instructions to get to the previous source +# line. +gdb_test "reverse-next" ".*b =3D 50;.*" "reverse next at b =3D 50, call fr= om GEP" + +# Turn off record to clear logs and turn on again +gdb_test "record stop" "Process record is stopped.*" \ + "turn off process record for test3" +gdb_test_no_output "record" "turn on process record for test4" + + +### TEST 4: reverse finish from the body of function 1 when calling using = the +### alternate entrypoint (GEP). +gdb_breakpoint $srcfile:$bp_GEP_test temporary + +# Continue to break point at funp call. +gdb_continue_to_breakpoint \ + "at function1 entry point instruction to step to body of funp call" \ + ".*$srcfile:$bp_GEP_test\r\n.*" + +# Step into body of funp, called via GEP. +gdb_test "step" ".*int ret =3D 0;.*" "step test 2" + +gdb_test "reverse-finish" ".*funp \\(a, b\\);.*" \ + "reverse-finish function1 GEP call, from function body " + +# Check to make sure we stopped at the first instruction in the source code +# line. It should only take one reverse next command to get to the previo= us +# source line. If GDB stops at the last instruction in the source code li= ne +# it will take two reverse next instructions to get to the previous source +# line. +gdb_test "reverse-next" ".*b =3D 50;.*" \ + "reverse next at b =3D 50 from function body"