From: Olaf Hering <olaf@aepfle.de>
To: "Philippe Mathieu-Daudé" <philmd@redhat.com>
Cc: qemu-devel@nongnu.org, Laszlo Ersek <lersek@redhat.com>,
"Michael S. Tsirkin" <mst@redhat.com>
Subject: Re: [Qemu-devel] edk2 fails to compile in v4.0.0-rc2
Date: Fri, 5 Apr 2019 13:27:53 +0200 [thread overview]
Message-ID: <20190405132753.6c5f2f88.olaf@aepfle.de> (raw)
In-Reply-To: <7fded1f3-7539-45b5-17c7-39fe7193dbc9@redhat.com>
[-- Attachment #1: Type: text/plain, Size: 1525 bytes --]
Am Fri, 5 Apr 2019 13:14:35 +0200
schrieb Philippe Mathieu-Daudé <philmd@redhat.com>:
> On 4/5/19 12:59 PM, Olaf Hering wrote:
> > Am Fri, 5 Apr 2019 12:49:15 +0200
> > schrieb Philippe Mathieu-Daudé <philmd@redhat.com>:
> >
> >> The EDK2 submodule was added for UEFI testing, you don't need to compile
> >> it to build/use QEMU.
> >>
> >> How did you end up compiling it?
> >
> > The qemu.spec file has this since a very long time:
> > make -C roms efirom ${unrelated_settings}
> I don't have any qemu.spec, is it a SUSE file?
Yes, it is part of the qemu package.
https://build.opensuse.org/package/show/Virtualization/qemu
> > This used to work still in January with c9d18c1c150c84e7a976df989ad04ddf01083f46.
> Before f590a812c210 the EfiRom tool was not available, now we compile it
> inconditionally.
I wonder why it must be compiled unconditionally.
This is my workaround:
sed -i 's@EFIROM@KAPUTT@g' roms/Makefile
efirom=
test -x "$(type -P EfiRom)" && KAPUTT="$_" efirom=efirom
make -C roms ${efirom} KAPUTT=${KAPUTT}
> So I assume your distribution already provides the EfiRom tool. Can you
> point me at the package which provides it? I wonder if your distribution
> use a non-upstreamed patch that change the EDK2 BaseTools PIE/PIC flags.
It is in ovmf-tools.rpm, which comes from ovmf. And looking at this package,
they also had to workaround the fact that edk2 does not cope with PIE.
https://build.opensuse.org/package/show/Virtualization/ovmf
Olaf
[-- Attachment #2: Digitale Signatur von OpenPGP --]
[-- Type: application/pgp-signature, Size: 195 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: Olaf Hering <olaf@aepfle.de>
To: "Philippe Mathieu-Daudé" <philmd@redhat.com>
Cc: Laszlo Ersek <lersek@redhat.com>,
qemu-devel@nongnu.org, "Michael S. Tsirkin" <mst@redhat.com>
Subject: Re: [Qemu-devel] edk2 fails to compile in v4.0.0-rc2
Date: Fri, 5 Apr 2019 13:27:53 +0200 [thread overview]
Message-ID: <20190405132753.6c5f2f88.olaf@aepfle.de> (raw)
Message-ID: <20190405112753.lQ5ViLpyGwdO6oXUmgueKqxqb0YVBK7D7RZq9K2hyF0@z> (raw)
In-Reply-To: <7fded1f3-7539-45b5-17c7-39fe7193dbc9@redhat.com>
[-- Attachment #1: Type: text/plain, Size: 1525 bytes --]
Am Fri, 5 Apr 2019 13:14:35 +0200
schrieb Philippe Mathieu-Daudé <philmd@redhat.com>:
> On 4/5/19 12:59 PM, Olaf Hering wrote:
> > Am Fri, 5 Apr 2019 12:49:15 +0200
> > schrieb Philippe Mathieu-Daudé <philmd@redhat.com>:
> >
> >> The EDK2 submodule was added for UEFI testing, you don't need to compile
> >> it to build/use QEMU.
> >>
> >> How did you end up compiling it?
> >
> > The qemu.spec file has this since a very long time:
> > make -C roms efirom ${unrelated_settings}
> I don't have any qemu.spec, is it a SUSE file?
Yes, it is part of the qemu package.
https://build.opensuse.org/package/show/Virtualization/qemu
> > This used to work still in January with c9d18c1c150c84e7a976df989ad04ddf01083f46.
> Before f590a812c210 the EfiRom tool was not available, now we compile it
> inconditionally.
I wonder why it must be compiled unconditionally.
This is my workaround:
sed -i 's@EFIROM@KAPUTT@g' roms/Makefile
efirom=
test -x "$(type -P EfiRom)" && KAPUTT="$_" efirom=efirom
make -C roms ${efirom} KAPUTT=${KAPUTT}
> So I assume your distribution already provides the EfiRom tool. Can you
> point me at the package which provides it? I wonder if your distribution
> use a non-upstreamed patch that change the EDK2 BaseTools PIE/PIC flags.
It is in ovmf-tools.rpm, which comes from ovmf. And looking at this package,
they also had to workaround the fact that edk2 does not cope with PIE.
https://build.opensuse.org/package/show/Virtualization/ovmf
Olaf
[-- Attachment #2: Digitale Signatur von OpenPGP --]
[-- Type: application/pgp-signature, Size: 195 bytes --]
next prev parent reply other threads:[~2019-04-05 11:40 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-04-05 10:39 [Qemu-devel] edk2 fails to compile in v4.0.0-rc2 Olaf Hering
2019-04-05 10:39 ` Olaf Hering
2019-04-05 10:49 ` Philippe Mathieu-Daudé
2019-04-05 10:59 ` Olaf Hering
2019-04-05 10:59 ` Olaf Hering
2019-04-05 11:14 ` Philippe Mathieu-Daudé
2019-04-05 11:14 ` Philippe Mathieu-Daudé
2019-04-05 11:24 ` Philippe Mathieu-Daudé
2019-04-05 11:24 ` Philippe Mathieu-Daudé
2019-04-05 11:27 ` Olaf Hering [this message]
2019-04-05 11:27 ` Olaf Hering
2019-04-05 11:16 ` Olaf Hering
2019-04-05 11:16 ` Olaf Hering
2019-04-05 11:29 ` Philippe Mathieu-Daudé
2019-04-05 11:29 ` Philippe Mathieu-Daudé
2019-04-05 11:31 ` Olaf Hering
2019-04-05 11:31 ` Olaf Hering
2019-04-08 9:04 ` Laszlo Ersek
2019-04-08 9:09 ` Olaf Hering
2019-04-08 9:42 ` Laszlo Ersek
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20190405132753.6c5f2f88.olaf@aepfle.de \
--to=olaf@aepfle.de \
--cc=lersek@redhat.com \
--cc=mst@redhat.com \
--cc=philmd@redhat.com \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.