[LTP] [PATCH v2] Reduce needed memory by the dio_read test

[LTP] [PATCH v2] Reduce needed memory by the dio_read test

[Andrea Cervesato]

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
-- 
2.34.1


-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

Re: [LTP] [PATCH v2] Reduce needed memory by the dio_read test

[Martin Doucha]

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

-- 
Martin Doucha   mdoucha@suse.cz
QA Engineer for Software Maintenance
SUSE LINUX, s.r.o.
CORSO IIa
Krizikova 148/34
186 00 Prague 8
Czech Republic

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

Re: [LTP] [PATCH v2] Reduce needed memory by the dio_read test

[Cyril Hrubis]

Hi!
I've adjusted the commit message a bit and pushed, thanks.

-- 
Cyril Hrubis
chrubis@suse.cz

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp