From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 23263 invoked by alias); 25 Sep 2013 14:27:31 -0000 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 Received: (qmail 23215 invoked by uid 89); 25 Sep 2013 14:27:30 -0000 Received: from relay1.mentorg.com (HELO relay1.mentorg.com) (192.94.38.131) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Wed, 25 Sep 2013 14:27:30 +0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.7 required=5.0 tests=AWL,BAYES_00,FROM_12LTRDOM,RDNS_NONE,SPF_HELO_FAIL autolearn=no version=3.3.2 X-HELO: relay1.mentorg.com Received: from svr-orw-exc-10.mgc.mentorg.com ([147.34.98.58]) by relay1.mentorg.com with esmtp id 1VOq3q-0005kk-Aj from Yao_Qi@mentor.com for gdb-patches@sourceware.org; Wed, 25 Sep 2013 07:27:26 -0700 Received: from SVR-ORW-FEM-04.mgc.mentorg.com ([147.34.97.41]) by SVR-ORW-EXC-10.mgc.mentorg.com with Microsoft SMTPSVC(6.0.3790.4675); Wed, 25 Sep 2013 07:27:26 -0700 Received: from qiyao.dyndns.org (147.34.91.1) by svr-orw-fem-04.mgc.mentorg.com (147.34.97.41) with Microsoft SMTP Server id 14.2.247.3; Wed, 25 Sep 2013 07:27:24 -0700 From: Yao Qi To: Subject: [PATCH 3/3] Test on solib load and unload Date: Wed, 25 Sep 2013 14:27:00 -0000 Message-ID: <1380119209-25975-4-git-send-email-yao@codesourcery.com> In-Reply-To: <1380119209-25975-1-git-send-email-yao@codesourcery.com> References: <1380119209-25975-1-git-send-email-yao@codesourcery.com> MIME-Version: 1.0 Content-Type: text/plain X-IsSubscribed: yes X-SW-Source: 2013-09/txt/msg00896.txt.bz2 This patch is to add a test case to on the performance of GDB handling load and unload of shared library. In V2, there are some changes, - A new proc gdb_produce_source to produce source files. I tried to move all source file generation code out of solib.exp, but compilation step still needs to know the generated file names. I have to hard-code the file names in compilation step, which is not good to me, so I give up on this moving. - SOLIB_NUMBER -> SOLIB_COUNT - New variable SOLIB_DLCLOSE_REVERSED_ORDER to control the order of iterating a list of shared libs to dlclose them. - New variable GDB_PERFORMANCE to enable these perf test cases. - Remove dlsym call in solib.c. - Update solib.py for the updated framework. gdb/testsuite/ * lib/gdb.exp (gdb_produce_source): New procedure. * gdb.perf/solib.c: New. * gdb.perf/solib.exp: New. * gdb.perf/solib.py: New. --- gdb/testsuite/gdb.perf/solib.c | 71 ++++++++++++++++++++++++++++++++++ gdb/testsuite/gdb.perf/solib.exp | 78 ++++++++++++++++++++++++++++++++++++++ gdb/testsuite/gdb.perf/solib.py | 45 ++++++++++++++++++++++ gdb/testsuite/lib/gdb.exp | 16 ++++++++ 4 files changed, 210 insertions(+), 0 deletions(-) create mode 100644 gdb/testsuite/gdb.perf/solib.c create mode 100644 gdb/testsuite/gdb.perf/solib.exp create mode 100644 gdb/testsuite/gdb.perf/solib.py diff --git a/gdb/testsuite/gdb.perf/solib.c b/gdb/testsuite/gdb.perf/solib.c new file mode 100644 index 0000000..82b1efa --- /dev/null +++ b/gdb/testsuite/gdb.perf/solib.c @@ -0,0 +1,71 @@ +/* This testcase is part of GDB, the GNU debugger. + + Copyright (C) 2013 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 . */ + +#include +#include + +#ifdef __WIN32__ +#include +#define dlopen(name, mode) LoadLibrary (TEXT (name)) +# define dlsym(handle, func) GetProcAddress (handle, func) +#define dlclose(handle) FreeLibrary (handle) +#else +#include +#endif + +void +do_test (int number) +{ + void **handles; + char libname[40]; + int i; + + handles = malloc (sizeof (void *) * number); + + for (i = 0; i < number; i++) + { + sprintf (libname, "solib-lib%d", i); + handles[i] = dlopen (libname, RTLD_LAZY); + if (handles[i] == NULL) + { + printf ("ERROR on dlopen %s\n", libname); + return; + } + } + + /* Unload shared libraries in different orders. */ +#ifndef SOLIB_DLCLOSE_REVERSED_ORDER + for (i = 0; i < number; i++) +#else + for (i = number - 1; i >= 0; i--) +#endif + dlclose (handles[i]); + + free (handles); +} + +static void +end (void) +{} + +int +main (void) +{ + end (); + + return 0; +} diff --git a/gdb/testsuite/gdb.perf/solib.exp b/gdb/testsuite/gdb.perf/solib.exp new file mode 100644 index 0000000..ab41b4e --- /dev/null +++ b/gdb/testsuite/gdb.perf/solib.exp @@ -0,0 +1,78 @@ +# Copyright (C) 2013 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 . + +# This test case is to test the performance of GDB when it is handling +# the shared libraries of inferior are loaded and unloaded. +load_lib perftest.exp + +if [skip_perf_tests] { + return 0 +} + +standard_testfile .c +set executable $testfile +set expfile $testfile.exp + +# make check-perf RUNTESTFLAGS='solib.exp SOLIB_COUNT=1024' +if ![info exists SOLIB_COUNT] { + set SOLIB_COUNT 128 +} + +PerfTest::assemble { + compile { + for {set i 0} {$i < $SOLIB_COUNT} {incr i} { + + # Produce source files. + set libname "solib-lib$i" + set src [standard_output_file $libname.c] + set exe [standard_output_file $libname] + + gdb_produce_source $src { "int shr$i (void) {return 0;}" } + + # Compile. + if { [gdb_compile_shlib $src $exe {debug}] != "" } { + untested "Couldn't compile $src." + return -1 + } + + # Delete object files to save some space. + file delete [standard_output_file "solib-lib$i.c.o"] + } + + set compile_flags {debug shlib_load} + global SOLIB_DLCLOSE_REVERSED_ORDER + + if [info exists SOLIB_DLCLOSE_REVERSED_ORDER] { + lappend compile_flags "additional_flags=-DSOLIB_DLCLOSE_REVERSED_ORDER" + } + + if { [gdb_compile "$srcdir/$subdir/$srcfile" ${binfile} executable $compile_flags] == "" } { + PerfTest::compiled + } + } + + setup_gdb { + clean_restart $binfile + + if ![runto_main] { + fail "Can't run to main" + return -1 + } + } +} { + run { + gdb_test_no_output "python SolibLoadUnload\($SOLIB_COUNT\).run()" + } +} diff --git a/gdb/testsuite/gdb.perf/solib.py b/gdb/testsuite/gdb.perf/solib.py new file mode 100644 index 0000000..9376814 --- /dev/null +++ b/gdb/testsuite/gdb.perf/solib.py @@ -0,0 +1,45 @@ +# Copyright (C) 2013 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 . + +# This test case is to test the speed of GDB when it is handling the +# shared libraries of inferior are loaded and unloaded. + +from perftest import perftest +from perftest import measure + +class SolibLoadUnload(perftest.SingleStatisticTestCase): + def __init__(self, solib_number): + # We want to measure time in this test. + super (SolibLoadUnload, self).__init__ ([measure.Measurement.TIME], + "solib") + self.solib_number = solib_number + + def warm_up(self): + do_test_command = "call do_test (%d)" % self.solib_number + gdb.execute(do_test_command) + gdb.execute(do_test_command) + + def execute_test(self): + num = self.solib_number + iteration = 5; + + while num > 0 and iteration > 0: + do_test_command = "call do_test (%d)" % num + func = lambda: gdb.execute (do_test_command) + + self.measure.measure(func, num) + + num = num / 2 + iteration -= 1 diff --git a/gdb/testsuite/lib/gdb.exp b/gdb/testsuite/lib/gdb.exp index 5e3331a..22349f7 100644 --- a/gdb/testsuite/lib/gdb.exp +++ b/gdb/testsuite/lib/gdb.exp @@ -1796,6 +1796,22 @@ proc supports_reverse {} { return 0 } +# Produce source file NAME and write SOURCES into it. + +proc gdb_produce_source { name sources } { + set index 0 + set f [open $name "w"] + + while { ${index} < [llength ${sources}] } { + set line [lindex ${sources} ${index}] + set index [expr ${index} + 1] + + set line [uplevel list $line] + puts $f $line + } + close $f +} + # Return 1 if target is ILP32. # This cannot be decided simply from looking at the target string, # as it might depend on externally passed compiler options like -m64. -- 1.7.7.6