Hi Heyi,
Thanks a lot for this fix.
Committed as SVN r18390.
/
Leif
On 28 August 2015 at 16:21, Heyi Guo <[email protected]> wrote:
> Hi Ard,
>
> Would you help to commit the patch? Sorry I didn't see it upstreamed.
>
> Thanks.
>
>
> On 08/17/2015 06:25 PM, Ard Biesheuvel wrote:
>>
>> On 13 August 2015 at 16:37, Heyi Guo <[email protected]> wrote:
>>>
>>> NewNode is the node we found, while Node is the last node in the
>>> list. Also update mFreedBufferSize.
>>>
>>> Contributed-under: TianoCore Contribution Agreement 1.0
>>> Signed-off-by: Heyi Guo <[email protected]>
>>> Cc: Leif Lindholm <[email protected]>
>>> Cc: Ard Biesheuvel <[email protected]>
>>> ---
>>> .../UncachedMemoryAllocationLib/UncachedMemoryAllocationLib.c | 7
>>> ++++---
>>> 1 file changed, 4 insertions(+), 3 deletions(-)
>>>
>> Reviewed-by: Ard Biesheuvel <[email protected]>
>>
>>
>>> diff --git
>>> a/ArmPkg/Library/UncachedMemoryAllocationLib/UncachedMemoryAllocationLib.c
>>> b/ArmPkg/Library/UncachedMemoryAllocationLib/UncachedMemoryAllocationLib.c
>>> index e70d877..b859f63 100644
>>> ---
>>> a/ArmPkg/Library/UncachedMemoryAllocationLib/UncachedMemoryAllocationLib.c
>>> +++
>>> b/ArmPkg/Library/UncachedMemoryAllocationLib/UncachedMemoryAllocationLib.c
>>> @@ -125,9 +125,10 @@ AllocatePagesFromList (
>>> }
>>> // Check if we have found a node that could contain our new
>>> allocation
>>> if (NewNode != NULL) {
>>> - NewNode->Allocated = TRUE;
>>> - Node->Allocation = (VOID*)(UINTN)Node->Base;
>>> - *Allocation = Node->Allocation;
>>> + NewNode->Allocated = TRUE;
>>> + NewNode->Allocation = (VOID*)(UINTN)NewNode->Base;
>>> + *Allocation = NewNode->Allocation;
>>> + mFreedBufferSize -= NewNode->Pages * EFI_PAGE_SIZE;
>>> return EFI_SUCCESS;
>>> }
>>>
>>> --
>>> 2.1.4
>>>
>
_______________________________________________
edk2-devel mailing list
[email protected]
https://lists.01.org/mailman/listinfo/edk2-devel
