On Fri, 12 Jan 2024 at 12:38, Gerd Hoffmann <kra...@redhat.com> wrote: > > It is possible to find variable entries with State being 0xff, > i.e. not updated since flash block erase. This indicates the > header write was not completed (and therefore State was not set > to VAR_HEADER_VALID_ONLY). Treat this as additional "end of > variable list" condition. >
Why? > Signed-off-by: Gerd Hoffmann <kra...@redhat.com> > --- > OvmfPkg/VirtNorFlashDxe/VirtNorFlashFvb.c | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/OvmfPkg/VirtNorFlashDxe/VirtNorFlashFvb.c > b/OvmfPkg/VirtNorFlashDxe/VirtNorFlashFvb.c > index 8fcd999ac6df..c8b5e0be1379 100644 > --- a/OvmfPkg/VirtNorFlashDxe/VirtNorFlashFvb.c > +++ b/OvmfPkg/VirtNorFlashDxe/VirtNorFlashFvb.c > @@ -302,6 +302,11 @@ ValidateFvHeader ( > break; > } > > + if (VarHeader->State == 0xff) { > + DEBUG ((DEBUG_INFO, "%a: end of var list (unwritten state)\n", > __func__)); > + break; > + } > + > VarName = NULL; > switch (VarHeader->State) { > // usage: State = VAR_HEADER_VALID_ONLY > -- > 2.43.0 > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#113729): https://edk2.groups.io/g/devel/message/113729 Mute This Topic: https://groups.io/mt/103680936/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-