Blame SOURCES/binutils-SHF_LINK_ORDER.patch

d1152b
diff -rup binutils.orig/bfd/bfd-in2.h binutils-2.35.1/bfd/bfd-in2.h
d1152b
--- binutils.orig/bfd/bfd-in2.h	2021-01-04 13:18:10.234368481 +0000
d1152b
+++ binutils-2.35.1/bfd/bfd-in2.h	2021-01-04 13:18:20.596301287 +0000
d1152b
@@ -1177,6 +1177,9 @@ typedef struct bfd_section
d1152b
   struct bfd_symbol *symbol;
d1152b
   struct bfd_symbol **symbol_ptr_ptr;
d1152b
 
d1152b
+  /* The matching section name pattern in linker script.  */
d1152b
+  const char *pattern;
d1152b
+
d1152b
   /* Early in the link process, map_head and map_tail are used to build
d1152b
      a list of input sections attached to an output section.  Later,
d1152b
      output sections use these fields for a list of bfd_link_order
d1152b
@@ -1370,8 +1373,8 @@ discarded_section (const asection *sec)
d1152b
   /* target_index, used_by_bfd, constructor_chain, owner,           */ \
d1152b
      0,            NULL,        NULL,              NULL,               \
d1152b
                                                                        \
d1152b
-  /* symbol,                    symbol_ptr_ptr,                     */ \
d1152b
-     (struct bfd_symbol *) SYM, &SEC.symbol,                           \
d1152b
+  /* symbol,                    symbol_ptr_ptr, pattern,            */ \
d1152b
+     (struct bfd_symbol *) SYM, &SEC.symbol,    NULL,                  \
d1152b
                                                                        \
d1152b
   /* map_head, map_tail, already_assigned                           */ \
d1152b
      { NULL }, { NULL }, NULL                                          \
