From owner-freebsd-current@freebsd.org Mon Jul 9 22:05:29 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 49CF01035ECB for ; Mon, 9 Jul 2018 22:05:29 +0000 (UTC) (envelope-from wlosh@bsdimp.com) Received: from mail-it0-x229.google.com (mail-it0-x229.google.com [IPv6:2607:f8b0:4001:c0b::229]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "Google Internet Authority G2" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id C34338A3D9 for ; Mon, 9 Jul 2018 22:05:28 +0000 (UTC) (envelope-from wlosh@bsdimp.com) Received: by mail-it0-x229.google.com with SMTP id g4-v6so11596334iti.1 for ; Mon, 09 Jul 2018 15:05:28 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bsdimp-com.20150623.gappssmtp.com; s=20150623; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=uIgercgav3VE7yA/uyyEWYm59CaiMV+WCiMoaj4UWLM=; b=N1fIV69KWbAdMxKpxwcsIYoynjusL2snNBzNVGorVFjqUn5nhIOtGM3EHAL6KA6tCJ tdaQIGzEhD1HpYRuNsiNkhyxPpwq3ztnWIf8kZL00FajrCp+IxlAmQ3zz6bc5omAGUoY 03Xo4k+MQPX/rCGZNz1Xa67h/afZWAN6HUUIHbI73xH2U0sz5bJRc8C5tmYOU2qpgs/d shMpCEDhSjaTZhC3E66fPg1o9JHDEYHptpB5oG0ZjwOac8j9AughCYfJwJZBOWoGtoVR Ic4uqHUB6EODQxiyMh9t0Hx0a4paM9abyUWvRUji8J5kibCm+8pcTd7ztfeK605LM1cG +LZA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=uIgercgav3VE7yA/uyyEWYm59CaiMV+WCiMoaj4UWLM=; b=qWmZnA0nYT0oZ0zhkrx3lkpU48smeNpWjBdEMjHIB0eb9jb1hrFWaLu0eNuM/LBDBQ Z9axwfZ8FKoV7/Y7qbOenlfsN7u2RuVvj4MAV0eUSxIdAigxWOq3Q8B1lIGpu4TWO8MG 2verSmMDYwMxSJWzHCaEw7RrlErdGx8UOsnL8EZ9wBCydwd7k+qCShiEI8kChY7KK1r3 0IoyTxsqwmd+nFRA28I31I/WP+cFWFjOGbDPUeRo+qZNyyrHXF/rv2zM35p6939j2trh SyI8xU0TjiGZfB1Qnax7rxqALzRfuQBt7YHiJT9s7KCjpHTX5TeWmbGZldrlqiuFZ+d3 rXIw== X-Gm-Message-State: APt69E2Ju1RGelcdFj6q5XzT8SIuZ1yidsc6caRhc4CMzIw+7NRC613F 79flPbXUJEQnrlZRKHVuQEafPzE/rLm7M2vUWdintw== X-Google-Smtp-Source: AAOMgpfa2AO8XolLYH+OcUY/1vN+0SyOKGLa781PrcHCMI3TFQnxEGq4tejY8wxNKL4LREtEu3pTtsu66f8MszKZu5w= X-Received: by 2002:a02:3344:: with SMTP id k4-v6mr3150639jak.45.1531173927924; Mon, 09 Jul 2018 15:05:27 -0700 (PDT) MIME-Version: 1.0 References: <0e75a2ba-9a59-8301-a678-68a822025bd6@metricspace.net> <9df63df2-9d61-4106-f360-347411869b41@metricspace.net> <5ba11024-e99b-86e1-48b7-125fb80b4001@metricspace.net> <1531078307.1336.22.camel@freebsd.org> <06cb8190-7a04-5c92-8fb9-637d1a80758f@freebsd.org> In-Reply-To: <06cb8190-7a04-5c92-8fb9-637d1a80758f@freebsd.org> From: Warner Losh Date: Mon, 9 Jul 2018 16:05:16 -0600 Message-ID: Subject: Re: GELI with UEFI supporting Boot Environments goes to HEAD when? To: Allan Jude Cc: Ian Lepore , Oliver Pinter , Eric McCorkle , Tommi Pernila , freebsd-current , Warner Losh Content-Type: text/plain; charset="UTF-8" X-Content-Filtered-By: Mailman/MimeDel 2.1.27 X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.27 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: Mon, 09 Jul 2018 22:05:29 -0000 I have this in my tree already... Warner On Mon, Jul 9, 2018, 10:28 AM Allan Jude wrote: > I will look at updating the rootgen.sh script this evening, to support > creating more flexible ESP partitions, so we can drop the loader.efi > into an msdosfs directly. > > On 07/08/2018 15:31, Ian Lepore wrote: > > On Sun, 2018-07-08 at 21:08 +0200, Oliver Pinter wrote: > >> Hi! > >> > >> Have you or Warner any update on this code? > >> > >> On Thursday, April 12, 2018, Eric McCorkle > >> wrote: > >> > > > > Are you aware of https://reviews.freebsd.org/D15743 ? > > > > That's my changes to add geli support to loader(8) in an architecture- > > agnostic way, so that "it just works" for all platforms and flavors of > > loader. It has been succesfully tested on armv6/7 (ubldr) and on x86 > > using qemu. The x86 tests cover ufs and zfs, legacy bios and uefi. The > > only variations that aren't tested yet are the uefi flavors, because > > the current rootgen.sh script for assembling test images is still using > > boot1.efi and I don't know enough about efi myself to update the script > > to make it assemble images the new way Warner envisions. > > > > -- Ian > > > >>> > >>> I'm in the middle of moving to a new apartment right now. It's > >>> going to > >>> be a bit before I can get to this. > >>> > >>> On 04/11/2018 20:31, Warner Losh wrote: > >>>> > >>>> OK. I've pushed in the main part of it. The additional work I > >>>> have > >>>> shouldn't affect any of this stuff. I was going to look at what > >>>> part(s) > >>>> of your open reviewed needed to be redone tomorrow and send you > >>>> feedback, but if you wanted to get a start before then, I'm happy > >>>> to > >>>> answer questions. All the rest of my work is going to be > >>>> selecting the > >>>> root partition when we're told to us a specific partition, so > >>>> will be > >>>> very constrained. > >>>> > >>>> Warner > >>>> > >>>> On Wed, Apr 11, 2018 at 6:02 PM, Eric McCorkle >>>> net > >>>> > wrote: > >>>> > >>>> 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. > >>>> > > >>>> > Warner > >>>> > > >>>> > On Wed, Apr 11, 2018 at 1:03 PM, Oliver Pinter > >>>> > >>>> > >>>> >>>> >> > >>>> > wrote: > >>>> > > >>>> > Hi! > >>>> > > >>>> > Is there any update regarding the rebase or the > >>>> inclusion to > >>> base > >>>> > >>>> > system? > >>>> > On 3/28/18, Eric McCorkle >>>> >>> eric@metricspace.net> > >>>> > >>>> > >>>> et>>> > >>> wrote: > >>>> > >>>> > > I'll do another rebase from head just to be sure > >>>> > > > >>>> > > On March 28, 2018 3:23:23 PM EDT, Warner Losh < > >>> imp@bsdimp.com > >>>> > >>>> > >> 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 have > >>>> some uefi > >>> boot > >>>> > >>>> > >>loader > >>>> > >>protocol changes that I need to get in. > >>>> > >> > >>>> > >>Warner > >>>> > >> > >>>> > >>On Feb 21, 2018 11:18 PM, "Tommi Pernila" < > >>> tommi.pernila@iki.fi > >>>> > >>>> > >>>> fi>>> > >>> 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 < > >>> eric@metricspace.net > >>>> > >>>> > >>>> et>>> > >>>> > >>wrote: > >>>> > >>> > >>>> > >>>> FYI, I just IFC'ed everything, and the current > >>>> patches > >>>> are still > >>>> > >>fine. > >>>> > >>>> > >>>> > >>>> Also, the full GELI + standalone loader has been > >>>> deployed > >>>> on one 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. 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. Toomas > >>>> Soome > >>>> identified > >>>> > some > >>>> > >>>> > problems, which I have addressed. He has not > >>>> re-reviewed it, > >>>> > >>however. > >>>> > >>>> > > >>>> > >>>> > > >>>> > >>>> > https://reviews.freebsd.org/D12692 > >>>> > >>>> > >>>> > > >>>> > >>>> > > >>>> > >>>> > This adds some crypto code needed for GELI. 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. It can be found on the geli branch on > >>>> my > >>>> github freebsd > >>>> > >>>> > repository, however. > >>>> > >>>> > > >>>> > >>>> > > >>>> > >>>> > Additionally, you need this patch, which allows > >>>> loader.efi to > >>>> > >>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" > >>>> > >>>> > >>>> et>> > >>>> > >>>> >> >>>> >>>> >>> > >>>> > wrote: > >>>> > >>>> >> > >>>> > >>>> >> Right, so basically, the remaining GELI > >>>> patches > >>>> are against > >>>> > >>>> loader, and > >>>> > >>>> >> most of them can go in independently of the > >>>> work > >>>> on removing > >>>> > >>boot1. > >>>> > >>>> >> There's a unanimous consensus on getting > >>>> rid of > >>>> boot1 which > >>>> > >>>> includes its > >>>> > >>>> >> original author, so that's going to happen. > >>>> > >>>> >> > >>>> > >>>> >> > >>>> > >>>> >> For GELI, we have the following (not > >>>> necessarily > >>>> in order): > >>>> > >>>> >> > >>>> > >>>> >> a) Adding the KMS interfaces, pseudo- > >>>> device, and > >>>> kernel > >>>> > >>keybuf > >>>> > >>>> >> interactions > >>>> > >>>> >> b) Modifications to the efipart driver > >>>> > >>>> >> c) boot crypto > >>>> > >>>> >> d) GELI partition types (not strictly > >>>> necessary) > >>>> > >>>> >> > >>>> > >>>> >> Then there's the GELI driver itself. (a) > >>>> and (c) > >>> are > >>>> > >>>> > good to > >>>> > >>>> land, (b) > >>>> > >>>> >> needs some more work after Toomas Soome > >>>> pointed > >>> out a > >>>> > >>>> > >>legitimate > >>>> > >>>> >> problem, and (d) actually needs a good bit > >>>> more > >>>> code (but > >>>> > >>again, > >>>> > >>>> it's > >>>> > >>>> >> more cosmetic). Additionally, the GELI > >>>> driver > >>>> will need > >>>> > >>further > >>>> > >>>> mods to > >>>> > >>>> >> efipart to be written (nothing too > >>>> big). But we > >>>> could go > >>>> > >>ahead > >>>> > >>>> with (a) > >>>> > >>>> >> and (c), as they've already been proven to > >>>> work. > >>>> > >>>> >> > >>>> > >>>> >> I'd wanted to have this stuff shaped up > >>>> sooner, > >>>> but I'm > >>>> > >>>> preoccupied with > >>>> > >>>> >> the 7th RISC-V workshop at the end of the > >>>> month. > >>>> > >>>> >> > >>>> > >>>> >> Once this stuff is all in, loader should > >>>> handle > >>>> any GELI > >>>> > >>volumes it > >>>> > >>>> >> finds, and it should Just Work once boot1 > >>>> is gone. > >>>> > >>>> >> > >>>> > >>>> >> > >>>> > >>>> > _______________________________________________ > >>>> > >>>> > freebsd-current@freebsd.org > >>>> > >>>> > >>>> > mailing list > >>>> > >>>> > https://lists.freebsd.org/mailman/listinfo/freeb > >>>> sd- > >>> current > >>>> > >>>> > >>>> > >>>> rent > >>>> > > >>>> > >>>> > To unsubscribe, send any mail to > >>> "freebsd-current-unsubscribe@ > >>>> > >>>> > >>>> freebsd.org > >>>> " > >>>> > >>>> > > >>>> > >>>> > >>>> > >>> > >>>> > > > >>>> > > -- > >>>> > > Sent from my Android device with K-9 Mail. Please > >>>> excuse my > >>> brevity. > >>>> > >>>> > > _______________________________________________ > >>>> > > freebsd-current@freebsd.org > >>>> > >>>> >>>> > > >>>> > mailing list > >>>> > > https://lists.freebsd.org/mailman/listinfo/freebsd-cu > >>>> rrent > >>>> > >>>> > >>>> rent > >>>> > > >>>> > > To unsubscribe, send any mail to > >>>> > "freebsd-current-unsubscribe@freebsd.org > >>>> > >>>> > >>>> >" > >>>> > > > >>>> > > >>>> > > >>>> > >>>> > >>> > >> _______________________________________________ > >> 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" > > -- > Allan Jude >