Differential D41795

stand/loader.efi: fix regression with ignoring nvstore
ClosedPublic
Actions

Authored by glebius on Sep 9 2023, 4:21 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Mar 28, 10:00 AM

	Unknown Object (File)
	Fri, Mar 27, 7:03 PM

	Unknown Object (File)
	Thu, Mar 26, 7:53 AM

	Unknown Object (File)
	Wed, Mar 18, 4:25 AM

	Unknown Object (File)
	Tue, Mar 17, 11:42 PM

	Unknown Object (File)
	Mon, Mar 16, 6:51 AM

	Unknown Object (File)
	Sun, Mar 8, 4:08 AM

	Unknown Object (File)
	Feb 8 2026, 11:22 AM

View All 91 Files

Subscribers

Details

Reviewers

tsoome
imp
manu

Commits

rGebdd5d72fffc: stand/loader.efi: fix regression with ignoring nvstore
rGd13550f0c9c2: stand/loader.efi: fix regression with ignoring nvstore

Summary

We always need to call zfs_attach_nvstore() regardless of whether we
use bootonce key in nvstore or the bootfs property of the pool.

In particular this fixes broken 'nextboot -k'.

Fixes: e3e2681d0ec28d6661fef6da76c9479049e2761c

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

glebius created this revision.Sep 9 2023, 4:21 AM

Herald added a reviewer: manu. · View Herald TranscriptSep 9 2023, 4:21 AM

Herald added subscribers: dab, bcran. · View Herald Transcript

glebius requested review of this revision.Sep 9 2023, 4:21 AM

Harbormaster completed remote builds in B53516: Diff 127165.Sep 9 2023, 4:21 AM

Please include why this is needed always in the commit message because this is counter intuitive to me and I think contradicts review feedback I got originally.

This revision is now accepted and ready to land.Sep 9 2023, 4:27 AM

Closed by commit rGd13550f0c9c2: stand/loader.efi: fix regression with ignoring nvstore (authored by glebius). · Explain WhySep 11 2023, 6:06 PM

This revision was automatically updated to reflect the committed changes.

glebius added a commit: rGd13550f0c9c2: stand/loader.efi: fix regression with ignoring nvstore.

imp added a commit: rGebdd5d72fffc: stand/loader.efi: fix regression with ignoring nvstore.Apr 16 2024, 2:35 AM

Revision Contents
Changeset List

Path

Size

stand/

efi/

loader/

2 lines

Diff 127237

stand/efi/loader/main.c

Loading...