d1152b
diff -rup binutils.orig/bfd/elflink.c binutils-2.35.1/bfd/elflink.c
d1152b
--- binutils.orig/bfd/elflink.c	2021-01-04 13:18:10.223368552 +0000
d1152b
+++ binutils-2.35.1/bfd/elflink.c	2021-01-04 13:18:20.599301268 +0000
d1152b
@@ -11662,8 +11662,21 @@ compare_link_order (const void *a, const
d1152b
   const struct bfd_link_order *blo = *(const struct bfd_link_order **) b;
d1152b
   asection *asec = elf_linked_to_section (alo->u.indirect.section);
d1152b
   asection *bsec = elf_linked_to_section (blo->u.indirect.section);
d1152b
-  bfd_vma apos = asec->output_section->lma + asec->output_offset;
d1152b
-  bfd_vma bpos = bsec->output_section->lma + bsec->output_offset;
d1152b
+  bfd_vma apos, bpos;
d1152b
+
d1152b
+  /* Check if any sections are unordered.  */
d1152b
+  if (asec == NULL || bsec == NULL)
d1152b
+    {
d1152b
+      /* Place ordered sections before unordered sections.  */
d1152b
+      if (bsec != NULL)
d1152b
+	return 1;
d1152b
+      else if (asec != NULL)
d1152b
+	return -1;
d1152b
+      return 0;
d1152b
+    }
d1152b
+
d1152b
+  apos = asec->output_section->lma + asec->output_offset;
d1152b
+  bpos = bsec->output_section->lma + bsec->output_offset;
d1152b
 
d1152b
   if (apos < bpos)
d1152b
     return -1;
d1152b
@@ -11698,14 +11711,14 @@ compare_link_order (const void *a, const
d1152b
    sections.  Ideally we'd do this in the linker proper.  */
d1152b
 
d1152b
 static bfd_boolean
d1152b
-elf_fixup_link_order (bfd *abfd, asection *o)
d1152b
+elf_fixup_link_order (struct bfd_link_info *info, bfd *abfd, asection *o)
d1152b
 {
d1152b
   size_t seen_linkorder;
d1152b
   size_t seen_other;
d1152b
   size_t n;
d1152b
   struct bfd_link_order *p;
d1152b
   bfd *sub;
d1152b
-  struct bfd_link_order **sections;
d1152b
+  struct bfd_link_order **sections, **indirect_sections;
d1152b
   asection *other_sec, *linkorder_sec;
d1152b
   bfd_vma offset;  /* Octets.  */
d1152b
 
d1152b
@@ -11736,7 +11749,9 @@ elf_fixup_link_order (bfd *abfd, asectio
d1152b
       else
d1152b
 	seen_other++;
d1152b
 
d1152b
-      if (seen_other && seen_linkorder)
d1152b
+      /* Allow mixed ordered and unordered input sections for
d1152b
+         non-relocatable link.  */
d1152b
+      if (bfd_link_relocatable (info) && seen_other && seen_linkorder)
d1152b
 	{
d1152b
 	  if (other_sec && linkorder_sec)
d1152b
 	    _bfd_error_handler
d1152b
@@ -11756,6 +11771,10 @@ elf_fixup_link_order (bfd *abfd, asectio
d1152b
   if (!seen_linkorder)
d1152b
     return TRUE;
d1152b
 
d1152b
+  /* Non-relocatable output can have both ordered and unordered input
d1152b
+     sections.  */
d1152b
+  seen_linkorder += seen_other;
d1152b
+
d1152b
   sections = bfd_malloc (seen_linkorder * sizeof (*sections));
d1152b
   if (sections == NULL)
d1152b
     return FALSE;
d1152b
@@ -11764,22 +11783,51 @@ elf_fixup_link_order (bfd *abfd, asectio
d1152b
   for (p = o->map_head.link_order; p != NULL; p = p->next)
d1152b
     sections[seen_linkorder++] = p;
d1152b
 
d1152b
-  /* Sort the input sections in the order of their linked section.  */
d1152b
-  qsort (sections, seen_linkorder, sizeof (*sections), compare_link_order);
d1152b
+  for (indirect_sections = sections, n = 0; n < seen_linkorder;)
d1152b
+    {
d1152b
+      /* Find the first bfd_indirect_link_order section.  */
d1152b
+      if (indirect_sections[0]->type == bfd_indirect_link_order)
d1152b
+	{
d1152b
+	  /* Count the consecutive bfd_indirect_link_order sections
d1152b
+	     with the same pattern.  */
d1152b
+	  size_t i, n_indirect;
d1152b
+	  const char *pattern
d1152b
+	    = indirect_sections[0]->u.indirect.section->pattern;
d1152b
+	  for (i = n + 1; i < seen_linkorder; i++)
d1152b
+	    if (sections[i]->type != bfd_indirect_link_order
d1152b
+		|| sections[i]->u.indirect.section->pattern != pattern)
d1152b
+	      break;
d1152b
+	  n_indirect = i - n;
d1152b
+	  /* Sort the bfd_indirect_link_order sections in the order of
d1152b
+	     their linked section.  */
d1152b
+	  qsort (indirect_sections, n_indirect, sizeof (*sections),
d1152b
+		 compare_link_order);
d1152b
+	  indirect_sections += n_indirect;
d1152b
+	  n += n_indirect;
d1152b
+	}
d1152b
+      else
d1152b
+	{
d1152b
+	  indirect_sections++;
d1152b
+	  n++;
d1152b
+	}
d1152b
+    }
d1152b
 
d1152b
-  /* Change the offsets of the sections.  */
d1152b
+  /* Change the offsets of the bfd_indirect_link_order sections.  */
d1152b
   offset = 0;
d1152b
   for (n = 0; n < seen_linkorder; n++)
d1152b
-    {
d1152b
-      bfd_vma mask;
d1152b
-      asection *s = sections[n]->u.indirect.section;
d1152b
-      unsigned int opb = bfd_octets_per_byte (abfd, s);
d1152b
-
d1152b
-      mask = ~(bfd_vma) 0 << s->alignment_power * opb;
d1152b
-      offset = (offset + ~mask) & mask;
d1152b
-      sections[n]->offset = s->output_offset = offset / opb;
d1152b
-      offset += sections[n]->size;
d1152b
-    }
d1152b
+    if (sections[n]->type == bfd_indirect_link_order)
d1152b
+      {
d1152b
+	bfd_vma mask;
d1152b
+	asection *s = sections[n]->u.indirect.section;
d1152b
+	unsigned int opb = bfd_octets_per_byte (abfd, s);
d1152b
+
d1152b
+	mask = ~(bfd_vma) 0 << s->alignment_power * opb;
d1152b
+	offset = (offset + ~mask) & mask;
d1152b
+	sections[n]->offset = s->output_offset = offset / opb;
d1152b
+	offset += sections[n]->size;
d1152b
+      }
d1152b
+    else
d1152b
+      offset = sections[n]->offset + sections[n]->size;
d1152b
 
d1152b
   free (sections);
d1152b
   return TRUE;
d1152b
@@ -12408,7 +12456,7 @@ bfd_elf_final_link (bfd *abfd, struct bf
d1152b
   /* Reorder SHF_LINK_ORDER sections.  */
d1152b
   for (o = abfd->sections; o != NULL; o = o->next)
d1152b
     {
d1152b
-      if (!elf_fixup_link_order (abfd, o))
d1152b
+      if (!elf_fixup_link_order (info, abfd, o))
d1152b
 	return FALSE;
d1152b
     }
d1152b
 
d1152b
diff -rup binutils.orig/bfd/section.c binutils-2.35.1/bfd/section.c
d1152b
--- binutils.orig/bfd/section.c	2021-01-04 13:18:10.233368487 +0000
d1152b
+++ binutils-2.35.1/bfd/section.c	2021-01-04 13:18:20.599301268 +0000
d1152b
@@ -541,6 +541,9 @@ CODE_FRAGMENT
d1152b
 .  struct bfd_symbol *symbol;
d1152b
 .  struct bfd_symbol **symbol_ptr_ptr;
d1152b
 .
d1152b
+.  {* The matching section name pattern in linker script.  *}
d1152b
+.  const char *pattern;
d1152b
+.
d1152b
 .  {* Early in the link process, map_head and map_tail are used to build
d1152b
 .     a list of input sections attached to an output section.  Later,
d1152b
 .     output sections use these fields for a list of bfd_link_order
d1152b
@@ -734,8 +737,8 @@ CODE_FRAGMENT
d1152b
 .  {* target_index, used_by_bfd, constructor_chain, owner,           *}	\
d1152b
 .     0,            NULL,        NULL,              NULL,		\
d1152b
 .									\
d1152b
-.  {* symbol,                    symbol_ptr_ptr,                     *}	\
d1152b
-.     (struct bfd_symbol *) SYM, &SEC.symbol,				\
d1152b
+.  {* symbol,                    symbol_ptr_ptr, pattern,            *}	\
d1152b
+.     (struct bfd_symbol *) SYM, &SEC.symbol,    NULL,			\
d1152b
 .									\
d1152b
 .  {* map_head, map_tail, already_assigned                           *}	\
d1152b
 .     { NULL }, { NULL }, NULL						\
d1152b
diff -rup binutils.orig/gas/config/obj-elf.c binutils-2.35.1/gas/config/obj-elf.c
d1152b
--- binutils.orig/gas/config/obj-elf.c	2021-01-04 13:18:09.942370375 +0000
d1152b
+++ binutils-2.35.1/gas/config/obj-elf.c	2021-01-04 13:18:20.599301268 +0000
d1152b
@@ -659,7 +659,9 @@ obj_elf_change_section (const char *name
d1152b
 	    }
d1152b
 	}
d1152b
 
d1152b
-      if (old_sec == NULL && ((attr & ~(SHF_MASKOS | SHF_MASKPROC))
d1152b
+      if (old_sec == NULL && ((attr & ~(SHF_LINK_ORDER
d1152b
+					| SHF_MASKOS
d1152b
+					| SHF_MASKPROC))
d1152b
 			      & ~ssect->attr) != 0)
d1152b
 	{
d1152b
 	  /* As a GNU extension, we permit a .note section to be
d1152b
diff -rup binutils.orig/ld/ldlang.c binutils-2.35.1/ld/ldlang.c
d1152b
--- binutils.orig/ld/ldlang.c	2021-01-04 13:18:09.691372002 +0000
d1152b
+++ binutils-2.35.1/ld/ldlang.c	2021-01-04 13:18:20.600301261 +0000
d1152b
@@ -7421,7 +7421,7 @@ lang_reset_memory_regions (void)
d1152b
 
d1152b
 static void
d1152b
 gc_section_callback (lang_wild_statement_type *ptr,
d1152b
-		     struct wildcard_list *sec ATTRIBUTE_UNUSED,
d1152b
+		     struct wildcard_list *sec,
d1152b
 		     asection *section,
d1152b
 		     struct flag_info *sflag_info ATTRIBUTE_UNUSED,
d1152b
 		     lang_input_statement_type *file ATTRIBUTE_UNUSED,
d1152b
@@ -7431,6 +7431,8 @@ gc_section_callback (lang_wild_statement
d1152b
      should be as well.  */
d1152b
   if (ptr->keep_sections)
d1152b
     section->flags |= SEC_KEEP;
d1152b
+  if (sec)
d1152b
+    section->pattern = sec->spec.name;
d1152b
 }
d1152b
 
d1152b
 /* Iterate over sections marking them against GC.  */