Sorry, it is a typo. the example should be
The first segment is located at 0x1000, and the desired address is 0x2000. 
The second segment is located at 0x2000, and the desired address is 0x1000.
The current ELF loader may copy first segment to 0x2000, which will erase the 
second segment.

Thanks
Zhiguang


> -----Original Message-----
> From: Ni, Ray <ray...@intel.com>
> Sent: Thursday, April 7, 2022 8:06 AM
> To: Liu, Zhiguang <zhiguang....@intel.com>; devel@edk2.groups.io
> Cc: Dong, Guo <guo.d...@intel.com>; Ma, Maurice <maurice...@intel.com>;
> You, Benjamin <benjamin....@intel.com>; Rhodes, Sean
> <sean@starlabs.systems>
> Subject: RE: [PATCH] UefiPayloadPkg: Fix bug that Elf loader doesn't relocate 
> in
> some case
> 
> 
> 
> > -----Original Message-----
> > From: Liu, Zhiguang <zhiguang....@intel.com>
> > Sent: Wednesday, April 6, 2022 9:52 PM
> > To: devel@edk2.groups.io
> > Cc: Dong, Guo <guo.d...@intel.com>; Ni, Ray <ray...@intel.com>; Ma,
> > Maurice <maurice...@intel.com>; You, Benjamin
> > <benjamin....@intel.com>; Rhodes, Sean <sean@starlabs.systems>
> > Subject: [PATCH] UefiPayloadPkg: Fix bug that Elf loader doesn't
> > relocate in some case
> >
> > Current implementation of the Elf loader reuses the same memory range
> > if the desired memory range is covered by [file base, file base + file 
> > size].
> > However, there is a potentil bug, for example:
> > If the first segment is located at 0x1000, and the desired address is
> > 0x2000. While the second segment is located at 0x2000, and the desired
> > address is 0x2000. When we parse and try to load the elf file, current
> 
> Why the desired address for both segments is 0x2000?



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#88490): https://edk2.groups.io/g/devel/message/88490
Mute This Topic: https://groups.io/mt/90289885/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to