change log for rtems (2011-05-05)
rtems-vc at rtems.org
rtems-vc at rtems.org
Thu May 5 17:10:38 UTC 2011
*joel*:
2011-05-05 Joel Sherrill <joel.sherrill at oarcorp.com>
* base_sp/apptask.c, base_sp/init.c, fileio/init.c, fileio/system.h,
nsecs/init.c, ticker/init.c, ticker/system.h, ticker/tasks.c: Remove
warnings.
M 1.188 testsuites/samples/ChangeLog
M 1.16 testsuites/samples/base_sp/apptask.c
M 1.13 testsuites/samples/base_sp/init.c
M 1.32 testsuites/samples/fileio/init.c
M 1.17 testsuites/samples/fileio/system.h
M 1.15 testsuites/samples/nsecs/init.c
M 1.15 testsuites/samples/ticker/init.c
M 1.24 testsuites/samples/ticker/system.h
M 1.16 testsuites/samples/ticker/tasks.c
diff -u rtems/testsuites/samples/ChangeLog:1.187 rtems/testsuites/samples/ChangeLog:1.188
--- rtems/testsuites/samples/ChangeLog:1.187 Tue Feb 22 08:48:25 2011
+++ rtems/testsuites/samples/ChangeLog Thu May 5 11:45:40 2011
@@ -1,3 +1,9 @@
+2011-05-05 Joel Sherrill <joel.sherrill at oarcorp.com>
+
+ * base_sp/apptask.c, base_sp/init.c, fileio/init.c, fileio/system.h,
+ nsecs/init.c, ticker/init.c, ticker/system.h, ticker/tasks.c: Remove
+ warnings.
+
2011-02-22 Ralf Corsépius <ralf.corsepius at rtems.org>
* configure.ac: Add AC_CONFIG_HEADER(config.h).
diff -u rtems/testsuites/samples/base_sp/apptask.c:1.15 rtems/testsuites/samples/base_sp/apptask.c:1.16
--- rtems/testsuites/samples/base_sp/apptask.c:1.15 Tue Feb 22 04:54:32 2011
+++ rtems/testsuites/samples/base_sp/apptask.c Thu May 5 11:45:40 2011
@@ -8,7 +8,7 @@
*
* Output parameters: NONE
*
- * COPYRIGHT (c) 1989-1999.
+ * COPYRIGHT (c) 1989-2011.
* On-Line Applications Research Corporation (OAR).
*
* The license and distribution terms for this file may be
@@ -36,9 +36,12 @@
rtems_status_code status;
status = rtems_task_ident( RTEMS_SELF, RTEMS_SEARCH_ALL_NODES, &tid );
+ directive_failed( status, "ident" );
- printf( "Application task was invoked with argument (%" PRIdrtems_task_argument ") "
- "and has id of 0x%" PRIxrtems_id "\n", argument, tid );
+ printf(
+ "Application task was invoked with argument (%" PRIdrtems_task_argument ") "
+ "and has id of 0x%" PRIxrtems_id "\n", argument, tid
+ );
printf( "*** END OF SAMPLE SINGLE PROCESSOR APPLICATION ***\n" );
exit( 0 );
diff -u rtems/testsuites/samples/base_sp/init.c:1.12 rtems/testsuites/samples/base_sp/init.c:1.13
--- rtems/testsuites/samples/base_sp/init.c:1.12 Tue Feb 22 04:54:32 2011
+++ rtems/testsuites/samples/base_sp/init.c Thu May 5 11:45:40 2011
@@ -10,7 +10,7 @@
*
* Output parameters: NONE
*
- * COPYRIGHT (c) 1989-1999.
+ * COPYRIGHT (c) 1989-2011.
* On-Line Applications Research Corporation (OAR).
*
* The license and distribution terms for this file may be
@@ -26,6 +26,7 @@
#define CONFIGURE_INIT
#include "system.h"
+#include "tmacros.h"
#include <stdio.h>
#define ARGUMENT 0
@@ -45,8 +46,11 @@
status = rtems_task_create( task_name, 1, RTEMS_MINIMUM_STACK_SIZE,
RTEMS_INTERRUPT_LEVEL(0), RTEMS_DEFAULT_ATTRIBUTES, &tid );
+ directive_failed( status, "create" );
status = rtems_task_start( tid, Application_task, ARGUMENT );
+ directive_failed( status, "start" );
status = rtems_task_delete( RTEMS_SELF );
+ directive_failed( status, "delete" );
}
diff -u rtems/testsuites/samples/fileio/init.c:1.31 rtems/testsuites/samples/fileio/init.c:1.32
--- rtems/testsuites/samples/fileio/init.c:1.31 Tue Feb 22 04:54:32 2011
+++ rtems/testsuites/samples/fileio/init.c Thu May 5 11:45:40 2011
@@ -10,7 +10,7 @@
*
* Output parameters: NONE
*
- * COPYRIGHT (c) 1989-2009.
+ * COPYRIGHT (c) 1989-2011.
* On-Line Applications Research Corporation (OAR).
*
* The license and distribution terms for this file may be
@@ -707,10 +707,13 @@
RTEMS_DEFAULT_MODES ,
RTEMS_FLOATING_POINT | RTEMS_DEFAULT_ATTRIBUTES, &Task_id
);
+ directive_failed( status, "create" );
status = rtems_task_start( Task_id, fileio_task, 1 );
+ directive_failed( status, "start" );
status = rtems_task_delete( RTEMS_SELF );
+ directive_failed( status, "delete" );
}
#if defined(USE_SHELL)
diff -u rtems/testsuites/samples/fileio/system.h:1.16 rtems/testsuites/samples/fileio/system.h:1.17
--- rtems/testsuites/samples/fileio/system.h:1.16 Wed Aug 12 15:50:37 2009
+++ rtems/testsuites/samples/fileio/system.h Thu May 5 11:45:40 2011
@@ -14,6 +14,7 @@
*/
#include <rtems.h>
+#include "tmacros.h"
/* functions */
@@ -63,56 +64,4 @@
#define CONFIGURE_UNIFIED_WORK_AREAS
#include <rtems/confdefs.h>
-/*
- * Handy macros and static inline functions
- */
-
-/*
- * Macro to hide the ugliness of printing the time.
- */
-
-#define print_time(_s1, _tb, _s2) \
- do { \
- printf( "%s%02d:%02d:%02d %02d/%02d/%04d%s", \
- _s1, (_tb)->hour, (_tb)->minute, (_tb)->second, \
- (_tb)->month, (_tb)->day, (_tb)->year, _s2 ); \
- fflush(stdout); \
- } while ( 0 )
-
-/*
- * Macro to print an task name that is composed of ASCII characters.
- *
- */
-
-#define put_name( _name, _crlf ) \
- do { \
- uint32_t c0, c1, c2, c3; \
- \
- c0 = ((_name) >> 24) & 0xff; \
- c1 = ((_name) >> 16) & 0xff; \
- c2 = ((_name) >> 8) & 0xff; \
- c3 = (_name) & 0xff; \
- putchar( (char)c0 ); \
- if ( c1 ) putchar( (char)c1 ); \
- if ( c2 ) putchar( (char)c2 ); \
- if ( c3 ) putchar( (char)c3 ); \
- if ( (_crlf) ) \
- putchar( '\n' ); \
- } while (0)
-
-/*
- * This allows us to view the "Test_task" instantiations as a set
- * of numbered tasks by eliminating the number of application
- * tasks created.
- *
- * In reality, this is too complex for the purposes of this
- * example. It would have been easier to pass a task argument. :)
- * But it shows how rtems_id's can sometimes be used.
- */
-
-#define task_number( tid ) \
- ( rtems_object_id_get_index( tid ) - \
- rtems_configuration_get_rtems_api_configuration()-> \
- number_of_initialization_tasks )
-
/* end of include file */
diff -u rtems/testsuites/samples/nsecs/init.c:1.14 rtems/testsuites/samples/nsecs/init.c:1.15
--- rtems/testsuites/samples/nsecs/init.c:1.14 Tue Feb 22 04:54:32 2011
+++ rtems/testsuites/samples/nsecs/init.c Thu May 5 11:45:40 2011
@@ -2,7 +2,7 @@
* Nanoseconds accuracy timestamp test
*/
-/* COPYRIGHT (c) 1989-2007.
+/* COPYRIGHT (c) 1989-2011.
* On-Line Applications Research Corporation (OAR).
*
* The license and distribution terms for this file may be
@@ -69,6 +69,7 @@
time.ticks = 0;
status = rtems_clock_set( &time );
+ directive_failed( status, "clock set" );
/*
* Iterate 10 times showing difference in TOD
@@ -91,7 +92,7 @@
my_ctime(stop.tv_sec), stop.tv_nsec
);
- printf( " --> %" PRItime_t ":%ld\n", diff.tv_sec, diff.tv_nsec );
+ printf( " --> %" PRItime_t ":%ld\n", diff.tv_sec, diff.tv_nsec );
}
/*
diff -u rtems/testsuites/samples/ticker/init.c:1.14 rtems/testsuites/samples/ticker/init.c:1.15
--- rtems/testsuites/samples/ticker/init.c:1.14 Tue Feb 22 04:54:32 2011
+++ rtems/testsuites/samples/ticker/init.c Thu May 5 11:45:40 2011
@@ -10,7 +10,7 @@
*
* Output parameters: NONE
*
- * COPYRIGHT (c) 1989-1999.
+ * COPYRIGHT (c) 1989-2011.
* On-Line Applications Research Corporation (OAR).
*
* The license and distribution terms for this file may be
@@ -52,6 +52,7 @@
time.ticks = 0;
status = rtems_clock_set( &time );
+ directive_failed( status, "clock get" );
Task_name[ 1 ] = rtems_build_name( 'T', 'A', '1', ' ' );
Task_name[ 2 ] = rtems_build_name( 'T', 'A', '2', ' ' );
@@ -61,18 +62,29 @@
Task_name[ 1 ], 1, RTEMS_MINIMUM_STACK_SIZE * 2, RTEMS_DEFAULT_MODES,
RTEMS_DEFAULT_ATTRIBUTES, &Task_id[ 1 ]
);
+ directive_failed( status, "create 1" );
+
status = rtems_task_create(
Task_name[ 2 ], 1, RTEMS_MINIMUM_STACK_SIZE * 2, RTEMS_DEFAULT_MODES,
RTEMS_DEFAULT_ATTRIBUTES, &Task_id[ 2 ]
);
+ directive_failed( status, "create 2" );
+
status = rtems_task_create(
Task_name[ 3 ], 1, RTEMS_MINIMUM_STACK_SIZE * 2, RTEMS_DEFAULT_MODES,
RTEMS_DEFAULT_ATTRIBUTES, &Task_id[ 3 ]
);
+ directive_failed( status, "create 3" );
status = rtems_task_start( Task_id[ 1 ], Test_task, 1 );
+ directive_failed( status, "start 1" );
+
status = rtems_task_start( Task_id[ 2 ], Test_task, 2 );
+ directive_failed( status, "start 2" );
+
status = rtems_task_start( Task_id[ 3 ], Test_task, 3 );
+ directive_failed( status, "start 3" );
status = rtems_task_delete( RTEMS_SELF );
+ directive_failed( status, "delete" );
}
diff -u rtems/testsuites/samples/ticker/system.h:1.23 rtems/testsuites/samples/ticker/system.h:1.24
--- rtems/testsuites/samples/ticker/system.h:1.23 Wed Aug 12 15:50:38 2009
+++ rtems/testsuites/samples/ticker/system.h Thu May 5 11:45:40 2011
@@ -16,6 +16,7 @@
#include <rtems.h>
#include <inttypes.h>
#include "../../support/include/buffer_test_io.h"
+#include "tmacros.h"
/* functions */
@@ -53,42 +54,6 @@
#include <rtems/confdefs.h>
/*
- * Handy macros and static inline functions
- */
-
-/*
- * Macro to hide the ugliness of printing the time.
- */
-
-#define print_time(_s1, _tb, _s2) \
- do { \
- printf( "%s%02" PRId32 ":%02" PRId32 ":%02" PRId32 " %02" PRId32 "/%02" PRId32 "/%04" PRId32 "%s", \
- _s1, (_tb)->hour, (_tb)->minute, (_tb)->second, \
- (_tb)->month, (_tb)->day, (_tb)->year, _s2 ); \
- } while ( 0 )
-
-/*
- * Macro to print an task name that is composed of ASCII characters.
- *
- */
-
-#define put_name( _name, _crlf ) \
- do { \
- uint32_t c0, c1, c2, c3; \
- \
- c0 = ((_name) >> 24) & 0xff; \
- c1 = ((_name) >> 16) & 0xff; \
- c2 = ((_name) >> 8) & 0xff; \
- c3 = (_name) & 0xff; \
- putchar( (char)c0 ); \
- if ( c1 ) putchar( (char)c1 ); \
- if ( c2 ) putchar( (char)c2 ); \
- if ( c3 ) putchar( (char)c3 ); \
- if ( (_crlf) ) \
- putchar( '\n' ); \
- } while (0)
-
-/*
* This allows us to view the "Test_task" instantiations as a set
* of numbered tasks by eliminating the number of application
* tasks created.
diff -u rtems/testsuites/samples/ticker/tasks.c:1.15 rtems/testsuites/samples/ticker/tasks.c:1.16
--- rtems/testsuites/samples/ticker/tasks.c:1.15 Tue Feb 22 04:54:32 2011
+++ rtems/testsuites/samples/ticker/tasks.c Thu May 5 11:45:40 2011
@@ -29,10 +29,12 @@
{
rtems_id tid;
rtems_time_of_day time;
- uint32_t task_index;
+ uint32_t task_index;
rtems_status_code status;
status = rtems_task_ident( RTEMS_SELF, RTEMS_SEARCH_ALL_NODES, &tid );
+ directive_failed( status, "task ident" );
+
task_index = task_number( tid );
for ( ; ; ) {
status = rtems_clock_get_tod( &time );
@@ -43,6 +45,8 @@
put_name( Task_name[ task_index ], FALSE );
print_time( " - rtems_clock_get_tod - ", &time, "\n" );
status = rtems_task_wake_after(
- task_index * 5 * rtems_clock_get_ticks_per_second() );
+ task_index * 5 * rtems_clock_get_ticks_per_second()
+ );
+ directive_failed( status, "wake after" );
}
}
*joel*:
2011-05-05 Joel Sherrill <joel.sherrill at oarcorp.com>
* devnullfatal01/testcase.h, malloc02/init.c, malloc04/init.c,
malloctest/init.c, termios03/termios_testdriver_polled.c,
termios04/termios_testdriver_intr.c: Remove warnings.
M 1.265 testsuites/libtests/ChangeLog
M 1.2 testsuites/libtests/devnullfatal01/testcase.h
M 1.4 testsuites/libtests/malloc02/init.c
M 1.5 testsuites/libtests/malloc04/init.c
M 1.46 testsuites/libtests/malloctest/init.c
M 1.4 testsuites/libtests/termios03/termios_testdriver_polled.c
M 1.4 testsuites/libtests/termios04/termios_testdriver_intr.c
diff -u rtems/testsuites/libtests/ChangeLog:1.264 rtems/testsuites/libtests/ChangeLog:1.265
--- rtems/testsuites/libtests/ChangeLog:1.264 Tue Apr 5 04:19:53 2011
+++ rtems/testsuites/libtests/ChangeLog Thu May 5 11:45:47 2011
@@ -1,3 +1,9 @@
+2011-05-05 Joel Sherrill <joel.sherrill at oarcorp.com>
+
+ * devnullfatal01/testcase.h, malloc02/init.c, malloc04/init.c,
+ malloctest/init.c, termios03/termios_testdriver_polled.c,
+ termios04/termios_testdriver_intr.c: Remove warnings.
+
2011-04-05 Sebastian Huber <sebastian.huber at embedded-brains.de>
* tar01/init.c: Fixed integer types.
diff -u rtems/testsuites/libtests/devnullfatal01/testcase.h:1.1 rtems/testsuites/libtests/devnullfatal01/testcase.h:1.2
--- rtems/testsuites/libtests/devnullfatal01/testcase.h:1.1 Tue Jul 27 08:57:17 2010
+++ rtems/testsuites/libtests/devnullfatal01/testcase.h Thu May 5 11:45:47 2011
@@ -16,12 +16,14 @@
#define FATAL_ERROR_EXPECTED_ERROR RTEMS_TOO_MANY
#include <rtems/devnull.h>
+#include "tmacros.h"
void force_error()
{
int status;
status = rtems_io_register_name( "/dev/null", 0, 0 );
+ directive_failed( status, "io register" );
status = null_initialize( 0, 0, NULL );
diff -u rtems/testsuites/libtests/malloc02/init.c:1.3 rtems/testsuites/libtests/malloc02/init.c:1.4
--- rtems/testsuites/libtests/malloc02/init.c:1.3 Tue Feb 22 01:34:07 2011
+++ rtems/testsuites/libtests/malloc02/init.c Thu May 5 11:45:47 2011
@@ -69,6 +69,7 @@
puts( "Free from ISR successfully processed" );
puts( "Now malloc'ing more memory to process the free" );
pointer2 = malloc(20);
+ rtems_test_assert( pointer2 );
puts( "*** END OF TEST MALLOC 02 ***" );
rtems_test_exit( 0 );
diff -u rtems/testsuites/libtests/malloc04/init.c:1.4 rtems/testsuites/libtests/malloc04/init.c:1.5
--- rtems/testsuites/libtests/malloc04/init.c:1.4 Tue Feb 22 01:34:07 2011
+++ rtems/testsuites/libtests/malloc04/init.c Thu May 5 11:45:47 2011
@@ -1,5 +1,5 @@
/*
- * COPYRIGHT (c) 1989-2010.
+ * COPYRIGHT (c) 1989-2011.
* On-Line Applications Research Corporation (OAR).
*
* The license and distribution terms for this file may be
@@ -111,15 +111,23 @@
sbrk_count = 0;
RTEMS_Malloc_Initialize( NULL, 0, 64 );
p1 = malloc(64);
+ rtems_test_assert( p1 );
+
p2 = malloc(64);
+ rtems_test_assert( p2 );
+
p3 = malloc(48);
+ rtems_test_assert( p3 );
+
p4 = malloc(48);
+ rtems_test_assert( p4 );
puts( "Initialize heap with no memory (sbrk aligned)" );
offset = 0;
sbrk_count = 0;
RTEMS_Malloc_Initialize( NULL, 0, 64 );
p1 = malloc(64);
+ rtems_test_assert( p1 );
puts( "Set sbrk amount in heap to 0" );
offset = 0;
@@ -127,6 +135,7 @@
RTEMS_Malloc_Initialize( NULL, 0, 64 );
RTEMS_Malloc_Sbrk_amount = 0;
p4 = malloc(48);
+ rtems_test_assert( p4 );
/* Restore information on real heap */
malloc_set_heap_pointer( TempHeap );
diff -u rtems/testsuites/libtests/malloctest/init.c:1.45 rtems/testsuites/libtests/malloctest/init.c:1.46
--- rtems/testsuites/libtests/malloctest/init.c:1.45 Wed Mar 9 09:39:58 2011
+++ rtems/testsuites/libtests/malloctest/init.c Thu May 5 11:45:47 2011
@@ -11,7 +11,7 @@
*
* Output parameters: NONE
*
- * COPYRIGHT (c) 1989-2009.
+ * COPYRIGHT (c) 1989-2011.
* On-Line Applications Research Corporation (OAR).
*
* Copyright (c) 2009, 2010 embedded brains GmbH.
@@ -420,15 +420,11 @@
uintptr_t previous_last_block_begin = 0;
uintptr_t previous_last_page_begin = 0;
- uintptr_t first_block_begin = 0;
-
uintptr_t last_block_begin = 0;
uintptr_t last_alloc_begin = 0;
test_heap_init( TEST_DEFAULT_PAGE_SIZE );
- first_block_begin = (uintptr_t) TestHeap.first_block;
-
last_block_begin = (uintptr_t) TestHeap.last_block;
last_alloc_begin = _Heap_Alloc_area_of_block( TestHeap.last_block );
@@ -467,6 +463,8 @@
alloc_size = test_page_size();
p1 = test_init_and_alloc_simple( alloc_size, alignment, boundary );
p2 = test_alloc_simple( alloc_size, alignment, boundary );
+ rtems_test_assert( p2 );
+
test_free( p1 );
alloc_size = 2 * alloc_size;
@@ -890,7 +888,11 @@
puts( "\t\tlet the next block be used alredy and try to get a size bigger than the actual block" );
test_heap_init( TEST_DEFAULT_PAGE_SIZE );
p1 = test_alloc_one_page();
+ rtems_test_assert( p1 );
+
p2 = test_alloc_one_page();
+ rtems_test_assert( p2 );
+
new_alloc_size = 3 * TEST_DEFAULT_PAGE_SIZE / 2;
test_simple_resize_block( p1, new_alloc_size, HEAP_RESIZE_UNSATISFIED );
@@ -903,8 +905,14 @@
puts( "\t\tlet the block after the next be used and try to allocate more then one pagesize more" );
test_heap_init( TEST_DEFAULT_PAGE_SIZE );
p1 = test_alloc_one_page();
+ rtems_test_assert( p1 );
+
p2 = test_alloc_one_page();
+ rtems_test_assert( p2 );
+
p3 = test_alloc_one_page();
+ rtems_test_assert( p3 );
+
_Heap_Free( &TestHeap, p2 );
new_alloc_size = 5 * TEST_DEFAULT_PAGE_SIZE / 2;
test_simple_resize_block( p1, new_alloc_size, HEAP_RESIZE_UNSATISFIED );
diff -u rtems/testsuites/libtests/termios03/termios_testdriver_polled.c:1.3 rtems/testsuites/libtests/termios03/termios_testdriver_polled.c:1.4
--- rtems/testsuites/libtests/termios03/termios_testdriver_polled.c:1.3 Tue Feb 22 01:34:08 2011
+++ rtems/testsuites/libtests/termios03/termios_testdriver_polled.c Thu May 5 11:45:47 2011
@@ -1,7 +1,7 @@
/*
* This file contains a test fixture termios device driver
*
- * COPYRIGHT (c) 1989-2010.
+ * COPYRIGHT (c) 1989-2011.
* On-Line Applications Research Corporation (OAR).
*
* The license and distribution terms for this file may be
@@ -134,6 +134,7 @@
}
sc = rtems_termios_open (major, minor, arg, &Callbacks);
+ directive_failed( sc, "termios open" );
return RTEMS_SUCCESSFUL;
}
diff -u rtems/testsuites/libtests/termios04/termios_testdriver_intr.c:1.3 rtems/testsuites/libtests/termios04/termios_testdriver_intr.c:1.4
--- rtems/testsuites/libtests/termios04/termios_testdriver_intr.c:1.3 Tue Feb 22 01:34:08 2011
+++ rtems/testsuites/libtests/termios04/termios_testdriver_intr.c Thu May 5 11:45:47 2011
@@ -1,7 +1,7 @@
/*
* This file contains a test fixture termios device driver
*
- * COPYRIGHT (c) 1989-2010.
+ * COPYRIGHT (c) 1989-2011.
* On-Line Applications Research Corporation (OAR).
*
* The license and distribution terms for this file may be
@@ -205,6 +205,7 @@
}
sc = rtems_termios_open (major, minor, arg, &Callbacks);
+ directive_failed( sc, "termios open" );
Ttyp = args->iop->data1; /* Keep cookie returned by termios_open */
*joel*:
2011-05-05 Joel Sherrill <joel.sherrill at oarcorp.com>
* psx04/task3.c, psxfile01/test.c, psxhdrs/Makefile.am,
psxmsgq01/init.c, psxreaddir/test.c, psxsignal01/init.c,
psxtimes01/init.c, psxualarm/init.c: Remove warnings.
M 1.338 testsuites/psxtests/ChangeLog
M 1.13 testsuites/psxtests/psx04/task3.c
M 1.32 testsuites/psxtests/psxfile01/test.c
M 1.18 testsuites/psxtests/psxhdrs/Makefile.am
M 1.26 testsuites/psxtests/psxmsgq01/init.c
M 1.21 testsuites/psxtests/psxreaddir/test.c
M 1.11 testsuites/psxtests/psxsignal01/init.c
M 1.4 testsuites/psxtests/psxtimes01/init.c
M 1.7 testsuites/psxtests/psxualarm/init.c
diff -u rtems/testsuites/psxtests/ChangeLog:1.337 rtems/testsuites/psxtests/ChangeLog:1.338
--- rtems/testsuites/psxtests/ChangeLog:1.337 Fri Apr 15 09:21:19 2011
+++ rtems/testsuites/psxtests/ChangeLog Thu May 5 11:45:51 2011
@@ -1,3 +1,9 @@
+2011-05-05 Joel Sherrill <joel.sherrill at oarcorp.com>
+
+ * psx04/task3.c, psxfile01/test.c, psxhdrs/Makefile.am,
+ psxmsgq01/init.c, psxreaddir/test.c, psxsignal01/init.c,
+ psxtimes01/init.c, psxualarm/init.c: Remove warnings.
+
2011-04-15 Sebastian Huber <Sebastian.Huber at embedded-brains.de>
* psxstat/test.c: Fixed integer type. Check status codes.
diff -u rtems/testsuites/psxtests/psx04/task3.c:1.12 rtems/testsuites/psxtests/psx04/task3.c:1.13
--- rtems/testsuites/psxtests/psx04/task3.c:1.12 Tue Feb 22 03:47:31 2011
+++ rtems/testsuites/psxtests/psx04/task3.c Thu May 5 11:45:52 2011
@@ -7,7 +7,7 @@
*
* Output parameters: NONE
*
- * COPYRIGHT (c) 1989-2009.
+ * COPYRIGHT (c) 1989-2011.
* On-Line Applications Research Corporation (OAR).
*
* The license and distribution terms for this file may be
@@ -112,6 +112,7 @@
printf( "Task_3: sleep so the Init task can reguest a signal\n" );
remaining = sleep( 1 );
rtems_test_assert( !status );
+ rtems_test_assert( remaining == 0 );
/* end of task 3 */
printf( "Task_3: exit\n" );
diff -u rtems/testsuites/psxtests/psxfile01/test.c:1.31 rtems/testsuites/psxtests/psxfile01/test.c:1.32
--- rtems/testsuites/psxtests/psxfile01/test.c:1.31 Tue Feb 22 03:47:32 2011
+++ rtems/testsuites/psxtests/psxfile01/test.c Thu May 5 11:45:52 2011
@@ -10,7 +10,7 @@
* /dev
* /dev/XXX [where XXX includes at least console]
*
- * COPYRIGHT (c) 1989-2010.
+ * COPYRIGHT (c) 1989-2011.
* On-Line Applications Research Corporation (OAR).
*
* The license and distribution terms for this file may be
@@ -162,6 +162,7 @@
build_time( &time, 12, 31, 1988, 9, 0, 0, 0 );
rtems_status = rtems_clock_set( &time );
+ directive_failed( rtems_status, "clock set" );
/*
* Dump an empty file system
@@ -656,6 +657,7 @@
rtems_test_assert( status == -1);
rtems_status = rtems_io_register_name( "/dev/console", 0, 0 );
+ directive_failed( rtems_status, "io register" );
test_case_reopen_append();
diff -u rtems/testsuites/psxtests/psxhdrs/Makefile.am:1.17 rtems/testsuites/psxtests/psxhdrs/Makefile.am:1.18
--- rtems/testsuites/psxtests/psxhdrs/Makefile.am:1.17 Mon Oct 4 10:53:45 2010
+++ rtems/testsuites/psxtests/psxhdrs/Makefile.am Thu May 5 11:45:52 2011
@@ -30,4 +30,5 @@
include $(top_srcdir)/../automake/compile.am
include $(top_srcdir)/../automake/leaf.am
+AM_CPPFLAGS += -Wno-unused-but-set-variable
include $(top_srcdir)/../automake/local.am
diff -u rtems/testsuites/psxtests/psxmsgq01/init.c:1.25 rtems/testsuites/psxtests/psxmsgq01/init.c:1.26
--- rtems/testsuites/psxtests/psxmsgq01/init.c:1.25 Tue Feb 22 03:47:35 2011
+++ rtems/testsuites/psxtests/psxmsgq01/init.c Thu May 5 11:45:52 2011
@@ -1205,6 +1205,7 @@
timeout.tv_nsec = tv1.tv_usec * 1000;
status = mq_timedsend( Test_q[que].mq, msg, len , 0, &timeout );
+ rtems_test_assert( status == 0 );
gettimeofday( &tv2, &tz2 );
tv3.tv_sec = tv2.tv_sec - tv1.tv_sec;
diff -u rtems/testsuites/psxtests/psxreaddir/test.c:1.20 rtems/testsuites/psxtests/psxreaddir/test.c:1.21
--- rtems/testsuites/psxtests/psxreaddir/test.c:1.20 Tue Feb 22 03:47:35 2011
+++ rtems/testsuites/psxtests/psxreaddir/test.c Thu May 5 11:45:52 2011
@@ -15,7 +15,7 @@
* implementation of this appears to seek to the ((off/DIRENT_SIZE) + 1)
* record where DIRENT_SIZE seems to be 12 bytes.
*
- * COPYRIGHT (c) 1989-2009.
+ * COPYRIGHT (c) 1989-2011.
* On-Line Applications Research Corporation (OAR).
*
* The license and distribution terms for this file may be
@@ -79,8 +79,10 @@
the_dir = opendir( path );
rtems_test_assert( the_dir );
+
printdir( the_dir );
status = closedir( the_dir );
+ rtems_test_assert( status );
}
char *many_files[] = {
@@ -145,13 +147,13 @@
"END"
};
-int select1 ( struct dirent *entry )
+int select1 ( const struct dirent *entry )
{
printf("SCANDIR SELECT1 accepts nodename: %s\n", entry->d_name );
return 1;
}
-int select2 ( struct dirent *entry )
+int select2 ( const struct dirent *entry )
{
if( strcmp( entry->d_name, "y") == 0 ) {
printf("SCANDIR SELECT accepted nodename: %s\n", entry->d_name );
@@ -273,6 +275,7 @@
directory_not = opendir( "/many" );
printdir ( directory_not );
d_not = readdir( directory_not );
+ rtems_test_assert( d_not == 0 );
printf("open /b/myfile\n");
fd = open ("/b/my_file", O_CREAT, S_IRWXU);
diff -u rtems/testsuites/psxtests/psxsignal01/init.c:1.10 rtems/testsuites/psxtests/psxsignal01/init.c:1.11
--- rtems/testsuites/psxtests/psxsignal01/init.c:1.10 Tue Feb 22 03:47:35 2011
+++ rtems/testsuites/psxtests/psxsignal01/init.c Thu May 5 11:45:52 2011
@@ -77,7 +77,9 @@
)
{
int status;
+
status = pthread_kill( pthread_self(), SIGUSR1 );
+ rtems_test_assert( status == 0 );
}
diff -u rtems/testsuites/psxtests/psxtimes01/init.c:1.3 rtems/testsuites/psxtests/psxtimes01/init.c:1.4
--- rtems/testsuites/psxtests/psxtimes01/init.c:1.3 Tue Feb 22 03:47:36 2011
+++ rtems/testsuites/psxtests/psxtimes01/init.c Thu May 5 11:45:52 2011
@@ -1,5 +1,5 @@
/*
- * COPYRIGHT (c) 1989-2010.
+ * COPYRIGHT (c) 1989-2011.
* On-Line Applications Research Corporation (OAR).
*
* The license and distribution terms for this file may be
@@ -57,6 +57,7 @@
puts( "_times( &start_tm ) -- OK" );
now = _times( &start_tm );
rtems_test_assert( start != 0 );
+ rtems_test_assert( now != 0 );
rtems_test_spin_for_ticks(5);
diff -u rtems/testsuites/psxtests/psxualarm/init.c:1.6 rtems/testsuites/psxtests/psxualarm/init.c:1.7
--- rtems/testsuites/psxtests/psxualarm/init.c:1.6 Tue Feb 22 03:47:36 2011
+++ rtems/testsuites/psxtests/psxualarm/init.c Thu May 5 11:45:52 2011
@@ -1,5 +1,5 @@
/*
- * COPYRIGHT (c) 1989-2009.
+ * COPYRIGHT (c) 1989-2011.
* On-Line Applications Research Corporation (OAR).
*
* The license and distribution terms for this file may be
@@ -45,7 +45,9 @@
)
{
int status;
+
status = kill( getpid(), SIGUSR1 );
+ rtems_test_assert( status == 0 );
}
@@ -79,7 +81,10 @@
puts( "Init: ualarm in 1 us" );
sleep(3);
result = ualarm(1,0);
+ rtems_test_assert( result == 0 );
+
status = sleep(10);
+ rtems_test_assert( status == 0 );
/* unblock Signal and see if it happened */
status = sigemptyset( &mask );
*joel*:
2011-05-05 Joel Sherrill <joel.sherrill at oarcorp.com>
* sp09/screen12.c, sp09/sp09.scn, sp21/Makefile.am, sp35/priinv.c,
sp39/init.c, sp50/init.c, sp57/init.c, sp72/init.c, sp73/init.c,
spintrcritical01/init.c, spprivenv01/init.c, spsimplesched01/init.c,
spsimplesched02/init.c: Remove warnings.
M 1.448 testsuites/sptests/ChangeLog
M 1.25 testsuites/sptests/sp09/screen12.c
M 1.28 testsuites/sptests/sp09/sp09.scn
M 1.26 testsuites/sptests/sp21/Makefile.am
M 1.18 testsuites/sptests/sp35/priinv.c
M 1.12 testsuites/sptests/sp39/init.c
M 1.6 testsuites/sptests/sp50/init.c
M 1.4 testsuites/sptests/sp57/init.c
M 1.3 testsuites/sptests/sp72/init.c
M 1.2 testsuites/sptests/sp73/init.c
M 1.7 testsuites/sptests/spintrcritical01/init.c
M 1.4 testsuites/sptests/spprivenv01/init.c
M 1.2 testsuites/sptests/spsimplesched01/init.c
M 1.2 testsuites/sptests/spsimplesched02/init.c
diff -u rtems/testsuites/sptests/ChangeLog:1.447 rtems/testsuites/sptests/ChangeLog:1.448
--- rtems/testsuites/sptests/ChangeLog:1.447 Fri Apr 15 08:21:09 2011
+++ rtems/testsuites/sptests/ChangeLog Thu May 5 11:45:57 2011
@@ -1,3 +1,10 @@
+2011-05-05 Joel Sherrill <joel.sherrill at oarcorp.com>
+
+ * sp09/screen12.c, sp09/sp09.scn, sp21/Makefile.am, sp35/priinv.c,
+ sp39/init.c, sp50/init.c, sp57/init.c, sp72/init.c, sp73/init.c,
+ spintrcritical01/init.c, spprivenv01/init.c, spsimplesched01/init.c,
+ spsimplesched02/init.c: Remove warnings.
+
2011-04-15 Sebastian Huber <sebastian.huber at embedded-brains.de>
* sp09/screen02.c: Use integer value that fits into one character.
diff -u rtems/testsuites/sptests/sp09/screen12.c:1.24 rtems/testsuites/sptests/sp09/screen12.c:1.25
--- rtems/testsuites/sptests/sp09/screen12.c:1.24 Tue Feb 22 01:36:58 2011
+++ rtems/testsuites/sptests/sp09/screen12.c Thu May 5 11:45:57 2011
@@ -6,7 +6,7 @@
*
* Output parameters: NONE
*
- * COPYRIGHT (c) 1989-2009.
+ * COPYRIGHT (c) 1989-2011.
* On-Line Applications Research Corporation (OAR).
*
* The license and distribution terms for this file may be
@@ -27,9 +27,6 @@
void *segment_address_1;
void *segment_address_2;
void *segment_address_3;
- uint32_t good_back_flag;
- uint32_t good_front_flag;
- uint32_t offset;
uintptr_t segment_size;
rtems_status_code status;
Heap_Information_block the_info;
@@ -400,48 +397,6 @@
puts( "TA1 - rtems_debug_disable - RTEMS_DEBUG_REGION" );
rtems_debug_disable( RTEMS_DEBUG_REGION );
-#if 0
-
- offset = (segment_address_1 - (void *)Region_good_area) / 4;
-
-/* bad FRONT_FLAG error */
-
- good_front_flag = Region_good_area[ offset - 1 ];
- Region_good_area[ offset - 1 ] = good_front_flag + 2;
-
- status = rtems_region_return_segment( Region_id[ 1 ], segment_address_1 );
- fatal_directive_status(
- status,
- RTEMS_INVALID_ADDRESS,
- "rtems_region_return_segment with back_flag != front_flag"
- );
- puts( "TA1 - rtems_region_return_segment - RTEMS_INVALID_ADDRESS" );
-
- Region_good_area[ offset - 1 ] = good_front_flag;
-
-/* bad FRONT_FLAG error */
-
- good_back_flag = Region_good_area[ offset - 2 ];
- Region_good_area[ offset - 2 ] = 1024;
-
- status = rtems_region_return_segment( Region_id[ 1 ], segment_address_1 );
- fatal_directive_status(
- status,
- RTEMS_INVALID_ADDRESS,
- "rtems_region_return_segment with back_flag != front_flag"
- );
- puts( "TA1 - rtems_region_return_segment - RTEMS_INVALID_ADDRESS" );
-
- Region_good_area[ offset - 2 ] = good_back_flag;
-
-#else
- offset = 0;
- good_front_flag = 0;
- good_back_flag = 0;
- puts( "TA1 - rtems_region_return_segment - RTEMS_INVALID_ADDRESS - SKIPPED" );
- puts( "TA1 - rtems_region_return_segment - RTEMS_INVALID_ADDRESS - SKIPPED" );
-#endif
-
puts( "TA1 - rtems_debug_enable - RTEMS_DEBUG_REGION" );
rtems_debug_enable( RTEMS_DEBUG_REGION );
diff -u rtems/testsuites/sptests/sp09/sp09.scn:1.27 rtems/testsuites/sptests/sp09/sp09.scn:1.28
--- rtems/testsuites/sptests/sp09/sp09.scn:1.27 Fri Mar 11 14:26:59 2011
+++ rtems/testsuites/sptests/sp09/sp09.scn Thu May 5 11:45:57 2011
@@ -273,8 +273,6 @@
TA1 - rtems_region_return_segment - RTEMS_INVALID_ID
TA1 - rtems_region_return_segment - RTEMS_INVALID_ADDRESS
TA1 - rtems_debug_disable - RTEMS_DEBUG_REGION
-TA1 - rtems_region_return_segment - RTEMS_INVALID_ADDRESS - SKIPPED
-TA1 - rtems_region_return_segment - RTEMS_INVALID_ADDRESS - SKIPPED
TA1 - rtems_debug_enable - RTEMS_DEBUG_REGION
TA1 - rtems_region_extend - RTEMS_INVALID_ID
TA1 - rtems_region_extend - within heap - RTEMS_INVALID_ADDRESS
diff -u rtems/testsuites/sptests/sp21/Makefile.am:1.25 rtems/testsuites/sptests/sp21/Makefile.am:1.26
--- rtems/testsuites/sptests/sp21/Makefile.am:1.25 Tue Jul 6 17:27:23 2010
+++ rtems/testsuites/sptests/sp21/Makefile.am Thu May 5 11:45:57 2011
@@ -13,6 +13,7 @@
include $(top_srcdir)/../automake/leaf.am
AM_CPPFLAGS += -I$(top_srcdir)/../support/include
+AM_CPPFLAGS += -Wno-deprecated-declarations
LINK_OBJS = $(sp21_OBJECTS) $(sp21_LDADD)
LINK_LIBS = $(sp21_LDLIBS)
diff -u rtems/testsuites/sptests/sp35/priinv.c:1.17 rtems/testsuites/sptests/sp35/priinv.c:1.18
--- rtems/testsuites/sptests/sp35/priinv.c:1.17 Tue Feb 22 01:37:00 2011
+++ rtems/testsuites/sptests/sp35/priinv.c Thu May 5 11:45:57 2011
@@ -243,17 +243,20 @@
{
rtems_status_code Sts;
- rtems_task_priority EnterPrio; /* Statistics log */
+#if defined(TEST_PRINT_STATISTICS)
rtems_task_priority AccessPrio; /* : */
- rtems_task_priority LeavePrio; /* : */
- uint32_t EnterCnt; /* : */
uint32_t AccessCnt; /* : */
+ rtems_task_priority EnterPrio; /* Statistics log */
+ uint32_t EnterCnt; /* : */
+ rtems_task_priority LeavePrio; /* : */
uint32_t LeaveCnt; /* : */
+#endif
+#if defined(TEST_PRINT_STATISTICS)
/* Store information about the current situation */
EnterPrio = _Thread_Executing->current_priority;
EnterCnt = _Thread_Executing->resource_count;
-
+#endif
printf(" AccessLocalHw called by %s\n", CallerName());
@@ -269,18 +272,20 @@
Sts = rtems_semaphore_obtain(LocalHwSync_S, RTEMS_WAIT, RTEMS_NO_TIMEOUT);
directive_failed( Sts, "rtems_semaphore_obtain(LocalHwAccess_R...)" );
+#if defined(TEST_PRINT_STATISTICS)
/* Store information about the current situation */
AccessPrio = _Thread_Executing->current_priority;
AccessCnt = _Thread_Executing->resource_count;
+#endif
Sts = rtems_semaphore_release(LocalHwAccess_R);
directive_failed( Sts, "rtems_semaphore_release(LocalHwAccess_R)" );
+#if defined(TEST_PRINT_STATISTICS)
/* Store information about the current situation */
LeavePrio = _Thread_Executing->current_priority;
LeaveCnt = _Thread_Executing->resource_count;
-#if defined(TEST_PRINT_STATISTICS)
printf(
" AccessLocalHw from %s statistics:\n"
" - Prio: %d -> %d -> %d\n - Cnt: %d -> %d -> %d\n",
@@ -304,17 +309,20 @@
{
rtems_status_code Sts;
+#if defined(TEST_PRINT_STATISTICS)
rtems_task_priority EnterPrio; /* Statistics log */
rtems_task_priority AccessPrio; /* : */
rtems_task_priority LeavePrio; /* : */
uint32_t EnterCnt; /* : */
uint32_t AccessCnt; /* : */
uint32_t LeaveCnt; /* : */
+#endif
+#if defined(TEST_PRINT_STATISTICS)
/* Store information about the current situation */
EnterPrio = _Thread_Executing->current_priority;
EnterCnt = _Thread_Executing->resource_count;
-
+#endif
printf("AccessRemoteHw called by %s\n", CallerName());
@@ -329,18 +337,20 @@
printf("AccessRemoteHw access local %s\n", CallerName());
AccessLocalHw();
+#if defined(TEST_PRINT_STATISTICS)
/* Store information about the current situation */
AccessPrio = _Thread_Executing->current_priority;
AccessCnt = _Thread_Executing->resource_count;
+#endif
Sts = rtems_semaphore_release(RemoteHwAccess_R);
directive_failed( Sts, "rtems_semaphore_release(RemoreHwAccess_R" );
+#if defined(TEST_PRINT_STATISTICS)
/* Store information about the current situation */
LeavePrio = _Thread_Executing->current_priority;
LeaveCnt = _Thread_Executing->resource_count;
-#if defined(TEST_PRINT_STATISTICS)
printf(
"\nAccessRemoteHw from %s statistics:\n"
" - Prio: %d -> %d -> %d\n - Cnt: %d -> %d -> %d\n",
diff -u rtems/testsuites/sptests/sp39/init.c:1.11 rtems/testsuites/sptests/sp39/init.c:1.12
--- rtems/testsuites/sptests/sp39/init.c:1.11 Tue Feb 22 01:37:00 2011
+++ rtems/testsuites/sptests/sp39/init.c Thu May 5 11:45:57 2011
@@ -1,7 +1,7 @@
/*
* Classic API Signal to Task from ISR
*
- * COPYRIGHT (c) 1989-2009.
+ * COPYRIGHT (c) 1989-2011.
* On-Line Applications Research Corporation (OAR).
*
* The license and distribution terms for this file may be
@@ -44,17 +44,20 @@
* This event send hits the critical section but sends to
* another task so doesn't impact this critical section.
*/
- rtems_event_send( other_task, 0x02 );
+ status = rtems_event_send( other_task, 0x02 );
+ directive_failed( status, "event send" );
/*
* This event send hits the main task but doesn't satisfy
* it's blocking condition so it will still block
*/
- rtems_event_send( main_task, 0x02 );
+ status = rtems_event_send( main_task, 0x02 );
+ directive_failed( status, "event send" );
case_hit = TRUE;
}
status = rtems_event_send( main_task, 0x01 );
+ directive_failed( status, "event send" );
}
rtems_timer_service_routine test_event_with_timeout_from_isr(
@@ -72,6 +75,7 @@
case_hit = TRUE;
}
status = rtems_event_send( main_task, 0x01 );
+ directive_failed( status, "event send" );
}
rtems_task Init(
diff -u rtems/testsuites/sptests/sp50/init.c:1.5 rtems/testsuites/sptests/sp50/init.c:1.6
--- rtems/testsuites/sptests/sp50/init.c:1.5 Tue Feb 22 01:37:01 2011
+++ rtems/testsuites/sptests/sp50/init.c Thu May 5 11:45:57 2011
@@ -1,5 +1,5 @@
/*
- * COPYRIGHT (c) 1989-2009.
+ * COPYRIGHT (c) 1989-2011.
* On-Line Applications Research Corporation (OAR).
*
* The license and distribution terms for this file may be
@@ -20,17 +20,19 @@
rtems_timer_service_routine Timer_Routine( rtems_id id, void *ignored )
{
- rtems_status_code sc;
+ rtems_status_code status;
Fired++;
timerRan = true;
- sc = rtems_timer_server_fire_after(
+ status = rtems_timer_server_fire_after(
id,
rtems_clock_get_ticks_per_second(),
Timer_Routine,
NULL
);
+ directive_failed( status, "fire after" );
+
}
rtems_task Init(
diff -u rtems/testsuites/sptests/sp57/init.c:1.3 rtems/testsuites/sptests/sp57/init.c:1.4
--- rtems/testsuites/sptests/sp57/init.c:1.3 Tue Feb 22 01:37:01 2011
+++ rtems/testsuites/sptests/sp57/init.c Thu May 5 11:45:57 2011
@@ -1,6 +1,6 @@
/* Restart a task which is delaying
*
- * COPYRIGHT (c) 1989-2009.
+ * COPYRIGHT (c) 1989-2011.
* On-Line Applications Research Corporation (OAR).
*
* The license and distribution terms for this file may be
@@ -24,6 +24,7 @@
puts( "Delay - rtems_task_wake_after - OK" );
status = rtems_task_wake_after( RTEMS_MILLISECONDS_TO_TICKS(2000) );
+ directive_failed( status, "wake after" );
puts( "ERROR - delay task woke up!!" );
rtems_test_exit(0);
diff -u rtems/testsuites/sptests/sp72/init.c:1.2 rtems/testsuites/sptests/sp72/init.c:1.3
--- rtems/testsuites/sptests/sp72/init.c:1.2 Tue Feb 22 01:37:01 2011
+++ rtems/testsuites/sptests/sp72/init.c Thu May 5 11:45:57 2011
@@ -1,5 +1,5 @@
/*
- * COPYRIGHT (c) 1989-2010.
+ * COPYRIGHT (c) 1989-2011.
* On-Line Applications Research Corporation (OAR).
*
* The license and distribution terms for this file may be
@@ -30,6 +30,8 @@
RTEMS_DEFAULT_OPTIONS,
rtems_clock_get_ticks_per_second() * 10
);
+ directive_failed( status, "obtain" );
+
}
rtems_task Init(
diff -u rtems/testsuites/sptests/sp73/init.c:1.1 rtems/testsuites/sptests/sp73/init.c:1.2
--- rtems/testsuites/sptests/sp73/init.c:1.1 Fri Mar 11 10:39:27 2011
+++ rtems/testsuites/sptests/sp73/init.c Thu May 5 11:45:57 2011
@@ -31,6 +31,8 @@
task_index = arg;
for ( ; ; ) {
status = rtems_clock_get_tod( &time );
+ directive_failed( status, "get tod" );
+
if ( time.second >= 15 ) {
puts( "*** END OF SP73 (YIELD) TEST ***" );
rtems_test_exit( 0 );
@@ -38,11 +40,12 @@
put_name( Task_name[ task_index ], FALSE );
print_time( " - rtems_clock_get_tod - ", &time, "\n" );
status = rtems_task_wake_after(
- task_index * 5 * rtems_clock_get_ticks_per_second() );
+ task_index * 5 * rtems_clock_get_ticks_per_second()
+ );
+ directive_failed( status, "wake after" );
}
}
-
rtems_task Init(
rtems_task_argument argument
)
@@ -71,25 +74,39 @@
Task_name[ 1 ], 1, RTEMS_MINIMUM_STACK_SIZE, RTEMS_DEFAULT_MODES,
RTEMS_DEFAULT_ATTRIBUTES, &Task_id[ 1 ]
);
+ directive_failed( status, "create 1" );
+
status = rtems_task_create(
Task_name[ 2 ], 1, RTEMS_MINIMUM_STACK_SIZE, RTEMS_DEFAULT_MODES,
RTEMS_DEFAULT_ATTRIBUTES, &Task_id[ 2 ]
);
+ directive_failed( status, "create 2" );
+
status = rtems_task_create(
Task_name[ 3 ], 1, RTEMS_MINIMUM_STACK_SIZE, RTEMS_DEFAULT_MODES,
RTEMS_DEFAULT_ATTRIBUTES, &Task_id[ 3 ]
);
+ directive_failed( status, "create 3" );
status = rtems_task_start( Task_id[ 1 ], Test_task, 1 );
+ directive_failed( status, "start 1" );
rtems_task_wake_after( RTEMS_YIELD_PROCESSOR );
+
status = rtems_task_start( Task_id[ 2 ], Test_task, 2 );
+ directive_failed( status, "start 2" );
rtems_task_wake_after( RTEMS_YIELD_PROCESSOR );
+
status = rtems_task_start( Task_id[ 3 ], Test_task, 3 );
+ directive_failed( status, "start 3" );
- rtems_task_set_priority(Task_id[1], 1, &old);
- rtems_task_wake_after( RTEMS_YIELD_PROCESSOR );
+ status = rtems_task_set_priority(Task_id[1], 1, &old);
+ directive_failed( status, "set priority" );
+
+ status = rtems_task_wake_after( RTEMS_YIELD_PROCESSOR );
+ directive_failed( status, "wake after" );
status = rtems_task_delete( RTEMS_SELF );
+ directive_failed( status, "delete" );
}
/* configuration information */
diff -u rtems/testsuites/sptests/spintrcritical01/init.c:1.6 rtems/testsuites/sptests/spintrcritical01/init.c:1.7
--- rtems/testsuites/sptests/spintrcritical01/init.c:1.6 Tue Feb 22 01:37:02 2011
+++ rtems/testsuites/sptests/spintrcritical01/init.c Thu May 5 11:45:57 2011
@@ -1,5 +1,5 @@
/*
- * COPYRIGHT (c) 1989-2009.
+ * COPYRIGHT (c) 1989-2011.
* On-Line Applications Research Corporation (OAR).
*
* The license and distribution terms for this file may be
@@ -82,7 +82,9 @@
if ( getState() == THREAD_BLOCKING_OPERATION_NOTHING_HAPPENED ) {
case_hit = true;
}
+
status = rtems_semaphore_release( Semaphore );
+ directive_failed( status, "release" );
}
diff -u rtems/testsuites/sptests/spprivenv01/init.c:1.3 rtems/testsuites/sptests/spprivenv01/init.c:1.4
--- rtems/testsuites/sptests/spprivenv01/init.c:1.3 Tue Feb 22 01:37:03 2011
+++ rtems/testsuites/sptests/spprivenv01/init.c Thu May 5 11:45:57 2011
@@ -1,5 +1,5 @@
/*
- * COPYRIGHT (c) 1989-2010.
+ * COPYRIGHT (c) 1989-2011.
* On-Line Applications Research Corporation (OAR).
*
* The license and distribution terms for this file may be
@@ -20,11 +20,12 @@
rtems_task task_routine( rtems_task_argument not_used )
{
- int sc = 0;
+ rtems_status_code sc;
puts( "task_routine - setting up a private environment" );
sc = rtems_libio_set_private_env();
+ directive_failed( sc, "set private env" );
sleep( 1 );
rtems_task_delete( RTEMS_SELF );
@@ -34,12 +35,12 @@
rtems_task_argument argument
)
{
- int sc = 0;
- bool status = 0;
- void *alloc_ptr = (void *)0;
- rtems_id current_task_id;
- rtems_id task_id;
- rtems_name another_task_name;
+ rtems_status_code sc;
+ bool status;
+ void *alloc_ptr;
+ rtems_id current_task_id;
+ rtems_id task_id;
+ rtems_name another_task_name;
Heap_Information_block Info;
puts( "\n\n*** TEST USER ENVIRONMENT ROUTINE - 01 ***" );
diff -u rtems/testsuites/sptests/spsimplesched01/init.c:1.1 rtems/testsuites/sptests/spsimplesched01/init.c:1.2
--- rtems/testsuites/sptests/spsimplesched01/init.c:1.1 Wed Mar 16 11:33:04 2011
+++ rtems/testsuites/sptests/spsimplesched01/init.c Thu May 5 11:45:57 2011
@@ -25,15 +25,18 @@
rtems_task_argument unused
)
{
- rtems_id tid;
- rtems_time_of_day time;
- uint32_t task_index;
- rtems_status_code status;
+ rtems_id tid;
+ rtems_time_of_day time;
+ uint32_t task_index;
+ rtems_status_code status;
status = rtems_task_ident( RTEMS_SELF, RTEMS_SEARCH_ALL_NODES, &tid );
+ directive_failed( status, "task ident" );
+
task_index = task_number( tid );
for ( ; ; ) {
status = rtems_clock_get_tod( &time );
+ directive_failed( status, "clock get tod" );
if ( time.second >= 35 ) {
puts( "*** END OF SIMPLE01 TEST ***" );
rtems_test_exit( 0 );
@@ -41,7 +44,9 @@
put_name( Task_name[ task_index ], FALSE );
print_time( " - rtems_clock_get_tod - ", &time, "\n" );
status = rtems_task_wake_after(
- task_index * 5 * rtems_clock_get_ticks_per_second() );
+ task_index * 5 * rtems_clock_get_ticks_per_second()
+ );
+ directive_failed( status, "wake after" );
}
}
@@ -73,28 +78,43 @@
Task_name[ 1 ], 1, RTEMS_MINIMUM_STACK_SIZE * 2, RTEMS_DEFAULT_MODES,
RTEMS_DEFAULT_ATTRIBUTES, &Task_id[ 1 ]
);
+ directive_failed( status, "create 1" );
+
status = rtems_task_create(
Task_name[ 2 ], 1, RTEMS_MINIMUM_STACK_SIZE * 2, RTEMS_DEFAULT_MODES,
RTEMS_DEFAULT_ATTRIBUTES, &Task_id[ 2 ]
);
+ directive_failed( status, "create 2" );
+
status = rtems_task_create(
Task_name[ 3 ], 1, RTEMS_MINIMUM_STACK_SIZE * 2, RTEMS_DEFAULT_MODES,
RTEMS_DEFAULT_ATTRIBUTES, &Task_id[ 3 ]
);
+ directive_failed( status, "create 3" );
status = rtems_task_start( Task_id[ 1 ], Test_task, 1 );
+ directive_failed( status, "start 1" );
rtems_task_wake_after( RTEMS_YIELD_PROCESSOR );
+
status = rtems_task_start( Task_id[ 2 ], Test_task, 2 );
+ directive_failed( status, "start 2" );
rtems_task_wake_after( RTEMS_YIELD_PROCESSOR );
+
status = rtems_task_start( Task_id[ 3 ], Test_task, 3 );
+ directive_failed( status, "start 3" );
- rtems_task_set_priority(Task_id[1], 2, &old);
- rtems_task_set_priority(Task_id[2], 2, &old);
- rtems_task_set_priority(Task_id[3], 2, &old);
+ status = rtems_task_set_priority( Task_id[1], 2, &old );
+ directive_failed( status, "set priority 1" );
+ status = rtems_task_set_priority( Task_id[2], 2, &old );
+ directive_failed( status, "set priority 2" );
+ status = rtems_task_set_priority( Task_id[3], 2, &old );
+ directive_failed( status, "set priority 3" );
- rtems_task_wake_after( RTEMS_YIELD_PROCESSOR );
+ status = rtems_task_wake_after( RTEMS_YIELD_PROCESSOR );
+ directive_failed( status, "yield" );
status = rtems_task_delete( RTEMS_SELF );
+ directive_failed( status, "delete self" );
}
/* configuration information */
diff -u rtems/testsuites/sptests/spsimplesched02/init.c:1.1 rtems/testsuites/sptests/spsimplesched02/init.c:1.2
--- rtems/testsuites/sptests/spsimplesched02/init.c:1.1 Wed Mar 16 11:33:04 2011
+++ rtems/testsuites/sptests/spsimplesched02/init.c Thu May 5 11:45:57 2011
@@ -28,14 +28,14 @@
)
{
rtems_id tid;
- uint32_t task_index;
rtems_status_code status;
status = rtems_task_ident( RTEMS_SELF, RTEMS_SEARCH_ALL_NODES, &tid );
- task_index = task_number( tid );
+ directive_failed( status, "wake after" );
for ( ; ; ) {
- rtems_task_wake_after( RTEMS_YIELD_PROCESSOR );
+ status = rtems_task_wake_after( RTEMS_YIELD_PROCESSOR );
+ directive_failed( status, "yield" );
}
}
*joel*:
2011-05-05 Joel Sherrill <joel.sherrill at oarcorp.com>
* tm01/task1.c, tm02/task1.c, tm03/task1.c, tm15/task1.c: Remove
warnings.
M 1.100 testsuites/tmtests/ChangeLog
M 1.20 testsuites/tmtests/tm01/task1.c
M 1.20 testsuites/tmtests/tm02/task1.c
M 1.20 testsuites/tmtests/tm03/task1.c
M 1.19 testsuites/tmtests/tm15/task1.c
diff -u rtems/testsuites/tmtests/ChangeLog:1.99 rtems/testsuites/tmtests/ChangeLog:1.100
--- rtems/testsuites/tmtests/ChangeLog:1.99 Wed Mar 16 12:11:32 2011
+++ rtems/testsuites/tmtests/ChangeLog Thu May 5 11:46:00 2011
@@ -1,3 +1,8 @@
+2011-05-05 Joel Sherrill <joel.sherrill at oarcorp.com>
+
+ * tm01/task1.c, tm02/task1.c, tm03/task1.c, tm15/task1.c: Remove
+ warnings.
+
2011-03-16 Jennifer Averett <jennifer.averett at OARcorp.com>
PR 1743/cpu
diff -u rtems/testsuites/tmtests/tm01/task1.c:1.19 rtems/testsuites/tmtests/tm01/task1.c:1.20
--- rtems/testsuites/tmtests/tm01/task1.c:1.19 Tue Feb 22 03:57:13 2011
+++ rtems/testsuites/tmtests/tm01/task1.c Thu May 5 11:46:00 2011
@@ -1,5 +1,5 @@
/*
- * COPYRIGHT (c) 1989-2009.
+ * COPYRIGHT (c) 1989-2011.
* On-Line Applications Research Corporation (OAR).
*
* The license and distribution terms for this file may be
@@ -115,6 +115,7 @@
RTEMS_NO_PRIORITY,
&smid
);
+ directive_failed( status, "rtems_task_create of TA1" );
for ( iterations=OPERATION_COUNT ; iterations ; iterations-- ) {
diff -u rtems/testsuites/tmtests/tm02/task1.c:1.19 rtems/testsuites/tmtests/tm02/task1.c:1.20
--- rtems/testsuites/tmtests/tm02/task1.c:1.19 Tue Feb 22 03:57:13 2011
+++ rtems/testsuites/tmtests/tm02/task1.c Thu May 5 11:46:00 2011
@@ -1,5 +1,5 @@
/*
- * COPYRIGHT (c) 1989-2009.
+ * COPYRIGHT (c) 1989-2011.
* On-Line Applications Research Corporation (OAR).
*
* The license and distribution terms for this file may be
@@ -121,10 +121,10 @@
rtems_task_argument argument
)
{
- rtems_status_code status;
+ /* start blocking rtems_semaphore_obtain time */
+ benchmark_timer_initialize();
- benchmark_timer_initialize(); /* start blocking rtems_semaphore_obtain time */
- status = rtems_semaphore_obtain(
+ (void) rtems_semaphore_obtain(
Semaphore_id,
RTEMS_DEFAULT_OPTIONS,
RTEMS_NO_TIMEOUT
@@ -135,9 +135,7 @@
rtems_task_argument argument
)
{
- rtems_status_code status;
-
- status = rtems_semaphore_obtain(
+ (void) rtems_semaphore_obtain(
Semaphore_id,
RTEMS_DEFAULT_OPTIONS,
RTEMS_NO_TIMEOUT
diff -u rtems/testsuites/tmtests/tm03/task1.c:1.19 rtems/testsuites/tmtests/tm03/task1.c:1.20
--- rtems/testsuites/tmtests/tm03/task1.c:1.19 Tue Feb 22 03:57:13 2011
+++ rtems/testsuites/tmtests/tm03/task1.c Thu May 5 11:46:00 2011
@@ -1,5 +1,5 @@
/*
- * COPYRIGHT (c) 1989-2009.
+ * COPYRIGHT (c) 1989-2011.
* On-Line Applications Research Corporation (OAR).
*
* The license and distribution terms for this file may be
@@ -118,24 +118,20 @@
rtems_task_argument argument
)
{
- rtems_status_code status;
-
- status = rtems_semaphore_obtain(
+ (void) rtems_semaphore_obtain(
Semaphore_id,
RTEMS_DEFAULT_OPTIONS,
RTEMS_NO_TIMEOUT
);
- status = rtems_semaphore_release( Semaphore_id );
+ (void) rtems_semaphore_release( Semaphore_id );
}
rtems_task High_task(
rtems_task_argument argument
)
{
- rtems_status_code status;
-
- status = rtems_semaphore_obtain(
+ (void) rtems_semaphore_obtain(
Semaphore_id,
RTEMS_DEFAULT_OPTIONS,
RTEMS_NO_TIMEOUT
diff -u rtems/testsuites/tmtests/tm15/task1.c:1.18 rtems/testsuites/tmtests/tm15/task1.c:1.19
--- rtems/testsuites/tmtests/tm15/task1.c:1.18 Tue Feb 22 03:57:13 2011
+++ rtems/testsuites/tmtests/tm15/task1.c Thu May 5 11:46:00 2011
@@ -1,5 +1,5 @@
/*
- * COPYRIGHT (c) 1989-2009.
+ * COPYRIGHT (c) 1989-2011.
* On-Line Applications Research Corporation (OAR).
*
* The license and distribution terms for this file may be
@@ -203,10 +203,8 @@
rtems_task_argument argument
)
{
- rtems_status_code status;
-
if ( time_set )
- status = rtems_event_receive(
+ (void) rtems_event_receive(
RTEMS_EVENT_16,
RTEMS_DEFAULT_OPTIONS,
RTEMS_NO_TIMEOUT,
@@ -214,8 +212,9 @@
);
else {
time_set = true;
- benchmark_timer_initialize(); /* start blocking rtems_event_receive time */
- status = rtems_event_receive(
+ /* start blocking rtems_event_receive time */
+ benchmark_timer_initialize();
+ (void) rtems_event_receive(
RTEMS_EVENT_16,
RTEMS_DEFAULT_OPTIONS,
RTEMS_NO_TIMEOUT,
--
Generated by Deluxe Loginfo [http://www.codewiz.org/projects/index.html#loginfo] 2.122 by Bernardo Innocenti <bernie at develer.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rtems.org/pipermail/vc/attachments/20110505/b2e4791e/attachment-0001.html>
More information about the vc
mailing list