public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* [PATCH] BaseTools/Scripts: discard .gnu.hash section in GCC builds
@ 2017-05-19 10:47 Ard Biesheuvel
  2017-05-19 10:58 ` Leif Lindholm
  2017-05-22  4:05 ` Gao, Liming
  0 siblings, 2 replies; 6+ messages in thread
From: Ard Biesheuvel @ 2017-05-19 10:47 UTC (permalink / raw)
  To: edk2-devel; +Cc: leif.lindholm, yonghong.zhu, liming.gao, Ard Biesheuvel

Some builds of GCC/binutils will default to using the GNU flavor of
the symbol hash table, and will emit it into a section called .gnu.hash
rather than .hash. We have no use for its contents, and GenFw ignores
it anyway, so it shouldn't really matter what we do with it.

However, due to a workaround for AARCH64 we have in GenFw to deal with
older GCCs that corrupt section-based relocations when merging sections
during the final link, we need the ELF and PE/COFF views of the binary
to be identical. Since we don't place the .gnu.hash section explicitly,
it may end up at the beginning of the ELF binary, causing other sections
to be shifted in the ELF view but not in the PE/COFF view.

So let's add .gnu.hash to the GCC linker script. We don't care about its
contents so add it to the /DISCARD/ section.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
---
 BaseTools/Scripts/GccBase.lds | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/BaseTools/Scripts/GccBase.lds b/BaseTools/Scripts/GccBase.lds
index 41e5c0b4a769..a43e0072f2b4 100644
--- a/BaseTools/Scripts/GccBase.lds
+++ b/BaseTools/Scripts/GccBase.lds
@@ -78,7 +78,7 @@ SECTIONS {
     *(.dynsym)
     *(.dynstr)
     *(.dynamic)
-    *(.hash)
+    *(.hash .gnu.hash)
     *(.comment)
     *(COMMON)
   }
-- 
2.9.3



^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH] BaseTools/Scripts: discard .gnu.hash section in GCC builds
  2017-05-19 10:47 [PATCH] BaseTools/Scripts: discard .gnu.hash section in GCC builds Ard Biesheuvel
@ 2017-05-19 10:58 ` Leif Lindholm
  2017-05-22  4:05 ` Gao, Liming
  1 sibling, 0 replies; 6+ messages in thread
From: Leif Lindholm @ 2017-05-19 10:58 UTC (permalink / raw)
  To: Ard Biesheuvel; +Cc: edk2-devel, yonghong.zhu, liming.gao

On Fri, May 19, 2017 at 11:47:40AM +0100, Ard Biesheuvel wrote:
> Some builds of GCC/binutils will default to using the GNU flavor of
> the symbol hash table, and will emit it into a section called .gnu.hash
> rather than .hash. We have no use for its contents, and GenFw ignores
> it anyway, so it shouldn't really matter what we do with it.
> 
> However, due to a workaround for AARCH64 we have in GenFw to deal with
> older GCCs that corrupt section-based relocations when merging sections
> during the final link, we need the ELF and PE/COFF views of the binary
> to be identical. Since we don't place the .gnu.hash section explicitly,
> it may end up at the beginning of the ELF binary, causing other sections
> to be shifted in the ELF view but not in the PE/COFF view.
> 
> So let's add .gnu.hash to the GCC linker script. We don't care about its
> contents so add it to the /DISCARD/ section.
> 
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>

Tested-by: Leif Lindholm <leif.lindholm@linaro.org>

