Re: [PATCH 0/3] memtest performance improvements

[Anshul Dalal <anshuld@ti.com>]

Hi Rasmus,

On Fri Aug 22, 2025 at 11:48 PM IST, Rasmus Villemoes wrote:
> The first two patches make memtest run ~40x faster (when, as it should
> be, dcache is disabled), with the second patch being responsible for
> most of that. At least on the beagleboneblack which I used for
> testing; other boards and configurations will likely see different
> numbers.
>
> This is for CONFIG_SYS_ALT_MEMTEST=y and
> CONFIG_SYS_ALT_MEMTEST_BITFLIP=n; one could probably get a similar
> improvement in the bitflip case since that also has a schedule() call
> in the inner loop.
>
> Rasmus Villemoes (3):
>   memtest: don't volatile-qualify local variables
>   memtest: only call schedule() once for every 256 words
>   memtest: remove use of vu_long typedef in mem_test_alt
>
>  cmd/mem.c | 40 +++++++++++++++++++++-------------------
>  1 file changed, 21 insertions(+), 19 deletions(-)

For the entire series:
Tested-by: Anshul Dalal <anshuld@ti.com>

Tested on TI AM62P EVM, with ~10x improvment.

Patch:

	diff --git a/configs/am62px_evm_a53_defconfig b/configs/am62px_evm_a53_defconfig
	index fa857e51137..896e98fb7ca 100644
	--- a/configs/am62px_evm_a53_defconfig
	+++ b/configs/am62px_evm_a53_defconfig
	@@ -150,6 +150,9 @@ CONFIG_SPL_USB_HOST=y
	 CONFIG_SPL_USB_STORAGE=y
	 CONFIG_FS_FAT_MAX_CLUSTSIZE=16384
	 CONFIG_EFI_SET_TIME=y
	+CONFIG_CMD_MEMTEST=y
	+CONFIG_SYS_ALT_MEMTEST=y
	+CONFIG_SYS_ALT_MEMTEST_BITFLIP=n

	 #include <configs/k3_efi_capsule.config>
	 #include <configs/am62x_a53_usbdfu.config>

Results:

Before:
	=> dcache off
	=> time mtest 0x81000000 0x82000000 0 1
	Testing 81000000 ... 82000000:
	Iteration:      1
	Tested 1 iteration(s) with 0 errors.

	time: 10.848 seconds

After:
	=> dcache off
	=> time mtest 0x81000000 0x82000000 0 1
	Testing 81000000 ... 82000000:
	Iteration:      1
	Tested 1 iteration(s) with 0 errors.

	time: 1.054 seconds

Regards,
Anshul