> On May 9, 2015, at 1:43 PM, Ard Biesheuvel <ard.biesheu...@linaro.org> wrote:
> 
> On 9 May 2015 at 22:39, Andrew Fish <af...@apple.com 
> <mailto:af...@apple.com>> wrote:
>> Ard,
>> 
>> Would removing the Reserved field and making Size a UINT64 also fix the 
>> issue?
>> 
> 
> Yes, but since the Size field is assigned to POOL_TAIL::Size as well,
> and used in comparisons in CoreFreePoolI(), I thought this would be
> the cleaner approach. But

I had not though about the need to update code in other places. If my fix would 
cause types to change in C functions I’m fine with doing it your way.

Thanks,

Andrew Fish

> if you prefer that, I am happy to update the
> patch.
> 
>> Contributed-under: TianoCore Contribution Agreement 1.0
>> Reviewed-by: Andrew Fish <af...@apple.com <mailto:af...@apple.com>>
>> 
>> Thanks,
>> 
>> Andrew Fish

------------------------------------------------------------------------------
One dashboard for servers and applications across Physical-Virtual-Cloud 
Widest out-of-the-box monitoring support with 50+ applications
Performance metrics, stats and reports that give you Actionable Insights
Deep dive visibility with transaction tracing using APM Insight.
http://ad.doubleclick.net/ddm/clk/290420510;117567292;y
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/edk2-devel

Reply via email to