From owner-svn-src-head@freebsd.org Tue Oct 17 14:48:09 2017 Return-Path: Delivered-To: svn-src-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 59107E3C2B6; Tue, 17 Oct 2017 14:48:09 +0000 (UTC) (envelope-from agh@fastmail.fm) Received: from out4-smtp.messagingengine.com (out4-smtp.messagingengine.com [66.111.4.28]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 27F7B6F830; Tue, 17 Oct 2017 14:48:08 +0000 (UTC) (envelope-from agh@fastmail.fm) Received: from compute3.internal (compute3.nyi.internal [10.202.2.43]) by mailout.nyi.internal (Postfix) with ESMTP id B5D6320548; Tue, 17 Oct 2017 10:48:01 -0400 (EDT) Received: from frontend2 ([10.202.2.161]) by compute3.internal (MEProxy); Tue, 17 Oct 2017 10:48:01 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=fastmail.fm; h= cc:content-type:date:from:in-reply-to:message-id:mime-version :references:subject:to:x-me-sender:x-me-sender:x-sasl-enc; s= fm1; bh=IxB89hXuRY6CxYxuKzBo6QckRtGlTt76CRTZkJMPjTw=; b=wDVsFiIF hfFCpEJWGZR2J9hKdD/eBuyKJMPzazCHG+D1y4E6yXvqOkJWakuMv1gwGr4Gey2H GmGuKabt4n3liBVA0w9iXMr8wI4EKG96vE1znvAYTdAeh3oDGWJryDKcucbsfWUp mHIaNlbMvra2vbZz4mDVGmaTWqBbKra9OpzBIiShPfu0MZXZIXW4xTLkdHBacwXE rict1dh1NpsUALIW7X6IJGKKe3o8kx6a/3iTg3H9ZKJ+QvXDIvjYzWnQQSRfd7oj Z26n6pmgWy937oDHpZJawQV1Ls7DcwONBE6Y8qjh8M27KG814qRp05Jd+TOXqAeY j6g2aiZxnaqxVA== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to:x-me-sender :x-me-sender:x-sasl-enc; s=fm1; bh=IxB89hXuRY6CxYxuKzBo6QckRtGlT t76CRTZkJMPjTw=; b=BLFskJSoijeJNEW+hFRe+Zv7LrXoPZMXE0UD6Xn0XdVd6 AirOLP9U/wABLszZFyNQ4i9yvnB82uE5S8DyVfmIiicX2MN7huJ7KFDFn54qI+bE oIqJswhyjQARcozNpNxsTx7JV6r1T2mdyEkPSich7OQaYQ0gGvDqik3SJ7g6Xkzz e8mTCgVMSuwtBuWhDP0izwCcSgHrMxmEuA9ZDT1BcIjnuHsM+BYmaGnn5t8H23zJ M1usQS9tIBUJJCiLIjzfKSV9YlJCLrwQhgpALARVSmk6UqEwiwtpVSyjl7tjuetk k6OrE9CHaJVvEwadfVO3K8KmYdj8asJI5J/1NQv0g== X-ME-Sender: Received: from localhost (c-59-101-49-202.hay.connect.net.au [59.101.49.202]) by mail.messagingengine.com (Postfix) with ESMTPA id 2DEEA24640; Tue, 17 Oct 2017 10:48:01 -0400 (EDT) From: Alastair Hogge To: svn-src-head@freebsd.org Cc: Warner Losh , src-committers@freebsd.org, svn-src-all@freebsd.org Subject: Re: svn commit: r324646 - in head/sys/boot/efi: boot1 libefi loader Date: Tue, 17 Oct 2017 22:47:38 +0800 Message-ID: <33612616.yJGDVEA9hj@direwolf.local.> User-Agent: KMail/4.14.10 (FreeBSD/12.0-CURRENT; KDE/4.14.30; amd64; ; ) In-Reply-To: <201710160359.v9G3xCCi087576@repo.freebsd.org> References: <201710160359.v9G3xCCi087576@repo.freebsd.org> MIME-Version: 1.0 Content-Type: multipart/signed; boundary="nextPart1857889.Am0gho80aa"; micalg="pgp-sha256"; protocol="application/pgp-signature" X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 17 Oct 2017 14:48:09 -0000 --nextPart1857889.Am0gho80aa Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" On Mon, 16 Oct 2017-10-16 03:59:12 Warner Losh wrote: > Author: imp > Date: Mon Oct 16 03:59:11 2017 > New Revision: 324646 > URL: https://svnweb.freebsd.org/changeset/base/324646 >=20 > Log: > Unify boot1 with loader >=20 > Refactor boot1 to use the same I/O code as /boot/loader uses. Refac= tor > to use the common efi_main.c. >=20 > Submitted by: Eric McCorkle > Differential Revision: https://reviews.freebsd.org/D10447 >=20 Hi Warner & Eric, thanks for the top stuff, however, I think at least o= ne ZFS compile time option was missed: cc -target x86_64-unknown-freebsd12.0 --sysroot=3D/tmp/direwolf/usr/src= /tmp -B/tmp/direwolf/usr/src/tmp/usr/bin -O2 -pipe -ffreestanding -Wfo= rmat -mno-mmx -mno-sse -mno-avx -msoft-float -fshort-wchar -mno-red-zon= e -mno-aes -march=3Dbd ver2 -DLOADER_UFS_SUPPORT -DLOADER_DISK_SUPPORT -DLOADER_GPT_SUPPORT -= DLOADER_MBR_SUPPORT -DLOADER_GELI_SUPPORT -I/usr/src/sys/boot/libsa -I/= usr/src/sys/boot/common -I. -I/usr/src/sys/boot/efi/boot1/../include -I= /usr/src/sys/boot/ efi/boot1/../include/amd64 -I/usr/src/sys/boot/efi/boot1/../../../contr= ib/dev/acpica/include -I/usr/src/sys/boot/efi/boot1/../../.. -DEFI_UFS_= BOOT -I/usr/src/sys/boot/efi/boot1/../../common -fPIC -ffreestanding -W= format -mno-mmx -m no-sse -mno-avx -msoft-float -fshort-wchar -mno-red-zone -mno-aes -MD = =2DMF.depend.boot1.o -MTboot1.o -std=3Dgnu99 -Wsystem-headers -Werror -Wa= ll -Wno-format-y2k -W -Wno-unused-parameter -Wstrict-prototypes -Wmissi= ng-prototypes -Wpoin ter-arith -Wno-uninitialized -Wno-pointer-sign -Wno-empty-body -Wno-str= ing-plus-int -Wno-unused-const-variable -Wno-tautological-compare -Wno-= unused-value -Wno-parentheses-equality -Wno-unused-function -Wno-enum-c= onversion -Wno-unu sed-local-typedef -Wno-address-of-packed-member -Wno-format -Qunused-ar= guments -c /usr/src/sys/boot/efi/boot1/boot1.c -o boot1.o = = =20 /usr/src/sys/boot/efi/boot1/boot1.c:269:1: error: no previous prototype= for function 'efi_zfs_is_preferred' [-Werror,-Wmissing-prototypes] = = =20 efi_zfs_is_preferred(EFI_HANDLE *h) > Added: > head/sys/boot/efi/libefi/efi_main.c (contents, props changed) > - copied, changed from r324645, head/sys/boot/efi/loader/efi_mai= n.c > Deleted: > head/sys/boot/efi/boot1/boot_module.h > head/sys/boot/efi/boot1/ufs_module.c > head/sys/boot/efi/boot1/zfs_module.c > head/sys/boot/efi/loader/efi_main.c > Modified: > head/sys/boot/efi/boot1/Makefile > head/sys/boot/efi/boot1/boot1.c > head/sys/boot/efi/libefi/Makefile > head/sys/boot/efi/loader/Makefile [leading diff removed] > -/* > - * devpath_last returns the last non-path end node in devpath. > - */ > -static EFI_DEVICE_PATH * > -devpath_last(EFI_DEVICE_PATH *devpath) > +bool > +efi_zfs_is_preferred(EFI_HANDLE *h) > { > + return (check_preferred(h)); > +} Should this function be conditional on #ifdef EFI_ZFS_BOOT? There could= be others, however, my buildworld has stumbled here for the moment, and I am hitti= ng the horizontal now. To good health =2D-=20 Only fools are quoted. =09=09-- Anonymous --nextPart1857889.Am0gho80aa Content-Type: application/pgp-signature; name="signature.asc" Content-Description: This is a digitally signed message part. Content-Transfer-Encoding: 7Bit -----BEGIN PGP SIGNATURE----- iQEzBAABCAAdFiEE863ZAMFLT4P9HbjBtzY+5ZChngEFAlnmGAsACgkQtzY+5ZCh ngGk/Qf/eFB+feK9NwQnwwB8yFH9hYsuM0ePdz0FThIgKZ4yRi7WLfJI0CcGhtj+ moxHwEXgCERTZBBmImTP61tGPH2B1Imc+pCthvr8WhHOkrC01wapm5VM/xQhRdlU UcvRUEA2eU0UHc2hDfMSEGoI9T3S8gDh2uqjH5rxlXJL6QiPEc61HjmYQ2g9OVYj 7SFU2q+E+gHEuDARlfUlVN9wCMUUsb3u/Pvo25KR2jukPrlO3ET69sTQ7MZfm7LE udN6pWAcDJNiOq46CKkXoQ1mpZZR4sPqt+v18REsuQnT/r9eKKgu5eLiSNJjTaef 9KPZDadlIEPjdyWK1lgQ6AlM90CJQQ== =X9Ju -----END PGP SIGNATURE----- --nextPart1857889.Am0gho80aa--