Page MenuHomeFreeBSD
Differential D55107

stand/efi: Allow RELAXED or STRICT boot policies
Needs ReviewPublic
Actions

Authored by imp on Wed, Feb 4, 5:46 PM.

Details

Reviewers
mav
glebius
manu
Summary

The RELAXED boot policy will automatically search for any root
filesystem (currently zfs only) on any device. STRICT policy only
searches on the boot device.

Sponsored by: Netflix

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 70415
Build 67298: arc lint + arc unit

Event Timeline

imp created this revision.Wed, Feb 4, 5:46 PM
Herald added a reviewer: manu. · View Herald TranscriptWed, Feb 4, 5:46 PM
Herald added subscribers: dab, bcran. · View Herald Transcript
imp requested review of this revision.Wed, Feb 4, 5:46 PM
Harbormaster completed remote builds in B70404: Diff 171160.Wed, Feb 4, 5:46 PM
imp mentioned this in D55094: loader.efi: try all ZFS pools found by efi_zfs_probe().Wed, Feb 4, 5:47 PM
imp updated this revision to Diff 171161.Wed, Feb 4, 6:20 PM

assume the boot dev are first on list.

Harbormaster completed remote builds in B70405: Diff 171161.Wed, Feb 4, 6:20 PM
imp updated this revision to Diff 171162.Wed, Feb 4, 6:27 PM

lookup table for values.

Harbormaster completed remote builds in B70406: Diff 171162.Wed, Feb 4, 6:28 PM
imp updated this revision to Diff 171171.Wed, Feb 4, 7:14 PM

Oh, I don't need the on_bootdev stuff. Simplify.

Harbormaster completed remote builds in B70414: Diff 171171.Wed, Feb 4, 7:14 PM
imp updated this revision to Diff 171172.Wed, Feb 4, 7:15 PM

Oh, don't make silly changes, so we're down to main.c only now.

Harbormaster completed remote builds in B70415: Diff 171172.Wed, Feb 4, 7:15 PM
glebius added inline comments.Wed, Feb 4, 8:13 PM
stand/efi/loader/main.c
151–154

This will make two declarations better coupled together.

611–615

Didn't you you want to skip entries that don't match the condition instead of breaking the cycle? What if next entry is actually the one we want?

glebius added inline comments.Wed, Feb 4, 8:21 PM
stand/efi/loader/main.c
611–615

Now that I've seen your change to D55094, I see that there is no difference between breaking the cycle or skipping entries. So whatever works. But in case if you prefer breaking the cycle, I will suggest to put a comment explaining why we sure that no good entry is there in the list, referencing the other file.

Revision Contents
Changeset List

PathSize
stand/
efi/
loader/
40 lines

Diff 171172

View Options

stand/efi/loader/main.c

Loading...