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]

SPU soft-icache fixes


This patch teaches ld to generate a smaller soft-icache call stub,
which will probably be the default.  For now though, enable the
smaller stub by passing --compact-stubs to ld.  In addition, ld no
longer allocates space for the __icache_ptr_handler<n> entries,
and icache calls from non-icache code now go via another manager
entry point.

bfd/
	* elf32-spu.h (struct spu_elf_params): ovly_flavour now only 1 bit.
	Add compact_stub.
	(emum _ovly_flavour): Delete ovly_compact, ovly_none.
	* elf32-spu.c (struct spu_link_hash_table): Replace ovly_load and
	ovly_return fields with ovly_entry[2].  Adjust all users.
	(spu_elf_find_overlays): Set ovly_entry[1] from __icache_call_handler
	when soft-icache.
	(spu_elf_build_stubs): Likewise.
	(ovl_stub_size): Change arg to spu_elf_params pointer.  Adjust for
	ovly_flavour changes.  Update all callers.
	(ovl_stub_size_log2): New function.
	(build_stub): Handle compact icache stubs.  Use different manager
	entry point for stubs in non-icache area.
	(spu_elf_size_stubs): Don't allocate space for indirect branch
	descriptors.
	(spu_elf_build_stubs): And don't built them.
ld/
	* emultempl/spu_icache.S: Add new entry to dummy handler.
	* emultempl/spu_icache.o_c: Regenerate.
	* emultempl/spuelf.em (params): Init new field.
	(no_overlays): New static var.
	(spu_before_allocation): Use it.
	(OPTION_SPU_COMPACT_STUBS): Define.
	(PARSE_AND_LIST_LONGOPTS, PARSE_AND_LIST_OPTIONS): Add compact-stubs.
	(PARSE_AND_LIST_ARGS_CASES): Handle compact-stubs.  Adjust no-overlays
	handling.
ld/testsuite/
	* ld-spu/icache1.d: Update for fixed set_id, new manager entry, and
	reduced data.

Index: bfd/elf32-spu.h
===================================================================
RCS file: /cvs/src/src/bfd/elf32-spu.h,v
retrieving revision 1.12
diff -u -p -r1.12 elf32-spu.h
--- bfd/elf32-spu.h	21 Jan 2009 02:27:12 -0000	1.12
+++ bfd/elf32-spu.h	17 Mar 2009 12:13:56 -0000
@@ -35,7 +35,8 @@ struct spu_elf_params
 #define OVERLAY_RODATA 4
 
   /* Type of overlays, enum _ovly_flavour.  */
-  unsigned int ovly_flavour : 2;
+  unsigned int ovly_flavour : 1;
+  unsigned int compact_stub : 1;
 
   /* Set if we should emit symbols for stubs.  */
   unsigned int emit_stub_syms : 1;
@@ -98,10 +99,8 @@ struct _spu_elf_section_data
 
 enum _ovly_flavour
 {
-  ovly_compact,
   ovly_normal,
-  ovly_soft_icache,
-  ovly_none
+  ovly_soft_icache
 };
 
 struct _ovl_stream
Index: bfd/elf32-spu.c
===================================================================
RCS file: /cvs/src/src/bfd/elf32-spu.c,v
retrieving revision 1.67
diff -u -p -r1.67 elf32-spu.c
--- bfd/elf32-spu.c	15 Mar 2009 15:15:29 -0000	1.67
+++ bfd/elf32-spu.c	17 Mar 2009 12:13:56 -0000
@@ -311,9 +311,7 @@ struct spu_link_hash_table
   /* The stub section for each overlay section.  */
   asection **stub_sec;
 
-  struct elf_link_hash_entry *ovly_load;
-  struct elf_link_hash_entry *ovly_return;
-  unsigned long ovly_load_r_symndx;
+  struct elf_link_hash_entry *ovly_entry[2];
 
   /* Number of overlay buffers.  */
   unsigned int num_buf;
