On Tue, Jun 07, 2022 at 12:38:46PM +0200, Jiri Slaby wrote: > Hi, > > On 07. 06. 22, 12:31, Gerd Hoffmann wrote: > > > The reason is TemporaryRamMigration() overwrites rbp unconditionally -- it > > > adds an offset to rbp even if rbp is NOT used as a frame pointer > > > > > Now, what is the right way to fix this? Do the SetJump/LongJump in > > > assembly > > > and wrap it into push rbp/pop rbp? > > > > push/pop rbp will break in case frame pointers are used, no? > > Yes, see the downstream bug at: > > https://bugzilla.suse.com/show_bug.cgi?id=1199597#c45 > > and read further. > > > I think essentially the code needs to know whenever frame pointers are > > used or not and then update (or not) rbp depending on that. Update > > compiler flags to explicitly set -f(no-)omit-frame-pointer, also add > > -D OMIT_FRAME_POINTER=1, the compile conditionally on OMIT_FRAME_POINTER? > > Yes, the comment above mentions this too (cf. CONFIG_FRAME_POINTER in the > kernel). So see the downstream bugzilla for discussion.
Ok. So what is the status here? Someone working on patches? > The upstream bugzilla needs an account which I don't have and cannot create > automatically. It needs manual intervention and I am too lazy to do so. It's just an email though: https://github.com/tianocore/tianocore.github.io/wiki/Reporting-Issues Used to be open in the past, as far I know it was closed to get the bugzilla spam bots under control, not to exclude people ... take care, Gerd -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#90286): https://edk2.groups.io/g/devel/message/90286 Mute This Topic: https://groups.io/mt/91216215/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
