From 2889e75bfb63b5638dffb3bf9ae598e85af86af9 Mon Sep 17 00:00:00 2001 From: Nick Clifton Date: Wed, 21 Feb 2007 16:43:50 +0000 Subject: Take into account MEMORY regions when creating a segment map. --- ld/ChangeLog | 7 +++++++ ld/ldlang.c | 36 ++++++++++++++++++++++++++++++++++++ ld/ldlang.h | 4 ++++ ld/ldmain.c | 3 ++- 4 files changed, 49 insertions(+), 1 deletion(-) (limited to 'ld') diff --git a/ld/ChangeLog b/ld/ChangeLog index cbe0544..7c6506b 100644 --- a/ld/ChangeLog +++ b/ld/ChangeLog @@ -1,3 +1,10 @@ +2007-02-21 Nick Clifton + + * ldlang.c (ldlang_override_segment_assignment): New function. + * ldlang.h (ldlang_override_segment_assignment): Prototype. + * ldmain.c (link_callbacks): Add + ldlang_override_segment_assignment. + 2007-02-20 Alan Modra * ldexp.c (fold_name ): Ensure result is always absolute. diff --git a/ld/ldlang.c b/ld/ldlang.c index 8a69c70..fa597f3 100644 --- a/ld/ldlang.c +++ b/ld/ldlang.c @@ -4704,6 +4704,42 @@ lang_size_sections_1 return dot; } +/* Callback routine that is used in _bfd_elf_map_sections_to_segments. */ + +bfd_boolean +ldlang_override_segment_assignment (struct bfd_link_info * info ATTRIBUTE_UNUSED, + bfd * abfd ATTRIBUTE_UNUSED, + asection * current_section, + asection * previous_section, + bfd_boolean new_segment) +{ + lang_output_section_statement_type * cur; + lang_output_section_statement_type * prev; + + if (new_segment) + return TRUE; + + /* Paranoia checks. */ + if (current_section == NULL || previous_section == NULL) + return new_segment; + + /* Find the memory regions associated with the two sections. + We call lang_output_section_find() here rather than scanning the list + of output sections looking for a matching section pointer because if + we have a large number of sections a hash lookup is faster. */ + cur = lang_output_section_find (current_section->name); + prev = lang_output_section_find (previous_section->name); + + if (cur == NULL || prev == NULL) + return new_segment; + + /* If the regions are different then force the sections to live in + different segments. See the email thread starting here for the + reasons why this is necessary: + http://sourceware.org/ml/binutils/2007-02/msg00216.html */ + return cur->region != prev->region; +} + void one_lang_size_sections_pass (bfd_boolean *relax, bfd_boolean check_regions) { diff --git a/ld/ldlang.h b/ld/ldlang.h index 0074159..d34ea68 100644 --- a/ld/ldlang.h +++ b/ld/ldlang.h @@ -623,4 +623,8 @@ extern void add_excluded_libs (const char *); extern bfd_boolean load_symbols (lang_input_statement_type *, lang_statement_list_type *); +extern bfd_boolean +ldlang_override_segment_assignment + (struct bfd_link_info *, bfd *, asection *, asection *, bfd_boolean); + #endif diff --git a/ld/ldmain.c b/ld/ldmain.c index 9e6a0c3..f47758f 100644 --- a/ld/ldmain.c +++ b/ld/ldmain.c @@ -161,7 +161,8 @@ static struct bfd_link_callbacks link_callbacks = reloc_dangerous, unattached_reloc, notice, - einfo + einfo, + ldlang_override_segment_assignment }; struct bfd_link_info link_info; -- cgit v1.1