On Fri, Mar 17, 2006 at 03:15:48PM -0800, James E Wilson wrote: > On Fri, 2006-03-17 at 13:52, H. J. Lu wrote: > > *(.gnu.linkonce.t*) > > This isn't something you added, but should be fixed just the same. This > will accidentally cause some thread-local data sections to be merged > into the text section. The thread-local sections may not have existed > when this code was written, but they do now. .gnu.linkonce.td.* is used > for linkonce thread local data, and .gnu.linkonce.tb.* is used for > linkonce thread local bss. I guess we never used TLS in EFI applications before. It doesn't really matter since TLS needs more than the linker script change. Here is the new patch. H.J. --- --- gnu-efi/gnuefi/elf_ia32_efi.lds.ia32 2004-12-22 07:52:28.000000000 -0800 +++ gnu-efi/gnuefi/elf_ia32_efi.lds 2006-03-17 13:48:13.000000000 -0800 @@ -10,6 +10,12 @@ SECTIONS .text : { *(.text) + *(.text.*) + *(.gnu.linkonce.t.*) + } + .reloc : + { + *(.reloc) } . = ALIGN(4096); .data : @@ -48,12 +54,9 @@ SECTIONS . = ALIGN(4096); .dynstr : { *(.dynstr) } . = ALIGN(4096); - .ignored.reloc : + /DISCARD/ : { *(.rel.reloc) - } - .ignored : - { *(.eh_frame) } } --- gnu-efi/gnuefi/elf_ia64_efi.lds.ia32 2004-12-22 10:18:10.000000000 -0800 +++ gnu-efi/gnuefi/elf_ia64_efi.lds 2006-03-17 15:27:48.000000000 -0800 @@ -10,7 +10,8 @@ SECTIONS .text : { *(.text) - *(.gnu.linkonce.t*) + *(.text.*) + *(.gnu.linkonce.t.*) } . = ALIGN(4096); __gp = ALIGN (8) + 0x200000; @@ -59,13 +60,10 @@ SECTIONS .dynsym : { *(.dynsym) } . = ALIGN(4096); .dynstr : { *(.dynstr) } - .ignored_reloc : + /DISCARD/ : { *(.rela.plabel) *(.rela.reloc) - } - /DISCARD/ : - { *(.IA_64.unwind*) *(.IA64.unwind*) } - : send the line "unsubscribe linux-ia64" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
![]() |