Re: [edk2] [PATCH] MdePkg/BaseMemoryLibOptDxe ARM AARCH64: fix thinko in SetMem##

2016-09-23 Thread Ard Biesheuvel
On 23 September 2016 at 06:03, Gao, Liming wrote: > Reviewed-by: Liming Gao > Pushed, thanks all >> -Original Message- >> From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] >> Sent: Thursday, September 22, 2016 4:54 PM >> To:

Re: [edk2] [PATCH] MdePkg/BaseMemoryLibOptDxe ARM AARCH64: fix thinko in SetMem##

2016-09-22 Thread Gao, Liming
Reviewed-by: Liming Gao > -Original Message- > From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] > Sent: Thursday, September 22, 2016 4:54 PM > To: edk2-devel@lists.01.org; Gao, Liming > Cc: ler...@redhat.com; leif.lindh...@linaro.org;

Re: [edk2] [PATCH] MdePkg/BaseMemoryLibOptDxe ARM AARCH64: fix thinko in SetMem##

2016-09-22 Thread Laszlo Ersek
On 09/22/16 10:54, Ard Biesheuvel wrote: > The new InternalMemSetMem##() implementations for ARM and AARCH64 in > BaseMemoryLibOptDxe fail to take into account that the 'length' argument > is not in bytes, but in number of items to be copied. So multiply by the > item size before proceeding. > >

[edk2] [PATCH] MdePkg/BaseMemoryLibOptDxe ARM AARCH64: fix thinko in SetMem##

2016-09-22 Thread Ard Biesheuvel
The new InternalMemSetMem##() implementations for ARM and AARCH64 in BaseMemoryLibOptDxe fail to take into account that the 'length' argument is not in bytes, but in number of items to be copied. So multiply by the item size before proceeding. Contributed-under: TianoCore Contribution Agreement