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

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

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 Skipped

Unit

Tests Skipped

Build Status

Buildable 53516
Build 50407: arc lint + arc unit

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/

main.c

2 lines

Diff 127165

View Options

stand/loader.efi: fix regression with ignoring nvstoreClosedPublicActions