From: Nick Clifton <nickc@redhat.com>
To: Jakub Jelinek <jakub@redhat.com>
Cc: djamel anonymous <djam8193ah@hotmail.com>,
binutils@sources.redhat.com, drepper@redhat.com,
michael.meeks@novell.com, libc-alpha@sources.redhat.com
Subject: Re: DT_GNU_HASH latest patches
Date: Mon, 10 Jul 2006 14:18:00 -0000 [thread overview]
Message-ID: <44B26190.7040509@redhat.com> (raw)
In-Reply-To: <20060707144406.GO3823@sunsite.mff.cuni.cz>
Hi Jakub,
> 2006-07-07 Jakub Jelinek <jakub@redhat.com>
>
> include/
> * bfdlink.h (struct bfd_link_info): Add emit_hash and
> emit_gnu_hash bitfields.
> include/elf/
> * common.h (SHT_GNU_HASH, DT_GNU_HASH): Define.
> ld/
> * scripttempl/elf.sc: Add .gnu.hash section.
> * emultempl/elf32.em (OPTION_HASH_STYLE): Define.
> (gld${EMULATION_NAME}_add_options): Register --hash-style option.
> (gld${EMULATION_NAME}_handle_option): Handle it.
> (gld${EMULATION_NAME}_list_options): Document it.
> * ldmain.c (main): Initialize emit_hash and emit_gnu_hash.
> * ld.texinfo: Document --hash-style option.
> bfd/
> * elf.c (_bfd_elf_print_private_bfd_data): Handle DT_GNU_HASH.
> (bfd_section_from_shdr, elf_fake_sections, assign_section_numbers):
> Handle SHT_GNU_HASH.
> (special_sections_g): Include .gnu.hash section.
> (bfd_elf_gnu_hash): New function.
> * elf-bfd.h (bfd_elf_gnu_hash, _bfd_elf_hash_symbol): New prototypes.
> (struct elf_backend_data): Add elf_hash_symbol method.
> * elflink.c (_bfd_elf_link_create_dynamic_sections): Create .hash
> only if info->emit_hash, create .gnu.hash section if
> info->emit_gnu_hash.
> (struct collect_gnu_hash_codes): New type.
> (elf_collect_gnu_hash_codes, elf_renumber_gnu_hash_syms,
> _bfd_elf_hash_symbol): New functions.
> (compute_bucket_count): Don't compute HASHCODES array, instead add
> that and NSYMS as arguments. Use bed->s->sizeof_hash_entry
> instead of bed->s->arch_size / 8. Fix .hash size estimation.
> When not optimizing, use the number of hashed symbols rather than
> dynsymcount.
> (bfd_elf_size_dynamic_sections): Only add DT_HASH if info->emit_hash,
> and ADD DT_GNU_HASH if info->emit_gnu_hash.
> (bfd_elf_size_dynsym_hash_dynstr): Size .hash only if info->emit_hash,
> adjust compute_bucket_count caller. Create and populate .gnu.hash
> section if info->emit_gnu_hash.
> (elf_link_output_extsym): Only populate .hash section if
> finfo->hash_sec != NULL.
> (bfd_elf_final_link): Adjust assertion. Handle DT_GNU_HASH.
> * elfxx-target.h (elf_backend_hash_symbol): Define if not yet defined.
> (elfNN_bed): Add elf_backend_hash_symbol.
> * elf64-x86-64.c (elf64_x86_64_hash_symbol): New function.
> (elf_backend_hash_symbol): Define.
> * elf32-i386.c (elf_i386_hash_symbol): New function.
> (elf_backend_hash_symbol): Define.
> binutils/
> * readelf.c (get_dynamic_type): Handle DT_GNU_HASH.
> (get_section_type_name): Handle SHT_GNU_HASH.
> (dynamic_info_DT_GNU_HASH): New variable.
> (process_dynamic_section): Handle DT_GNU_HASH.
> (process_symbol_table): Print also DT_GNU_HASH histogram.
Approved - please apply.
Cheers
Nick
next prev parent reply other threads:[~2006-07-10 14:18 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <44A9D4EB.2040107@redhat.com>
[not found] ` <BAY22-F13B0B8E34EA055F9BF6E50C7710@phx.gbl>
[not found] ` <20060705090438.GJ3823@sunsite.mff.cuni.cz>
2006-07-06 11:26 ` DT_GNU_HASH benchmark results Jakub Jelinek
2006-07-06 18:22 ` DT_GNU_HASH latest patches Jakub Jelinek
2006-07-06 19:53 ` djamel anonymous
2006-07-07 14:45 ` Jakub Jelinek
2006-07-07 15:57 ` djamel anonymous
2006-07-09 22:27 ` Jakub Jelinek
2006-07-08 2:45 ` Eric Christopher
2006-07-09 22:35 ` Jakub Jelinek
2006-07-10 14:18 ` Nick Clifton [this message]
2006-07-10 14:18 ` Nick Clifton
2006-07-10 14:20 ` Nick Clifton
2006-07-10 21:52 ` Jakub Jelinek
2006-07-11 13:35 ` Hans-Peter Nilsson
2006-07-11 13:58 ` Jakub Jelinek
2006-07-07 10:07 ` Michael Meeks
2006-07-07 13:43 ` Ulrich Drepper
2006-07-07 13:49 ` Jakub Jelinek
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=44B26190.7040509@redhat.com \
--to=nickc@redhat.com \
--cc=binutils@sources.redhat.com \
--cc=djam8193ah@hotmail.com \
--cc=drepper@redhat.com \
--cc=jakub@redhat.com \
--cc=libc-alpha@sources.redhat.com \
--cc=michael.meeks@novell.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).