From owner-dev-commits-src-all@freebsd.org Wed Mar 3 16:21:58 2021 Return-Path: Delivered-To: dev-commits-src-all@mailman.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.nyi.freebsd.org (Postfix) with ESMTP id 1CC6656FA3D; Wed, 3 Mar 2021 16:21:58 +0000 (UTC) (envelope-from andrew@freebsd.org) Received: from fry.fubar.geek.nz (fry.fubar.geek.nz [139.59.165.16]) by mx1.freebsd.org (Postfix) with ESMTP id 4DrK5K6r7Kz4VNk; Wed, 3 Mar 2021 16:21:57 +0000 (UTC) (envelope-from andrew@freebsd.org) Received: from [192.168.42.21] (cpc91232-cmbg18-2-0-cust554.5-4.cable.virginm.net [82.2.126.43]) by fry.fubar.geek.nz (Postfix) with ESMTPSA id 75F054E716; Wed, 3 Mar 2021 16:21:24 +0000 (UTC) Content-Type: text/plain; charset=utf-8 Mime-Version: 1.0 (Mac OS X Mail 12.4 \(3445.104.17\)) Subject: Re: git: c8db60c0673d - main - Split out the loader efifb setup to a new function From: Andrew Turner In-Reply-To: <20210303164913.010b2f03@thor.intern.walstatt.dynvpn.de> Date: Wed, 3 Mar 2021 16:21:19 +0000 Cc: "src-committers@freebsd.org" , "dev-commits-src-all@freebsd.org" , "dev-commits-src-main@freebsd.org" Content-Transfer-Encoding: quoted-printable Message-Id: <6989F292-2908-4E97-A50E-19C80648A6F8@freebsd.org> References: <202103031426.123EQmU4082875@gitrepo.freebsd.org> <20210303164913.010b2f03@thor.intern.walstatt.dynvpn.de> To: "O. Hartmann" X-Mailer: Apple Mail (2.3445.104.17) X-Rspamd-Queue-Id: 4DrK5K6r7Kz4VNk X-Spamd-Bar: ---- Authentication-Results: mx1.freebsd.org; none X-Spamd-Result: default: False [-4.00 / 15.00]; REPLY(-4.00)[] X-BeenThere: dev-commits-src-all@freebsd.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Commit messages for all branches of the src repository List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 03 Mar 2021 16:21:58 -0000 > On 3 Mar 2021, at 15:48, O. Hartmann wrote: >=20 > -----BEGIN PGP SIGNED MESSAGE----- > Hash: SHA256 >=20 > Am Wed, 3 Mar 2021 14:26:48 GMT > Andrew Turner schrieb: >=20 >> The branch main has been updated by andrew: >>=20 >> URL: = https://cgit.FreeBSD.org/src/commit/?id=3Dc8db60c0673d4bb7a3a3e2c043804e1e= d1108c2f >>=20 >> commit c8db60c0673d4bb7a3a3e2c043804e1ed1108c2f >> Author: Andrew Turner >> AuthorDate: 2021-02-26 11:47:34 +0000 >> Commit: Andrew Turner >> CommitDate: 2021-03-03 14:18:02 +0000 >>=20 >> Split out the loader efifb setup to a new function >>=20 >> This makes bi_load_efi_data cleaner to add common acpi setup code. >>=20 >> Reviewed by: imp, tsoome >> Sponsored by: Innovate UK >> Differential Revision: https://reviews.freebsd.org/D28936 >> --- >> stand/efi/loader/bootinfo.c | 22 ++++++++++++++++++++++ >> 1 file changed, 22 insertions(+) >>=20 >> diff --git a/stand/efi/loader/bootinfo.c = b/stand/efi/loader/bootinfo.c >> index 9924901d29e6..327751e32f0b 100644 >> --- a/stand/efi/loader/bootinfo.c >> +++ b/stand/efi/loader/bootinfo.c >> @@ -299,6 +299,9 @@ bi_load_efi_data(struct preloaded_file *kfp, bool = exit_bs) >> bool do_vmap; >>=20 >> #if defined(__amd64__) || defined(__aarch64__) >> +static void >> +bi_load_efifb(struct preloaded_file *kfp) >> +{ >> struct efi_fb efifb; >>=20 >> efifb.fb_addr =3D gfx_state.tg_fb.fb_addr; >> @@ -321,6 +324,25 @@ bi_load_efi_data(struct preloaded_file *kfp, = bool exit_bs) >>=20 >> if (efifb.fb_addr !=3D 0) >> file_addmetadata(kfp, MODINFOMD_EFI_FB, sizeof(efifb), = &efifb); >> +} >> +#endif >> + >> +static int >> +bi_load_efi_data(struct preloaded_file *kfp, bool exit_bs) >> +{ >> + EFI_MEMORY_DESCRIPTOR *mm; >> + EFI_PHYSICAL_ADDRESS addr =3D 0; >> + EFI_STATUS status; >> + const char *efi_novmap; >> + size_t efisz; >> + UINTN efi_mapkey; >> + UINTN dsz, pages, retry, sz; >> + UINT32 mmver; >> + struct efi_map_header *efihdr; >> + bool do_vmap; >> + >> +#if defined(__amd64__) || defined(__aarch64__) >> + bi_load_efifb(kfp); >> #endif >>=20 >> do_vmap =3D true; >> _______________________________________________ >> dev-commits-src-main@freebsd.org mailing list >> https://lists.freebsd.org/mailman/listinfo/dev-commits-src-main >> To unsubscribe, send any mail to = "dev-commits-src-main-unsubscribe@freebsd.org" >=20 > The commit seems to render "make buildworld" failing with the error = shown below: >=20 > [...] > - --- all_subdir_stand --- > - --- bootinfo.o --- > cc -target x86_64-unknown-freebsd14.0 = --sysroot=3D/usr/obj/usr/src/amd64.amd64/tmp > - -B/usr/obj/usr/src/amd64.amd64/tmp/usr/bin -O2 -pipe -O3 = -fno-common -march=3Dnative -Wformat > - -fshort-wchar -mno-red-zone -nostdinc = -I/usr/obj/usr/src/amd64.amd64/stand/libsa > - -I/usr/src/stand/libsa -D_STANDALONE -I/usr/src/sys = -Ddouble=3Djagged-little-pill > - -Dfloat=3Dfloaty-mcfloatface -ffunction-sections -fdata-sections = -DLOADER_GELI_SUPPORT > - -I/usr/src/stand/libsa/geli -DLOADER_DISK_SUPPORT -ffreestanding = -mno-mmx -mno-sse -mno-avx > - -mno-avx2 -msoft-float -fPIC -mno-red-zone -mno-relax -I. -Iinclude > - -I/usr/src/stand/efi/loader_4th/../loader -I/usr/src/stand/libsa/zfs > - -I/usr/src/sys/contrib/openzfs/include = -I/usr/src/sys/contrib/openzfs/include/os/freebsd/zfs > - -DEFI_ZFS_BOOT -fPIC -I/usr/src/stand/efi/loader_4th > - -I/usr/src/stand/efi/loader_4th/arch/amd64 = -I/usr/src/stand/efi/include > - -I/usr/src/stand/efi/include/amd64 = -I/usr/src/sys/contrib/dev/acpica/include > - -I/usr/src/stand/i386/libi386 -DEFI -DEFI_SECUREBOOT = -I/usr/src/stand/common -fPIC > - -I/usr/src/stand/ficl -I/usr/src/stand/ficl/amd64 = -I/usr/src/stand/common -DBF_DICTSIZE=3D30000 > - -DLOADER_VERIEXEC -I/usr/src/lib/libsecureboot/h -DLOADER_VERIEXEC > - -I/usr/src/lib/libsecureboot/h -DLOADER_VERIEXEC_VECTX = -DLOADER_MSDOS_SUPPORT > - -DLOADER_UFS_SUPPORT -DLOADER_NET_SUPPORT -DLOADER_GPT_SUPPORT = -DLOADER_MBR_SUPPORT > - -DLOADER_ZFS_SUPPORT -I/usr/src/stand/libsa/zfs = -I/usr/src/sys/cddl/boot/zfs > - -I/usr/src/sys/cddl/contrib/opensolaris/uts/common -DNDEBUG -fPIE = -MD -MF.depend.bootinfo.o > - -MTbootinfo.o -std=3Dgnu99 -Wno-format-zero-length -Wsystem-headers = -Wno-pointer-sign > - -Wno-empty-body -Wno-string-plus-int -Wno-unused-const-variable = -Wno-tautological-compare > - -Wno-unused-value -Wno-parentheses-equality -Wno-unused-function = -Wno-enum-conversion > - -Wno-unused-local-typedef -Wno-address-of-packed-member -Wno-switch = -Wno-switch-enum > - -Wno-knr-promoted-parameter -Wno-parentheses -Oz -Qunused-arguments = -I/usr/src/sys/teken > - -I/usr/src/contrib/pnglite -c = /usr/src/stand/efi/loader_4th/../loader/bootinfo.c -o > bootinfo.o --- all_subdir_share --- --- = all_subdir_share/i18n/csmapper/BIG5 --- =3D=3D=3D> > share/i18n/csmapper/BIG5 (all) --- all_subdir_share/i18n/csmapper/CNS = --- =3D=3D=3D> > share/i18n/csmapper/CNS (all) --- all_subdir_share/i18n/csmapper/CP = --- =3D=3D=3D> > share/i18n/csmapper/CP (all) --- all_subdir_stand --- > /usr/src/stand/efi/loader_4th/../loader/bootinfo.c:304:1: error: = function definition is not > allowed here --- all_subdir_share --- --- all_subdir_share/examples = --- --- > all_subdir_share/examples/ipfilter --- =3D=3D=3D> = share/examples/ipfilter (all) >=20 > make[1]: stopped in /usr/src It looks like it might have been broken by a rebase. I=E2=80=99ve = reverted until I have time to fix. Andrew