public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
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

  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).