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: edk2-devel@lists.01.org; Gao, Liming >> Cc: ler...@redh

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; Ard Biesheuvel > > Subject: [PATCH] MdePkg/Ba

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. > > C