Abstract ZFS preferred device detection
ClosedPublic
Actions

Authored by eric_metricspace.net on Oct 2 2017, 11:31 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Nov 3, 1:22 AM

	Unknown Object (File)
	Oct 24 2024, 4:54 PM

	Unknown Object (File)
	Oct 14 2024, 5:49 PM

	Unknown Object (File)
	Sep 24 2024, 8:39 AM

	Unknown Object (File)
	Sep 20 2024, 2:58 PM

	Unknown Object (File)
	Aug 31 2024, 10:27 AM

	Unknown Object (File)
	Aug 23 2024, 5:29 PM

	Unknown Object (File)
	Aug 14 2024, 8:32 PM

View All 52 Files

Subscribers

None

Details

Reviewers

imp
allanjude

Commits

rS324360: Encapsulate ZFS preferences into efi_zfs_is_preferred
rS324359: Add efi_devpath_is_prefix

Summary

This is a part of https://reviews.freebsd.org/D10447, and a precursor to that patch.

This abstracts the way that preferred ZFS devices are detected, paving the way for the boot1 refactor.

Test Plan

As of creation, this needs to be tested. The original change did introduce an error, so it needs to be tested with ZFS to ensure that separating it from D10447 didn't introduce new ones.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

eric_metricspace.net created this revision.Oct 2 2017, 11:31 AM

eric_metricspace.net added a child revision: D10447: Unify boot1 with loader.

Confirmed this works in QEMU

Closed by commit rS324359: Add efi_devpath_is_prefix (authored by imp). · Explain WhyOct 6 2017, 12:20 PM

This revision was automatically updated to reflect the committed changes.

These looked good. I've committed it. efi_devpath_is_prefix is a bit of a duplication, but a useful one. It's also not yet used, but I assume further patches will use it.

Revision Contents
Changeset List

Path

Size

head/

sys/

boot/

efi/

include/

efilib.h

1 line

libefi/

devpath.c

29 lines

Diff 33746

View Options

head/sys/boot/efi/include/efilib.h