Please commit your RTEMS_UNREACHABLE so I can use it. ::)
--joel
On Tue, Mar 22, 2016 at 10:21 AM, Joel Sherrill <j...@rtems.org> wrote:
> I don't know how these extra pieces got in that patch. :(
>
> I will remove the pieces other than cpuusagetop.c.
>
> For threadglobalconstruction.c, I am switching to just RTEMS_UNREACHABLE()
> in place of this:
>
> #ifdef __GNUC__
> __builtin_unreachable();
> #endif
> _Assert_Not_reached();
>
> I will send a new series of patches once the test builds complete.
>
> On Tue, Mar 22, 2016 at 9:46 AM, Sebastian Huber <
> sebastian.hu...@embedded-brains.de> wrote:
>
>> The commit message doesn't cover all changes.
>>
>>
>> On 22/03/16 15:40, Joel Sherrill wrote:
>>
>>> ---
>>> cpukit/libmisc/cpuuse/cpuusagetop.c | 1 -
>>> cpukit/score/src/threadglobalconstruction.c | 3 +++
>>> testsuites/sptests/Makefile.am | 2 +-
>>> testsuites/sptests/sp13/task1.c | 9 +++++----
>>> 4 files changed, 9 insertions(+), 6 deletions(-)
>>>
>>> diff --git a/cpukit/libmisc/cpuuse/cpuusagetop.c
>>> b/cpukit/libmisc/cpuuse/cpuusagetop.c
>>> index 9b92858..cf896e3 100644
>>> --- a/cpukit/libmisc/cpuuse/cpuusagetop.c
>>> +++ b/cpukit/libmisc/cpuuse/cpuusagetop.c
>>> @@ -465,7 +465,6 @@ rtems_cpuusage_top_thread (rtems_task_argument arg)
>>> for (i = 0; i < data->task_count; i++)
>>> {
>>> Thread_Control* thread = data->tasks[i];
>>> - Timestamp_Control last;
>>> Timestamp_Control usage;
>>> Timestamp_Control current_usage;
>>> diff --git a/cpukit/score/src/threadglobalconstruction.c
>>> b/cpukit/score/src/threadglobalconstruction.c
>>> index 1e84124..bc96317 100644
>>> --- a/cpukit/score/src/threadglobalconstruction.c
>>> +++ b/cpukit/score/src/threadglobalconstruction.c
>>> @@ -62,5 +62,8 @@ void _Thread_Global_construction(
>>> _Thread_Restart( executing, executing, entry );
>>> _Thread_Enable_dispatch();
>>> +#ifdef __GNUC__
>>> + __builtin_unreachable();
>>> +#endif
>>> _Assert_Not_reached();
>>> }
>>>
>>
>> Maybe use new RTEMS_UNREACHABLE(). Maybe the RTEMS_UNREACHABLE() should
>> generate a fatal error in case RTEMS_DEBUG is defined to avoid the
>> undefined behaviour.
>>
>> diff --git a/testsuites/sptests/Makefile.am
>>> b/testsuites/sptests/Makefile.am
>>> index 43f3d82..53adfc1 100644
>>> --- a/testsuites/sptests/Makefile.am
>>> +++ b/testsuites/sptests/Makefile.am
>>> @@ -65,7 +65,7 @@ _SUBDIRS += sptls03
>>> _SUBDIRS += spcpucounter01
>>> if HAS_CPLUSPLUS
>>> _SUBDIRS += spglobalcon01
>>> -_SUBDIRS += sptls02
>>> +## _SUBDIRS += sptls02
>>> endif
>>> _SUBDIRS += sptls01
>>> _SUBDIRS += spintrcritical20
>>>
>>
>> Conflict with later patch?
>>
>> diff --git a/testsuites/sptests/sp13/task1.c
>>> b/testsuites/sptests/sp13/task1.c
>>> index e104e8d..58859c0 100644
>>> --- a/testsuites/sptests/sp13/task1.c
>>> +++ b/testsuites/sptests/sp13/task1.c
>>> @@ -352,10 +352,11 @@ rtems_test_pause();
>>> for (cp = (big_receive_buffer + size);
>>> cp < (big_receive_buffer + sizeof(big_receive_buffer));
>>> - cp++)
>>> - if (*cp != 'Z') {
>>> - puts("TA1 - exact size overrun match failed");
>>> - rtems_test_exit(1);
>>> + cp++) {
>>> + if (*cp != 'Z') {
>>> + puts("TA1 - exact size overrun match failed");
>>> + rtems_test_exit(1);
>>> + }
>>> }
>>> /* all done with this one; delete it */
>>>
>>
>> Conflict with later patch?
>>
>> --
>> Sebastian Huber, embedded brains GmbH
>>
>> Address : Dornierstr. 4, D-82178 Puchheim, Germany
>> Phone : +49 89 189 47 41-16
>> Fax : +49 89 189 47 41-09
>> E-Mail : sebastian.hu...@embedded-brains.de
>> PGP : Public key available on request.
>>
>> Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.
>>
>>
>
_______________________________________________
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel
