Revert my last change since it has not been approved.
authorJie Zhang <jie.zhang@analog.com>
Sat, 12 Jul 2008 08:54:13 +0000 (08:54 +0000)
committerJie Zhang <jie.zhang@analog.com>
Sat, 12 Jul 2008 08:54:13 +0000 (08:54 +0000)
18 files changed:
bfd/ChangeLog
bfd/elf.c
bfd/elf32-bfin.c
binutils/ChangeLog
binutils/readelf.c
include/ChangeLog
include/bfdlink.h
include/elf/bfin.h
ld/ChangeLog
ld/Makefile.am
ld/Makefile.in
ld/emulparams/bfin.sh
ld/emulparams/elf32bfinfd.sh
ld/emultempl/bfin.em [deleted file]
ld/gen-doc.texi
ld/ld.texinfo
ld/ldmain.c
ld/lexsup.c

index c37f13e0d52d72f2f290436545630c532c473537..5c02856972cfcec0a8aaa367c0f0346167fdeaf3 100644 (file)
@@ -1,3 +1,17 @@
+2008-07-12  Jie Zhang  <jie.zhang@analog.com>
+
+       Revert
+       2008-07-12  Jie Zhang  <jie.zhang@analog.com>
+       * elf.c (_bfd_elf_map_sections_to_segments): Don't put
+       executable sections into the same segment with other
+       read only sections if --sep-code.
+       * elf32-bfin.c (elf32_bfin_code_in_l1): New variable.
+       (elf32_bfin_data_in_l1): New variable.
+       (elf32_bfin_final_write_processing): New.
+       (elf32_bfin_special_sections[]): New.
+       (elf_backend_final_write_processing): Define.
+       (elf_backend_special_sections): Define.
+
 2008-07-12  Jie Zhang  <jie.zhang@analog.com>
 
        * elf.c (_bfd_elf_map_sections_to_segments): Don't put
index f066c7ca58cceb2f5fa220dc87b97bed90b527d2..d65c78d963a413792fb4bbd254894e695cc736c9 100644 (file)
--- a/bfd/elf.c
+++ b/bfd/elf.c
@@ -3605,7 +3605,6 @@ _bfd_elf_map_sections_to_segments (bfd *abfd, struct bfd_link_info *info)
       asection **hdrpp;
       bfd_boolean phdr_in_segment = TRUE;
       bfd_boolean writable;
-      bfd_boolean executable;
       int tls_count = 0;
       asection *first_tls = NULL;
       asection *dynsec, *eh_frame_hdr;
@@ -3677,7 +3676,6 @@ _bfd_elf_map_sections_to_segments (bfd *abfd, struct bfd_link_info *info)
       phdr_index = 0;
       maxpagesize = bed->maxpagesize;
       writable = FALSE;
-      executable = FALSE;
       dynsec = bfd_get_section_by_name (abfd, ".dynamic");
       if (dynsec != NULL
          && (dynsec->flags & SEC_LOAD) == 0)
@@ -3759,14 +3757,6 @@ _bfd_elf_map_sections_to_segments (bfd *abfd, struct bfd_link_info *info)
                 ends precisely on a page boundary.  */
              new_segment = TRUE;
            }
