On 08/19/13 12:00, Michael Chang wrote: > On Sat, Aug 17, 2013 at 11:16:16PM -0700, Jordan Justen wrote: >> The volatile 'NvVars' variable indicates that the variables do >> not need to be loaded from the file again. After we write the >> variables out to the file, there is clearly no need to load >> them back from the file. >> >> Contributed-under: TianoCore Contribution Agreement 1.0 >> Signed-off-by: Jordan Justen <[email protected]> >> Cc: Michael Chang <[email protected]> >> Cc: Laszlo Ersek <[email protected]> >> --- >> Michael, >> >> Does this patch fix the issue you highlighted (way back) on June >> 21st in your patch: >> * OvmfPkg/NvVarsFileLib: handle the inital file not found > > It seems not fix the problem if I use regular openSUSE 12.3 > installation to a new virtual disk or chooses to recreate the ESP > if it already exists.
I believe the problem is that Jordan's new SetNvVarsVariable() call is not reached in SaveNvVarsToFs() when the latter runs for the very first time. At that point the (virtual) disk is not even partitioned. Hence SaveNvVarsToFs() bails out earlier, with the message FsAccess.c: Unable to open file to saved NV Variables Laszlo ------------------------------------------------------------------------------ Get 100% visibility into Java/.NET code with AppDynamics Lite! It's a free troubleshooting tool designed for production. Get down to code-level detail for bottlenecks, with <2% overhead. Download for free and get started troubleshooting in minutes. http://pubads.g.doubleclick.net/gampad/clk?id=48897031&iu=/4140/ostg.clktrk _______________________________________________ edk2-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/edk2-devel
