On Wed, Sep 09, 2020 at 12:10:00PM -0400, Ralph Siemsen wrote: > The bitflip test uses two equal sized memory buffers. This is achieved > by splitting the range of memory into two pieces. The address of the > second buffer, as well as the length of each buffer, were not correctly > calculated. This caused bitflip test to access beyond the end of range. > This patch fixes the pointer arithmetic problem. > > A second problem arises because u-boot "mtest" command expects the > ending address to be inclusive. When computing (end - start) this > results in missing 1 byte of the requested length. The bitflip test > expects a count rather than an "ending" address. Thus it fails to test > the last word of the requested range. Fixed by using (end - start + 1). > > Added Kconfig option to optionally disable the bitflip test, since it > does add significantly to the time taken for "mtest". > > Fixes: 8e434cb705d463bc8cff935160e4fb4c77cb99ab ("cmd: mem: Add bitflip > memory test to alternate mtest") > > Signed-off-by: Ralph Siemsen <ralph.siem...@linaro.org> > Reviewed-by: Stefan Roese <s...@denx.de>
Applied to u-boot/master, thanks! -- Tom
signature.asc
Description: PGP signature