Page MenuHomeFreeBSD
Differential D19686

Update message displaying during `zpool upgrade`
Needs ReviewPublic
Actions

Authored by allanjude on Mar 23 2019, 6:13 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 4, 2:20 AM
Unknown Object (File)
Mon, Nov 3, 5:43 PM
Unknown Object (File)
Mon, Nov 3, 1:57 PM
Unknown Object (File)
Mon, Nov 3, 5:06 AM
Unknown Object (File)
Tue, Oct 21, 12:34 AM
Unknown Object (File)
Tue, Oct 21, 12:34 AM
Unknown Object (File)
Mon, Oct 20, 2:45 PM
Unknown Object (File)
Thu, Oct 16, 5:05 AM
View All 39 Files
Subscribers
delphij
imp

Details

Reviewers
trasz
bcr
Group Reviewers
ZFS
Summary

Many users are running the default command that installs the BIOS bootcode overtop of their EFI ESP partition and getting an unbootable system.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 23260
Build 22297: arc lint + arc unit

Event Timeline

allanjude created this revision.Mar 23 2019, 6:13 AM
Herald added subscribers: delphij, imp. · View Herald TranscriptMar 23 2019, 6:13 AM
Harbormaster completed remote builds in B23260: Diff 55380.Mar 23 2019, 6:13 AM
delphij added a comment.Mar 23 2019, 7:04 AM

Why is the partition index changed from 1 to 2? (The change looks otherwise fine to me).

allanjude added a comment.Mar 23 2019, 8:05 AM
In D19686#421604, @delphij wrote:

Why is the partition index changed from 1 to 2? (The change looks otherwise fine to me).

bsdinstall since April 2016 has created both an EFI and BIOS boot partition by default, so the freebsd-boot partition is index 2 on most people's machines.

bcr added a comment.Mar 24 2019, 7:16 AM

A few minor suggestions to be more precise.

cddl/contrib/opensolaris/cmd/zpool/zpool_main.c
5859

I'd use a colon at the end of the sentence.

5863

Same here for the EFI instructions.

5866

No sentence stop (or even an exclamation mark) here?

cddl/contrib/opensolaris/lib/libzfs/common/libzfs_pool.c
2886

I think you need to mention "gpart show" here, so that users know what you are referring to when talking about freebsd-boot (where is that output coming from).

bcr added a comment.Jun 25 2019, 7:33 PM

Any update on this one or are we waiting for upstream here?

Revision Contents
Changeset List

PathSize
cddl/
contrib/
opensolaris/
cmd/
zpool/
16 lines
lib/
libzfs/
common/
16 lines

Diff 55380

View Options

cddl/contrib/opensolaris/cmd/zpool/zpool_main.c

Loading...
View Options

cddl/contrib/opensolaris/lib/libzfs/common/libzfs_pool.c

Loading...