From owner-dev-commits-src-main@freebsd.org Mon Mar 1 16:50:24 2021 Return-Path: <owner-dev-commits-src-main@freebsd.org> Delivered-To: dev-commits-src-main@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 BC943564273; Mon, 1 Mar 2021 16:50:24 +0000 (UTC) (envelope-from kevans@freebsd.org) Received: from smtp.freebsd.org (smtp.freebsd.org [IPv6:2610:1c1:1:606c::24b:4]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "smtp.freebsd.org", Issuer "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4Dq5q44nJbz3CMR; Mon, 1 Mar 2021 16:50:24 +0000 (UTC) (envelope-from kevans@freebsd.org) Received: from mail-qv1-f54.google.com (mail-qv1-f54.google.com [209.85.219.54]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "GTS CA 1O1" (verified OK)) (Authenticated sender: kevans) by smtp.freebsd.org (Postfix) with ESMTPSA id 8E37E34A05; Mon, 1 Mar 2021 16:50:24 +0000 (UTC) (envelope-from kevans@freebsd.org) Received: by mail-qv1-f54.google.com with SMTP id dj14so829331qvb.1; Mon, 01 Mar 2021 08:50:24 -0800 (PST) X-Gm-Message-State: AOAM533rheAmQNdoGEDl0x2vlYvlnkAjcDxtK4gIHAoZs/VRPy4Dy8ZZ ZumnoBE2vD4AMA7+jYC9kmXB1xkj+bqTOspnG1g= X-Google-Smtp-Source: ABdhPJxxtvD84NcW4djlS83mCTjJicYJU1oKQuYygD+jaxWl/tvsCAed5xJwDmwV81mdlHqge784reCPNtxJ87LC7HY= X-Received: by 2002:a05:6214:1424:: with SMTP id o4mr15704344qvx.34.1614617424200; Mon, 01 Mar 2021 08:50:24 -0800 (PST) MIME-Version: 1.0 References: <202102232124.11NLOT27012354@gitrepo.freebsd.org> <dba58a1f-5eb0-da87-7d18-d930aa5d422a@freebsd.org> <CANCZdfoL3QFg9NvpkPW8pfVrNvUK-cTPD2ZGbBr2xxh-0dv=eA@mail.gmail.com> <dc61048c-2cfa-0ff5-0470-e93aa5233aae@freebsd.org> <583f83d8-c78b-d961-d2c5-9693bd36563b@freebsd.org> <743fd126-2077-49b1-9c85-7ccc61616b98@www.fastmail.com> <460ceb98-e0c1-42b3-8e3c-2587c5ce8398@www.fastmail.com> <F4E1978D-DE96-4082-885F-D6686A5D005C@freebsd.org> <CANCZdfpw1+sjHVPH4zahywMsLHJ1jKWbnd5A3RjGinu=nQFSGA@mail.gmail.com> <a30870ef-00b1-40b6-b378-033d8737c7e4@www.fastmail.com> <CANCZdfqPNf-RW=-T9qk5a4o-rX3CWFVGz0H_hTav-CqnW9JJ=Q@mail.gmail.com> <CAK7dMtD9k5FscvG754tLO6ZeTm5+ru87F06O8+H3bVJqVjf9iQ@mail.gmail.com> <CANCZdfrej=mY0uZnY4Cg0PxTtPjJdCUVsM2trBO83z--Qks7JA@mail.gmail.com> <8215cd95-6905-49da-ab07-65796845613c@www.fastmail.com> <c9c1db40-4aff-e0c6-284f-96a4c1482cd8@freebsd.org> <CACNAnaEJUL+kNbEYKVfWkt0A8gK26MkukST_PXYeK27Y_1ZrsA@mail.gmail.com> <82f307a1-d7e2-0950-510d-148a1a7e61a2@freebsd.org> In-Reply-To: <82f307a1-d7e2-0950-510d-148a1a7e61a2@freebsd.org> From: Kyle Evans <kevans@freebsd.org> Date: Mon, 1 Mar 2021 10:50:10 -0600 X-Gmail-Original-Message-ID: <CACNAnaHkN5+skX4WUJuVhHoe11sp7iSeh9q9hg4jxrHBBCb_1Q@mail.gmail.com> Message-ID: <CACNAnaHkN5+skX4WUJuVhHoe11sp7iSeh9q9hg4jxrHBBCb_1Q@mail.gmail.com> Subject: Re: git: 0b7472b3d8d2 - main - Mount the EFI system partition (ESP) on newly-installed systems. To: Nathan Whitehorn <nwhitehorn@freebsd.org> Cc: Brandon Bergren <bdragon@freebsd.org>, Warner Losh <imp@bsdimp.com>, Kevin Bowling <kevin.bowling@kev009.com>, Jessica Clarke <jrtc27@freebsd.org>, Colin Percival <cperciva@tarsnap.com>, src-committers <src-committers@freebsd.org>, "<dev-commits-src-all@freebsd.org>" <dev-commits-src-all@freebsd.org>, dev-commits-src-main@freebsd.org Content-Type: text/plain; charset="UTF-8" X-BeenThere: dev-commits-src-main@freebsd.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Commit messages for the main branch of the src repository <dev-commits-src-main.freebsd.org> List-Unsubscribe: <https://lists.freebsd.org/mailman/options/dev-commits-src-main>, <mailto:dev-commits-src-main-request@freebsd.org?subject=unsubscribe> List-Archive: <http://lists.freebsd.org/pipermail/dev-commits-src-main/> List-Post: <mailto:dev-commits-src-main@freebsd.org> List-Help: <mailto:dev-commits-src-main-request@freebsd.org?subject=help> List-Subscribe: <https://lists.freebsd.org/mailman/listinfo/dev-commits-src-main>, <mailto:dev-commits-src-main-request@freebsd.org?subject=subscribe> X-List-Received-Date: Mon, 01 Mar 2021 16:50:24 -0000 On Mon, Mar 1, 2021 at 10:45 AM Nathan Whitehorn <nwhitehorn@freebsd.org> wrote: > > > > On 3/1/21 11:42 AM, Kyle Evans wrote: > > On Mon, Mar 1, 2021 at 10:39 AM Nathan Whitehorn <nwhitehorn@freebsd.org> wrote: > >> > >> > >> On 2/28/21 3:44 PM, Brandon Bergren wrote: > >>> On Sun, Feb 28, 2021, at 2:25 PM, Warner Losh wrote: > >>>> Yes. I agree as well. I was just hoping to say just that: EFI is barely > >>>> theoretically possible, but in reality we'll likely never use it.... > >>>> > >>>> The net effect is that we don't want to install efi on powerpc on freebsd. > >>>> > >>>> Warner > >>>> > >>> Yeah. The code before the change excluded mips and powerpc platforms, and it should continue to do so instead of using the existence of a /boot/efi directory as the only clue. > >>> > >>> Currently bsdinstall bails out and leaves powerpc* in a half-installed state because the die in the uname case propagates to the main script, so it never runs the bits after the bootconfig. > >>> > >> So that was a deliberate choice to keep the list of places that know > >> about efi vs. non-EFI centralized. I'd prefer to just not make that > >> directory on systems where it doesn't apply rather than messing with the > >> installer. Do you know where it is being made? > >> -Nathan > > It's part of the hierarchy in ^/etc/mtree/BSD.root.dist > > > > Is there a reason it needs to be? The installer bits all make it when > needed already, so just removing it there seems like the simplest path. I can't think of a reason, as long as both the release(7) scripts and the installer create it as needed -- I note that vmimage.subr seems to create it itself, but arm.subr seems to get it wrong atm. It'll need to create /boot/efi for PART_SCHEME == GPT, and /boot/msdos should probably be scoped down to PART_SCHEME == MBR where it's used at the moment.