diff mbox series

[v2] Reduce needed memory by the dio_read test

Message ID 20220121091500.8264-1-andrea.cervesato@suse.de
State Accepted
Headers show
Series [v2] Reduce needed memory by the dio_read test | expand

Commit Message

Andrea Cervesato Jan. 21, 2022, 9:15 a.m. UTC
The previous runtest file was requiring half an hour to complete using
quite amount of RAM. For 100 threads, the test would easily use ~3.5GB
of memory making really difficult to test on system with few resources.

With this modification we reduce both read and write buffer size, as
well as the file size, so test will take few minutes to complete and
we will reduce memory allocation.

Signed-off-by: Andrea Cervesato <andrea.cervesato@suse.de>
---
 runtest/ltp-aiodio.part4 | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

Comments

Martin Doucha Jan. 21, 2022, 9:24 a.m. UTC | #1
Hi,
Acked-by: Martin Doucha <mdoucha@suse.cz>

On 21. 01. 22 10:15, Andrea Cervesato wrote:
> The previous runtest file was requiring half an hour to complete using
> quite amount of RAM. For 100 threads, the test would easily use ~3.5GB
> of memory making really difficult to test on system with few resources.
> 
> With this modification we reduce both read and write buffer size, as
> well as the file size, so test will take few minutes to complete and
> we will reduce memory allocation.
> 
> Signed-off-by: Andrea Cervesato <andrea.cervesato@suse.de>
> ---
>  runtest/ltp-aiodio.part4 | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/runtest/ltp-aiodio.part4 b/runtest/ltp-aiodio.part4
> index fc1e1d479..54019d47b 100644
> --- a/runtest/ltp-aiodio.part4
> +++ b/runtest/ltp-aiodio.part4
> @@ -62,7 +62,7 @@ DIT002 dio_truncate
>  #Running read_checkzero
>  #gread_checkzero
>  #Running dio_read
> -DOR000 dio_read -n 1 -i 100
> -DOR001 dio_read -n 10 -i 30
> -DOR002 dio_read -n 20 -i 15
> -DOR003 dio_read -n 100 -i 4
> +DOR000 dio_read -n 1 -i 100 -r 512k -w 512k -s 32M
> +DOR001 dio_read -n 10 -i 30 -r 512k -w 512k -s 32M
> +DOR002 dio_read -n 20 -i 15 -r 512k -w 512k -s 32M
> +DOR003 dio_read -n 100 -i 4 -r 512k -w 512k -s 32M
Cyril Hrubis Jan. 21, 2022, 9:50 a.m. UTC | #2
Hi!
I've adjusted the commit message a bit and pushed, thanks.
diff mbox series

Patch

diff --git a/runtest/ltp-aiodio.part4 b/runtest/ltp-aiodio.part4
index fc1e1d479..54019d47b 100644
--- a/runtest/ltp-aiodio.part4
+++ b/runtest/ltp-aiodio.part4
@@ -62,7 +62,7 @@  DIT002 dio_truncate
 #Running read_checkzero
 #gread_checkzero
 #Running dio_read
-DOR000 dio_read -n 1 -i 100
-DOR001 dio_read -n 10 -i 30
-DOR002 dio_read -n 20 -i 15
-DOR003 dio_read -n 100 -i 4
+DOR000 dio_read -n 1 -i 100 -r 512k -w 512k -s 32M
+DOR001 dio_read -n 10 -i 30 -r 512k -w 512k -s 32M
+DOR002 dio_read -n 20 -i 15 -r 512k -w 512k -s 32M
+DOR003 dio_read -n 100 -i 4 -r 512k -w 512k -s 32M