<div dir="ltr"><div>How do i report the findings of the  psx and  tests? <br></div><div><br></div><div><br></div><div>Thanks <br></div><div>Zack <br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, 5 Aug 2021 at 19:23, Gedare Bloom <<a href="mailto:gedare@rtems.org">gedare@rtems.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Thu, Aug 5, 2021 at 12:36 PM Zacchaeus Leung<br>
<<a href="mailto:zakthertemsdev@gmail.com" target="_blank">zakthertemsdev@gmail.com</a>> wrote:<br>
><br>
> the timer_create() method can use CLOCK_MONOTONIC but there was  no test for this<br>
><br>
The commit message needs to be improved, because this patch is doing<br>
more than adding a "test", it is implementing the functionality to<br>
create a CLOCK_MONOTONIC timer and to gettime() on it.<br>
<br>
> Closes #3888<br>
<a href="https://devel.rtems.org/ticket/3888" rel="noreferrer" target="_blank">https://devel.rtems.org/ticket/3888</a>  ??<br>
<br>
><br>
> ---<br>
<br>
[...]<br>
<br>
> +<br>
> + if ( rtems_timespec_less_than( &now, &expire ) ) {<br>
> +      rtems_timespec_subtract( &now, &expire, &result );<br>
> +    } else {<br>
> +      result.tv_nsec = 0;<br>
> +      result.tv_sec  = 0;<br>
> +    }<br>
The indentation level is wrong in this block.<br>
<br>
> +<br>
> +  value->it_value = result;<br>
> +  value->it_interval = ptimer->timer_data.it_interval;<br>
> +<br>
> +  _POSIX_Timer_Release( cpu, &lock_context );<br>
> +  return 0;<br>
> +}<br>
> \ No newline at end of file<br>
> diff --git a/testsuites/psxtests/psxtimer02/psxtimer.c b/testsuites/psxtests/psxtimer02/psxtimer.c<br>
> index 9f79d33c42..1a79369efb 100644<br>
> --- a/testsuites/psxtests/psxtimer02/psxtimer.c<br>
> +++ b/testsuites/psxtests/psxtimer02/psxtimer.c<br>
> @@ -126,6 +126,32 @@ void *POSIX_Init (<br>
>    puts( "timer_delete - bad id - EINVAL" );<br>
>    status = timer_delete( timer );<br>
>    fatal_posix_service_status_errno( status, EINVAL, "bad id" );<br>
> +<br>
> +  puts( "timer_create (monotonic) - bad timer id pointer - EINVAL" );<br>
> +  status = timer_create( CLOCK_MONOTONIC, &event, NULL );<br>
> +  fatal_posix_service_status_errno( status, EINVAL, "bad timer id" );<br>
> +<br>
> +  puts( "timer_create (monotonic) - OK" );<br>
> +  status = timer_create( CLOCK_MONOTONIC, NULL, &timer );<br>
> +  posix_service_failed( status, "timer_create OK" );<br>
> +<br>
> +  puts( "timer_create (monotonic) - too many - EAGAIN" );<br>
> +  status = timer_create( CLOCK_MONOTONIC, NULL, &timer1 );<br>
> +  fatal_posix_service_status_errno( status, EAGAIN, "too many" );<br>
> +<br>
> +  clock_gettime( CLOCK_MONOTONIC, &now );<br>
> +  itimer.it_value = now;<br>
> +  itimer.it_value.tv_sec = itimer.it_value.tv_sec - 1;<br>
> +  puts( "timer_settime (monotonic) - bad itimer value - previous time - EINVAL" );<br>
> +  status = timer_settime( timer, TIMER_ABSTIME, &itimer, NULL );<br>
> +  fatal_posix_service_status_errno( status, EINVAL, "bad itimer value #3" );<br>
> +<br>
> +  clock_gettime( CLOCK_MONOTONIC, &now );<br>
> +  itimer.it_value = now;<br>
> +  itimer.it_value.tv_sec = itimer.it_value.tv_sec + 1;<br>
> +  puts( "timer_settime (monotonic) - bad id - EINVAL" );<br>
> +  status = timer_settime( timer1, TIMER_ABSTIME, &itimer, NULL );<br>
> +  fatal_posix_service_status_errno( status, EINVAL, "bad id" );<br>
><br>
<br>
Please provide updated psxtimer02.scn and report the results for<br>
running rtems-test for at least the sparc/erc32 with sis including the<br>
sptests and psxtests. If you need help how to run rtems-test, consult<br>
the documentation and ask questions.<br>
<a href="https://docs.rtems.org/branches/master/user/testing/index.html" rel="noreferrer" target="_blank">https://docs.rtems.org/branches/master/user/testing/index.html</a><br>
<br>
>    TEST_END();<br>
>    rtems_test_exit (0);<br>
> --<br>
> 2.32.0<br>
><br>
> _______________________________________________<br>
> devel mailing list<br>
> <a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a><br>
> <a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
</blockquote></div>