From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 10038 invoked by alias); 25 Aug 2009 08:47:39 -0000 Received: (qmail 10026 invoked by uid 22791); 25 Aug 2009 08:47:37 -0000 X-SWARE-Spam-Status: No, hits=-2.4 required=5.0 tests=BAYES_00,SARE_SUB_OBFU_Q1,SPF_HELO_PASS,SPF_PASS X-Spam-Check-By: sourceware.org Received: from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Tue, 25 Aug 2009 08:47:29 +0000 Received: from int-mx08.intmail.prod.int.phx2.redhat.com (int-mx08.intmail.prod.int.phx2.redhat.com [10.5.11.21]) by mx1.redhat.com (8.13.8/8.13.8) with ESMTP id n7P8lRSm007316 for ; Tue, 25 Aug 2009 04:47:27 -0400 Received: from host0.dyn.jankratochvil.net (ovpn01.gateway.prod.ext.phx2.redhat.com [10.5.9.1]) by int-mx08.intmail.prod.int.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id n7P8lOJ8030860 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO) for ; Tue, 25 Aug 2009 04:47:26 -0400 Received: from host0.dyn.jankratochvil.net (localhost [127.0.0.1]) by host0.dyn.jankratochvil.net (8.14.3/8.14.3) with ESMTP id n7P8lNhJ024513 for ; Tue, 25 Aug 2009 10:47:23 +0200 Received: (from jkratoch@localhost) by host0.dyn.jankratochvil.net (8.14.3/8.14.3/Submit) id n7P8lNZe024512 for gdb-patches@sourceware.org; Tue, 25 Aug 2009 10:47:23 +0200 Date: Tue, 25 Aug 2009 09:40:00 -0000 From: Jan Kratochvil To: gdb-patches@sourceware.org Subject: [patch] [new testcase] Regression on qsort_cmp [Re: status of Darwin support] Message-ID: <20090825084723.GA24462@host0.dyn.jankratochvil.net> References: <200908041804.30240.thiago.bauermann@gmail.com> <20090805044504.GJ4370@adacore.com> <4A7ABAF0.7010004@Sun.COM> <4A7FFAB3.7080903@Sun.COM> <8ac60eac0908101320k6cc20e9age92656cba254a5d4@mail.gmail.com> <4A812B8B.8040202@Sun.COM> <20090825084613.GA22242@host0.dyn.jankratochvil.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20090825084613.GA22242@host0.dyn.jankratochvil.net> User-Agent: Mutt/1.5.19 (2009-01-05) X-IsSubscribed: yes Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2009-08/txt/msg00410.txt.bz2 Forgotten post to gdb-patches: On Tue, 11 Aug 2009 10:27:55 +0200, Christian Thalinger wrote: > objfiles.c:793: internal-error: qsort_cmp: Assertion `obj_section_endaddr (sect1) <= sect2_addr' failed. Getting randomly this error or: objfiles.c:817: internal-error: preferred_obj_section: Assertion `(a->objfile->separate_debug_objfile == b->objfile) || (b->objfile->separate_debug_objfile == a->objfile)' failed. which have both the same reason that solibs overlap in VMAs. IMO these assertions are right and GDB should rather refuse to load overlapping sections. Still they may(?) be required for overlays, actively in use for IBM Cell (as I was considering overlays as obsolete before myself). First bugreport of this crash by the courtesy of Mamoru Tasaka in RH Bug 515434. Requesting approval for the testcase check-in. Tested on {x86_64,i686}-fedorarawhide-linux-gnu: FAIL: gdb.base/solib-overlap.exp: 0x40000000: attach (GDB internal error) FAIL: gdb.base/solib-overlap.exp: 0x50000000: attach (GDB internal error) Thanks, Jan 2009-08-25 Jan Kratochvil * gdb.base/solib-overlap.exp, gdb.base/solib-overlap-lib.c, gdb.base/solib-overlap-main.c: New. --- /dev/null +++ b/gdb/testsuite/gdb.base/solib-overlap-lib.c @@ -0,0 +1,27 @@ +/* Copyright 2009 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 . + + Contributed by Jan Kratochvil . */ + +void +libsym (void) +{ +} + +#ifdef SYMB +void +libsymb (void) +{ +} +#endif --- /dev/null +++ b/gdb/testsuite/gdb.base/solib-overlap-main.c @@ -0,0 +1,25 @@ +/* Copyright 2009 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 . + + Contributed by Jan Kratochvil . */ + +#include + +int +main (void) +{ + sleep (60); + + return 1; +} --- /dev/null +++ b/gdb/testsuite/gdb.base/solib-overlap.exp @@ -0,0 +1,135 @@ +# Copyright 2009 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 . +# +# Contributed by Jan Kratochvil . + +# Test GDB can cope with two libraries loaded with overlapping VMA ranges. +# Prelink libraries first so they can be loaded and their native address. +# In such case `struct linkmap'.l_addr will be zero. Provide different +# unprelinked library files on the disk which have zero-based VMAs. These +# different files should have their .dynamic section at a different offset in +# page size so that we get for +# warning: .dynamic section for "..." is not at the expected address +# the reason +# (wrong library or version mismatch?) +# and not: +# difference appears to be caused by prelink, adjusting expectations +# In such case both disk libraries will be loaded at VMAs starting at zero. + +if [skip_shlib_tests] { + return 0 +} + +# Are we on a target board? It is required for attaching to a process. +if [is_remote target] { + return 0 +} + +# Library file. +set libname "solib-overlap-lib" +set srcfile_lib ${srcdir}/${subdir}/${libname}.c +# Binary file. +set testfile "solib-overlap-main" +set srcfile ${srcdir}/${subdir}/${testfile}.c + +# Base addresses for `prelink -r' which should be compatible with both -m32 and +# -m64 targets. If it clashes with system prelinked libraries it would give +# false PASS. +# Prelink first lib1 at 0x40000000 and lib2 at 0x41000000. +# During second pass try lib1 at 0x50000000 and lib2 at 0x51000000. +foreach prelink_lib1 {0x40000000 0x50000000} { + set prelink_lib2 [format "0x%x" [expr $prelink_lib1 + 0x01000000]] + + set old_prefix $pf_prefix + lappend pf_prefix "$prelink_lib1:" + + # Library file. + set binfile_lib1 ${objdir}/${subdir}/${libname}1-${prelink_lib1}.so + set binfile_lib2 ${objdir}/${subdir}/${libname}2-${prelink_lib1}.so + set lib_flags {debug} + # Binary file. + set binfile_base ${testfile}-${prelink_lib1} + set binfile ${objdir}/${subdir}/${binfile_base} + set bin_flags [list debug shlib=${binfile_lib1} shlib=${binfile_lib2}] + set escapedbinfile [string_to_regexp ${binfile}] + + if { [gdb_compile_shlib ${srcfile_lib} ${binfile_lib1} $lib_flags] != "" + || [gdb_compile_shlib ${srcfile_lib} ${binfile_lib2} $lib_flags] != "" + || [gdb_compile ${srcfile} ${binfile} executable $bin_flags] != "" } { + untested "Could not compile ${binfile_lib1}, ${binfile_lib2} or ${binfile}." + return -1 + } + + if {[catch "system \"prelink -N -r ${prelink_lib1} ${binfile_lib1}\""] != 0 + || [catch "system \"prelink -N -r ${prelink_lib2} ${binfile_lib2}\""] != 0} { + # Maybe we don't have prelink. + untested "Could not prelink ${binfile_lib1} or ${binfile_lib2}." + return -1 + } + + # Start the program running and then wait for a bit, to be sure + # that it can be attached to. + + set testpid [eval exec $binfile &] + sleep 2 + if { [istarget "*-*-cygwin*"] } { + # testpid is the Cygwin PID, GDB uses the Windows PID, which might be + # different due to the way fork/exec works. + set testpid [ exec ps -e | gawk "{ if (\$1 == $testpid) print \$4; }" ] + } + + remote_exec build "mv -f ${binfile_lib1} ${binfile_lib1}-running" + remote_exec build "mv -f ${binfile_lib2} ${binfile_lib2}-running" + + # Provide another exported function name to cause different sizes of sections. + lappend lib_flags additional_flags=-DSYMB + + if { [gdb_compile_shlib ${srcfile_lib} ${binfile_lib1} $lib_flags] != "" + || [gdb_compile_shlib ${srcfile_lib} ${binfile_lib2} $lib_flags] != ""} { + untested "Could not recompile ${binfile_lib1} or ${binfile_lib2}." + remote_exec build "kill -9 ${testpid}" + return -1 + } + + clean_restart ${binfile_base} + # This testcase currently does not support remote targets. + # gdb_load_shlibs ${binfile_lib1} ${binfile_lib2} + + # Here we should get: + # warning: .dynamic section for ".../solib-overlap-lib1.so" is not at the expected address (wrong library or version mismatch?) + # warning: .dynamic section for ".../solib-overlap-lib2.so" is not at the expected address (wrong library or version mismatch?) + + set test attach + gdb_test_multiple "attach $testpid" $test { + -re "Attaching to program.*`?$escapedbinfile'?, process $testpid.*$gdb_prompt $" { + pass $test + } + -re "Attaching to program.*`?$escapedbinfile\.exe'?, process $testpid.*\[Switching to thread $testpid\..*\].*$gdb_prompt $" { + # Response expected on Cygwin + pass $test + } + } + + # Detach the process. + + gdb_test "detach" "Detaching from program: .*$escapedbinfile, process $testpid" + + # Wait a bit for gdb to finish detaching + + sleep 5 + + remote_exec build "kill -9 ${testpid}" + + set pf_prefix $old_prefix +}