#11022: Invalid ELF "note" section format ---------------------------+---------------------------------------- Reporter: hsyl20 | Owner: Type: bug | Status: new Priority: normal | Milestone: 8.0.1 Component: Driver | Version: 7.10.2 Resolution: | Keywords: Operating System: POSIX | Architecture: Unknown/Multiple Type of failure: Other | Test Case: Blocked By: | Blocking: Related Tickets: | Differential Rev(s): Wiki Page: | ---------------------------+---------------------------------------- Changes (by bgamari): * milestone: => 8.0.1 Comment: Sounds like the right thing to do. Are you offering a patch? -- Ticket URL: http://ghc.haskell.org/trac/ghc/ticket/11022#comment:1 GHC http://www.haskell.org/ghc/ The Glasgow Haskell Compiler

#11022: Invalid ELF "note" section format ---------------------------+---------------------------------------- Reporter: hsyl20 | Owner: Type: bug | Status: patch Priority: normal | Milestone: 8.0.1 Component: Driver | Version: 7.10.2 Resolution: | Keywords: Operating System: POSIX | Architecture: Unknown/Multiple Type of failure: Other | Test Case: Blocked By: | Blocking: Related Tickets: | Differential Rev(s): Wiki Page: | ---------------------------+---------------------------------------- Changes (by hsyl20): * status: new => patch -- Ticket URL: http://ghc.haskell.org/trac/ghc/ticket/11022#comment:2 GHC http://www.haskell.org/ghc/ The Glasgow Haskell Compiler

#11022: Invalid ELF "note" section format ---------------------------+---------------------------------------- Reporter: hsyl20 | Owner: Type: bug | Status: patch Priority: normal | Milestone: 8.0.1 Component: Driver | Version: 7.10.2 Resolution: | Keywords: Operating System: POSIX | Architecture: Unknown/Multiple Type of failure: Other | Test Case: Blocked By: | Blocking: Related Tickets: | Differential Rev(s): Phab:D1375 Wiki Page: | ---------------------------+---------------------------------------- Comment (by Ben Gamari ): In [changeset:"f78b477bd0ba1f85089c515259c9e3145abd1f7b/ghc" f78b477b/ghc]: {{{ #!CommitTicketReference repository="ghc" revision="f78b477bd0ba1f85089c515259c9e3145abd1f7b" driver: use PROGBITS type for .debug-ghc-link-info section Previously the `.debug-ghc-link-info` section was of type `SHT_NOTE` but this is not compliant with the ELF specification, which requires that `NOTE` sections are in a particular record-based format. We mark this section as `PROGBITS` instead, which is defined as implying no particular format. Fixes #11022. Reviewers: bgamari, austin Reviewed By: bgamari, austin Subscribers: thomie, hsyl20 Differential Revision: https://phabricator.haskell.org/D1375 GHC Trac Issues: #11022 }}} -- Ticket URL: http://ghc.haskell.org/trac/ghc/ticket/11022#comment:4 GHC http://www.haskell.org/ghc/ The Glasgow Haskell Compiler

#11022: Invalid ELF "note" section format ---------------------------+---------------------------------------- Reporter: hsyl20 | Owner: Type: bug | Status: closed Priority: normal | Milestone: 8.0.1 Component: Driver | Version: 7.10.2 Resolution: fixed | Keywords: Operating System: POSIX | Architecture: Unknown/Multiple Type of failure: Other | Test Case: Blocked By: | Blocking: Related Tickets: | Differential Rev(s): Phab:D1375 Wiki Page: | ---------------------------+---------------------------------------- Comment (by Ben Gamari ): In [changeset:"109d7ce85aadbd9fb7a322a0a83548e5d4e83926/ghc" 109d7ce/ghc]: {{{ #!CommitTicketReference repository="ghc" revision="109d7ce85aadbd9fb7a322a0a83548e5d4e83926" Systools: read ELF section without calling readelf This patch tackles two issues: 1) GHC stores a "link info" string into a ELF section. Initially a section with type "note" was used but GHC didn't follow the ELF specification which specifies a record-based format for these sections. With D1375 we switched to a "progbits" section type for which there isn't any format constraint. This is an issue for D1242 which use GCC's --gc-sections which collects "unused" sections, such as our section containing link info... In this patch, we fall back to a section with type "note" but we respect the specified format. 2) Reading back the ELF section was done by parsing the result of a call to "readelf". Calling readelf is problematic because the program may not be available or it may be renamed on some platforms (see D1326). Moreover we have no garanty that its output layout will stay the same in future releases of readelf. Finally we would need to fix the parsing to support "note" sections because of 1. Instead, this patch proposes to use Data.Binary.Get to directly read the "link info" note into its section. ELF has a specification, hence it should work on every conforming platform. This patch "reverts" D1375, hence it supersedes D1432. It makes D1326 not necessary anymore. Test Plan: - recomp011 should pass (test that relinking is avoided when both "link info" match) - we should add a test for ELF objects with more than 0xff00 sections => added test "recomp015" - we should check that GAS generates 32-bit words with .int on every supported platform using ELF (or find a place where this is documented). harbomaster and I (@hsyl20) only tested on x86-64. On platforms where it is not true, it should make recomp011 fail. => tested to work on Linux/amd64, Solaris/i386 and OpenBSD/amd64 Reviewers: olsner, ony, thomie, kgardas, austin, bgamari Reviewed By: thomie, bgamari Subscribers: kgardas, rwbarton, thomie Differential Revision: https://phabricator.haskell.org/D1381 GHC Trac Issues: #10974, #11022 }}} -- Ticket URL: http://ghc.haskell.org/trac/ghc/ticket/11022#comment:6 GHC http://www.haskell.org/ghc/ The Glasgow Haskell Compiler

#11022: Invalid ELF "note" section format ---------------------------+---------------------------------------- Reporter: hsyl20 | Owner: Type: bug | Status: closed Priority: normal | Milestone: 8.0.1 Component: Driver | Version: 7.10.2 Resolution: fixed | Keywords: Operating System: POSIX | Architecture: Unknown/Multiple Type of failure: Other | Test Case: Blocked By: | Blocking: Related Tickets: | Differential Rev(s): Phab:D1375 Wiki Page: | ---------------------------+---------------------------------------- Comment (by bgamari): I've also had to disable this test on ARM due to differences in the assembly syntax. -- Ticket URL: http://ghc.haskell.org/trac/ghc/ticket/11022#comment:8 GHC http://www.haskell.org/ghc/ The Glasgow Haskell Compiler