From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:46590) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Yglog-0002ve-2F for qemu-devel@nongnu.org; Fri, 10 Apr 2015 23:10:43 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Yglob-0002QN-9f for qemu-devel@nongnu.org; Fri, 10 Apr 2015 23:10:41 -0400 Received: from cantor2.suse.de ([195.135.220.15]:45353 helo=mx2.suse.de) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Yglob-0002Q2-0I for qemu-devel@nongnu.org; Fri, 10 Apr 2015 23:10:37 -0400 Message-ID: <552890A9.9010109@suse.de> Date: Sat, 11 Apr 2015 05:10:33 +0200 From: =?ISO-8859-15?Q?Andreas_F=E4rber?= MIME-Version: 1.0 References: <1428675432-31433-1-git-send-email-kraxel@redhat.com> <1428675432-31433-4-git-send-email-kraxel@redhat.com> In-Reply-To: <1428675432-31433-4-git-send-email-kraxel@redhat.com> Content-Type: text/plain; charset=iso-8859-15 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH 3/7] ipxe: add local patches List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Gerd Hoffmann , qemu-devel@nongnu.org Cc: ipxe-devel@ipxe.org Am 10.04.2015 um 16:17 schrieb Gerd Hoffmann: > There are two ipxe patches needed to make efi pxe boots work. > They didn't made it upstream yet, and I don't want to wait any > longer with updating qemu. So add them here, with some logic > to apply them before building ipxe. >=20 > /me still hopes I can revert that patch some day. >=20 > Signed-off-by: Gerd Hoffmann > --- > roms/Makefile | 12 +- > ...rove-compliance-with-the-EFI_SIMPLE_NETWO.patch | 160 +++++++++++++= ++++++++ > ...0002-efi-make-load-file-protocol-optional.patch | 102 +++++++++++++ > 3 files changed, 271 insertions(+), 3 deletions(-) > create mode 100644 roms/ipxe-patches/0001-efi_snp-improve-compliance-w= ith-the-EFI_SIMPLE_NETWO.patch > create mode 100644 roms/ipxe-patches/0002-efi-make-load-file-protocol-= optional.patch >=20 > diff --git a/roms/Makefile b/roms/Makefile > index 461cb49..ab4532c 100644 > --- a/roms/Makefile > +++ b/roms/Makefile > @@ -115,12 +115,12 @@ efi-rom-%: build-pxe-roms build-efi-roms > -ec ipxe/src/bin-x86_64-efi/$(VID)$(DID).efidrv \ > -o ../pc-bios/efi-$*.rom > =20 > -build-pxe-roms: ipxe/src/config/local/general.h > +build-pxe-roms: ipxe/qemu-patches ipxe/src/config/local/general.h > $(MAKE) -C ipxe/src GITVERSION=3D"" \ > CROSS_COMPILE=3D$(x86_64_cross_prefix) \ > $(patsubst %,bin/%.rom,$(pxerom_targets)) > =20 > -build-efi-roms: build-pxe-roms ipxe/src/config/local/general.h > +build-efi-roms: ipxe/qemu-patches build-pxe-roms ipxe/src/config/local= /general.h > $(MAKE) -C ipxe/src GITVERSION=3D"" \ > CROSS_COMPILE=3D$(x86_64_cross_prefix) \ > $(patsubst %,bin-i386-efi/%.efidrv,$(pxerom_targets)) \ > @@ -129,6 +129,12 @@ build-efi-roms: build-pxe-roms ipxe/src/config/loc= al/general.h > ipxe/src/config/local/%: config.ipxe.% > cp $< $@ > =20 > +ipxe/qemu-patches: Looks like this is only ever removed by clean? Should depend on the patch files, in case they change with an update. But why are you adding patch files in the first place? Can't we just push the commits to a branch on git.qemu-project.org and update the submodule config accordingly? Regards, Andreas > + for patch in ipxe-patches/*; do \ > + echo "# applying $$patch"; \ > + cat $$patch | (cd ipxe; patch -p1); \ > + done > + touch $@ > =20 > slof: > $(MAKE) -C SLOF CROSS=3D$(powerpc64_cross_prefix) qemu > @@ -148,6 +154,6 @@ clean: > $(MAKE) -C sgabios clean > rm -f sgabios/.depend > $(MAKE) -C ipxe/src veryclean > - (cd ipxe; rm -f src/config/local/*.h) > + (cd ipxe; git reset --hard; rm -f qemu-patches src/config/local/*.h) > $(MAKE) -C SLOF clean > rm -rf u-boot/build.e500 [snip] --=20 SUSE Linux GmbH, Maxfeldstr. 5, 90409 N=FCrnberg, Germany GF: Felix Imend=F6rffer, Jane Smithard, Jennifer Guild, Dilip Upmanyu, Graham Norton; HRB 21284 (AG N=FCrnberg)