> ---
>  BaseTools/Scripts/GccBase.lds | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/BaseTools/Scripts/GccBase.lds b/BaseTools/Scripts/GccBase.lds
> index 41e5c0b4a769..a43e0072f2b4 100644
> --- a/BaseTools/Scripts/GccBase.lds
> +++ b/BaseTools/Scripts/GccBase.lds
> @@ -78,7 +78,7 @@ SECTIONS {
>      *(.dynsym)
>      *(.dynstr)
>      *(.dynamic)
> -    *(.hash)
> +    *(.hash .gnu.hash)
>      *(.comment)
>      *(COMMON)
>    }
> -- 
> 2.9.3
> 


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] BaseTools/Scripts: discard .gnu.hash section in GCC builds
  2017-05-19 10:47 [PATCH] BaseTools/Scripts: discard .gnu.hash section in GCC builds Ard Biesheuvel
  2017-05-19 10:58 ` Leif Lindholm
@ 2017-05-22  4:05 ` Gao, Liming
  2017-05-24 13:13   ` Ard Biesheuvel
  1 sibling, 1 reply; 6+ messages in thread
From: Gao, Liming @ 2017-05-22  4:05 UTC (permalink / raw)
  To: Ard Biesheuvel, edk2-devel@lists.01.org
  Cc: leif.lindholm@linaro.org, Zhu, Yonghong

Reviewed-by: Liming Gao <liming.gao@intel.com>

>-----Original Message-----
>From: Ard Biesheuvel [mailto:ard.biesheuvel@linaro.org]
>Sent: Friday, May 19, 2017 6:48 PM
>To: edk2-devel@lists.01.org
>Cc: leif.lindholm@linaro.org; Zhu, Yonghong <yonghong.zhu@intel.com>; Gao,
>Liming <liming.gao@intel.com>; Ard Biesheuvel <ard.biesheuvel@linaro.org>
>Subject: [PATCH] BaseTools/Scripts: discard .gnu.hash section in GCC builds
>
>Some builds of GCC/binutils will default to using the GNU flavor of
>the symbol hash table, and will emit it into a section called .gnu.hash
>rather than .hash. We have no use for its contents, and GenFw ignores
>it anyway, so it shouldn't really matter what we do with it.
>
>However, due to a workaround for AARCH64 we have in GenFw to deal with
>older GCCs that corrupt section-based relocations when merging sections
>during the final link, we need the ELF and PE/COFF views of the binary
>to be identical. Since we don't place the .gnu.hash section explicitly,
>it may end up at the beginning of the ELF binary, causing other sections
>to be shifted in the ELF view but not in the PE/COFF view.
>
>So let's add .gnu.hash to the GCC linker script. We don't care about its
>contents so add it to the /DISCARD/ section.
>
>Contributed-under: TianoCore Contribution Agreement 1.0
>Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
>---
> BaseTools/Scripts/GccBase.lds | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/BaseTools/Scripts/GccBase.lds b/BaseTools/Scripts/GccBase.lds
>index 41e5c0b4a769..a43e0072f2b4 100644
>--- a/BaseTools/Scripts/GccBase.lds
>+++ b/BaseTools/Scripts/GccBase.lds
>@@ -78,7 +78,7 @@ SECTIONS {
>     *(.dynsym)
>     *(.dynstr)
>     *(.dynamic)
>-    *(.hash)
>+    *(.hash .gnu.hash)
>     *(.comment)
>     *(COMMON)
>   }
>--
>2.9.3



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] BaseTools/Scripts: discard .gnu.hash section in GCC builds
  2017-05-22  4:05 ` Gao, Liming
@ 2017-05-24 13:13   ` Ard Biesheuvel
  2017-05-27  9:05     ` Gao, Liming
  0 siblings, 1 reply; 6+ messages in thread
From: Ard Biesheuvel @ 2017-05-24 13:13 UTC (permalink / raw)
  To: Gao, Liming
  Cc: edk2-devel@lists.01.org, leif.lindholm@linaro.org, Zhu, Yonghong

On 21 May 2017 at 21:05, Gao, Liming <liming.gao@intel.com> wrote:
> Reviewed-by: Liming Gao <liming.gao@intel.com>
>

Thanks. Pushed as 00b00cc57bfe0fca54c904d4dd44a263e243c88b

