diff mbox series

[v2,2/2] dio_sparse: Set timeout 30 min

Message ID 20220121134052.30048-2-pvorel@suse.cz
State Accepted
Headers show
Series [v2,1/2] aiodio: Skip tests on tmpfs | expand

Commit Message

Petr Vorel Jan. 21, 2022, 1:40 p.m. UTC
Originally only ADSP075 failed (dio_sparse  -w 518192k -s 518192k -n 1000)
but just in case increase timeout for all newly rewritten tests.

Signed-off-by: Petr Vorel <pvorel@suse.cz>
---
Changes v1->v2:
Prolong the timeout only for dio_sparse (Martin)

Kind regards,
Petr

 testcases/kernel/io/ltp-aiodio/dio_sparse.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Andrea Cervesato Jan. 21, 2022, 2:17 p.m. UTC | #1
Hi,

Reviewed-by: Andrea Cervesato <andrea.cervesato@suse.com>

On 1/21/22 14:40, Petr Vorel wrote:
> Originally only ADSP075 failed (dio_sparse  -w 518192k -s 518192k -n 1000)
> but just in case increase timeout for all newly rewritten tests.
>
> Signed-off-by: Petr Vorel<pvorel@suse.cz>
> ---
> Changes v1->v2:
> Prolong the timeout only for dio_sparse (Martin)
>
> Kind regards,
> Petr
>
>   testcases/kernel/io/ltp-aiodio/dio_sparse.c | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/testcases/kernel/io/ltp-aiodio/dio_sparse.c b/testcases/kernel/io/ltp-aiodio/dio_sparse.c
> index 39fc895d65..0039daa8d1 100644
> --- a/testcases/kernel/io/ltp-aiodio/dio_sparse.c
> +++ b/testcases/kernel/io/ltp-aiodio/dio_sparse.c
> @@ -135,4 +135,5 @@ static struct tst_test test = {
>   		"tmpfs",
>   		NULL
>   	},
> +	.timeout = 1800,
>   };
Martin Doucha Jan. 21, 2022, 3:25 p.m. UTC | #2
Hi,
the commit message needs an update due to changes but otherwise:

Acked-by: Martin Doucha <mdoucha@suse.cz>

On 21. 01. 22 14:40, Petr Vorel wrote:
> Originally only ADSP075 failed (dio_sparse  -w 518192k -s 518192k -n 1000)
> but just in case increase timeout for all newly rewritten tests.
> 
> Signed-off-by: Petr Vorel <pvorel@suse.cz>
> ---
> Changes v1->v2:
> Prolong the timeout only for dio_sparse (Martin)
> 
> Kind regards,
> Petr
> 
>  testcases/kernel/io/ltp-aiodio/dio_sparse.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/testcases/kernel/io/ltp-aiodio/dio_sparse.c b/testcases/kernel/io/ltp-aiodio/dio_sparse.c
> index 39fc895d65..0039daa8d1 100644
> --- a/testcases/kernel/io/ltp-aiodio/dio_sparse.c
> +++ b/testcases/kernel/io/ltp-aiodio/dio_sparse.c
> @@ -135,4 +135,5 @@ static struct tst_test test = {
>  		"tmpfs",
>  		NULL
>  	},
> +	.timeout = 1800,
>  };
Petr Vorel Jan. 21, 2022, 3:30 p.m. UTC | #3
Hi Martin,

> Hi,
> the commit message needs an update due to changes but otherwise:
Thanks, again today!

Kind regards,
Petr
diff mbox series

Patch

diff --git a/testcases/kernel/io/ltp-aiodio/dio_sparse.c b/testcases/kernel/io/ltp-aiodio/dio_sparse.c
index 39fc895d65..0039daa8d1 100644
--- a/testcases/kernel/io/ltp-aiodio/dio_sparse.c
+++ b/testcases/kernel/io/ltp-aiodio/dio_sparse.c
@@ -135,4 +135,5 @@  static struct tst_test test = {
 		"tmpfs",
 		NULL
 	},
+	.timeout = 1800,
 };