From owner-freebsd-current@freebsd.org Thu Apr 12 00:02:19 2018 Return-Path: Delivered-To: freebsd-current@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id E4479F9CA5A for ; Thu, 12 Apr 2018 00:02:18 +0000 (UTC) (envelope-from eric@metricspace.net) Received: from mail.metricspace.net (mail.metricspace.net [IPv6:2001:470:1f11:617::107]) by mx1.freebsd.org (Postfix) with ESMTP id 62C2786ABD; Thu, 12 Apr 2018 00:02:18 +0000 (UTC) (envelope-from eric@metricspace.net) Received: from [IPv6:2001:470:1f11:617:3210:b3ff:fe77:ca3f] (unknown [IPv6:2001:470:1f11:617:3210:b3ff:fe77:ca3f]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (Client did not present a certificate) (Authenticated sender: eric) by mail.metricspace.net (Postfix) with ESMTPSA id 63F899747; Thu, 12 Apr 2018 00:02:12 +0000 (UTC) Subject: Re: GELI with UEFI supporting Boot Environments goes to HEAD when? To: Warner Losh , Oliver Pinter Cc: Tommi Pernila , "[ScaleEngine] Allan Jude" , freebsd-current , Warner Losh References: <0e75a2ba-9a59-8301-a678-68a822025bd6@metricspace.net> <9df63df2-9d61-4106-f360-347411869b41@metricspace.net> From: Eric McCorkle Openpgp: preference=signencrypt Autocrypt: addr=eric@metricspace.net; prefer-encrypt=mutual; keydata= mDMEWQXo2BYJKwYBBAHaRw8BAQdAYZJt/w5CCUvp4v5Mssy0JwiO21sDxKfa27YLD5uQVc60 JEVyaWMgTWNDb3JrbGUgPGVyaWNAbWV0cmljc3BhY2UubmV0PoiZBBMWCABBAhsDBQkB4TOA BQsJCAcCBhUICQoLAgQWAgMBAh4BAheAFiEE6eoVp0R+Jx2/f0Df5CP6Oqs6upgFAlkF6gcC GQEACgkQ5CP6Oqs6upiK5gEA2rdFpRnPct2O6IWIIkiiDSxVcQumGJNLpl5+wvjcOgsA/iHL kDE4v0RLg6w8b8KWzSJVFtim9Zoff66iUzkEVNQNtChFcmljIE1jQ29ya2xlIDxlcmljX21j Y29ya2xlQGtleWJhc2UuaW8+iJYEExYIAD4WIQTp6hWnRH4nHb9/QN/kI/o6qzq6mAUCWQXq AQIbAwUJAeEzgAULCQgHAgYVCAkKCwIEFgIDAQIeAQIXgAAKCRDkI/o6qzq6mC1TAP49VcmC wwa0/X/jU8EeLCjFQL+3U1UvKKRaN7tvt8U76QD+IzjI3LoSWzc3F/Zqu+8NE/LSbzTbFezC f0VUJiuEhgu0K0VyaWMgTWNDb3JrbGUgPGVyaWNtY2NvcmtsZUBwcm90b25tYWlsLmNvbT6I lgQTFggAPhYhBOnqFadEficdv39A3+Qj+jqrOrqYBQJZLrJoAhsDBQkB4TOABQsJCAcCBhUI CQoLAgQWAgMBAh4BAheAAAoJEOQj+jqrOrqYAGsBANlpTq+GF+7N9o5iVkDwuuO7ZBFZlxsO CdA/dIhh3oBCAQC2ipgR/mE6eab1akzRa5PsEAHA/z3bDxLYtCZzCBXdBrg4BFkF6NgSCisG AQQBl1UBBQEBB0BXtYyAeWPqTL7aosi48FCkwH7+w17y3wMv2kCTLStqPgMBCAeIfgQYFggA JhYhBOnqFadEficdv39A3+Qj+jqrOrqYBQJZBejYAhsMBQkB4TOAAAoJEOQj+jqrOrqY2H8A /1tdtmFg6evmfC6Hf4+kTd76Dj+Kb7DfDyGrcYDy8cmuAQCGwHh+Za5U1zptnKCSgvKcjBgS EuvfTgXZTaIXaZOnBA== Message-ID: Date: Wed, 11 Apr 2018 20:02:08 -0400 User-Agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0 MIME-Version: 1.0 In-Reply-To: Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="j6rsGUR5EYtrs6ssNwWRg2jQQpYzSHXaK" X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.25 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 12 Apr 2018 00:02:19 -0000 This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --j6rsGUR5EYtrs6ssNwWRg2jQQpYzSHXaK Content-Type: multipart/mixed; boundary="ztiM6EPjGKnm5abadnKD9yHa5yHRwthwq"; protected-headers="v1" From: Eric McCorkle To: Warner Losh , Oliver Pinter Cc: Tommi Pernila , "[ScaleEngine] Allan Jude" , freebsd-current , Warner Losh Message-ID: Subject: Re: GELI with UEFI supporting Boot Environments goes to HEAD when? References: <0e75a2ba-9a59-8301-a678-68a822025bd6@metricspace.net> <9df63df2-9d61-4106-f360-347411869b41@metricspace.net> In-Reply-To: --ztiM6EPjGKnm5abadnKD9yHa5yHRwthwq Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: quoted-printable I think the thing to do at this point is to wait for the current work on loader.efi to land, then adapt my patches to apply against that work. On 04/11/2018 15:06, Warner Losh wrote: > Still reviewing the code. I'm worried it's too i386 specific and it > conflicts with some work I'm doing. I'll have a list of actionable > critiques this week. >=20 > Warner >=20 > On Wed, Apr 11, 2018 at 1:03 PM, Oliver Pinter > > > wrote: >=20 > Hi! >=20 > Is there any update regarding the rebase or the inclusion to base > system? > On 3/28/18, Eric McCorkle > wrote: > > I'll do another rebase from head just to be sure > > > > On March 28, 2018 3:23:23 PM EDT, Warner Losh > wrote: > >>It's on my list for nexr, finally. I have an alternate patch for > >>loader.efi > >>from ESP, but i don't think it will affect the GELI stuff. I have= some > >>time > >>slotted for integration issues though. > >> > >>I am quite mindful of the freeze dates.... I=C2=A0 have some uefi= boot > >>loader > >>protocol changes that I need to get in. > >> > >>Warner > >> > >>On Feb 21, 2018 11:18 PM, "Tommi Pernila" > wrote: > >> > >>> Awesome, thanks for the update and the work that you have done!= > >>> > >>> Now we just need some more reviewers eyes on the code :) > >>> > >>> Br, > >>> > >>> Tommi > >>> > >>> On Thu, 22 Feb 2018 at 2.03, Eric McCorkle > > >>wrote: > >>> > >>>> FYI, I just IFC'ed everything, and the current patches are sti= ll > >>fine. > >>>> > >>>> Also, the full GELI + standalone loader has been deployed on o= ne of > >>my > >>>> laptops for some time now. > >>>> > >>>> On 02/21/2018 18:15, Eric McCorkle wrote: > >>>> > The GELI work could be merged at this point, though it won't= be > >>usable > >>>> > without an additional patch to enable loader-only operation.= =C2=A0 The > >>>> > patches are currently up for review: > >>>> > > >>>> > This is the order in which they'd need to be merged: > >>>> > > >>>> > > >>>> > https://reviews.freebsd.org/D12732 > > >>>> > > >>>> > This one changes the efipart device.=C2=A0 Toomas Soome iden= tified > some > >>>> > problems, which I have addressed.=C2=A0 He has not re-review= ed it, > >>however. > >>>> > > >>>> > > >>>> > https://reviews.freebsd.org/D12692 > > >>>> > > >>>> > This adds some crypto code needed for GELI.=C2=A0 It simply = adds new > >>code, > >>>> > and doesn't conflict with anything. > >>>> > > >>>> > > >>>> > https://reviews.freebsd.org/D12698 > > >>>> > > >>>> > This adds the EFI KMS interface code, and has the EFI loader= pass > >>keys > >>>> > into the keybuf interface. > >>>> > > >>>> > > >>>> > I can't post the main GELI driver until those get merged, as= it > >>depends > >>>> > on them.=C2=A0 It can be found on the geli branch on my gith= ub freebsd > >>>> > repository, however. > >>>> > > >>>> > > >>>> > Additionally, you need this patch, which allows loader.efi t= o > >>function > >>>> > when installed directly to the ESP: > >>>> > > >>>> > https://reviews.freebsd.org/D13497 > > >>>> > > >>>> > On 02/20/2018 22:56, Tommi Pernila wrote: > >>>> >> Hi Eric, > >>>> >> > >>>> >> could you provide a brief update how the work is going? > >>>> >> > >>>> >> > >>>> >> Br, > >>>> >> > >>>> >> Tommi > >>>> >> > >>>> >> > >>>> >> On Nov 16, 2017 04:29, "Eric McCorkle" > >>>> >> >= > > wrote: > >>>> >> > >>>> >>=C2=A0 =C2=A0 =C2=A0Right, so basically, the remaining GELI = patches are against > >>>> loader, and > >>>> >>=C2=A0 =C2=A0 =C2=A0most of them can go in independently of = the work on removing > >>boot1. > >>>> >>=C2=A0 =C2=A0 =C2=A0There's a unanimous consensus on getting= rid of boot1 which > >>>> includes its > >>>> >>=C2=A0 =C2=A0 =C2=A0original author, so that's going to happ= en. > >>>> >> > >>>> >> > >>>> >>=C2=A0 =C2=A0 =C2=A0For GELI, we have the following (not nec= essarily in order): > >>>> >> > >>>> >>=C2=A0 =C2=A0 =C2=A0a) Adding the KMS interfaces, pseudo-dev= ice, and kernel > >>keybuf > >>>> >>=C2=A0 =C2=A0 =C2=A0interactions > >>>> >>=C2=A0 =C2=A0 =C2=A0b) Modifications to the efipart driver > >>>> >>=C2=A0 =C2=A0 =C2=A0c) boot crypto > >>>> >>=C2=A0 =C2=A0 =C2=A0d) GELI partition types (not strictly ne= cessary) > >>>> >> > >>>> >>=C2=A0 =C2=A0 =C2=A0Then there's the GELI driver itself.=C2=A0= (a) and (c) are > good to > >>>> land, (b) > >>>> >>=C2=A0 =C2=A0 =C2=A0needs some more work after Toomas Soome = pointed out a > >>legitimate > >>>> >>=C2=A0 =C2=A0 =C2=A0problem, and (d) actually needs a good b= it more code (but > >>again, > >>>> it's > >>>> >>=C2=A0 =C2=A0 =C2=A0more cosmetic).=C2=A0 Additionally, the = GELI driver will need > >>further > >>>> mods to > >>>> >>=C2=A0 =C2=A0 =C2=A0efipart to be written (nothing too big).= =C2=A0 But we could go > >>ahead > >>>> with (a) > >>>> >>=C2=A0 =C2=A0 =C2=A0and (c), as they've already been proven = to work. > >>>> >> > >>>> >>=C2=A0 =C2=A0 =C2=A0I'd wanted to have this stuff shaped up = sooner, but I'm > >>>> preoccupied with > >>>> >>=C2=A0 =C2=A0 =C2=A0the 7th RISC-V workshop at the end of th= e month. > >>>> >> > >>>> >>=C2=A0 =C2=A0 =C2=A0Once this stuff is all in, loader should= handle any GELI > >>volumes it > >>>> >>=C2=A0 =C2=A0 =C2=A0finds, and it should Just Work once boot= 1 is gone. > >>>> >> > >>>> >> > >>>> > _______________________________________________ > >>>> > freebsd-current@freebsd.org > mailing list > >>>> > https://lists.freebsd.org/mailman/listinfo/freebsd-current > > >>>> > To unsubscribe, send any mail to "freebsd-current-unsubscrib= e@ > >>>> freebsd.org " > >>>> > > >>>> > >>> > > > > -- > > Sent from my Android device with K-9 Mail. Please excuse my brevi= ty. > > _______________________________________________ > > freebsd-current@freebsd.org > mailing list > > https://lists.freebsd.org/mailman/listinfo/freebsd-current > > > To unsubscribe, send any mail to > "freebsd-current-unsubscribe@freebsd.org > " > > >=20 >=20 --ztiM6EPjGKnm5abadnKD9yHa5yHRwthwq-- --j6rsGUR5EYtrs6ssNwWRg2jQQpYzSHXaK Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEARYIAB0WIQTp6hWnRH4nHb9/QN/kI/o6qzq6mAUCWs6iAwAKCRDkI/o6qzq6 mFpoAP98KbfVs4FCPI9ncl7DXRMHofYE4kIyLPPeediPPdKpdwD+K6jx2AfyhLlI hGNrVvoSXC3U2YsTk5E41g5cnP4T/Ac= =STKF -----END PGP SIGNATURE----- --j6rsGUR5EYtrs6ssNwWRg2jQQpYzSHXaK--