This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Ping Re: Patch to enhance MIPS GOT page estimates


Richard Sandiford wrote:
> Thiemo Seufer <ths@networkno.de> writes:
> > Joseph S. Myers wrote:
> >> I'd like to ping this patch of Richard's 
> >> <http://sourceware.org/ml/binutils/2007-09/msg00308.html> (plus the 
> >> testsuite fixes we discussed in that thread) as it doesn't appear to have 
> >> been reviewed.
> >
> > Please (re-)send an updated version which includes those bits.
> 
> bfd/
> 2007-09-18  Richard Sandiford  <richard@codesourcery.com>
> 
> 	* elfxx-mips.c (mips_got_page_range): New structure.
> 	(mips_got_page_entry): Likewise.
> 	(mips_got_info): Add page_gotno and got_page_entries fields.
> 	(mips_elf_got_per_bfd_arg): Add max_pages field.  Delete
> 	primary_count and current_count fields.
> 	(mips_got_page_entry_hash, mips_got_page_entry_eq): New functions.
> 	(mips_elf_pages_for_range, mips_elf_record_got_page_entry): Likewise.
> 	(mips_elf_get_got_for_bfd): New function, split out from
> 	mips_elf_make_got_per_bfd.  Initialize the page_gotno and
> 	got_page_entries fields when creating a new GOT structure.
> 	(mips_elf_make_got_pages_per_bfd): New function.
> 	(mips_elf_merge_got_with): New function, split out from
> 	mips_elf_make_got_per_bfd.  Merge page entries as well as
> 	non-page entries.  Use the minimum of max_pages and the sum
> 	of the page_gotnos to estimate the number of page entries.
> 	(mips_elf_merge_gots): Use the minimum of max_pages and the
> 	bfd's page_gotno to estimate the number of page entries.
> 	Use the above functions.
> 	(mips_elf_multi_got): Add page entries as well as non-page entries
> 	to the individual per-bfd GOTs.  Initialize got_per_bfd_arg.max_pages.
> 	Initialize the page_gotno and got_page_entries fields when creating
> 	a new primary GOT.  Use the minimum of pages and page_gotno when
> 	adding the number of pages entries to local_gotno.
> 	(mips_elf_create_got_section): Initialize the page_gotno and
> 	got_page_entries fields of the GOT structure.
> 	(mips_elf_rel_relocation_p, mips_elf_read_rel_addend)
> 	(mips_elf_add_lo16_rel_addend, mips_elf_get_section_contents): New
> 	functions, split out from...
> 	(_bfd_mips_elf_relocate_section): ...here.
> 	(_bfd_mips_elf_check_relocs): Record GOT page entries too.
> 	(_bfd_mips_relax_section): Use mips_elf_get_section_contents.
> 	(_bfd_mips_elf_always_size_sections): Use the smaller of the
> 	loadable_size- and page_gotno-derived estimates.
> 
> ld/testsuite/
> 2007-09-18  Richard Sandiford  <richard@codesourcery.com>
>             Joseph Myers  <joseph@codesourcery.com>
> 
> 	* ld-mips-elf/got-page-1.d, ld-mips-elf/got-page-1.s,
> 	* ld-mips-elf/got-page-2.d, ld-mips-elf/got-page-2.s,
> 	* ld-mips-elf/got-page-3.d, ld-mips-elf/got-page-3a.s,
> 	* ld-mips-elf/got-page-3b.s, ld-mips-elf/got-page-3c.s,
> 	* ld-mips-elf/got-page-1.ld: New tests.
> 	* ld-mips-elf/mips-elf.exp: Run them.
> 	* ld-mips-elf/multi-got-1.d, ld-mips-elf/multi-got-no-shared.d,
> 	* ld-mips-elf/tls-hidden2-got.d, ld-mips-elf/tls-hidden2.d,
> 	* ld-mips-elf/tls-hidden3.d, ld-mips-elf/tls-hidden3.got,
> 	* ld-mips-elf/tls-hidden3.r, ld-mips-elf/tls-hidden4.got,
> 	* ld-mips-elf/tls-hidden4.r, ld-mips-elf/tls-multi-got-1.d,
> 	* ld-mips-elf/tls-multi-got-1.got, ld-mips-elf/tls-multi-got-1.r,
> 	* ld-mips-elf/tlsbin-o32.d, ld-mips-elf/tlsbin-o32.got,
> 	* ld-mips-elf/tlsdyn-o32-1.d, ld-mips-elf/tlsdyn-o32-1.got,
> 	* ld-mips-elf/tlsdyn-o32-2.d, ld-mips-elf/tlsdyn-o32-2.got,
> 	* ld-mips-elf/tlsdyn-o32-3.d, ld-mips-elf/tlsdyn-o32-3.got,
> 	* ld-mips-elf/tlsdyn-o32.d, ld-mips-elf/tlsdyn-o32.got,
> 	* ld-mips-elf/tlslib-o32-hidden.got, ld-mips-elf/tlslib-o32-ver.got,
> 	* ld-mips-elf/tlslib-o32.d, ld-mips-elf/tlslib-o32.got: Update for
> 	GOT allocation changes.

This is ok. Sorry for the long delay to review it.


Thiemo


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]