On 2015-01-15 14:30:20, Laszlo Ersek wrote: > Your patch changes the initialization such that when this chunk of > memory, allocated by PlatformPei, is empty & lacks the headers, > EmuVariableFvbRuntimeDxe no longer populates it with just the headers & > structure, but actually copies the 4GB-2MB .. 4GB-2MB+56KB range into > it. Because, that's where the data you have prepared with your tool is > mapped, by -bios. > > At the first sight it seems reasonable, but EmuVariableFvbRuntimeDxe is > deprecated at this point (Jordan, please feel free to chime in). I'd > rather not see a driver being patched that I recommend users to stay > away from.
I think the portion of the stack that attempts to save / restore variables in the FAT partition should be dropped at this point. But, we need to leave the portion that handles variable in RAM for the ROM case. I think it is fine to initialize the RAM from the ROM area rather than via code. As far as I know, the variable store format is not specified in a manner than can be relied upon. -Jordan ------------------------------------------------------------------------------ New Year. New Location. New Benefits. New Data Center in Ashburn, VA. GigeNET is offering a free month of service with a new server in Ashburn. Choose from 2 high performing configs, both with 100TB of bandwidth. Higher redundancy.Lower latency.Increased capacity.Completely compliant. http://p.sf.net/sfu/gigenet _______________________________________________ edk2-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/edk2-devel