@@ -773,10 +771,12 @@ spu_elf_find_overlays (struct bfd_link_i
   ovly_mgr_entry = "__ovly_load";
   if (htab->params->ovly_flavour == ovly_soft_icache)
     ovly_mgr_entry = "__icache_br_handler";
-  htab->ovly_load = elf_link_hash_lookup (&htab->elf, ovly_mgr_entry,
-					  FALSE, FALSE, FALSE);
-  if (htab->params->ovly_flavour != ovly_soft_icache)
-    htab->ovly_return = elf_link_hash_lookup (&htab->elf, "__ovly_return",
+  htab->ovly_entry[0] = elf_link_hash_lookup (&htab->elf, ovly_mgr_entry,
+					      FALSE, FALSE, FALSE);
+  ovly_mgr_entry = "__ovly_return";
+  if (htab->params->ovly_flavour == ovly_soft_icache)
+    ovly_mgr_entry = "__icache_call_handler";
+  htab->ovly_entry[1] = elf_link_hash_lookup (&htab->elf, ovly_mgr_entry,
 					      FALSE, FALSE, FALSE);
   return ovl_index != 0;
 }
@@ -897,7 +897,7 @@ needs_ovl_stub (struct elf_link_hash_ent
   if (h != NULL)
     {
       /* Ensure no stubs for user supplied overlay manager syms.  */
-      if (h == htab->ovly_load || h == htab->ovly_return)
+      if (h == htab->ovly_entry[0]|| h == htab->ovly_entry[1])
 	return ret;
 
       /* setjmp always goes via an overlay stub, because then the return
@@ -1100,12 +1100,19 @@ count_stub (struct spu_link_hash_table *
 }
 
 /* Support two sizes of overlay stubs, a slower more compact stub of two
-   intructions, and a faster stub of four instructions.  */
+   intructions, and a faster stub of four instructions.
+   Soft-icache stubs are four or eight words.  */
 
 static unsigned int
-ovl_stub_size (enum _ovly_flavour ovly_flavour)
+ovl_stub_size (struct spu_elf_params *params)
 {
-  return 8 << ovly_flavour;
+  return 16 << params->ovly_flavour >> params->compact_stub;
+}
+
+static unsigned int
+ovl_stub_size_log2 (struct spu_elf_params *params)
+{
+  return 4 + params->ovly_flavour - params->compact_stub;
 }
 
 /* Two instruction overlay stubs look like:
@@ -1195,9 +1202,9 @@ build_stub (struct bfd_link_info *info,
   dest += dest_sec->output_offset + dest_sec->output_section->vma;
   from = sec->size + sec->output_offset + sec->output_section->vma;
   g->stub_addr = from;
-  to = (htab->ovly_load->root.u.def.value
-	+ htab->ovly_load->root.u.def.section->output_offset
-	+ htab->ovly_load->root.u.def.section->output_section->vma);
+  to = (htab->ovly_entry[0]->root.u.def.value
+	+ htab->ovly_entry[0]->root.u.def.section->output_offset
+	+ htab->ovly_entry[0]->root.u.def.section->output_section->vma);
 
   if (((dest | to | from) & 3) != 0)
     {
@@ -1206,9 +1213,9 @@ build_stub (struct bfd_link_info *info,
     }
   dest_ovl = spu_elf_section_data (dest_sec->output_section)->u.o.ovl_index;
 
-  switch (htab->params->ovly_flavour)
+  if (htab->params->ovly_flavour == ovly_normal
+      && !htab->params->compact_stub)
     {
-    case ovly_normal:
       bfd_put_32 (sec->owner, ILA + ((dest_ovl << 7) & 0x01ffff80) + 78,
 		  sec->contents + sec->size);
       bfd_put_32 (sec->owner, LNOP,
@@ -1221,9 +1228,10 @@ build_stub (struct bfd_link_info *info,
       else
 	bfd_put_32 (sec->owner, BRA + ((to << 5) & 0x007fff80),
 		    sec->contents + sec->size + 12);
-      break;
-
-    case ovly_compact:
+    }
+  else if (htab->params->ovly_flavour == ovly_normal
+	   && htab->params->compact_stub)
+    {
       if (!BRA_STUBS)
 	bfd_put_32 (sec->owner, BRSL + (((to - from) << 5) & 0x007fff80) + 75,
 		    sec->contents + sec->size);
@@ -1232,9 +1240,9 @@ build_stub (struct bfd_link_info *info,
 		    sec->contents + sec->size);
       bfd_put_32 (sec->owner, (dest & 0x3ffff) | (dest_ovl << 18),
 		  sec->contents + sec->size + 4);
-      break;
-
-    case ovly_soft_icache:
+    }
+  else if (htab->params->ovly_flavour == ovly_soft_icache)
+    {
       lrlive = 0;
       if (stub_type == nonovl_stub)
 	;
@@ -1314,43 +1322,75 @@ build_stub (struct bfd_link_info *info,
       if (stub_type > br000_ovl_stub)
 	lrlive = stub_type - br000_ovl_stub;
 
-      /* The branch that uses this stub goes to stub_addr + 12.  We'll
-         set up an xor pattern that can be used by the icache manager
-	 to modify this branch to go directly to its destination.  */
-      g->stub_addr += 12;
-      br_dest = g->stub_addr;
-      if (irela == NULL)
-	{
-	  /* Except in the case of _SPUEAR_ stubs, the branch in
-	     question is the one in the stub itself.  */
-	  BFD_ASSERT (stub_type == nonovl_stub);
-	  g->br_addr = g->stub_addr;
-	  br_dest = to;
-	}
-
-      bfd_put_32 (sec->owner, dest_ovl - 1,
-		  sec->contents + sec->size + 0);
-      set_id = ((dest_ovl - 1) >> htab->num_lines_log2) + 1;
-      bfd_put_32 (sec->owner, (set_id << 18) | (dest & 0x3ffff),
-		  sec->contents + sec->size + 4);
-      bfd_put_32 (sec->owner, (lrlive << 29) | (g->br_addr & 0x3ffff),
-		  sec->contents + sec->size + 8);
-      bfd_put_32 (sec->owner, BRASL + ((to << 5) & 0x007fff80) + 75,
-		  sec->contents + sec->size + 12);
-      patt = dest ^ br_dest;
-      if (irela != NULL && ELF32_R_TYPE (irela->r_info) == R_SPU_REL16)
-	patt = (dest - g->br_addr) ^ (br_dest - g->br_addr);
-      bfd_put_32 (sec->owner, (patt << 5) & 0x007fff80,
-		  sec->contents + sec->size + 16 + (g->br_addr & 0xf));
+      if (ovl == 0)
+	to = (htab->ovly_entry[1]->root.u.def.value
+	      + htab->ovly_entry[1]->root.u.def.section->output_offset
+	      + htab->ovly_entry[1]->root.u.def.section->output_section->vma);
+
+      if (!htab->params->compact_stub)
+	{
+	  /* The branch that uses this stub goes to stub_addr + 12.  We'll
+	     set up an xor pattern that can be used by the icache manager
+	     to modify this branch to go directly to its destination.  */
+	  g->stub_addr += 12;
+	  br_dest = g->stub_addr;
+	  if (irela == NULL)
+	    {
+	      /* Except in the case of _SPUEAR_ stubs, the branch in
+		 question is the one in the stub itself.  */
+	      BFD_ASSERT (stub_type == nonovl_stub);
+	      g->br_addr = g->stub_addr;
+	      br_dest = to;
+	    }
+
+	  bfd_put_32 (sec->owner, dest_ovl - 1,
+		      sec->contents + sec->size + 0);
+	  set_id = ((dest_ovl - 1) >> htab->num_lines_log2) + 1;
+	  bfd_put_32 (sec->owner, (set_id << 18) | (dest & 0x3ffff),
+		      sec->contents + sec->size + 4);
+	  bfd_put_32 (sec->owner, (lrlive << 29) | (g->br_addr & 0x3ffff),
+		      sec->contents + sec->size + 8);
+	  bfd_put_32 (sec->owner, BRASL + ((to << 5) & 0x007fff80) + 75,
+		      sec->contents + sec->size + 12);
+	  patt = dest ^ br_dest;
+	  if (irela != NULL && ELF32_R_TYPE (irela->r_info) == R_SPU_REL16)
+	    patt = (dest - g->br_addr) ^ (br_dest - g->br_addr);
+	  bfd_put_32 (sec->owner, (patt << 5) & 0x007fff80,
+		      sec->contents + sec->size + 16 + (g->br_addr & 0xf));
+	}
+      else
+	{
+	  g->stub_addr += 4;
+	  br_dest = g->stub_addr;
+	  if (irela == NULL)
+	    {
+	      BFD_ASSERT (stub_type == nonovl_stub);
+	      g->br_addr = g->stub_addr;
+	      br_dest = to;
+	    }
+
+	  set_id = ((dest_ovl - 1) >> htab->num_lines_log2) + 1;
+	  bfd_put_32 (sec->owner, (set_id << 18) | (dest & 0x3ffff),
+		      sec->contents + sec->size);
+	  bfd_put_32 (sec->owner, BRASL + ((to << 5) & 0x007fff80) + 75,
+		      sec->contents + sec->size + 4);
+	  bfd_put_32 (sec->owner, (lrlive << 29) | (g->br_addr & 0x3ffff),
+		      sec->contents + sec->size + 8);
+	  patt = dest ^ br_dest;
+	  if (irela != NULL && ELF32_R_TYPE (irela->r_info) == R_SPU_REL16)
+	    patt = (dest - g->br_addr) ^ (br_dest - g->br_addr);
+	  bfd_put_32 (sec->owner, (patt << 5) & 0x007fff80,
+		      sec->contents + sec->size + 12);
+	}
+
       if (ovl == 0)
 	/* Extra space for linked list entries.  */
 	sec->size += 16;
-      break;
-
-    default:
-      abort ();
     }
-  sec->size += ovl_stub_size (htab->params->ovly_flavour);
+  else
+    abort ();
+
+  sec->size += ovl_stub_size (htab->params);
 
   if (htab->params->emit_stub_syms)
     {
@@ -1390,7 +1430,7 @@ build_stub (struct bfd_link_info *info,
 	{
 	  h->root.type = bfd_link_hash_defined;
 	  h->root.u.def.section = sec;
-	  h->size = ovl_stub_size (htab->params->ovly_flavour);
+	  h->size = ovl_stub_size (htab->params);
 	  h->root.u.def.value = sec->size - h->size;
 	  h->type = STT_FUNC;
 	  h->ref_regular = 1;
@@ -1587,7 +1627,8 @@ process_stubs (struct bfd_link_info *inf
   return TRUE;
 }
 
-/* Allocate space for overlay call and return stubs.  */
+/* Allocate space for overlay call and return stubs.
+   Return 0 on error, 1 if no stubs, 2 otherwise.  */
 
 int
 spu_elf_size_stubs (struct bfd_link_info *info)
@@ -1623,9 +1664,9 @@ spu_elf_size_stubs (struct bfd_link_info
   htab->stub_sec[0] = stub;
   if (stub == NULL
       || !bfd_set_section_alignment (ibfd, stub,
-				     htab->params->ovly_flavour + 3))
+				     ovl_stub_size_log2 (htab->params)))
     return 0;
-  stub->size = htab->stub_count[0] * ovl_stub_size (htab->params->ovly_flavour);
+  stub->size = htab->stub_count[0] * ovl_stub_size (htab->params);
   if (htab->params->ovly_flavour == ovly_soft_icache)
     /* Extra space for linked list entries.  */
     stub->size += htab->stub_count[0] * 16;
@@ -1639,9 +1680,9 @@ spu_elf_size_stubs (struct bfd_link_info
       htab->stub_sec[ovl] = stub;
       if (stub == NULL
 	  || !bfd_set_section_alignment (ibfd, stub,
-					 htab->params->ovly_flavour + 3))
+					 ovl_stub_size_log2 (htab->params)))
 	return 0;
-      stub->size = htab->stub_count[ovl] * ovl_stub_size (htab->params->ovly_flavour);
+      stub->size = htab->stub_count[ovl] * ovl_stub_size (htab->params);
       (*htab->params->place_spu_section) (stub, osec, NULL);
     }
 
@@ -1656,11 +1697,9 @@ spu_elf_size_stubs (struct bfd_link_info
     {
       /* Space for icache manager tables.
 	 a) Tag array, one quadword per cache line.
-	 b) Linked list elements, max_branch per line quadwords.
-	 c) Indirect branch descriptors, 8 quadwords.  */
-      htab->ovtab->size = 16 * (((1 + htab->params->max_branch)
-				 << htab->num_lines_log2)
-				+ 8);
+	 b) Linked list elements, max_branch per line quadwords.  */
+      htab->ovtab->size = 16 * ((1 + htab->params->max_branch)
+				<< htab->num_lines_log2);
 
       htab->init = bfd_make_section_anyway_with_flags (ibfd, ".ovini", flags);
       if (htab->init == NULL
@@ -1824,7 +1863,7 @@ spu_elf_build_stubs (struct bfd_link_inf
 	htab->stub_sec[i]->size = 0;
       }
 
-  h = htab->ovly_load;
+  h = htab->ovly_entry[0];
   if (h == NULL)
     {
       const char *ovly_mgr_entry = "__ovly_load";
@@ -1833,7 +1872,7 @@ spu_elf_build_stubs (struct bfd_link_inf
 	ovly_mgr_entry = "__icache_br_handler";
       h = elf_link_hash_lookup (&htab->elf, ovly_mgr_entry,
 				FALSE, FALSE, FALSE);
-      htab->ovly_load = h;
+      htab->ovly_entry[0] = h;
     }
   BFD_ASSERT (h != NULL
 	      && (h->root.type == bfd_link_hash_defined
@@ -1849,12 +1888,29 @@ spu_elf_build_stubs (struct bfd_link_inf
       return FALSE;
     }
 
-  h = htab->ovly_return;
-  if (h == NULL && htab->params->ovly_flavour != ovly_soft_icache)
+  h = htab->ovly_entry[1];
+  if (h == NULL)
     {
-      h = elf_link_hash_lookup (&htab->elf, "__ovly_return",
+      const char *ovly_mgr_entry = "__ovly_return";
+
+      if (htab->params->ovly_flavour == ovly_soft_icache)
+	ovly_mgr_entry = "__icache_call_handler";
+      h = elf_link_hash_lookup (&htab->elf, ovly_mgr_entry,
 				FALSE, FALSE, FALSE);
-      htab->ovly_return = h;
+      htab->ovly_entry[1] = h;
+    }
+  BFD_ASSERT (h != NULL
+	      && (h->root.type == bfd_link_hash_defined
+		  || h->root.type == bfd_link_hash_defweak)
+	      && h->def_regular);
+
+  s = h->root.u.def.section->output_section;
+  if (spu_elf_section_data (s)->u.o.ovl_index)
+    {
+      (*_bfd_error_handler) (_("%s in overlay section"),
+			     h->root.root.string);
+      bfd_set_error (bfd_error_bad_value);
+      return FALSE;
     }
 
   /* Fill in all the stubs.  */
@@ -1890,9 +1946,7 @@ spu_elf_build_stubs (struct bfd_link_inf
   p = htab->ovtab->contents;
   if (htab->params->ovly_flavour == ovly_soft_icache)
     {
-#define BI_HANDLER "__icache_ptr_handler0"
-      char name[sizeof (BI_HANDLER)];
-      bfd_vma off, icache_base, linklist, bihand;
+      bfd_vma off, icache_base, linklist;
 
       h = define_ovtab_symbol (htab, "__icache_tagbase");
       if (h == NULL)
@@ -1927,31 +1981,6 @@ spu_elf_build_stubs (struct bfd_link_inf
       off += h->size;
       p += h->size;
 
-      h = elf_link_hash_lookup (&htab->elf, "__icache_bi_handler",
-				 FALSE, FALSE, FALSE);
-      bihand = 0;
-      if (h != NULL
-	  && (h->root.type == bfd_link_hash_defined
-	      || h->root.type == bfd_link_hash_defweak)
-	  && h->def_regular)
-	bihand = (h->root.u.def.value
-		  + h->root.u.def.section->output_offset
-		  + h->root.u.def.section->output_section->vma);
-      memcpy (name, BI_HANDLER, sizeof (BI_HANDLER));
-      for (i = 0; i < 8; i++)
-	{
-	  name[sizeof (BI_HANDLER) - 2] = '0' + i;
-	  h = define_ovtab_symbol (htab, name);
-	  if (h == NULL)
-	    return FALSE;
-	  h->root.u.def.value = off;
-	  h->size = 16;
-	  bfd_put_32 (htab->ovtab->owner, bihand, p);
-	  bfd_put_32 (htab->ovtab->owner, i << 28, p + 8);
-	  p += 16;
-	  off += 16;
-	}
-
       h = define_ovtab_symbol (htab, "__icache_base");
       if (h == NULL)
 	return FALSE;
@@ -3652,7 +3681,7 @@ auto_ovl_lib_functions (struct bfd_link_
 		    if (p->fun == call->fun)
 		      break;
 		  if (!p)
-		    stub_size += ovl_stub_size (htab->params->ovly_flavour);
+		    stub_size += ovl_stub_size (htab->params);
 		}
 	}
       if (tmp + stub_size < lib_size)
@@ -3670,7 +3699,7 @@ auto_ovl_lib_functions (struct bfd_link_
 	  while ((p = *pp) != NULL)
 	    if (!p->fun->sec->linker_mark)
 	      {
-		lib_size += ovl_stub_size (htab->params->ovly_flavour);
+		lib_size += ovl_stub_size (htab->params);
 		*pp = p->next;
 		free (p);
 	      }
@@ -4200,7 +4229,7 @@ spu_elf_auto_overlay (struct bfd_link_in
   free (bfd_arr);
 
   fixed_size += htab->reserved;
-  fixed_size += htab->non_ovly_stub * ovl_stub_size (htab->params->ovly_flavour);
+  fixed_size += htab->non_ovly_stub * ovl_stub_size (htab->params);
   if (fixed_size + mos_param.max_overlay_size <= htab->local_store)
     {
       if (htab->params->ovly_flavour == ovly_soft_icache)
@@ -4362,7 +4391,7 @@ spu_elf_auto_overlay (struct bfd_link_in
 	  if (htab->params->ovly_flavour == ovly_soft_icache
 	      && num_stubs > htab->params->max_branch)
 	    break;
-	  if (tmp + num_stubs * ovl_stub_size (htab->params->ovly_flavour)
+	  if (tmp + num_stubs * ovl_stub_size (htab->params)
 	      > overlay_size)
 	    break;
 	  size = tmp;
Index: ld/emultempl/spu_icache.S
===================================================================
RCS file: /cvs/src/src/ld/emultempl/spu_icache.S,v
retrieving revision 1.1
diff -u -p -r1.1 spu_icache.S
--- ld/emultempl/spu_icache.S	12 Jan 2009 00:23:57 -0000	1.1
+++ ld/emultempl/spu_icache.S	17 Mar 2009 12:14:11 -0000
@@ -2,3 +2,6 @@
 	.global __icache_br_handler
 __icache_br_handler:
 	stop
+	.global __icache_call_handler
+__icache_call_handler:
+	stop
Index: ld/emultempl/spuelf.em
===================================================================
RCS file: /cvs/src/src/ld/emultempl/spuelf.em,v
retrieving revision 1.34
diff -u -p -r1.34 spuelf.em
--- ld/emultempl/spuelf.em	16 Feb 2009 07:25:52 -0000	1.34
+++ ld/emultempl/spuelf.em	17 Mar 2009 12:14:12 -0000
@@ -37,11 +37,12 @@ static struct spu_elf_params params =
   &spu_elf_load_ovl_mgr,
   &spu_elf_open_overlay_script,
   &spu_elf_relink,
-  0, ovly_normal, 0, 0, 0, 0, 0, 0,
+  0, ovly_normal, 0, 0, 0, 0, 0, 0, 0,
   0, 0x3ffff,
   1, 0, 16, 0, 0, 2000
 };
-  
+
+static unsigned int no_overlays = 0;  
 static unsigned int num_lines_set = 0;
 static unsigned int line_size_set = 0;
 static char *auto_overlay_file = 0;
@@ -263,7 +264,7 @@ spu_before_allocation (void)
 {
   if (is_spu_target ()
       && !link_info.relocatable
-      && params.ovly_flavour != ovly_none)
+      && !no_overlays)
     {
       /* Size the sections.  This is premature, but we need to know the
 	 rough layout so that overlays can be found.  */
@@ -584,7 +585,8 @@ fi
 PARSE_AND_LIST_PROLOGUE='
 #define OPTION_SPU_PLUGIN		301
 #define OPTION_SPU_NO_OVERLAYS		(OPTION_SPU_PLUGIN + 1)
-#define OPTION_SPU_STUB_SYMS		(OPTION_SPU_NO_OVERLAYS + 1)
+#define OPTION_SPU_COMPACT_STUBS	(OPTION_SPU_NO_OVERLAYS + 1)
+#define OPTION_SPU_STUB_SYMS		(OPTION_SPU_COMPACT_STUBS + 1)
 #define OPTION_SPU_NON_OVERLAY_STUBS	(OPTION_SPU_STUB_SYMS + 1)
 #define OPTION_SPU_LOCAL_STORE		(OPTION_SPU_NON_OVERLAY_STUBS + 1)
 #define OPTION_SPU_STACK_ANALYSIS	(OPTION_SPU_LOCAL_STORE + 1)
@@ -611,6 +613,7 @@ PARSE_AND_LIST_LONGOPTS='
   { "line-size", required_argument, NULL, OPTION_SPU_LINE_SIZE },
   { "non-ia-text", no_argument, NULL, OPTION_SPU_NON_IA_TEXT },
   { "no-overlays", no_argument, NULL, OPTION_SPU_NO_OVERLAYS },
+  { "compact-stubs", no_argument, NULL, OPTION_SPU_COMPACT_STUBS },
   { "emit-stub-syms", no_argument, NULL, OPTION_SPU_STUB_SYMS },
   { "extra-overlay-stubs", no_argument, NULL, OPTION_SPU_NON_OVERLAY_STUBS },
   { "local-store", required_argument, NULL, OPTION_SPU_LOCAL_STORE },
@@ -631,6 +634,7 @@ PARSE_AND_LIST_OPTIONS='
   fprintf (file, _("\
   --plugin                    Make SPU plugin.\n\
   --no-overlays               No overlay handling.\n\
+  --compact-stubs             Use smaller and possibly slower call stubs.\n\
   --emit-stub-syms            Add symbols on overlay call stubs.\n\
   --extra-overlay-stubs       Add stubs on all calls out of overlay regions.\n\
   --local-store=lo:hi         Valid address range.\n\
@@ -662,7 +666,11 @@ PARSE_AND_LIST_ARGS_CASES='
       break;
 
     case OPTION_SPU_NO_OVERLAYS:
-      params.ovly_flavour = ovly_none;
+      no_overlays = 1;
+      break;
+
+    case OPTION_SPU_COMPACT_STUBS:
+      params.compact_stub = 1;
       break;
 
     case OPTION_SPU_STUB_SYMS:
Index: ld/testsuite/ld-spu/icache1.d
===================================================================
RCS file: /cvs/src/src/ld/testsuite/ld-spu/icache1.d,v
retrieving revision 1.2
diff -u -p -r1.2 icache1.d
--- ld/testsuite/ld-spu/icache1.d	21 Jan 2009 02:28:22 -0000	1.2
+++ ld/testsuite/ld-spu/icache1.d	17 Mar 2009 12:14:14 -0000
@@ -8,7 +8,7 @@
 Disassembly of section .ovl.init:
 00000800 <__icache_fileoff>:
 .*	00 00 00 00.*
-.*	00 00 07 80.*
+.*	00 00 07 00.*
 	\.\.\.
 
 Disassembly of section \.ovly1:
@@ -32,19 +32,19 @@ Disassembly of section \.ovly2:
 .*	32 00 16 80 	br	fec .*
 	\.\.\.
      fa0:	00 00 00 02.*
-     fa4:	00 00 11 04.*
+     fa4:	00 04 11 04.*
      fa8:	a0 00 0c 10.*
      fac:	31 03 0a 4b 	brasl	\$75,1850 <__icache_br_handler>
      fb0:	00 00 ed 00.*
 	\.\.\.
      fc0:	00 00 00 02.*
-     fc4:	00 00 10 00.*
+     fc4:	00 04 10 00.*
      fc8:	a0 00 0c 14.*
      fcc:	31 03 0a 4b 	brasl	\$75,1850 <__icache_br_handler>
      fd0:	00 00 00 00.*
      fd4:	00 00 0a 80.*
 	\.\.\.
-     fe4:	00 00 08 00.*
+     fe4:	00 04 08 00.*
      fe8:	20 00 0f 38.*
      fec:	31 03 0a 4b 	brasl	\$75,1850 <__icache_br_handler>
 	\.\.\.
@@ -89,7 +89,7 @@ Disassembly of section \.ovly5:
 .*	30 01 7d 80 	bra	bec .*
 	\.\.\.
  be0:	00 00 00 03.*
- be4:	00 00 14 00.*
+ be4:	00 04 14 00.*
  be8:	a0 00 0b 2c.*
  bec:	31 03 0a 4b 	brasl	\$75,1850 <__icache_br_handler>
 	\.\.\.
@@ -104,20 +104,20 @@ Disassembly of section \.ovly6:
 .*	32 00 18 80 	br	fec .*
 	\.\.\.
      fa0:	00 00 00 07.*
-     fa4:	00 04 14 00.*
+     fa4:	00 08 14 00.*
      fa8:	a0 00 0c 00.*
      fac:	31 03 0a 4b 	brasl	\$75,1850 <__icache_br_handler>
      fb0:	00 03 75 80.*
 	\.\.\.
      fc0:	00 00 00 07.*
-     fc4:	00 04 14 00.*
+     fc4:	00 08 14 00.*
      fc8:	a0 00 0c 04.*
      fcc:	31 03 0a 4b 	brasl	\$75,1850 <__icache_br_handler>
      fd0:	00 00 00 00.*
      fd4:	00 00 86 80.*
 	\.\.\.
      fe0:	00 00 00 04.*
-     fe4:	00 04 08 00.*
+     fe4:	00 08 08 00.*
      fe8:	20 00 0f 28.*
      fec:	31 03 0a 4b 	brasl	\$75,1850 <__icache_br_handler>
 	\.\.\.
@@ -140,20 +140,20 @@ Disassembly of section \.ovly7:
 .*	32 00 15 00 	br	13ec .*
 	\.\.\.
     13a0:	00 00 00 03.*
-    13a4:	00 00 14 00.*
+    13a4:	00 04 14 00.*
     13a8:	a0 00 10 18.*
     13ac:	31 03 0a 4b 	brasl	\$75,1850 <__icache_br_handler>
 	\.\.\.
     13b8:	00 00 0f 80.*
     13bc:	00 00 00 00.*
     13c0:	00 00 00 07.*
-    13c4:	00 04 14 00.*
+    13c4:	00 08 14 00.*
     13c8:	a0 00 10 1c.*
     13cc:	31 03 0a 4b 	brasl	\$75,1850 <__icache_br_handler>
 	\.\.\.
     13dc:	00 00 0a 80.*
     13e0:	00 00 00 05.*
-    13e4:	00 04 0c 00.*
+    13e4:	00 08 0c 00.*
     13e8:	20 00 13 44.*
     13ec:	31 03 0a 4b 	brasl	\$75,1850 <__icache_br_handler>
     13f0:	00 00 00 00.*
@@ -171,13 +171,13 @@ Disassembly of section \.ovly8:
 .*	32 00 17 80 	br	17ec .*
 	\.\.\.
     17c0:	00 00 00 02.*
-    17c4:	00 00 11 04.*
+    17c4:	00 04 11 04.*
     17c8:	a0 00 14 0c.*
     17cc:	31 03 0a 4b 	brasl	\$75,1850 <__icache_br_handler>
 	\.\.\.
     17dc:	00 00 d9 00.*
     17e0:	00 00 00 06.*
-    17e4:	00 04 10 00.*
+    17e4:	00 08 10 00.*
     17e8:	20 00 17 30.*
     17ec:	31 03 0a 4b 	brasl	\$75,1850 <__icache_br_handler>
     17f0:	00 7f 0d 80.*
@@ -186,14 +186,14 @@ Disassembly of section \.ovly8:
 Disassembly of section \.text:
 
 00001800 <_start>:
-.*	41 00 00 03 	ilhu	\$3,0
+.*	41 00 02 03 	ilhu	\$3,4
 .*	60 8a 00 03 	iohl	\$3,5120	# 1400
 .*	32 00 04 80 	br	182c.*
 	\.\.\.
     1820:	00 00 00 01.*
-    1824:	00 00 0c 00.*
+    1824:	00 04 0c 00.*
     1828:	a0 00 18 08.*
-    182c:	31 03 0a 4b 	brasl	\$75,1850 <__icache_br_handler>
+    182c:	31 03 0a cb 	brasl	\$75,1854 <__icache_call_handler>
 	\.\.\.
     1838:	00 7e 7b 80.*
 	\.\.\.

-- 
Alan Modra
Australia Development Lab, IBM


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