diff mbox series

tst-tzset: raise timeout to 10 seconds

Message ID 20180730152844.21999-1-abrodkin@synopsys.com
State New
Headers show
Series tst-tzset: raise timeout to 10 seconds | expand

Commit Message

Alexey Brodkin July 30, 2018, 3:28 p.m. UTC 
On ARC HSDK board even though it operates @ 1GHz this test runs
~7 seconds which gives us failures all the time.

That's what happens now:
| # time .../build/timezone/tst-tzset
| Timed out: killed the child process
| Command exited with non-zero status 1
| real    0m 5.11s
| user    0m 3.59s
| sys     0m 1.45s

And that's what happens with longer time-out:
| # time .../build/timezone/tst-tzset
| real    0m 6.95s
| user    0m 4.97s
| sys     0m 1.99s
|
| # echo $?
| 0
---
 timezone/tst-tzset.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Joseph Myers July 30, 2018, 3:58 p.m. UTC | #1 
On Mon, 30 Jul 2018, Alexey Brodkin wrote:

> diff --git a/timezone/tst-tzset.c b/timezone/tst-tzset.c
> index 27f986b56dab..534f7acdf75f 100644
> --- a/timezone/tst-tzset.c
> +++ b/timezone/tst-tzset.c
> @@ -25,7 +25,7 @@
>  #include <time.h>
>  #include <unistd.h>
>  
> -#define TIMEOUT 5
> +#define TIMEOUT 10

The timeout was set to 5 in commit 
112f28e88c4ea5afbd6050e71416a23db11e4adc to increase it from a default 2 
seconds.  That was before the default timeout was changed to 20 seconds in 
commit a28605b22946c708f0a5c4f06307e1a17650ced8.  In such cases I think we 
should remove the timeout setting - explicit timeouts less than the 
default are only appropriate where there is a strong reason to want to 
*decrease* the default timeout.
Andreas Schwab July 30, 2018, 4:07 p.m. UTC | #2 
On Jul 30 2018, Joseph Myers <joseph@codesourcery.com> wrote:

> On Mon, 30 Jul 2018, Alexey Brodkin wrote:
>
>> diff --git a/timezone/tst-tzset.c b/timezone/tst-tzset.c
>> index 27f986b56dab..534f7acdf75f 100644
>> --- a/timezone/tst-tzset.c
>> +++ b/timezone/tst-tzset.c
>> @@ -25,7 +25,7 @@
>>  #include <time.h>
>>  #include <unistd.h>
>>  
>> -#define TIMEOUT 5
>> +#define TIMEOUT 10
>
> The timeout was set to 5 in commit 
> 112f28e88c4ea5afbd6050e71416a23db11e4adc to increase it from a default 2 
> seconds.  That was before the default timeout was changed to 20 seconds in 
> commit a28605b22946c708f0a5c4f06307e1a17650ced8.  In such cases I think we 
> should remove the timeout setting - explicit timeouts less than the 
> default are only appropriate where there is a strong reason to want to 
> *decrease* the default timeout.

There are a lot of tests that set TIMEOUT to less than 20.

Andreas.
Florian Weimer July 30, 2018, 4:51 p.m. UTC | #3 
On 07/30/2018 06:07 PM, Andreas Schwab wrote:
> On Jul 30 2018, Joseph Myers <joseph@codesourcery.com> wrote:
> 
>> On Mon, 30 Jul 2018, Alexey Brodkin wrote:
>>
>>> diff --git a/timezone/tst-tzset.c b/timezone/tst-tzset.c
>>> index 27f986b56dab..534f7acdf75f 100644
>>> --- a/timezone/tst-tzset.c
>>> +++ b/timezone/tst-tzset.c
>>> @@ -25,7 +25,7 @@
>>>   #include <time.h>
>>>   #include <unistd.h>
>>>   
>>> -#define TIMEOUT 5
>>> +#define TIMEOUT 10
>>
>> The timeout was set to 5 in commit
>> 112f28e88c4ea5afbd6050e71416a23db11e4adc to increase it from a default 2
>> seconds.  That was before the default timeout was changed to 20 seconds in
>> commit a28605b22946c708f0a5c4f06307e1a17650ced8.  In such cases I think we
>> should remove the timeout setting - explicit timeouts less than the
>> default are only appropriate where there is a strong reason to want to
>> *decrease* the default timeout.
> 
> There are a lot of tests that set TIMEOUT to less than 20.

We should fix all of them.

Thanks,
Florian
Joseph Myers July 30, 2018, 5:11 p.m. UTC | #4 
On Mon, 30 Jul 2018, Florian Weimer wrote:

> > There are a lot of tests that set TIMEOUT to less than 20.
> 
> We should fix all of them.

It's conceivable some are deliberately less than 20 (if e.g. the test 
sleeps for a certain number of seconds and does nothing else that should 
take a significant amount of time), but certainly in the absence of a 
reason for a short timeout I think such settings should be removed.
diff mbox series

Patch

diff --git a/timezone/tst-tzset.c b/timezone/tst-tzset.c
index 27f986b56dab..534f7acdf75f 100644
--- a/timezone/tst-tzset.c
+++ b/timezone/tst-tzset.c
@@ -25,7 +25,7 @@ 
 #include <time.h>
 #include <unistd.h>
 
-#define TIMEOUT 5
+#define TIMEOUT 10
 static int do_test (void);
 #define TEST_FUNCTION do_test ()
 #include "../test-skeleton.c"