On Wed, Mar 23, 2022 at 11:54 AM Philippe Mathieu-Daudé
wrote:
>
> On 23/3/22 10:07, christian.ehrha...@canonical.com wrote:
> > From: Christian Ehrhardt
> >
> > Some of the roms build with -march=i486 -m16 which is incompatible
> > with -fcf-protection. That in turn is can be set by default, for
> > example in Ubuntu [1].
> > That causes:
> > cc1: error: ‘-fcf-protection’ is not compatible with this target
> >
> > This won't work on -march=i486 -m16 and no matter if set or not we can
> > override it to "none" if the option is known to the compiler to be
> > able to build reliably.
> >
> > Fixes: https://gitlab.com/qemu-project/qemu/-/issues/889
> >
> > [1]: https://wiki.ubuntu.com/ToolChain/CompilerFlags#A-fcf-protection
> >
> > Signed-off-by: Christian Ehrhardt
> > ---
> > pc-bios/optionrom/Makefile | 4
> > 1 file changed, 4 insertions(+)
>
> Reviewed-by: Philippe Mathieu-Daudé
Thank you for the review Thomas and Philippe!
For the sake of testing other than my local build checks, the CI jobs
on [1] on gitlab also all passed for this.
[1]: https://gitlab.com/paelzer/qemu/-/pipelines/498917375
--
Christian Ehrhardt
Staff Engineer, Ubuntu Server
Canonical Ltd