>>-----Original Message-----
>>From: Ard Biesheuvel [mailto:ard.biesheuvel@linaro.org]
>>Sent: Friday, May 19, 2017 6:48 PM
>>To: edk2-devel@lists.01.org
>>Cc: leif.lindholm@linaro.org; Zhu, Yonghong <yonghong.zhu@intel.com>; Gao,
>>Liming <liming.gao@intel.com>; Ard Biesheuvel <ard.biesheuvel@linaro.org>
>>Subject: [PATCH] BaseTools/Scripts: discard .gnu.hash section in GCC builds
>>
>>Some builds of GCC/binutils will default to using the GNU flavor of
>>the symbol hash table, and will emit it into a section called .gnu.hash
>>rather than .hash. We have no use for its contents, and GenFw ignores
>>it anyway, so it shouldn't really matter what we do with it.
>>
>>However, due to a workaround for AARCH64 we have in GenFw to deal with
>>older GCCs that corrupt section-based relocations when merging sections
>>during the final link, we need the ELF and PE/COFF views of the binary
>>to be identical. Since we don't place the .gnu.hash section explicitly,
>>it may end up at the beginning of the ELF binary, causing other sections
>>to be shifted in the ELF view but not in the PE/COFF view.
>>
>>So let's add .gnu.hash to the GCC linker script. We don't care about its
>>contents so add it to the /DISCARD/ section.
>>
>>Contributed-under: TianoCore Contribution Agreement 1.0
>>Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
>>---
>> BaseTools/Scripts/GccBase.lds | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>>diff --git a/BaseTools/Scripts/GccBase.lds b/BaseTools/Scripts/GccBase.lds
>>index 41e5c0b4a769..a43e0072f2b4 100644
>>--- a/BaseTools/Scripts/GccBase.lds
>>+++ b/BaseTools/Scripts/GccBase.lds
>>@@ -78,7 +78,7 @@ SECTIONS {
>>     *(.dynsym)
>>     *(.dynstr)
>>     *(.dynamic)
>>-    *(.hash)
>>+    *(.hash .gnu.hash)
>>     *(.comment)
>>     *(COMMON)
>>   }
>>--
>>2.9.3
>


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] BaseTools/Scripts: discard .gnu.hash section in GCC builds
  2017-05-24 13:13   ` Ard Biesheuvel
@ 2017-05-27  9:05     ` Gao, Liming
  2017-05-28 12:08       ` Ard Biesheuvel
  0 siblings, 1 reply; 6+ messages in thread
From: Gao, Liming @ 2017-05-27  9:05 UTC (permalink / raw)
  To: Ard Biesheuvel
  Cc: edk2-devel@lists.01.org, leif.lindholm@linaro.org, Zhu, Yonghong

Ard:
  I just run BaseTools\Scripts\PatchCheck.py to check the latest patches. It reports one issue in this patch. Could you fix it?

The commit message format passed all checks.
Code format is not valid:
 * Line ending ('\n') is not CRLF
   File: BaseTools/Scripts/GccBase.lds
   Line:     *(.hash .gnu.hash)

Thanks
Liming
> -----Original Message-----
> From: Ard Biesheuvel [mailto:ard.biesheuvel@linaro.org]
> Sent: Wednesday, May 24, 2017 9:13 PM
> To: Gao, Liming <liming.gao@intel.com>
> Cc: edk2-devel@lists.01.org; leif.lindholm@linaro.org; Zhu, Yonghong <yonghong.zhu@intel.com>
> Subject: Re: [PATCH] BaseTools/Scripts: discard .gnu.hash section in GCC builds
> 
> On 21 May 2017 at 21:05, Gao, Liming <liming.gao@intel.com> wrote:
> > Reviewed-by: Liming Gao <liming.gao@intel.com>
> >
> 
> Thanks. Pushed as 00b00cc57bfe0fca54c904d4dd44a263e243c88b
> 
> >>-----Original Message-----
> >>From: Ard Biesheuvel [mailto:ard.biesheuvel@linaro.org]
> >>Sent: Friday, May 19, 2017 6:48 PM
> >>To: edk2-devel@lists.01.org
> >>Cc: leif.lindholm@linaro.org; Zhu, Yonghong <yonghong.zhu@intel.com>; Gao,
> >>Liming <liming.gao@intel.com>; Ard Biesheuvel <ard.biesheuvel@linaro.org>
> >>Subject: [PATCH] BaseTools/Scripts: discard .gnu.hash section in GCC builds
> >>
> >>Some builds of GCC/binutils will default to using the GNU flavor of
> >>the symbol hash table, and will emit it into a section called .gnu.hash
> >>rather than .hash. We have no use for its contents, and GenFw ignores
> >>it anyway, so it shouldn't really matter what we do with it.
> >>
> >>However, due to a workaround for AARCH64 we have in GenFw to deal with
> >>older GCCs that corrupt section-based relocations when merging sections
> >>during the final link, we need the ELF and PE/COFF views of the binary
> >>to be identical. Since we don't place the .gnu.hash section explicitly,
> >>it may end up at the beginning of the ELF binary, causing other sections
> >>to be shifted in the ELF view but not in the PE/COFF view.
> >>
> >>So let's add .gnu.hash to the GCC linker script. We don't care about its
> >>contents so add it to the /DISCARD/ section.
> >>
> >>Contributed-under: TianoCore Contribution Agreement 1.0
> >>Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> >>---
> >> BaseTools/Scripts/GccBase.lds | 2 +-
> >> 1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >>diff --git a/BaseTools/Scripts/GccBase.lds b/BaseTools/Scripts/GccBase.lds
> >>index 41e5c0b4a769..a43e0072f2b4 100644
> >>--- a/BaseTools/Scripts/GccBase.lds
> >>+++ b/BaseTools/Scripts/GccBase.lds
> >>@@ -78,7 +78,7 @@ SECTIONS {
> >>     *(.dynsym)
> >>     *(.dynstr)
> >>     *(.dynamic)
> >>-    *(.hash)
> >>+    *(.hash .gnu.hash)
> >>     *(.comment)
> >>     *(COMMON)
> >>   }
> >>--
> >>2.9.3
> >

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] BaseTools/Scripts: discard .gnu.hash section in GCC builds
  2017-05-27  9:05     ` Gao, Liming
