Re: svn commit: r365578 - head/stand/efi/loader/arch/arm64

2020-09-10 Thread Andrew Turner 
We already discard it via objcopy when converting from elf -> EFI as we only 
copy the sections needed in the final EFI file.

Andrew

> On 10 Sep 2020, at 15:22, Brandon Bergren  wrote:
> 
> You can truly get rid of it with /DISCARD/ and shave the bytes off entirely, 
> by the way.
> 
> On Thu, Sep 10, 2020, at 9:13 AM, Andrew Turner wrote:
>> Author: andrew
>> Date: Thu Sep 10 14:13:49 2020
>> New Revision: 365578
>> URL: https://svnweb.freebsd.org/changeset/base/365578
>> 
>> Log:
>>  Ignore the .interp section in the arm64 EFI loader
>> 
>>  When building the loader an unneeded .interp section may be added. Move
>>  this to the unused section region so offsets of used sections don't
>>  change.
>> 
>>  Obtained from:  CheriBSD
>>  Sponsored by:   Innovate UK
>> 
>> Modified:
>>  head/stand/efi/loader/arch/arm64/ldscript.arm64
>> 
>> Modified: head/stand/efi/loader/arch/arm64/ldscript.arm64
>> ==
>> --- head/stand/efi/loader/arch/arm64/ldscript.arm64  Thu Sep 10 14:12:25 
>> 2020 (r365577)
>> +++ head/stand/efi/loader/arch/arm64/ldscript.arm64  Thu Sep 10 14:13:49 
>> 2020 (r365578)
>> @@ -80,6 +80,7 @@ SECTIONS
>>   _edata = .;
>> 
>>   /* Unused sections */
>> +  .interp   : { *(.interp) }
>>   .dynstr: { *(.dynstr) }
>>   .hash  : { *(.hash) }
>> }
>> 
> 
> -- 
>  Brandon Bergren
>  bdra...@freebsd.org
> 

___
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"

Re: svn commit: r365578 - head/stand/efi/loader/arch/arm64

2020-09-10 Thread Brandon Bergren 
You can truly get rid of it with /DISCARD/ and shave the bytes off entirely, by 
the way.

On Thu, Sep 10, 2020, at 9:13 AM, Andrew Turner wrote:
> Author: andrew
> Date: Thu Sep 10 14:13:49 2020
> New Revision: 365578
> URL: https://svnweb.freebsd.org/changeset/base/365578
> 
> Log:
>   Ignore the .interp section in the arm64 EFI loader
>   
>   When building the loader an unneeded .interp section may be added. Move
>   this to the unused section region so offsets of used sections don't
>   change.
>   
>   Obtained from:  CheriBSD
>   Sponsored by:   Innovate UK
> 
> Modified:
>   head/stand/efi/loader/arch/arm64/ldscript.arm64
> 
> Modified: head/stand/efi/loader/arch/arm64/ldscript.arm64
> ==
> --- head/stand/efi/loader/arch/arm64/ldscript.arm64   Thu Sep 10 14:12:25 
> 2020  (r365577)
> +++ head/stand/efi/loader/arch/arm64/ldscript.arm64   Thu Sep 10 14:13:49 
> 2020  (r365578)
> @@ -80,6 +80,7 @@ SECTIONS
>_edata = .;
>  
>/* Unused sections */
> +  .interp: { *(.interp) }
>.dynstr: { *(.dynstr) }
>.hash  : { *(.hash) }
>  }
>

-- 
  Brandon Bergren
  bdra...@freebsd.org
___
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"

svn commit: r365578 - head/stand/efi/loader/arch/arm64

2020-09-10 Thread Andrew Turner 
Author: andrew
Date: Thu Sep 10 14:13:49 2020
New Revision: 365578
URL: https://svnweb.freebsd.org/changeset/base/365578

Log:
  Ignore the .interp section in the arm64 EFI loader
  
  When building the loader an unneeded .interp section may be added. Move
  this to the unused section region so offsets of used sections don't
  change.
  
  Obtained from:CheriBSD
  Sponsored by: Innovate UK

Modified:
  head/stand/efi/loader/arch/arm64/ldscript.arm64

Modified: head/stand/efi/loader/arch/arm64/ldscript.arm64
==
--- head/stand/efi/loader/arch/arm64/ldscript.arm64 Thu Sep 10 14:12:25 
2020(r365577)
+++ head/stand/efi/loader/arch/arm64/ldscript.arm64 Thu Sep 10 14:13:49 
2020(r365578)
@@ -80,6 +80,7 @@ SECTIONS
   _edata = .;
 
   /* Unused sections */
+  .interp  : { *(.interp) }
   .dynstr  : { *(.dynstr) }
   .hash: { *(.hash) }
 }
___
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"