WIP bootnext: Next Generation

[tsoome]

Summary

First take on bootnext update.

This work is based on https://github.com/openzfs/zfs/commit/108a454a4604df6ea3be817f3cf076726df2c67a

Test Plan

incidentally it seems to work already:)

[tsoome]

zfsbootcfg should output the bootonce value when run without any
arguments.

[kevans]

If you could sneak in an update to lib/libbe/libbe.3 to include be_deactivate (between be_activate and be_destroy, I suppose), i'd appreciate it. =)

lib/libbe/be.c

1233

I somewhat wonder if we should check if bootonce was set and return a special error to indicate that's the case here and let the caller decide if that's a problem or not, but I don't insist and can always revise later if we decide to care.

sbin/bectl/bectl.8

109

New sentence should start on a new line.

sbin/bectl/bectl.c

74–75

Let's bump -T to a new line, bectl activate -T

169

and check here that argc != 1 && (!reset || argc != 0)

175

then just pass in NULL for be_deactivate, because ds is unused for this scenario and I think it's misleading to require an argument that we're not going to use. :-)

[tsoome]

lib/libbe/be.c

1233

I am not really sure if we should care about if bootonce was set or not; we might just want to ensure we do have sane state on disk -- for example when the bootenv area was not cleared on pool creation, or some unsupported state was stored.

sbin/zfsbootcfg/zfsbootcfg.c

58

Actually it did not, it did require argc == 2 and only wrote the argument string to pad2:) but the idea is still OK

[tsoome]

Updates as requested by kevans.

[tsoome]

Update zfsbootcfg.8

[kevans]

I am happy with the change; thanks!

sbin/bectl/bectl.c

169

Missing closing parenthesis here

[tsoome]

restore version field in bootenv. It does help us to keep old stage2
not to freak on nvlist stream data.

[tsoome]

userboot should also set zfs-bootonce environment variable

[tsoome]

Add zfs_bootenv.h with key values for nvlist pairs.

[tsoome]

whitespace cleanup

[tsoome]

Missed few bits..

[tsoome]

make bootonce BE permanenty active, if zfs_bootonce_activate is YES.

[tsoome]

libzfsbootenv should have major version 1

[tsoome]

Fix zfs_ioc_nextboot()

[imp]

quick sweep. This is kinda large and hard to review, so I also suggested a few 'carve outs' that could be committed independently.

Makefile.inc1

2738

Does this need to be conditional on cddl like the other libraries?

cddl/contrib/opensolaris/cmd/zinject/translate.c

23

You'll want to make sure that the commit message when this goes in states you are pulling in Delphix code so that people know that any copyright updates by them come from that...

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

398

This change could be peeled off, as it's 100% independent of the rest.

lib/libzfsbootenv/libzfsbootenv.h

34

Typically in our tree we use
BEGIN_DECLS
and
END_DECLS
for this.

sys/cddl/contrib/opensolaris/uts/common/fs/zfs/sys/zfs_bootenv.h

24

BEGIN_DECL / END_DECL

[tsoome]

Rebase on OpenZFS update:
Add support for boot environment data to be stored in the label

[tsoome]

Makefile.inc1

2738

Yes. Since we want libzfsbootenv to be integrated to upstream, I need to move it to cddl tree and fix accordingly.

cddl/contrib/opensolaris/cmd/zinject/translate.c

23

Since the Delphix code is nicely self contained, I did commit it and now we get much cleaner "base".

[tsoome]

move libzfsbootenv to cddl tree.

[tsoome]

move libzfsbootenv to cddl tree

[tsoome]

add nvstore framework and command

[tsoome]

add nextboot bits.

[tsoome]

defaults/loader.conf should not have nextboot_enable
fix lua
update forth nextboot

[tsoome]

keep if statement simple.

[tsoome]

add lzbe_flags_t to notify if lzbe_set_boot_device() should add data to
current bootenv, or create new bootenv.

return VB_RAW even when there is no data

DATA_TYPE_BOOLEAN is deprecated

[tsoome]

continuation should be indented 4 spaces

[tsoome]

rebase after openzfs import

[tsoome]

update manuals.

[oskar.holmlund_ohdata.se]

If src.conf contains
WITHOUT_LOADER_ZFS=true
WITHOUT_ZFS=true

This commit (r365938) will break buildworld.

• all_subdir_stand ---

/usr/src/stand/efi/loader_4th/../loader/main.c:39:10: fatal error: 'sys/zfs_bootenv.h' file not found
#include <sys/zfs_bootenv.h>

^~~~~~~~~~~~~~~~~~~

My target is ARM with GENERIC conf and src.conf as above.
Sorry if it has already been reported.

[tsoome]

In D25512#590650, @oskar.holmlund_ohdata.se wrote:

If src.conf contains
WITHOUT_LOADER_ZFS=true
WITHOUT_ZFS=true

This commit (r365938) will break buildworld.

• all_subdir_stand ---

/usr/src/stand/efi/loader_4th/../loader/main.c:39:10: fatal error: 'sys/zfs_bootenv.h' file not found
#include <sys/zfs_bootenv.h>

^~~~~~~~~~~~~~~~~~~

My target is ARM with GENERIC conf and src.conf as above.
Sorry if it has already been reported.

Thanks, should be better now.

[oskar.holmlund_ohdata.se]

Thanks, should be better now.

Thank you, it works fine.