This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [RFC/TileGX 2/6] simplify the handling of skip prologue for plt stub


On 02/20/2013 12:10 PM, Yao Qi wrote:
Is it possible that the section layouts on two pages? I mean, if is possible that NEXT_ADDR is within section FOO and page A, but the end of section FOO is within page A + 1. If this is true, we need to check to the min (page boundary, section boundary), otherwise, we don't have to worry about it.

sure, and I guess the page boundary check is for the risk that the next page is not in memory that need to be paged in.
but some of the time, we just need to analyze a few instructions then we could get the result, so we only cross a page when necessary, but this do not make sense for disk file access.


after a second think, I fell it's reasonable that "section_table_xfer_memory_partial" do not handle those gap between sections, because there is no bit on the disk file for those gap, while if the debuggee is loaded and under running, then target_read_memory will use ptrace to fetch runtime memory, then those gap has physical map in memory, and set to zero.

for x86, this is a issue also. for a simple testcase

char *fmt = "x%d\n";
int main(int argc, char **argv)
{
        printf(fmt, argc);
        return 0;
}

gcc test.c
gdb a.out
(gdb) x/10 fmt
0x4005c0 <__dso_handle+8>: 174335352 Cannot access memory at address 0x4005c4
(gdb) b main
Breakpoint 1 at 0x4004e0
(gdb) r
Starting program: /home/jiwang/GDB-TEST/a.out
Breakpoint 1, 0x00000000004004e0 in main ()
(gdb) x/10 fmt
0x4005c0 <__dso_handle+8>: 174335352 0 990059265 44
0x4005d0: 4 -552 72 -236
0x4005e0: 112 -184


so, I think fix this issue by checking section boundary simultaneously is a bit strange, the clean and proper way is to stop skip_prologue analysis when the pc is in plt stub.

below is the old patch, any one comments on this?


gdb/ChangeLog:


       * tilegx-tdep.c (tilegx_skip_prologue): simplify the handling for
       plt stub.

--
Regards,
Jiong. Wang
Tilera Corporation.

---
 gdb/tilegx-tdep.c |    4 ++++
 1 file changed, 4 insertions(+)

diff --git a/gdb/tilegx-tdep.c b/gdb/tilegx-tdep.c
index 2c4e349..2452232 100644
--- a/gdb/tilegx-tdep.c
+++ b/gdb/tilegx-tdep.c
@@ -758,6 +758,10 @@ tilegx_skip_prologue (struct gdbarch *gdbarch, CORE_ADDR start_pc)
         return max (start_pc, post_prologue_pc);
     }
 
+  /* for plt stub, just return the start pc */
+  if (in_plt_section (start_pc, NULL))
+    return start_pc;
+
   /* Otherwise, try to skip prologue the hard way.  */
   return tilegx_analyze_prologue (gdbarch,
 				  start_pc,
-- 
1.7.10.4


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]