@ 2017-05-28 12:08       ` Ard Biesheuvel
  0 siblings, 0 replies; 6+ messages in thread
From: Ard Biesheuvel @ 2017-05-28 12:08 UTC (permalink / raw)
  To: Gao, Liming
  Cc: edk2-devel@lists.01.org, leif.lindholm@linaro.org, Zhu, Yonghong

On 27 May 2017 at 09:05, Gao, Liming <liming.gao@intel.com> wrote:
> Ard:
>   I just run BaseTools\Scripts\PatchCheck.py to check the latest patches. It reports one issue in this patch. Could you fix it?
>

Pushed as f4d3ba87bb8f

-- 
Ard.

> The commit message format passed all checks.
> Code format is not valid:
>  * Line ending ('\n') is not CRLF
>    File: BaseTools/Scripts/GccBase.lds
>    Line:     *(.hash .gnu.hash)
>
> Thanks
> Liming
>> -----Original Message-----
>> From: Ard Biesheuvel [mailto:ard.biesheuvel@linaro.org]
>> Sent: Wednesday, May 24, 2017 9:13 PM
>> To: Gao, Liming <liming.gao@intel.com>
>> Cc: edk2-devel@lists.01.org; leif.lindholm@linaro.org; Zhu, Yonghong <yonghong.zhu@intel.com>
>> Subject: Re: [PATCH] BaseTools/Scripts: discard .gnu.hash section in GCC builds
>>
>> On 21 May 2017 at 21:05, Gao, Liming <liming.gao@intel.com> wrote:
>> > Reviewed-by: Liming Gao <liming.gao@intel.com>
>> >
>>
>> Thanks. Pushed as 00b00cc57bfe0fca54c904d4dd44a263e243c88b
>>
>> >>-----Original Message-----
>> >>From: Ard Biesheuvel [mailto:ard.biesheuvel@linaro.org]
>> >>Sent: Friday, May 19, 2017 6:48 PM
>> >>To: edk2-devel@lists.01.org
>> >>Cc: leif.lindholm@linaro.org; Zhu, Yonghong <yonghong.zhu@intel.com>; Gao,
>> >>Liming <liming.gao@intel.com>; Ard Biesheuvel <ard.biesheuvel@linaro.org>
>> >>Subject: [PATCH] BaseTools/Scripts: discard .gnu.hash section in GCC builds
>> >>
>> >>Some builds of GCC/binutils will default to using the GNU flavor of
>> >>the symbol hash table, and will emit it into a section called .gnu.hash
>> >>rather than .hash. We have no use for its contents, and GenFw ignores
>> >>it anyway, so it shouldn't really matter what we do with it.
>> >>
>> >>However, due to a workaround for AARCH64 we have in GenFw to deal with
>> >>older GCCs that corrupt section-based relocations when merging sections
>> >>during the final link, we need the ELF and PE/COFF views of the binary
>> >>to be identical. Since we don't place the .gnu.hash section explicitly,
>> >>it may end up at the beginning of the ELF binary, causing other sections
>> >>to be shifted in the ELF view but not in the PE/COFF view.
>> >>
>> >>So let's add .gnu.hash to the GCC linker script. We don't care about its
>> >>contents so add it to the /DISCARD/ section.
>> >>
>> >>Contributed-under: TianoCore Contribution Agreement 1.0
>> >>Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
>> >>---
>> >> BaseTools/Scripts/GccBase.lds | 2 +-
>> >> 1 file changed, 1 insertion(+), 1 deletion(-)
>> >>
>> >>diff --git a/BaseTools/Scripts/GccBase.lds b/BaseTools/Scripts/GccBase.lds
>> >>index 41e5c0b4a769..a43e0072f2b4 100644
>> >>--- a/BaseTools/Scripts/GccBase.lds
>> >>+++ b/BaseTools/Scripts/GccBase.lds
>> >>@@ -78,7 +78,7 @@ SECTIONS {
>> >>     *(.dynsym)
>> >>     *(.dynstr)
>> >>     *(.dynamic)
>> >>-    *(.hash)
>> >>+    *(.hash .gnu.hash)
>> >>     *(.comment)
>> >>     *(COMMON)
>> >>   }
>> >>--
>> >>2.9.3
>> >


^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2017-05-28 12:07 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-05-19 10:47 [PATCH] BaseTools/Scripts: discard .gnu.hash section in GCC builds Ard Biesheuvel
2017-05-19 10:58 ` Leif Lindholm
2017-05-22  4:05 ` Gao, Liming
2017-05-24 13:13   ` Ard Biesheuvel
2017-05-27  9:05     ` Gao, Liming
2017-05-28 12:08       ` Ard Biesheuvel

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox