El-Haj-Mahmoud, Samer [mailto:samer.el-haj-mahm...@hp.com] wrote:
]Sent: Monday, June 22, 2015 12:07 PM ]To: edk2-devel@lists.sourceforge.net ]Subject: Re: [edk2] [PATCH 1/2] SecurityPkg: Add RandLib from existing RngDxe code ] ]Thanks Qin and Scott. ] ]Since Jonathan's task is based on the existing EDK2 code, and involves code refactoring, and moving pieces ]of code from the DXE Driver to a library, I propose we proceed with that without making any modifications ]to the code. Once that patch is approved and checked in, we can then submit additional individual patches ]to address your feedback (removing the Intel vendor check, and fixing the IA32 assembly), as well as Mike's ](moving rdrand to BaseLib, and using TimerLib for the timeout). ] ]Thanks, ]--Samer Sounds good to me. Making the IA32 fix and cpuid vendor check removal into separate patches will make following the change history easier. Thanks, Scott ------------------------------------------------------------------------------ Monitor 25 network devices or servers for free with OpManager! OpManager is web-based network management software that monitors network devices and physical & virtual servers, alerts via email & sms for fault. Monitor 25 devices for free with no restriction. Download now http://ad.doubleclick.net/ddm/clk/292181274;119417398;o _______________________________________________ edk2-devel mailing list edk2-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/edk2-devel
