<div dir="ltr">I've applied the patch and will run the tests. If OK, then I should be <div>pushing soon.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, May 30, 2018 at 1:50 PM, Gedare Bloom <span dir="ltr"><<a href="mailto:gedare@rtems.org" target="_blank">gedare@rtems.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I'm having trouble standing up a sparc-rtems5/erc32 build right now,<br>
so I can't test this.<br>
<div class="HOEnZb"><div class="h5"><br>
On Wed, May 30, 2018 at 1:44 PM, Joel Sherrill <<a href="mailto:joel@rtems.org">joel@rtems.org</a>> wrote:<br>
><br>
><br>
> On Wed, May 30, 2018 at 10:33 AM, Gedare Bloom <<a href="mailto:gedare@rtems.org">gedare@rtems.org</a>> wrote:<br>
>><br>
>> This looks fine to me, and I'll commit after I can test it. (We<br>
>> currently don't apply any significance to --signed-off-by in RTEMS<br>
>> Project.) I'm investigating your inquiry on the 0/1 cover letter.<br>
><br>
><br>
> I am ok with it if it tests OK with you.<br>
><br>
>><br>
>><br>
>> On Tue, May 15, 2018 at 5:21 AM, Martin Erik Werner<br>
>> <<a href="mailto:martinerikwerner.aac@gmail.com">martinerikwerner.aac@gmail.<wbr>com</a>> wrote:<br>
>> > _Timespec_Equal_to() does not set errno, hence avoid using perror(),<br>
>> > instead use fprintf() to stderr, and extend the error message to provide<br>
>> > information about what the error is (measured timer value after<br>
>> > re-arming is not equal to the configured interval), and how large of a<br>
>> > difference was measured.<br>
>> ><br>
>> > Signed-off-by: Martin Erik Werner <<a href="mailto:martin.werner@aacmicrotec.com">martin.werner@aacmicrotec.com</a><wbr>><br>
>> > ---<br>
>> >  testsuites/psxtests/<wbr>psxtimer01/psxtimer.c | 14 ++++++++++++--<br>
>> >  1 file changed, 12 insertions(+), 2 deletions(-)<br>
>> ><br>
>> > diff --git a/testsuites/psxtests/<wbr>psxtimer01/psxtimer.c<br>
>> > b/testsuites/psxtests/<wbr>psxtimer01/psxtimer.c<br>
>> > index 032e9f8e6c..6ec049ac54 100644<br>
>> > --- a/testsuites/psxtests/<wbr>psxtimer01/psxtimer.c<br>
>> > +++ b/testsuites/psxtests/<wbr>psxtimer01/psxtimer.c<br>
>> > @@ -143,7 +143,12 @@ void * task_a (void *arg)<br>
>> >         rtems_test_exit(0);<br>
>> >       }<br>
>> >       if (! _Timespec_Equal_to( &timerdata.it_value, &my_period )){<br>
>> > -       perror ("Error in Task A timer_gettime\n");<br>
>> > +       fprintf(<br>
>> > +           stderr, "Error in Task A timer_gettime:\n"<br>
>> > +           "  re-armed timer: %" PRIdtime_t ":%ld does not match<br>
>> > interval: %" PRIdtime_t ":%ld\n",<br>
>> > +           timerdata.it_value.tv_sec, timerdata.it_value.tv_nsec,<br>
>> > +           my_period.tv_sec, my_period.tv_nsec<br>
>> > +      );<br>
>> >       }<br>
>> >       clock = time(NULL);<br>
>> >       printf("Executing task A with count = %2i %s", params->count,<br>
>> > ctime(&clock));<br>
>> > @@ -291,7 +296,12 @@ void * task_c (void *arg)<br>
>> >         rtems_test_exit(0);<br>
>> >       }<br>
>> >       if (! _Timespec_Equal_to( &timerdata.it_value, &my_period) ){<br>
>> > -       perror ("Error in Task C timer_gettime\n");<br>
>> > +       fprintf(<br>
>> > +           stderr, "Error in Task A timer_gettime:\n"<br>
>> > +           "  re-armed timer: %" PRIdtime_t ":%ld does not match<br>
>> > interval: %" PRIdtime_t ":%ld\n",<br>
>> > +           timerdata.it_value.tv_sec, timerdata.it_value.tv_nsec,<br>
>> > +           my_period.tv_sec, my_period.tv_nsec<br>
>> > +       );<br>
>> >       }<br>
>> >       pthread_mutex_lock (&data.mutex);<br>
>> >       while (data.updated == FALSE) {<br>
>> > --<br>
>> > 2.11.0<br>
>> ><br>
>> > ______________________________<wbr>_________________<br>
>> > devel mailing list<br>
>> > <a href="mailto:devel@rtems.org">devel@rtems.org</a><br>
>> > <a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/<wbr>mailman/listinfo/devel</a><br>
>> ______________________________<wbr>_________________<br>
>> devel mailing list<br>
>> <a href="mailto:devel@rtems.org">devel@rtems.org</a><br>
>> <a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/<wbr>mailman/listinfo/devel</a><br>
><br>
><br>
</div></div></blockquote></div><br></div>