From e965d5147e216ceb4157ad478cd271af96866e77 Mon Sep 17 00:00:00 2001 From: Roland McGrath Date: Sat, 28 Feb 2009 02:26:51 +0000 Subject: [PATCH] * Makeconfig (%.v.i): Depend on Makeconfig. Exclude % lines from initial #-comment removal. --- ChangeLog | 5 +++++ nptl/ChangeLog | 5 +++++ nptl/init.c | 2 +- nptl/pthread_create.c | 6 +++--- nptl_db/ChangeLog | 10 +++++++++ nptl_db/Makefile | 12 +++++++++-- nptl_db/db-symbols.awk | 45 ++++++++++++++++++++++++++++++++++++++ nptl_db/db-symbols.h | 56 ++++++++++++++++++++++++++++++++++++++++++++++++ nptl_db/td_symbol_list.c | 36 ++++++------------------------- 9 files changed, 141 insertions(+), 36 deletions(-) create mode 100644 nptl_db/db-symbols.awk create mode 100644 nptl_db/db-symbols.h diff --git a/ChangeLog b/ChangeLog index b30b987..e361da5 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,8 @@ +2009-02-27 Roland McGrath + + * Makeconfig (%.v.i): Depend on Makeconfig. + Exclude % lines from initial #-comment removal. + 2009-02-27 Ulrich Drepper * po/ko.po: Update from translation team. diff --git a/nptl/ChangeLog b/nptl/ChangeLog index 3c3ce82..c906c66 100644 --- a/nptl/ChangeLog +++ b/nptl/ChangeLog @@ -1,3 +1,8 @@ +2009-02-27 Roland McGrath + + * init.c (__nptl_initial_report_events): Mark __attribute_used__. + * pthread_create.c (__nptl_threads_events, __nptl_last_event): Likewise. + 2009-02-26 Ulrich Drepper * sysdeps/unix/sysv/linux/bits/posix_opt.h: Define diff --git a/nptl/init.c b/nptl/init.c index d0f1fc3..ba3caee 100644 --- a/nptl/init.c +++ b/nptl/init.c @@ -260,7 +260,7 @@ extern void **__libc_dl_error_tsd (void) __attribute__ ((const)); /* This can be set by the debugger before initialization is complete. */ -static bool __nptl_initial_report_events; +static bool __nptl_initial_report_events __attribute_used__; void __pthread_initialize_minimal_internal (void) diff --git a/nptl/pthread_create.c b/nptl/pthread_create.c index 5fcc72c..c693979 100644 --- a/nptl/pthread_create.c +++ b/nptl/pthread_create.c @@ -1,4 +1,4 @@ -/* Copyright (C) 2002-2007, 2008 Free Software Foundation, Inc. +/* Copyright (C) 2002-2007,2008,2009 Free Software Foundation, Inc. This file is part of the GNU C Library. Contributed by Ulrich Drepper , 2002. @@ -40,10 +40,10 @@ static int start_thread (void *arg); int __pthread_debug; /* Globally enabled events. */ -static td_thr_events_t __nptl_threads_events; +static td_thr_events_t __nptl_threads_events __attribute_used__; /* Pointer to descriptor with the last event. */ -static struct pthread *__nptl_last_event; +static struct pthread *__nptl_last_event __attribute_used__; /* Number of threads running. */ unsigned int __nptl_nthreads = 1; diff --git a/nptl_db/ChangeLog b/nptl_db/ChangeLog index 5166d3d..610ca9d 100644 --- a/nptl_db/ChangeLog +++ b/nptl_db/ChangeLog @@ -1,3 +1,13 @@ +2009-02-27 Roland McGrath + + * td_symbol_list.c (symbol_list_arr): Move initializer guts to ... + * db-symbols.h: ... here, new file. + * db-symbols.awk: New file. + * Makefile (distribute): Add them. + ($(objpfx)db-symbols.out): New target. + (tests): Depend on it. + ($(objpfx)db-symbols.v.i): New dependent rule. + 2009-02-06 Ulrich Drepper * td_thr_get_info.c (td_thr_get_info): Initialize schedpolicy in diff --git a/nptl_db/Makefile b/nptl_db/Makefile index 5c73ff2..af542dc 100644 --- a/nptl_db/Makefile +++ b/nptl_db/Makefile @@ -1,4 +1,4 @@ -# Copyright (C) 2002, 2003 Free Software Foundation, Inc. +# Copyright (C) 2002,2003,2009 Free Software Foundation, Inc. # This file is part of the GNU C Library. # The GNU C Library is free software; you can redistribute it and/or @@ -50,7 +50,8 @@ libthread_db-inhibit-o = $(filter-out .os,$(object-suffixes)) # The ps_* callback functions are not defined. libthread_db.so-no-z-defs = yes -distribute = thread_dbP.h shlib-versions proc_service.h db_info.c structs.def +distribute = thread_dbP.h shlib-versions proc_service.h \ + db_info.c structs.def db-symbols.h db-symbols.awk include ../Rules # Depend on libc.so so a DT_NEEDED is generated in the shared objects. @@ -58,3 +59,10 @@ include ../Rules # a statically-linked program that hasn't already loaded it. $(objpfx)libthread_db.so: $(common-objpfx)libc.so \ $(common-objpfx)libc_nonshared.a + +tests: $(objpfx)db-symbols.out +$(objpfx)db-symbols.out: $(objpfx)db-symbols.v.i \ + $(common-objpfx)nptl/libpthread.so + readelf -W -s $(filter %.so,$^) | $(AWK) -f $< > $@ + +$(objpfx)db-symbols.v.i: db-symbols.awk diff --git a/nptl_db/db-symbols.awk b/nptl_db/db-symbols.awk new file mode 100644 index 0000000..3327213 --- /dev/null +++ b/nptl_db/db-symbols.awk @@ -0,0 +1,45 @@ +# This script processes the output of 'readelf -W -s' on the libpthread.so +# we've just built. It checks for all the symbols used in td_symbol_list. + +BEGIN { +%define DB_LOOKUP_NAME(idx, name) required[STRINGIFY (name)] = 1; +%define DB_LOOKUP_NAME_TH_UNIQUE(idx, name) th_unique[STRINGIFY (name)] = 1; +%include "db-symbols.h" + + in_symtab = 0; +} + +/Symbol table '.symtab'/ { in_symtab=1; next } +NF == 0 { in_symtab=0; next } + +!in_symtab { next } + +NF >= 8 && $7 != "UND" { seen[$8] = 1 } + +END { + status = 0; + + for (s in required) { + if (s in seen) print s, "ok"; + else { + status = 1; + print s, "***MISSING***"; + } + } + + any = ""; + for (s in th_unique) { + if (s in seen) { + any = s; + break; + } + } + if (any) + print "th_unique:", any; + else { + status = 1; + print "th_unique:", "***MISSING***"; + } + + exit(status); +} diff --git a/nptl_db/db-symbols.h b/nptl_db/db-symbols.h new file mode 100644 index 0000000..d6c3525 --- /dev/null +++ b/nptl_db/db-symbols.h @@ -0,0 +1,56 @@ +/* List of symbols in libpthread examined by libthread_db. + Copyright (C) 2009 Free Software Foundation, Inc. + This file is part of the GNU C Library. + + The GNU C Library is free software; you can redistribute it and/or + modify it under the terms of the GNU Lesser General Public + License as published by the Free Software Foundation; either + version 2.1 of the License, or (at your option) any later version. + + The GNU C Library 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 + Lesser General Public License for more details. + + You should have received a copy of the GNU Lesser General Public + License along with the GNU C Library; if not, write to the Free + Software Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA + 02111-1307 USA. */ + +#ifdef HAVE_ASM_GLOBAL_DOT_NAME +# define DOT(x) .##x /* PPC64 requires . prefix on code symbols. */ +#else +# define DOT(x) x /* No prefix. */ +#endif + +#define STRINGIFY(name) STRINGIFY_1(name) +#define STRINGIFY_1(name) #name + +#define DB_STRUCT(type) \ + DB_LOOKUP_NAME (SYM_SIZEOF_##type, _thread_db_sizeof_##type) +#define DB_STRUCT_FIELD(type, field) \ + DB_LOOKUP_NAME (SYM_##type##_FIELD_##field, _thread_db_##type##_##field) +#define DB_SYMBOL(name) \ + DB_LOOKUP_NAME (SYM_##name, name) +#define DB_FUNCTION(name) \ + DB_LOOKUP_NAME (SYM_##name, DOT (name)) +#define DB_VARIABLE(name) \ + DB_LOOKUP_NAME (SYM_##name, name) \ + DB_LOOKUP_NAME (SYM_DESC_##name, _thread_db_##name) + +# include "structs.def" + +# undef DB_STRUCT +# undef DB_FUNCTION +# undef DB_SYMBOL +# undef DB_VARIABLE +# undef DOT + +DB_LOOKUP_NAME_TH_UNIQUE (SYM_TH_UNIQUE_REGISTER64, _thread_db_register64) +DB_LOOKUP_NAME_TH_UNIQUE (SYM_TH_UNIQUE_REGISTER32, _thread_db_register32) +DB_LOOKUP_NAME_TH_UNIQUE (SYM_TH_UNIQUE_CONST_THREAD_AREA, + _thread_db_const_thread_area) +DB_LOOKUP_NAME_TH_UNIQUE (SYM_TH_UNIQUE_REGISTER32_THREAD_AREA, + _thread_db_register32_thread_area) +DB_LOOKUP_NAME_TH_UNIQUE (SYM_TH_UNIQUE_REGISTER64_THREAD_AREA, + _thread_db_register64_thread_area) diff --git a/nptl_db/td_symbol_list.c b/nptl_db/td_symbol_list.c index 5d6c94e..7949840 100644 --- a/nptl_db/td_symbol_list.c +++ b/nptl_db/td_symbol_list.c @@ -1,5 +1,5 @@ /* Return list of symbols the library can request. - Copyright (C) 2001, 2002, 2003 Free Software Foundation, Inc. + Copyright (C) 2001,2002,2003,2009 Free Software Foundation, Inc. This file is part of the GNU C Library. Contributed by Ulrich Drepper , 2001. @@ -22,37 +22,13 @@ #include #include "thread_dbP.h" - -#ifdef HAVE_ASM_GLOBAL_DOT_NAME -# define DOT "." /* PPC64 requires . prefix on code symbols. */ -#else -# define DOT /* No prefix. */ -#endif - static const char *symbol_list_arr[] = { -# define DB_STRUCT(type) \ - [SYM_SIZEOF_##type] = "_thread_db_sizeof_" #type, -# define DB_STRUCT_FIELD(type, field) \ - [SYM_##type##_FIELD_##field] = "_thread_db_" #type "_" #field, -# define DB_SYMBOL(name) \ - [SYM_##name] = #name, -# define DB_FUNCTION(name) \ - [SYM_##name] = DOT #name, -# define DB_VARIABLE(name) \ - [SYM_##name] = #name, \ - [SYM_DESC_##name] = "_thread_db_" #name, -# include "structs.def" -# undef DB_STRUCT -# undef DB_FUNCTION -# undef DB_SYMBOL -# undef DB_VARIABLE - - [SYM_TH_UNIQUE_CONST_THREAD_AREA] = "_thread_db_const_thread_area", - [SYM_TH_UNIQUE_REGISTER64] = "_thread_db_register64", - [SYM_TH_UNIQUE_REGISTER32] = "_thread_db_register32", - [SYM_TH_UNIQUE_REGISTER32_THREAD_AREA] = "_thread_db_register32_thread_area", - [SYM_TH_UNIQUE_REGISTER64_THREAD_AREA] = "_thread_db_register64_thread_area", +# define DB_LOOKUP_NAME(idx, name) [idx] = #name, +# define DB_LOOKUP_NAME_TH_UNIQUE(idx, name) [idx] = #name, +# include "db-symbols.h" +# undef DB_LOOKUP_NAME +# undef DB_LOOKUP_NAME_TH_UNIQUE [SYM_NUM_MESSAGES] = NULL }; -- 2.7.4