-         else if (info->sep_code
-                  && ((! executable && (hdr->flags & SEC_CODE) != 0)
-                      || (executable && (hdr->flags & SEC_CODE) == 0)))
-           {
-             /* We don't want to put a executable section in a non-executable
-                segment.  */
-             new_segment = TRUE;
-           }
          else
            {
              /* Otherwise, we can use the same segment.  */
@@ -3781,8 +3771,6 @@ _bfd_elf_map_sections_to_segments (bfd *abfd, struct bfd_link_info *info)
            {
              if ((hdr->flags & SEC_READONLY) == 0)
                writable = TRUE;
-             if ((hdr->flags & SEC_CODE) != 0)
-               executable = TRUE;
              last_hdr = hdr;
              /* .tbss sections effectively have zero size.  */
              if ((hdr->flags & (SEC_THREAD_LOCAL | SEC_LOAD))
@@ -3808,11 +3796,6 @@ _bfd_elf_map_sections_to_segments (bfd *abfd, struct bfd_link_info *info)
          else
            writable = FALSE;
 
-         if ((hdr->flags & SEC_CODE) != 0)
-           executable = TRUE;
-         else
-           executable = FALSE;
-
          last_hdr = hdr;
          /* .tbss sections effectively have zero size.  */
          if ((hdr->flags & (SEC_THREAD_LOCAL | SEC_LOAD)) != SEC_THREAD_LOCAL)
index 480feb2d30e6d4383b743e53da8aa839f74bba45..f83abd28acc6d89d2bd9298d43d8f8026741e8ff 100644 (file)
@@ -1119,22 +1119,6 @@ bfin_reloc_type_lookup (bfd * abfd ATTRIBUTE_UNUSED,
   return (reloc_howto_type *) NULL;
 }
 
-/* Set by ld emulation if --code-in-l1.  */
-bfd_boolean elf32_bfin_code_in_l1 = 0;
-
-/* Set by ld emulation if --data-in-l1.  */
-bfd_boolean elf32_bfin_data_in_l1 = 0;
-
-static void
-elf32_bfin_final_write_processing (bfd *abfd,
-                                  bfd_boolean linker ATTRIBUTE_UNUSED)
-{
-  if (elf32_bfin_code_in_l1)
-    elf_elfheader (abfd)->e_flags |= EF_BFIN_CODE_IN_L1;
-  if (elf32_bfin_data_in_l1)
-    elf_elfheader (abfd)->e_flags |= EF_BFIN_DATA_IN_L1;
-}
-
 /* Return TRUE if the name is a local label.
    bfin local labels begin with L$.  */
 static bfd_boolean
@@ -5588,14 +5572,6 @@ error_return:
     free (internal_relocs);
   return FALSE;
 }
-
-struct bfd_elf_special_section const elf32_bfin_special_sections[] =
-{
-  { ".l1.text",                8, -2, SHT_PROGBITS, SHF_ALLOC + SHF_EXECINSTR },
-  { ".l1.data",                8, -2, SHT_PROGBITS, SHF_ALLOC + SHF_WRITE },
-  { NULL,              0,  0, 0,            0 }
-};
-
 \f
 #define TARGET_LITTLE_SYM              bfd_elf32_bfin_vec
 #define TARGET_LITTLE_NAME             "elf32-bfin"
@@ -5642,11 +5618,8 @@ struct bfd_elf_special_section const elf32_bfin_special_sections[] =
                                         elf32_bfin_set_private_flags
 #define bfd_elf32_bfd_print_private_bfd_data \
                                         elf32_bfin_print_private_bfd_data
-#define elf_backend_final_write_processing \
-                                       elf32_bfin_final_write_processing
 #define elf_backend_reloc_type_class    elf32_bfin_reloc_type_class
 #define elf_backend_can_gc_sections 1
-#define elf_backend_special_sections   elf32_bfin_special_sections
 #define elf_backend_can_refcount 1
 #define elf_backend_want_got_plt 0
 #define elf_backend_plt_readonly 1
index e0f1d8c12aa67c2723877e853baa1bc6a01379c2..d0f96bfc2120130f6f9bc0c361797ee75b54a084 100644 (file)
@@ -1,3 +1,10 @@
+2008-07-12  Jie Zhang  <jie.zhang@analog.com>
+
+       Revert
+       2008-07-12  Jie Zhang  <jie.zhang@analog.com>
+       * readelf.c (get_machine_flags): Deal with Blackfin specific
+       flags.
+
 2008-07-12  Jie Zhang  <jie.zhang@analog.com>
 
        * readelf.c (get_machine_flags): Deal with Blackfin specific
index 789287f3785c63026e0f6d84dbd5000614035246..8a38db747af2d38b74e1ae53ef87d1d11ce52c94 100644 (file)
@@ -2140,21 +2140,6 @@ get_machine_flags (unsigned e_flags, unsigned e_machine)
          decode_ARM_machine_flags (e_flags, buf);
          break;
 
-       case EM_BLACKFIN:
-         if (e_flags & EF_BFIN_PIC)
-           strcat (buf, ", PIC");
-
-         if (e_flags & EF_BFIN_FDPIC)
-           strcat (buf, ", FDPIC");
-
-         if (e_flags & EF_BFIN_CODE_IN_L1)
-           strcat (buf, ", code in L1");
-
-         if (e_flags & EF_BFIN_DATA_IN_L1)
-           strcat (buf, ", data in L1");
-
-         break;
-
        case EM_CYGNUS_FRV:
          switch (e_flags & EF_FRV_CPU_MASK)
            {
index 1cc6fc5f7aced5eb6f73f7b063a979b040ce214d..bf5901f9e7f8bfa6a4950c0a950ba43b9bf713e2 100644 (file)
@@ -1,3 +1,12 @@
+2008-07-12  Jie Zhang  <jie.zhang@analog.com>
+
+       Revert
+       2008-07-12  Jie Zhang  <jie.zhang@analog.com>
+       * bfdlink.h (struct bfd_link_info): Add sep_code member
+       variable.
+       * elf/bfin.h (EF_BFIN_CODE_IN_L1): Define.
+       (EF_BFIN_DATA_IN_L1): Define.
+
 2008-07-12  Jie Zhang  <jie.zhang@analog.com>
 
        * bfdlink.h (struct bfd_link_info): Add sep_code member
index d22519a1d11dd115cf68c94878382b3c4178eee0..e68331028396acd91c7e2ae8a069cff216904b76 100644 (file)
@@ -346,9 +346,6 @@ struct bfd_link_info
   /* Non-NULL if .note.gnu.build-id section should be created.  */
   char *emit_note_gnu_build_id;
 
-  /* TRUE if putting code into separate segment.  */
-  unsigned int sep_code: 1;
-
   /* What to do with unresolved symbols in an object file.
      When producing executables the default is GENERATE_ERROR.
      When producing shared libraries the default is IGNORE.  The
index 523db9b53a359d87fe0f1fa4c0b4fc42f5c64cd0..3c07cd166c88f395f221fd8117f2e0081a081da4 100644 (file)
@@ -88,8 +88,5 @@ END_RELOC_NUMBERS (R_max)
 #define EF_BFIN_PIC            0x00000001      /* -fpic */
 #define EF_BFIN_FDPIC          0x00000002      /* -mfdpic */
 
-#define EF_BFIN_CODE_IN_L1     0x00000010      /* --code-in-l1 */
-#define EF_BFIN_DATA_IN_L1     0x00000020      /* --data-in-l1 */
-
 #define        EF_BFIN_PIC_FLAGS       (EF_BFIN_PIC | EF_BFIN_FDPIC)
 #endif /* _ELF_BFIN_H */
index 91bae6728e0f47c50367fe62f90111690024ed1f..02bcfd002adba3f9ca3f5bbeade74de293888368 100644 (file)
@@ -1,3 +1,21 @@
+2008-07-12  Jie Zhang  <jie.zhang@analog.com>
+
+       Revert
+       2008-07-12  Jie Zhang  <jie.zhang@analog.com>
+       * Makefile.am (eelf32bfin.c): Depend on bfin.em.
+       (eelf32bfinfd.c): Likewise.
+       * Makefile.in: Regenerate.
+       * gen-doc.texi: Set Blackfin.
+       * ld.texinfo: Document --sep-code and Blackfin specific
+       options.
+       * ldmain.c (main): Initialize link_info.sep_code.
+       * lexsup.c (enum option_values): Add OPTION_SEP_CODE.
+       (ld_options[]): Add --sep-code.
+       (parse_args): Deal with --sep-code.
+       * emulparams/bfin.sh (EXTRA_EM_FILE): Define.
+       * emulparams/elf32bfinfd.sh (OTHER_SECTIONS): Define.
+       * emultempl/bfin.em: New file.
+
 2008-07-12  Jie Zhang  <jie.zhang@analog.com>
 
        * Makefile.am (eelf32bfin.c): Depend on bfin.em.
index fe8edf0973212a30226e42e2a76c28b0ab2bd029..80c53d85954d8ad41a0a6491ee44ec049b90b86f 100644 (file)
@@ -653,11 +653,11 @@ edelta68.c: $(srcdir)/emulparams/delta68.sh \
   $(srcdir)/emultempl/generic.em $(srcdir)/scripttempl/delta68.sc ${GEN_DEPENDS}
        ${GENSCRIPTS} delta68 "$(tdir_delta68)"
 eelf32bfin.c: $(srcdir)/emulparams/bfin.sh \
-  $(ELF_DEPS) $(srcdir)/emultempl/bfin.em \
+  $(ELF_DEPS) \
   $(srcdir)/scripttempl/elf.sc ${GEN_DEPENDS}
        ${GENSCRIPTS} elf32bfin "$(tdir_elf32bfin)" bfin
 eelf32bfinfd.c: $(srcdir)/emulparams/elf32bfinfd.sh $(srcdir)/emulparams/bfin.sh \
-  $(ELF_DEPS) $(srcdir)/emultempl/bfin.em \
+  $(ELF_DEPS) \
   $(srcdir)/scripttempl/elf.sc ${GEN_DEPENDS}
        ${GENSCRIPTS} elf32bfinfd "$(tdir_elf32bfinfd)" elf32bfinfd
 eelf32_dlx.c: $(srcdir)/emulparams/elf32_dlx.sh \
index 06b48469117cc8f082b827c81d5c1244b3979a4e..94e5d68ec1492298c2f044ad525087a147d6427d 100644 (file)
@@ -1482,11 +1482,11 @@ edelta68.c: $(srcdir)/emulparams/delta68.sh \
   $(srcdir)/emultempl/generic.em $(srcdir)/scripttempl/delta68.sc ${GEN_DEPENDS}
        ${GENSCRIPTS} delta68 "$(tdir_delta68)"
 eelf32bfin.c: $(srcdir)/emulparams/bfin.sh \
-  $(ELF_DEPS) $(srcdir)/emultempl/bfin.em \
+  $(ELF_DEPS) \
   $(srcdir)/scripttempl/elf.sc ${GEN_DEPENDS}
        ${GENSCRIPTS} elf32bfin "$(tdir_elf32bfin)" bfin
 eelf32bfinfd.c: $(srcdir)/emulparams/elf32bfinfd.sh $(srcdir)/emulparams/bfin.sh \
-  $(ELF_DEPS) $(srcdir)/emultempl/bfin.em \
+  $(ELF_DEPS) \
   $(srcdir)/scripttempl/elf.sc ${GEN_DEPENDS}
        ${GENSCRIPTS} elf32bfinfd "$(tdir_elf32bfinfd)" elf32bfinfd
 eelf32_dlx.c: $(srcdir)/emulparams/elf32_dlx.sh \
index f33baf56059a77aa6ca6a34700d35b31d29c8f57..1c764e8f9af40633ade7a7066c11a081ad8e8e38 100755 (executable)
@@ -10,4 +10,3 @@ TEMPLATE_NAME=elf32
 GENERATE_SHLIB_SCRIPT=yes
 EMBEDDED=yes
 USER_LABEL_PREFIX=_
-EXTRA_EM_FILE=bfin
index 588a53f9b2428feb14ddaeff60c7d6121d98be07..19ec748b7fb983ba675bf981b76c011e8c42a3ac 100644 (file)
@@ -14,24 +14,3 @@ OTHER_READONLY_SECTIONS="
     ${RELOCATING+__ROFIXUP_END__ = .;}
   }
 "
-# 0xff700000, 0xff800000, 0xff900000 and 0xffa00000 are also used in
-# Dynamic linker and linux kernel. They need to be keep synchronized.
-OTHER_SECTIONS="
-  .l1.data 0xff700000  :
-  {
-    *(.l1.data)
-  }
-  .l1.data.A 0xff800000        :
-  {
-    *(.l1.data.A)
-  }
-  .l1.data.B 0xff900000        :
-  {
-    *(.l1.data.B)
-  }
-  .l1.text  0xffa00000 :
-  {
-    *(.l1.text)
-  }
-"
-EXTRA_EM_FILE=bfin
diff --git a/ld/emultempl/bfin.em b/ld/emultempl/bfin.em
deleted file mode 100644 (file)
index 326537d..0000000
+++ /dev/null
@@ -1,64 +0,0 @@
-# This shell script emits a C file. -*- C -*-
-#   Copyright 2006 Free Software Foundation, Inc.
-#
-# This file is part of GLD, the Gnu Linker.
-#
-# This program is free software; you can redistribute it and/or modify
-# it under the terms of the GNU General Public License as published by
-# the Free Software Foundation; either version 2 of the License, or
-# (at your option) any later version.
-#
-# This program is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program; if not, write to the Free Software
-# Foundation, Inc., 51 Franklin Street - Fifth Floor, Boston, MA 02110-1301, USA.
-#
-
-# This file is sourced from elf32.em, and defines extra bfin-elf
-# specific routines.
-#
-cat >>e${EMULATION_NAME}.c <<EOF
-
-#include "elf-bfd.h"
-
-/* Whether to put code in Blackfin L1 SRAM.  */
-extern bfd_boolean elf32_bfin_code_in_l1;
-
-/* Whether to put (writable) data in Blackfin L1 SRAM.  */
-extern bfd_boolean elf32_bfin_data_in_l1;
-
-EOF
-
-
-# Define some shell vars to insert bits of code into the standard elf
-# parse_args and list_options functions.
-#
-PARSE_AND_LIST_PROLOGUE='
-#define OPTION_CODE_IN_L1              300
-#define OPTION_DATA_IN_L1              301
-'
-
-PARSE_AND_LIST_LONGOPTS='
-  { "code-in-l1", no_argument, NULL, OPTION_CODE_IN_L1 },
-  { "data-in-l1", no_argument, NULL, OPTION_DATA_IN_L1 },
-'
-
-PARSE_AND_LIST_OPTIONS='
-  fprintf (file, _("\
-  --code-in-l1          Put code in l1.\n\
-  --data-in-l1          Put data in l1.\n"
-                  ));
-'
-
-PARSE_AND_LIST_ARGS_CASES='
-    case OPTION_CODE_IN_L1:
-      elf32_bfin_code_in_l1 = TRUE;
-      break;
-    case OPTION_DATA_IN_L1:
-      elf32_bfin_data_in_l1 = TRUE;
-      break;
-'
index f7e75cfe78cbe935a03c55513ef6e1849946c8a7..7434edb772675f6cd00cd0d9bb70ff0b0cd14be7 100644 (file)
@@ -4,7 +4,6 @@
 
 @c 2. Specific target machines
 @set ARM
-@set Blackfin
 @set H8300
 @set HPPA
 @set I960
index a37d7d5eb2f3d599154ea972272532c8f08ec15f..4400524cea0ac2048c2baba693c5ca97db0cd45f 100644 (file)
@@ -21,7 +21,6 @@
 @set UsesEnvVars
 @set GENERIC
 @set ARM
-@set Blackfin
 @set H8300
 @set HPPA
 @set I960
@@ -141,9 +140,6 @@ section entitled ``GNU Free Documentation License''.
 @ifset ARM
 * ARM::                                ld and the ARM family
 @end ifset
-@ifset Blackfin
-* Blackfin::                   ld and the Blackfin family
-@end ifset
 @ifset HPPA
 * HPPA ELF32::                  ld and HPPA 32-bit ELF
 @end ifset
@@ -829,11 +825,6 @@ For compatibility with other ELF linkers, if the @option{-R} option is
 followed by a directory name, rather than a file name, it is treated as
 the @option{-rpath} option.
 
-@kindex --sep-code
-@cindex input files, displaying
-@itemx --sep-code
-Put code in a seperate segment, not along with other read only data.
-
 @kindex -s
 @kindex --strip-all
 @cindex strip all symbols
@@ -5371,9 +5362,6 @@ functionality are not listed.
 @ifset ARM
 * ARM::                                @command{ld} and the ARM family
 @end ifset
-@ifset Blackfin
-* Blackfin::                   @command{ld} and the Blackfin family
-@end ifset
 @ifset HPPA
 * HPPA ELF32::                  @command{ld} and HPPA 32-bit ELF
 @end ifset
@@ -5764,30 +5752,6 @@ otherwise.
 @end ifclear
 @end ifset
 
-@ifset Blackfin
-@ifclear GENERIC
-@raisesections
-@end ifclear
-
-@node Blackfin
-@section @command{ld} and the Blackfin family
-
-@cindex Put code in L1 instruction SRAM
-@kindex --code-in-l1
-The @samp{--code-in-l1} option adds a specific flag in the ELF header.
-This flag tells loader to put the code segments into Blackfin
-L1 instruction SRAM.
-
-@cindex Put code in L1 data SRAM
-@kindex --data-in-l1
-The @samp{--data-in-l1} option adds a specific flag in the ELF header.
-This flag tells loader to put the data segments into Blackfin L1 data SRAM.
-
-@ifclear GENERIC
-@lowersections
-@end ifclear
-@end ifset
-
 @ifset HPPA
 @ifclear GENERIC
 @raisesections
index d094afa571c6a8242211bf3996f73d5d31ba3291..e7b9c0f9d5cc6f3b4bb35c41fb9632da60486f7a 100644 (file)
@@ -277,7 +277,6 @@ main (int argc, char **argv)
   link_info.pei386_auto_import = -1;
   link_info.spare_dynamic_tags = 5;
   link_info.path_separator = ':';
-  link_info.sep_code = FALSE;
 
   ldfile_add_arch ("");
   emulation = get_emulation (argc, argv);
index 3f6306082d1eade570d1630d2fad72bb625dc9a2..60a8167d5dc6d3c96afb8f63d69800d603380a93 100644 (file)
@@ -92,7 +92,6 @@ enum option_values
   OPTION_RETAIN_SYMBOLS_FILE,
   OPTION_RPATH,
   OPTION_RPATH_LINK,
-  OPTION_SEP_CODE,
   OPTION_SHARED,
   OPTION_SONAME,
   OPTION_SORT_COMMON,
@@ -478,8 +477,6 @@ static const struct ld_option ld_options[] =
     '\0', NULL, N_("Create a position independent executable"), ONE_DASH },
   { {"pic-executable", no_argument, NULL, OPTION_PIE},
     '\0', NULL, NULL, TWO_DASHES },
-  { {"sep-code", no_argument, NULL, OPTION_SEP_CODE},
-    '\0', NULL, N_("Put code into separate segment"), TWO_DASHES },
   { {"sort-common", optional_argument, NULL, OPTION_SORT_COMMON},
     '\0', N_("[=ascending|descending]"), 
     N_("Sort common symbols by alignment [in specified order]"), 
@@ -1146,9 +1143,6 @@ parse_args (unsigned argc, char **argv)
          else
            einfo (_("%P%F: -pie not supported\n"));
          break;
-       case OPTION_SEP_CODE:
-         link_info.sep_code = TRUE;
-         break;
        case 'h':               /* Used on Solaris.  */
        case OPTION_SONAME:
          command_line.soname = optarg;