[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [PATCH 6/7] x86/efi: do not merge all .init sections
On 18.03.2025 18:35, Roger Pau Monne wrote: > As a result of relocations now being applied after the trampoline has been > copied into the low 1MB region, there's no need for a single .init section > that's writable, as .init.text is no longer modified. This builds on the confusion of the two different types of relocations that started in the previous patch. The change here may be okay once that other aspect was clarified; the description would need extending then, though, to cover both kinds or relocations. > Remove the bodge and fallback to the layout used by ELF images with an > .init.text and .init.data section. > > The resulting PE sections are: > > Sections: > Idx Name Size VMA LMA File off Algn > 0 .text 0019072c ffff82d040200000 ffff82d040200000 00000440 2**4 > CONTENTS, ALLOC, LOAD, READONLY, CODE > 1 .rodata 000884c8 ffff82d040400000 ffff82d040400000 00190b80 2**2 > CONTENTS, ALLOC, LOAD, DATA > 2 .buildid 00000035 ffff82d0404884c8 ffff82d0404884c8 00219060 2**2 > CONTENTS, ALLOC, LOAD, READONLY, DATA > 3 .init.text 00052866 ffff82d040600000 ffff82d040600000 002190a0 2**2 > CONTENTS, ALLOC, LOAD, READONLY, CODE > 4 .init.data 00059730 ffff82d040658000 ffff82d040658000 0026b920 2**2 > CONTENTS, ALLOC, LOAD, DATA > [...] Just to mention it, also because Demi raised concern: This will leave us with yet more sections with long names. We may want to consider to e.g. use .init.t and .init.d instead. (Of course there's nothing we can really do about the various .debug_* sections, as those can only be identified by name. The only option I see there is to strip the binary.) Jan
|
![]() |
Lists.xenproject.org is hosted with RackSpace, monitoring our |