Re: [edk2-devel] [PATCH V2 8/9] MdeModulePkg/Variable: Add RT GetNextVariableName() cache support

2019-10-03 Thread Kubacki, Michael A
, Michael D ; Ni, Ray ; Wang, Jian J ; Yao, Jiewen Subject: Re: [edk2-devel] [PATCH V2 8/9] MdeModulePkg/Variable: Add RT GetNextVariableName() cache support On Oct 3, 2019, at 1:52 PM, Kubacki, Michael A mailto:michael.a.kuba...@intel.com>> wrote: -Done: + mVariableRuntimeCacheRe

Re: [edk2-devel] [PATCH V2 8/9] MdeModulePkg/Variable: Add RT GetNextVariableName() cache support

2019-10-03 Thread Andrew Fish via Groups.Io
> On Oct 3, 2019, at 1:52 PM, Kubacki, Michael A > wrote: > >>> -Done: >>> + mVariableRuntimeCacheReadLock = FALSE; >> >> >> Similar to the previous patch (7/9), >> if timeout occurs when acquiring the read lock, should this flag be set to >> FALSE in such case? >> > > Given that the

Re: [edk2-devel] [PATCH V2 8/9] MdeModulePkg/Variable: Add RT GetNextVariableName() cache support

2019-10-03 Thread Kubacki, Michael A
> > -Done: > > + mVariableRuntimeCacheReadLock = FALSE; > > > Similar to the previous patch (7/9), > if timeout occurs when acquiring the read lock, should this flag be set to > FALSE in such case? > Given that the runtime service can be invoked in a multi-threaded OS environment, it is

Re: [edk2-devel] [PATCH V2 8/9] MdeModulePkg/Variable: Add RT GetNextVariableName() cache support

2019-10-03 Thread Wu, Hao A
> -Original Message- > From: Kubacki, Michael A > Sent: Saturday, September 28, 2019 9:47 AM > To: devel@edk2.groups.io > Cc: Bi, Dandan; Ard Biesheuvel; Dong, Eric; Laszlo Ersek; Gao, Liming; Kinney, > Michael D; Ni, Ray; Wang, Jian J; Wu, Hao A; Yao, Jiewen > Subject: [PATCH V2 8/9]

[edk2-devel] [PATCH V2 8/9] MdeModulePkg/Variable: Add RT GetNextVariableName() cache support

2019-09-27 Thread Kubacki, Michael A
https://bugzilla.tianocore.org/show_bug.cgi?id=2220 This change implements the Runtime Service GetNextVariableName() using the Runtime Cache in VariableSmmRuntimeDxe. Runtime Service calls to GetNextVariableName() will no longer trigger a SW SMI. Overall system performance and stability will be