Differential D20513

Fix regression in r347193 in selecting which drive to boot.
ClosedPublic
Actions

Authored by imp on Jun 4 2019, 2:55 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Dec 13, 6:41 AM

	Unknown Object (File)
	Sun, Dec 7, 8:46 AM

	Unknown Object (File)
	Thu, Nov 27, 10:10 AM

	Unknown Object (File)
	Thu, Nov 27, 12:11 AM

	Unknown Object (File)
	Wed, Nov 26, 4:57 AM

	Unknown Object (File)
	Nov 25 2025, 3:43 AM

	Unknown Object (File)
	Nov 25 2025, 12:41 AM

	Unknown Object (File)
	Nov 20 2025, 10:05 PM

View All 93 Files

Subscribers

bcran

Details

Reviewers

tsoome
kevans

Commits

rS348659: Use newly minted efi_devpath_same_disk() instead of
rS348658: Introduce efi_devpath_same_disk

Summary

Introduce efi_devpath_same_disk

This is like efi_devpath_match, but allows differing device media
paths. Those just specify the partition information.

Use newly minted efi_devpath_same_disk() instead of efi_devpath_match(). This fixes a regression in r347193.

Reported by: Tomoaki AOKI

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

imp created this revision.Jun 4 2019, 2:55 PM

Herald added a subscriber: bcran. · View Herald TranscriptJun 4 2019, 2:55 PM

Harbormaster completed remote builds in B24678: Diff 58227.Jun 4 2019, 2:55 PM

imp added a reviewer: tsoome.Jun 4 2019, 3:32 PM

tsoome accepted this revision.Jun 4 2019, 3:53 PM

This revision is now accepted and ready to land.Jun 4 2019, 3:53 PM

kevans accepted this revision.Jun 4 2019, 5:05 PM

Closed by commit rS348658: Introduce efi_devpath_same_disk (authored by imp). · Explain WhyJun 4 2019, 6:36 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

stand/

efi/

include/

efilib.h

1 line

libefi/

devpath.c

28 lines

Diff 58240

View Options

head/stand/efi/include/efilib.h