From cb7db0f2a917b4fe945f091f665f5eb1800e0e20 Mon Sep 17 00:00:00 2001 From: Mike Frysinger Date: Fri, 16 Apr 2010 22:47:42 +0000 Subject: [PATCH] gdb: fdpic/frv: fix shared library loading The recent change to reload_shared_libraries() broke FDPIC shared libraries as the solib-frv.c code was implicitly relying on the initial order of calls (first solib_addr() and then solib_create_inferior_hook()). It was maintaining internal state via enable_break{1,2}_done to handle this. While I could tweak these values a bit more, the original code wasn't terribly bullet proof -- if during the initial debug you attempted to view shared libraries, the enable2_break() code would whine about the ldso internal debug addresses being unfetchable (and would actually attempt to read address 0x8 on the target). So I've dropped this implicit dependency on order (i.e. enable_break1_done) and updated the ldso poking code (i.e. enable_break2) to silently return when the internal debug address is still set to 0. It will remain this way until the ldso gets a chance to initialize at which point the code will act the same as before. While I have no way of testing the FRV, the Blackfin FDPIC code is using this same base in a 100% copy & paste method since we implemented FDPIC the same way as the FRV guys (I'll address this in the future). This fix was required in order to handle shared libraries with Blackfin FDPIC properly, and I see no reason why it wouldn't also work for FRV (since the uClibc ldso FDPIC code is the same too and that's really what this is poking). Signed-off-by: Mike Frysinger --- gdb/ChangeLog | 10 ++++++++++ gdb/solib-frv.c | 33 +++++++++++++++++++++++---------- 2 files changed, 33 insertions(+), 10 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 1b5286682e7..a22cdebcea5 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,13 @@ +2010-04-16 Mike Frysinger + + * solib-frv.c (enable_break1_done): Delete. + (enable_break2): Do not check enable_break1_done. Move the + enable_break2_done setting and call to + remove_solib_event_breakpoints() to the end. Return without + warning when the contents of _dl_debug_addr are 0. + (enable_break): Do not set enable_break1_done. + (frv_clear_solib): Likewise. + 2010-04-16 Kevin Buettner * m32c-tdep.c (m32c_m16c_address_to_pointer): Print warning diff --git a/gdb/solib-frv.c b/gdb/solib-frv.c index 245ca84f763..bf4ac515e08 100644 --- a/gdb/solib-frv.c +++ b/gdb/solib-frv.c @@ -631,7 +631,6 @@ enable_break_failure_warning (void) */ -static int enable_break1_done = 0; static int enable_break2_done = 0; static int @@ -642,15 +641,9 @@ enable_break2 (void) char **bkpt_namep; asection *interp_sect; - if (!enable_break1_done || enable_break2_done) + if (enable_break2_done) return 1; - enable_break2_done = 1; - - /* First, remove all the solib event breakpoints. Their addresses - may have changed since the last time we ran the program. */ - remove_solib_event_breakpoints (); - interp_text_sect_low = interp_text_sect_high = 0; interp_plt_sect_low = interp_plt_sect_high = 0; @@ -771,6 +764,22 @@ enable_break2 (void) } addr = extract_unsigned_integer (addr_buf, sizeof addr_buf, byte_order); + if (solib_frv_debug) + fprintf_unfiltered (gdb_stdlog, + "enable_break: _dl_debug_addr[0..3] = %s\n", + hex_string_custom (addr, 8)); + + /* If it's zero, then the ldso hasn't initialized yet, and so + there are no shared libs yet loaded. */ + if (addr == 0) + { + if (solib_frv_debug) + fprintf_unfiltered (gdb_stdlog, + "enable_break: ldso not yet initialized\n"); + /* Do not warn, but mark to run again. */ + return 0; + } + /* Fetch the r_brk field. It's 8 bytes from the start of _dl_debug_addr. */ if (target_read_memory (addr + 8, addr_buf, sizeof addr_buf) != 0) @@ -800,9 +809,15 @@ enable_break2 (void) /* We're also done with the loadmap. */ xfree (ldm); + /* Remove all the solib event breakpoints. Their addresses + may have changed since the last time we ran the program. */ + remove_solib_event_breakpoints (); + /* Now (finally!) create the solib breakpoint. */ create_solib_event_breakpoint (target_gdbarch, addr); + enable_break2_done = 1; + return 1; } @@ -847,7 +862,6 @@ enable_break (void) return 0; } - enable_break1_done = 1; create_solib_event_breakpoint (target_gdbarch, symfile_objfile->ei.entry_point); @@ -997,7 +1011,6 @@ static void frv_clear_solib (void) { lm_base_cache = 0; - enable_break1_done = 0; enable_break2_done = 0; main_lm_addr = 0; if (main_executable_lm_info != 0) -- 2.30.2