On Wed, Jan 03, 2024 at 12:44:39PM +0000, Daniel P. Berrangé wrote: > On Tue, Nov 28, 2023 at 09:36:47AM -0500, Cole Robinson wrote: > > binutils 2.39 shows some warnings when building pvh.img > > > > /usr/bin/ld: warning: pvh.o: missing .note.GNU-stack section implies > > executable stack > > /usr/bin/ld: NOTE: This behaviour is deprecated and will be removed in a > > future version of the linker > > /usr/bin/ld: warning: pvh.img has a LOAD segment with RWX permissions > > > > The latter of which is fatal on Fedora rawhide for some reason. > > > > Add linker options to suppress the errors > > This makes it silent, but I guess someone needs to confirm that this > option ROM code genuinely does NOT need to have executable stack, > otherwise the future change that is being warned about could impact > it ?
Skimming the code it does not look like it depends on a execute-able stack. Beside that the option rom will be loaded as raw binary by seabios and run without paging. There is nothing which could actually setup and enforce an NX stack ... Acked-by: Gerd Hoffmann <kra...@redhat.com> take care, Gerd
