This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [RFC] Detect loops in the solib chain
- From: Tom Tromey <tromey at redhat dot com>
- To: Jan Kratochvil <jan dot kratochvil at redhat dot com>
- Cc: gdb-patches at sourceware dot org, Paul Pluzhnikov <ppluzhnikov at google dot com>, Kevin Buettner <kevinb at redhat dot com>
- Date: Fri, 23 Apr 2010 14:09:43 -0600
- Subject: Re: [RFC] Detect loops in the solib chain
- References: <20080718160210.66b480e9@mesquite.lan> <20080717220959.GA10652@caradoc.them.org> <20100409154123.GA8218@host0.dyn.jankratochvil.net> <20100409210255.GA26135@host0.dyn.jankratochvil.net>
- Reply-to: tromey at redhat dot com
>>>>> "Jan" == Jan Kratochvil <jan.kratochvil@redhat.com> writes:
Jan> 2010-04-09 Daniel Jacobowitz <dan@codesourcery.com>
Jan> Paul Pluzhnikov <ppluzhnikov@google.com>
Jan> Jan Kratochvil <jan.kratochvil@redhat.com>
Jan> Fix deadlock on looped list of loaded shared objects.
Jan> * solib-svr4.c (LM_PREV): New function.
Jan> (IGNORE_FIRST_LINK_MAP_ENTRY): Use it.
Jan> (svr4_current_sos): Check for correct l_prev. New variables prev_lm
Jan> and next_lm. Clear prev_lm for solib_svr4_r_ldsomap.
Jan> * config/djgpp/fnchange.lst: Add translation for solib-corrupted.exp.
Kevin already ok'd this part.
(FWIW I agree it is ok.)
Jan> 2010-04-09 Jan Kratochvil <jan.kratochvil@redhat.com>
Jan> Fix deadlock on looped list of loaded shared objects.
Jan> * gdb.base/solib-corrupted.exp: New.
Jan> + # glibc debug info is not available and it is too difficult to find and
Jan> + # parse it from this testcase without the gdb supporting functions.
Jan> + xfail "$test (no _r_debug symbol)"
Jan> + untested ${testfile}.exp
Jan> + return
Jan> + }
Jan> + -re " = 0x\[0-9a-f\]+\r\n$gdb_prompt $" {
Jan> + pass $test
I think it is preferable to have the test name the same, regardless of
whether it passes or xfails. Extra info can be logged with verbose.
This is ok with that change.
thanks,
Tom