From owner-freebsd-hackers@FreeBSD.ORG Tue Mar 31 01:24:14 2015 Return-Path: Delivered-To: freebsd-hackers@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 8DE4EBD8 for ; Tue, 31 Mar 2015 01:24:14 +0000 (UTC) Received: from st11p02mm-asmtp001.mac.com (st11p02mm-asmtpout001.mac.com [17.172.220.236]) (using TLSv1.2 with cipher DHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 617E58AE for ; Tue, 31 Mar 2015 01:24:13 +0000 (UTC) Received: from fukuyama.hsd1.ca.comcast.net (c-73-162-13-215.hsd1.ca.comcast.net [73.162.13.215]) by st11p02mm-asmtp001.mac.com (Oracle Communications Messaging Server 7.0.5.35.0 64bit (built Dec 4 2014)) with ESMTPSA id <0NM1001A0YJCH530@st11p02mm-asmtp001.mac.com> for freebsd-hackers@freebsd.org; Tue, 31 Mar 2015 01:23:38 +0000 (GMT) X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:5.13.68,1.0.33,0.0.0000 definitions=2015-03-30_04:2015-03-30,2015-03-30,1970-01-01 signatures=0 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 spamscore=0 suspectscore=0 phishscore=0 adultscore=0 bulkscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=7.0.1-1412110000 definitions=main-1503310011 Content-type: text/plain; charset=us-ascii MIME-version: 1.0 (Mac OS X Mail 8.2 \(2070.6\)) Subject: Re: ZFS support for EFI From: Rui Paulo In-reply-to: <5519D4D3.6080707@metricspace.net> Date: Mon, 30 Mar 2015 18:23:35 -0700 Content-transfer-encoding: quoted-printable Message-id: References: <55189CBA.9040107@metricspace.net> <543637C0-A4FF-4801-BE5C-859F2D968D48@me.com> <5519D4D3.6080707@metricspace.net> To: Eric McCorkle X-Mailer: Apple Mail (2.2070.6) Cc: freebsd-hackers@freebsd.org X-BeenThere: freebsd-hackers@freebsd.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: Technical Discussions relating to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 31 Mar 2015 01:24:14 -0000 On Mar 30, 2015, at 15:57, Eric McCorkle wrote: >=20 > On 03/30/2015 12:28 AM, Rui Paulo wrote: >=20 >>> First, I've converted boot1.c (the EFI boot block) to use an FS = module >>> framework. This facilitates the addition of ZFS, and should also = come >>> in handy if someone wants to add other functionality later (ie. = crypto, >>> netboot, etc.) >>=20 >> Good. :-) >=20 > Actually, would that be a good patch in its own right? I could > certainly strip out the ZFS-related stuff for folks with UFS+EFI = systems > to test. (I only have ZFS drives myself) This might actually be useful to split UFS1 and UFS2. Adding Warner. >> I have a crazy idea: why not use getopt() in loader.efi ? getopt() = is already part of libstand, so it should be easy to use it. >>=20 >> Alternatively you can just use key value pairs. >>=20 >=20 > I did a little more lookung. Turns out, I had it wrong. The old ZFS > loader loads and parses /boot/config, not /boot/loader.conf. >=20 > It appears that the existing EFI loader is ignoring /boot/config. Is > that intentional, or just missing functionality? (It would probably > make more sense to stash /boot/config or its analog on the ESP = anyway.) That's just a mistake in the EFI loader. /boot/config is the same thing = as /boot.config and both are missing from EFI. I don't quite understand = how this is related to the previous problem. -- Rui Paulo