Go to:
Gentoo Home
Documentation
Forums
Lists
Bugs
Planet
Store
Wiki
Get Gentoo!
Gentoo's Bugzilla – Attachment 86630 Details for
Bug 114008
-Wl,-Bdirect speed-up ...
Home
|
New
–
[Ex]
|
Browse
|
Search
|
Privacy Policy
|
[?]
|
Reports
|
Requests
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
A -bdirect.patch that will apply to binutils-2.16.92 and later.
binutils-suse-bdirect.diff (text/plain), 16.72 KB, created by
Peter Levine
on 2006-05-11 20:16:40 UTC
(
hide
)
Description:
A -bdirect.patch that will apply to binutils-2.16.92 and later.
Filename:
MIME Type:
Creator:
Peter Levine
Created:
2006-05-11 20:16:40 UTC
Size:
16.72 KB
patch
obsolete
>diff -Naur binutils-20101010/bfd/elf-bfd.h binutils-new/bfd/elf-bfd.h >--- binutils-20101010/bfd/elf-bfd.h 2006-03-16 07:20:15.000000000 -0500 >+++ binutils-new/bfd/elf-bfd.h 2006-05-11 06:10:13.721152250 -0400 >@@ -170,6 +170,14 @@ > matters. */ > unsigned int pointer_equality_needed : 1; > >+ /* FIXME: these 2 bits consume another 4 bytes. */ >+ /* Symbol is concrete, ie. non-vague if we can detect that, from the >+ .direct section. */ >+ unsigned int concrete_ref : 1; >+ /* Symbol is certainly vague (if we can detect that), from the >+ .direct section. */ >+ unsigned int vague_ref : 1; >+ > /* String table index in .dynstr if this is a dynamic symbol. */ > unsigned long dynstr_index; > >@@ -401,6 +409,9 @@ > asection *tls_sec; > bfd_size_type tls_size; > >+ /* Direct linkage output section. */ >+ asection *direct_sec; >+ > /* A linked list of BFD's loaded in the link. */ > struct elf_link_loaded_list *loaded; > >@@ -1294,6 +1305,7 @@ > name actually used, which will be the DT_SONAME entry if there is > one. */ > const char *dt_name; >+ int dt_needed_idx; > > /* Records the result of `get_program_header_size'. */ > bfd_size_type program_header_size; >@@ -1394,6 +1406,7 @@ > #define elf_local_got_offsets(bfd) (elf_tdata(bfd) -> local_got.offsets) > #define elf_local_got_ents(bfd) (elf_tdata(bfd) -> local_got.ents) > #define elf_dt_name(bfd) (elf_tdata(bfd) -> dt_name) >+#define elf_dt_needed_idx(bfd) (elf_tdata(bfd) -> dt_needed_idx) > #define elf_dyn_lib_class(bfd) (elf_tdata(bfd) -> dyn_lib_class) > #define elf_bad_symtab(bfd) (elf_tdata(bfd) -> bad_symtab) > #define elf_flags_init(bfd) (elf_tdata(bfd) -> flags_init) >diff -Naur binutils-20101010/bfd/elf.c binutils-new/bfd/elf.c >--- binutils-20101010/bfd/elf.c 2006-05-03 10:26:40.000000000 -0400 >+++ binutils-new/bfd/elf.c 2006-05-11 06:10:13.725152500 -0400 >@@ -1232,6 +1232,7 @@ > case DT_RELCOUNT: name = "RELCOUNT"; break; > case DT_FLAGS_1: name = "FLAGS_1"; break; > case DT_VERSYM: name = "VERSYM"; break; >+ case DT_SUSE_DIRECT: name = "SUSE_DIRECT"; break; > case DT_VERDEF: name = "VERDEF"; break; > case DT_VERDEFNUM: name = "VERDEFNUM"; break; > case DT_VERNEED: name = "VERNEED"; break; >@@ -1586,6 +1587,7 @@ > table->runpath = NULL; > table->tls_sec = NULL; > table->tls_size = 0; >+ table->direct_sec = NULL; > table->loaded = NULL; > table->is_relocatable_executable = FALSE; > >diff -Naur binutils-20101010/bfd/elflink.c binutils-new/bfd/elflink.c >--- binutils-20101010/bfd/elflink.c 2006-05-10 00:55:43.000000000 -0400 >+++ binutils-new/bfd/elflink.c 2006-05-11 06:26:27.970039000 -0400 >@@ -195,6 +195,16 @@ > elf_hash_table (info)->eh_info.hdr_sec = s; > } > >+ if ( info->direct && !info->executable ) >+ { >+ s = bfd_make_section (abfd, ".suse.direct"); >+ if (s == NULL >+ || ! bfd_set_section_flags (abfd, s, flags | SEC_READONLY) >+ || ! bfd_set_section_alignment (abfd, s, 2)) >+ return FALSE; >+ elf_hash_table (info)->direct_sec = s; >+ } >+ > /* Create sections to hold version informations. These are removed > if they are not needed. */ > s = bfd_make_section_with_flags (abfd, ".gnu.version_d", >@@ -2888,6 +2898,7 @@ > > if (!_bfd_elf_add_dynamic_entry (info, DT_NEEDED, strindex)) > return -1; >+ elf_dt_needed_idx (abfd) = ++(info->dt_needed_index); > } > else > /* We were just checking for existence of the tag. */ >@@ -3083,6 +3094,8 @@ > bfd_boolean add_needed; > struct elf_link_hash_table *htab; > bfd_size_type amt; >+ asection *direct_sec; >+ unsigned char *direct_data; > void *alloc_mark = NULL; > void *old_tab = NULL; > void *old_hash; >@@ -3096,6 +3109,9 @@ > htab = elf_hash_table (info); > bed = get_elf_backend_data (abfd); > >+ direct_sec = NULL; >+ direct_data = NULL; >+ > if ((abfd->flags & DYNAMIC) == 0) > dynamic = FALSE; > else >@@ -3115,6 +3131,16 @@ > bfd_set_error (bfd_error_wrong_format); > goto error_return; > } >+ if (info->direct) >+ direct_sec = bfd_get_section_by_name (abfd, ".suse.direct"); >+ if (direct_sec != NULL) >+ { >+ direct_data = bfd_alloc (abfd, direct_sec->size); >+ if (direct_data == NULL >+ || ! bfd_get_section_contents (abfd, direct_sec, >+ direct_data, 0, direct_sec->size)) >+ goto error_return; >+ } > } > > /* As a GNU extension, any input sections which are named >@@ -3822,6 +3848,23 @@ > && vernum > 1 > && definition) > h->verinfo.verdef = &elf_tdata (abfd)->verdef[vernum - 1]; >+ >+ if (direct_sec != NULL) >+ { >+ unsigned long d_idx = isym - isymbuf; >+ unsigned long direct_entry; >+ d_idx += extsymoff; >+ d_idx *= 2; >+ BFD_ASSERT (d_idx <= bfd_get_section_size (direct_sec)); >+ direct_entry = bfd_get_16 (abfd, direct_data + d_idx); >+ if ((direct_entry & DT_SUSE_DIRECT_VAGUE) != 0) >+ h->vague_ref = 1; >+ if (!h->vague_ref >+ && (direct_entry & DT_SUSE_DIRECT_MASK) != >+ DT_SUSE_DIRECT_UNKNOWN) >+ h->concrete_ref = 1; >+ } >+ > } > > if (! (_bfd_generic_link_add_one_symbol >@@ -4490,6 +4533,8 @@ > if (isymbuf != NULL) > free (isymbuf); > error_return: >+ if (direct_data != NULL) >+ free (direct_data); > return FALSE; > } > >@@ -5739,6 +5784,24 @@ > return FALSE; > } > >+ /* Create the direct bindings section - 1 entry per dynsym. */ >+ s = bfd_get_section_by_name (dynobj, ".suse.direct"); >+ if (s != NULL) >+ { >+ if (dynsymcount == 0) >+ s->flags |= SEC_EXCLUDE; >+ else >+ { >+ s->size = dynsymcount * sizeof (Elf_External_Direct); >+ s->contents = bfd_zalloc (output_bfd, s->size); >+ if (s->contents == NULL) >+ return FALSE; >+ memset (s->contents, 0xff, s->size); >+ if (!_bfd_elf_add_dynamic_entry (info, DT_SUSE_DIRECT, 0)) >+ return FALSE; >+ } >+ } >+ > /* Set the size of the .dynsym and .hash sections. We counted > the number of dynamic symbols in elf_link_add_object_symbols. > We will build the contents of .dynsym and .hash when we build >@@ -5813,6 +5876,8 @@ > asection *hash_sec; > /* symbol version section (.gnu.version). */ > asection *symver_sec; >+ /* .direct linkage section. */ >+ asection *direct_sec; > /* Buffer large enough to hold contents of any section. */ > bfd_byte *contents; > /* Buffer large enough to hold external relocs of any section. */ >@@ -6702,6 +6767,46 @@ > eversym += h->dynindx; > _bfd_elf_swap_versym_out (finfo->output_bfd, &iversym, eversym); > } >+ >+ if (finfo->direct_sec != NULL) >+ { >+ int dt_index = DT_SUSE_DIRECT_UNKNOWN; >+ bfd_vma offset = 2 * h->dynindx; >+ BFD_ASSERT (offset <= finfo->direct_sec->size); >+ >+ if ((h->root.type == bfd_link_hash_defined >+ || h->root.type == bfd_link_hash_defweak) >+ && h->root.u.def.section != NULL >+ && h->root.u.def.section->owner != NULL) >+ { >+ asection *sec = h->root.u.def.section; >+ >+ dt_index = elf_dt_needed_idx (sec->owner); >+ >+ if ((sec->owner->flags & DYNAMIC) == 0) >+ { >+ if (sec->name == NULL >+ || strncmp (sec->name, ".gnu.linkonce", >+ sizeof ".gnu.linkonce" - 1) == 0) >+ dt_index |= DT_SUSE_DIRECT_VAGUE; >+ >+ /* app -> library data references get turned into copy >+ relocs, so objects can migrate unpredictably into the >+ application itself. */ >+ else if (h->type == STT_OBJECT) >+ dt_index |= DT_SUSE_DIRECT_VAGUE; >+ } >+ else if (h->vague_ref) >+ dt_index |= DT_SUSE_DIRECT_VAGUE; >+ >+ else if (!h->concrete_ref) >+ dt_index = DT_SUSE_DIRECT_UNKNOWN; >+ >+ } >+ bfd_put_16 (finfo->output_bfd, >+ dt_index, >+ finfo->direct_sec->contents + offset); >+ } > } > > /* If we're stripping it, then it was just a dynamic symbol, and >@@ -7836,12 +7941,14 @@ > > if (! dynamic) > { >+ finfo.direct_sec = NULL; > finfo.dynsym_sec = NULL; > finfo.hash_sec = NULL; > finfo.symver_sec = NULL; > } > else > { >+ finfo.direct_sec = bfd_get_section_by_name (dynobj, ".suse.direct"); > finfo.dynsym_sec = bfd_get_section_by_name (dynobj, ".dynsym"); > finfo.hash_sec = bfd_get_section_by_name (dynobj, ".hash"); > BFD_ASSERT (finfo.dynsym_sec != NULL && finfo.hash_sec != NULL); >@@ -8602,6 +8709,9 @@ > case DT_VERNEED: > name = ".gnu.version_r"; > goto get_vma; >+ case DT_SUSE_DIRECT: >+ name = ".suse.direct"; >+ goto get_vma; > case DT_VERSYM: > name = ".gnu.version"; > get_vma: >diff -Naur binutils-20101010/binutils/readelf.c binutils-new/binutils/readelf.c >--- binutils-20101010/binutils/readelf.c 2006-04-26 11:43:17.000000000 -0400 >+++ binutils-new/binutils/readelf.c 2006-05-11 06:10:13.741153500 -0400 >@@ -145,6 +145,7 @@ > static int do_dynamic; > static int do_syms; > static int do_reloc; >+static int do_direct; > static int do_sections; > static int do_section_groups; > static int do_section_details; >@@ -1481,6 +1482,7 @@ > case DT_SYMINFO: return "SYMINFO"; /* aka ADDRRNGHI */ > > case DT_VERSYM: return "VERSYM"; >+ case DT_SUSE_DIRECT: return "SUSE_DIRECT"; > > case DT_TLSDESC_GOT: return "TLSDESC_GOT"; > case DT_TLSDESC_PLT: return "TLSDESC_PLT"; >@@ -2644,6 +2646,7 @@ > {"symbols", no_argument, 0, 's'}, > {"syms", no_argument, 0, 's'}, > {"relocs", no_argument, 0, 'r'}, >+ {"direct", no_argument, 0, 'y'}, > {"notes", no_argument, 0, 'n'}, > {"dynamic", no_argument, 0, 'd'}, > {"arch-specific", no_argument, 0, 'A'}, >@@ -2681,6 +2684,7 @@ > --symbols An alias for --syms\n\ > -n --notes Display the core notes (if present)\n\ > -r --relocs Display the relocations (if present)\n\ >+ -y --direct Display direct linkage table (if present)\n\ > -u --unwind Display the unwind info (if present)\n\ > -d --dynamic Display the dynamic section (if present)\n\ > -V --version-info Display the version sections (if present)\n\ >@@ -2770,7 +2774,7 @@ > usage (); > > while ((c = getopt_long >- (argc, argv, "ersuahnldSDAINtgw::x:i:vVWH", options, NULL)) != EOF) >+ (argc, argv, "erysuahnldSDAINtgw::x:i:vVWH", options, NULL)) != EOF) > { > char *cp; > int section; >@@ -2787,6 +2791,7 @@ > case 'a': > do_syms++; > do_reloc++; >+ do_direct++; > do_unwind++; > do_dynamic++; > do_header++; >@@ -2820,6 +2825,9 @@ > case 'r': > do_reloc++; > break; >+ case 'y': >+ do_direct++; >+ break; > case 'u': > do_unwind++; > break; >@@ -3029,7 +3037,7 @@ > if (!do_dynamic && !do_syms && !do_reloc && !do_unwind && !do_sections > && !do_segments && !do_header && !do_dump && !do_version > && !do_histogram && !do_debugging && !do_arch && !do_notes >- && !do_section_groups) >+ && !do_section_groups && !do_direct) > usage (); > else if (argc < 3) > { >@@ -4667,6 +4675,132 @@ > return 1; > } > >+static void >+print_dt_needed_name (int idx) >+{ >+ const char *name = NULL; >+ Elf_Internal_Dyn *entry; >+ >+ idx &= DT_SUSE_DIRECT_MASK; >+ >+ switch (idx) >+ { >+ case 0: >+ name = _("<self>"); >+ break; >+ case DT_SUSE_DIRECT_UNKNOWN: >+ name = _("<unknown>"); >+ break; >+ default: >+ idx--; >+ >+ for (entry = dynamic_section; >+ entry < dynamic_section + dynamic_nent && idx > 0; >+ entry++) >+ if (entry->d_tag == DT_NEEDED) >+ idx--; >+ >+ if (idx == 0) >+ { >+ if (VALID_DYNAMIC_NAME (entry->d_un.d_val)) >+ name = GET_DYNAMIC_NAME (entry->d_un.d_val); >+ else >+ name = NULL; >+ } >+ break; >+ } >+ >+ if (!name) >+ name = _("<out-of-range>"); >+ if (do_wide) >+ printf ("%s", name); >+ else >+ printf ("%-25s", name); >+} >+ >+static int >+process_direct (FILE *file) >+{ >+ unsigned int i; >+ unsigned int si; >+ char *strtab; >+ unsigned char *directtab; >+ Elf_Internal_Sym *symtab = NULL; >+ Elf_Internal_Sym *psym; >+ Elf_Internal_Shdr *direct = NULL; >+ Elf_Internal_Shdr *dynsym = NULL; >+ Elf_Internal_Shdr *section; >+ >+ if (!do_direct) >+ return 1; >+ if (!dynamic_symbols || !dynamic_section || !dynamic_strings) >+ return 1; >+ >+ for (i = 0, section = section_headers; >+ i < elf_header.e_shnum; >+ i++, section++) >+ { >+ if (section->sh_type == SHT_DYNSYM) >+ dynsym = section; >+ if (!strcmp (SECTION_NAME (section), ".suse.direct")) >+ direct = section; >+ } >+ if (!dynsym || !direct) >+ return 1; >+ >+ symtab = GET_ELF_SYMBOLS (file, dynsym); >+ if (!symtab) >+ return 1; >+ >+ directtab = get_data (NULL, file, direct->sh_offset, >+ direct->sh_size, 1, _("direct linkage table")); >+ if (!symtab) >+ { >+ free (symtab); >+ return 1; >+ } >+ >+ if (dynsym->sh_link == elf_header.e_shstrndx) >+ strtab = string_table; >+ else >+ { >+ Elf_Internal_Shdr *string_sec; >+ >+ string_sec = SECTION_HEADER (dynsym->sh_link); >+ >+ strtab = get_data (NULL, file, string_sec->sh_offset, >+ string_sec->sh_size, 1, _("string table")); >+ } >+ >+ assert (strtab != NULL); >+ >+ printf (_("\nDirect relocations for image:\n")); >+ printf (_(" Num: Index Vague Binding Symbol\n")); >+ for (si = 0, psym = symtab; >+ si < dynsym->sh_size / dynsym->sh_entsize; >+ si++, psym++) >+ { >+ unsigned int field; >+ printf ("%6d: ", si); >+ field = byte_get (directtab + (si * 2), 2); >+ printf ("[0x%.4x] ", field); >+ printf ("%s", (field & DT_SUSE_DIRECT_VAGUE) ? "Vague" : " "); >+ printf (" "); >+ print_dt_needed_name (field); >+ printf (" "); >+ print_symbol (25, strtab + psym->st_name); >+ printf ("\n"); >+ } >+ >+ >+ free (symtab); >+ if (strtab != string_table) >+ free (strtab); >+ >+ return 1; >+} >+ >+ > /* Process the unwind section. */ > > #include "unwind-ia64.h" >@@ -9003,6 +9137,8 @@ > > process_relocs (file); > >+ process_direct (file); >+ > process_unwind (file); > > process_symbol_table (file); >diff -Naur binutils-20101010/include/bfdlink.h binutils-new/include/bfdlink.h >--- binutils-20101010/include/bfdlink.h 2006-04-06 14:52:45.000000000 -0400 >+++ binutils-new/include/bfdlink.h 2006-05-11 06:10:13.745153750 -0400 >@@ -244,6 +244,9 @@ > /* TRUE if BFD should pre-bind symbols in a shared object. */ > unsigned int symbolic: 1; > >+ /* TRUE if BFD should bind symbols directly to resolved at link time. */ >+ unsigned int direct: 1; >+ > /* TRUE if BFD should export all symbols in the dynamic symbol table > of an executable, rather than only those used. */ > unsigned int export_dynamic: 1; >@@ -422,6 +425,9 @@ > > /* Start and end of RELRO region. */ > bfd_vma relro_start, relro_end; >+ >+ /* current max dtinfo section number. */ >+ int dt_needed_index; > }; > > /* This structures holds a set of callback functions. These are >diff -Naur binutils-20101010/include/elf/common.h binutils-new/include/elf/common.h >--- binutils-20101010/include/elf/common.h 2006-02-17 09:36:26.000000000 -0500 >+++ binutils-new/include/elf/common.h 2006-05-11 06:10:13.745153750 -0400 >@@ -648,6 +648,11 @@ > #define DF_BIND_NOW (1 << 3) > #define DF_STATIC_TLS (1 << 4) > >+/* Constants for the DT_DIRECT entries. */ >+#define DT_SUSE_DIRECT_VAGUE (1<<15) >+#define DT_SUSE_DIRECT_MASK 0xfff >+#define DT_SUSE_DIRECT_UNKNOWN DT_SUSE_DIRECT_MASK >+ > /* These constants are used for the version number of a Elf32_Verdef > structure. */ > >diff -Naur binutils-20101010/include/elf/external.h binutils-new/include/elf/external.h >--- binutils-20101010/include/elf/external.h 2005-06-17 09:37:23.000000000 -0400 >+++ binutils-new/include/elf/external.h 2006-05-11 06:10:13.745153750 -0400 >@@ -213,6 +213,10 @@ > unsigned char vd_next[4]; > } Elf_External_Verdef; > >+typedef struct { >+ unsigned char dir_libidx[2]; >+} Elf_External_Direct; >+ > /* This structure appears in a SHT_GNU_verdef section. */ > > typedef struct { >diff -Naur binutils-20101010/ld/lexsup.c binutils-new/ld/lexsup.c >--- binutils-20101010/ld/lexsup.c 2005-10-30 13:08:52.000000000 -0500 >+++ binutils-new/ld/lexsup.c 2006-05-11 06:17:36.348814750 -0400 >@@ -71,6 +71,7 @@ > OPTION_CREF, > OPTION_DEFSYM, > OPTION_DEMANGLE, >+ OPTION_DIRECT, > OPTION_DYNAMIC_LINKER, > OPTION_SYSROOT, > OPTION_EB, >@@ -342,6 +343,8 @@ > '\0', NULL, NULL, ONE_DASH }, > { {"Bsymbolic", no_argument, NULL, OPTION_SYMBOLIC}, > '\0', NULL, N_("Bind global references locally"), ONE_DASH }, >+ { {"Bdirect", no_argument, NULL, OPTION_DIRECT}, >+ '\0', NULL, N_("Direct-linkage the the shared libraries"), ONE_DASH }, > { {"check-sections", no_argument, NULL, OPTION_CHECK_SECTIONS}, > '\0', NULL, N_("Check section addresses for overlaps (default)"), > TWO_DASHES }, >@@ -747,6 +750,9 @@ > cplus_demangle_set_style (style); > } > break; >+ case OPTION_DIRECT: >+ link_info.direct = TRUE; >+ break; > case 'I': /* Used on Solaris. */ > case OPTION_DYNAMIC_LINKER: > command_line.interpreter = optarg;
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
Attachments on
bug 114008
:
74762
|
75104
|
77828
|
77830
|
77840
|
78188
|
78189
|
78190
|
78191
|
78192
|
78193
|
78194
|
78195
|
78196
|
78213
|
80806
|
80807
|
80808
|
80809
|
84585
|
84615
|
84616
|
84617
| 86630 |
89027
|
451452
|
451454