From e7ebb214834628b2b0d9d3233febc9fef2912515 Mon Sep 17 00:00:00 2001 From: Jan Beulich Date: Wed, 2 Nov 2011 14:53:04 +0000 Subject: Building EFI binaries, particularly larger ones (like e.g. Xen does), on Linux (where relocatable objects are in ELF format) so far led to all local (aka static) symbols to be discarded, making debugging quite a bit more difficult (like Linux, Xen builds an internal symbol lookup table from nm output generated on the binary produced by an earlier linking pass). Therefore, this patch arranges to insert all (relevant) local symbols from non-COFF objects into the final executable's symbol table between those coming from COFF input files and the global ones. bfd/ 2011-11-02 Jan Beulich * coffgen.c (coff_write_alien_symbol): Make public. Add 'struct internal_syment *' parameter. Extend 'dummy' to an array with two elements. Set n_numaux early. Handle BSF_FILE. (coff_write_symbols): Pass NULL as new third argument to coff_write_alien_symbol(). * cofflink.c (_bfd_coff_final_link): Don't use COFF-specific obj_raw_syment_count() on non-COFF input BFD. Insert local symbols from non-COFF input BFDs. * libcoff-in.h (coff_write_alien_symbol): Declare. * libcoff.h (coff_write_alien_symbol): Re-generate. --- bfd/coffgen.c | 34 +++++++++++++++++++++++++--------- 1 file changed, 25 insertions(+), 9 deletions(-) (limited to 'bfd/coffgen.c') diff --git a/bfd/coffgen.c b/bfd/coffgen.c index 87a3bfa..c0505c6 100644 --- a/bfd/coffgen.c +++ b/bfd/coffgen.c @@ -983,31 +983,36 @@ coff_write_symbol (bfd *abfd, file originally. This symbol may have been created by the linker, or we may be linking a non COFF file to a COFF file. */ -static bfd_boolean +bfd_boolean coff_write_alien_symbol (bfd *abfd, asymbol *symbol, + struct internal_syment *isym, bfd_vma *written, bfd_size_type *string_size_p, asection **debug_string_section_p, bfd_size_type *debug_string_size_p) { combined_entry_type *native; - combined_entry_type dummy; + combined_entry_type dummy[2]; asection *output_section = symbol->section->output_section ? symbol->section->output_section : symbol->section; struct bfd_link_info *link_info = coff_data (abfd)->link_info; + bfd_boolean ret; if ((!link_info || link_info->strip_discarded) && !bfd_is_abs_section (symbol->section) && symbol->section->output_section == bfd_abs_section_ptr) { symbol->name = ""; + if (isym != NULL) + memset (isym, 0, sizeof(*isym)); return TRUE; } - native = &dummy; + native = dummy; native->u.syment.n_type = T_NULL; native->u.syment.n_flags = 0; + native->u.syment.n_numaux = 0; if (bfd_is_und_section (symbol->section)) { native->u.syment.n_scnum = N_UNDEF; @@ -1018,6 +1023,11 @@ coff_write_alien_symbol (bfd *abfd, native->u.syment.n_scnum = N_UNDEF; native->u.syment.n_value = symbol->value; } + else if (symbol->flags & BSF_FILE) + { + native->u.syment.n_scnum = N_DEBUG; + native->u.syment.n_numaux = 1; + } else if (symbol->flags & BSF_DEBUGGING) { /* There isn't much point to writing out a debugging symbol @@ -1025,6 +1035,8 @@ coff_write_alien_symbol (bfd *abfd, format. So, we just ignore them. We must clobber the symbol name to keep it from being put in the string table. */ symbol->name = ""; + if (isym != NULL) + memset (isym, 0, sizeof(*isym)); return TRUE; } else @@ -1045,16 +1057,20 @@ coff_write_alien_symbol (bfd *abfd, } native->u.syment.n_type = 0; - if (symbol->flags & BSF_LOCAL) + if (symbol->flags & BSF_FILE) + native->u.syment.n_sclass = C_FILE; + else if (symbol->flags & BSF_LOCAL) native->u.syment.n_sclass = C_STAT; else if (symbol->flags & BSF_WEAK) native->u.syment.n_sclass = obj_pe (abfd) ? C_NT_WEAK : C_WEAKEXT; else native->u.syment.n_sclass = C_EXT; - native->u.syment.n_numaux = 0; - return coff_write_symbol (abfd, symbol, native, written, string_size_p, - debug_string_section_p, debug_string_size_p); + ret = coff_write_symbol (abfd, symbol, native, written, string_size_p, + debug_string_section_p, debug_string_size_p); + if (isym != NULL) + *isym = native->u.syment; + return ret; } /* Write a native symbol to a COFF file. */ @@ -1170,8 +1186,8 @@ coff_write_symbols (bfd *abfd) if (c_symbol == (coff_symbol_type *) NULL || c_symbol->native == (combined_entry_type *) NULL) { - if (!coff_write_alien_symbol (abfd, symbol, &written, &string_size, - &debug_string_section, + if (!coff_write_alien_symbol (abfd, symbol, NULL, &written, + &string_size, &debug_string_section, &debug_string_size)) return FALSE; } -- cgit v1.1