change log for rtems (2010-08-10)

rtems-vc at rtems.org rtems-vc at rtems.org
Tue Aug 10 23:11:11 UTC 2010


 *joel*:
2010-08-09	Bharath Suri <bharath.s.jois at gmail.com>

	PR 1662/filesystem
	* libcsupport/src/termios.c: osem is now released before attempt
	to delete in rtems_termios_close.

M 1.2559  cpukit/ChangeLog
M   1.64  cpukit/libcsupport/src/termios.c

diff -u rtems/cpukit/ChangeLog:1.2558 rtems/cpukit/ChangeLog:1.2559
--- rtems/cpukit/ChangeLog:1.2558	Tue Aug 10 12:41:29 2010
+++ rtems/cpukit/ChangeLog	Tue Aug 10 17:15:32 2010
@@ -1,5 +1,11 @@
 2010-08-09	Bharath Suri <bharath.s.jois at gmail.com>
 
+	PR 1662/filesystem
+	* libcsupport/src/termios.c: osem is now released before attempt
+	to delete in rtems_termios_close.
+
+2010-08-09	Bharath Suri <bharath.s.jois at gmail.com>
+
 	PR 1660/filesystem
 	* libfs/src/pipe/fifo.c, libfs/src/pipe/pipe.h: Changed
 	int pipe_release to void pipe_release.

diff -u rtems/cpukit/libcsupport/src/termios.c:1.63 rtems/cpukit/libcsupport/src/termios.c:1.64
--- rtems/cpukit/libcsupport/src/termios.c:1.63	Mon Jul  5 16:31:56 2010
+++ rtems/cpukit/libcsupport/src/termios.c	Tue Aug 10 17:15:32 2010
@@ -382,6 +382,7 @@
         rtems_fatal_error_occurred (sc);
       }
       drainOutput (tty);
+      rtems_semaphore_release (tty->osem);
     }
 
     if (tty->device.outputUsesInterrupts == TERMIOS_TASK_DRIVEN) {


 *joel*:
2010-08-10	Bharath Suri <bharath.s.jois at gmail.com>

	PR 1661/testing
	* termios01/init.c, termios01/termios01.scn: Added a test case to
	hit a case in rtems_termios_read.

M  1.236  testsuites/libtests/ChangeLog
M   1.14  testsuites/libtests/termios01/init.c
M    1.7  testsuites/libtests/termios01/termios01.scn

diff -u rtems/testsuites/libtests/ChangeLog:1.235 rtems/testsuites/libtests/ChangeLog:1.236
--- rtems/testsuites/libtests/ChangeLog:1.235	Tue Aug 10 09:45:22 2010
+++ rtems/testsuites/libtests/ChangeLog	Tue Aug 10 17:31:48 2010
@@ -1,3 +1,9 @@
+2010-08-10	Bharath Suri <bharath.s.jois at gmail.com>
+
+	PR 1661/testing
+	* termios01/init.c, termios01/termios01.scn: Added a test case to
+	hit a case in rtems_termios_read.
+
 2010-08-10	Joel Sherrill <joel.sherrill at oarcorp.com>
 
 	PR 1664/testing

diff -u rtems/testsuites/libtests/termios01/init.c:1.13 rtems/testsuites/libtests/termios01/init.c:1.14
--- rtems/testsuites/libtests/termios01/init.c:1.13	Tue Aug 10 09:45:22 2010
+++ rtems/testsuites/libtests/termios01/init.c	Tue Aug 10 17:31:49 2010
@@ -462,6 +462,7 @@
   rtems_device_major_number registered;
   int                       test;
   struct termios            t;
+  int index = 0;
 
   puts( "\n\n*** TEST TERMIOS 01 ***" );
 
@@ -591,22 +592,16 @@
     rtems_test_exit(0);
   }
 
-  /*
-  TODO: This must be enabled, but is facing a strange problem 
-  where the code dies off at rtems_termios_open.
-  */
 
-  /*
   puts( "Multiple open of the device" );
   for( ; index < 26; ++index ) {
-    printf( "...%d ", index );
     test = open( TERMIOS_TEST_DRIVER_DEVICE_NAME, O_RDWR );
     rtems_test_assert( test != -1 );
     rc = close( test );
     rtems_test_assert( rc == 0 );
   }
   puts( "" );
-  */
+
   puts( "*** END OF TEST TERMIOS 01 ***" );
   rtems_test_exit(0);
 }

diff -u rtems/testsuites/libtests/termios01/termios01.scn:1.6 rtems/testsuites/libtests/termios01/termios01.scn:1.7
--- rtems/testsuites/libtests/termios01/termios01.scn:1.6	Mon Aug  9 09:36:47 2010
+++ rtems/testsuites/libtests/termios01/termios01.scn	Tue Aug 10 17:31:49 2010
@@ -302,4 +302,58 @@
 cfsetispeed(B460800) - OK
 cfgetispeed(B460800) - OK
 Init - close - /dev/test - OK
+Multiple open of the device
+Termios_test_driver - rtems_set_initial_baud - bad baud - OK
+Termios_test_driver - rtems_set_initial_baud - 38400 - OK
+Termios_test_driver - rtems_set_initial_baud - bad baud - OK
+Termios_test_driver - rtems_set_initial_baud - 38400 - OK
+Termios_test_driver - rtems_set_initial_baud - bad baud - OK
+Termios_test_driver - rtems_set_initial_baud - 38400 - OK
+Termios_test_driver - rtems_set_initial_baud - bad baud - OK
+Termios_test_driver - rtems_set_initial_baud - 38400 - OK
+Termios_test_driver - rtems_set_initial_baud - bad baud - OK
+Termios_test_driver - rtems_set_initial_baud - 38400 - OK
+Termios_test_driver - rtems_set_initial_baud - bad baud - OK
+Termios_test_driver - rtems_set_initial_baud - 38400 - OK
+Termios_test_driver - rtems_set_initial_baud - bad baud - OK
+Termios_test_driver - rtems_set_initial_baud - 38400 - OK
+Termios_test_driver - rtems_set_initial_baud - bad baud - OK
+Termios_test_driver - rtems_set_initial_baud - 38400 - OK
+Termios_test_driver - rtems_set_initial_baud - bad baud - OK
+Termios_test_driver - rtems_set_initial_baud - 38400 - OK
+Termios_test_driver - rtems_set_initial_baud - bad baud - OK
+Termios_test_driver - rtems_set_initial_baud - 38400 - OK
+Termios_test_driver - rtems_set_initial_baud - bad baud - OK
+Termios_test_driver - rtems_set_initial_baud - 38400 - OK
+Termios_test_driver - rtems_set_initial_baud - bad baud - OK
+Termios_test_driver - rtems_set_initial_baud - 38400 - OK
+Termios_test_driver - rtems_set_initial_baud - bad baud - OK
+Termios_test_driver - rtems_set_initial_baud - 38400 - OK
+Termios_test_driver - rtems_set_initial_baud - bad baud - OK
+Termios_test_driver - rtems_set_initial_baud - 38400 - OK
+Termios_test_driver - rtems_set_initial_baud - bad baud - OK
+Termios_test_driver - rtems_set_initial_baud - 38400 - OK
+Termios_test_driver - rtems_set_initial_baud - bad baud - OK
+Termios_test_driver - rtems_set_initial_baud - 38400 - OK
+Termios_test_driver - rtems_set_initial_baud - bad baud - OK
+Termios_test_driver - rtems_set_initial_baud - 38400 - OK
+Termios_test_driver - rtems_set_initial_baud - bad baud - OK
+Termios_test_driver - rtems_set_initial_baud - 38400 - OK
+Termios_test_driver - rtems_set_initial_baud - bad baud - OK
+Termios_test_driver - rtems_set_initial_baud - 38400 - OK
+Termios_test_driver - rtems_set_initial_baud - bad baud - OK
+Termios_test_driver - rtems_set_initial_baud - 38400 - OK
+Termios_test_driver - rtems_set_initial_baud - bad baud - OK
+Termios_test_driver - rtems_set_initial_baud - 38400 - OK
+Termios_test_driver - rtems_set_initial_baud - bad baud - OK
+Termios_test_driver - rtems_set_initial_baud - 38400 - OK
+Termios_test_driver - rtems_set_initial_baud - bad baud - OK
+Termios_test_driver - rtems_set_initial_baud - 38400 - OK
+Termios_test_driver - rtems_set_initial_baud - bad baud - OK
+Termios_test_driver - rtems_set_initial_baud - 38400 - OK
+Termios_test_driver - rtems_set_initial_baud - bad baud - OK
+Termios_test_driver - rtems_set_initial_baud - 38400 - OK
+Termios_test_driver - rtems_set_initial_baud - bad baud - OK
+Termios_test_driver - rtems_set_initial_baud - 38400 - OK
+
 *** END OF TEST TERMIOS 01 ***


 *joel*:
2010-08-10	Joel Sherrill <joel.sherrill at oarcorp.com>

	PR 1658/cpukit
	* sapi/src/exshutdown.c, score/include/rtems/score/interr.h: If the
	system is not up, consider it a fatal error.

M 1.2560  cpukit/ChangeLog
M    1.4  cpukit/sapi/src/exshutdown.c
M   1.32  cpukit/score/include/rtems/score/interr.h

diff -u rtems/cpukit/ChangeLog:1.2559 rtems/cpukit/ChangeLog:1.2560
--- rtems/cpukit/ChangeLog:1.2559	Tue Aug 10 17:15:32 2010
+++ rtems/cpukit/ChangeLog	Tue Aug 10 17:56:31 2010
@@ -1,3 +1,9 @@
+2010-08-10	Joel Sherrill <joel.sherrill at oarcorp.com>
+
+	PR 1658/cpukit
+	* sapi/src/exshutdown.c, score/include/rtems/score/interr.h: If the
+	system is not up, consider it a fatal error.
+
 2010-08-09	Bharath Suri <bharath.s.jois at gmail.com>
 
 	PR 1662/filesystem

diff -u rtems/cpukit/sapi/src/exshutdown.c:1.3 rtems/cpukit/sapi/src/exshutdown.c:1.4
--- rtems/cpukit/sapi/src/exshutdown.c:1.3	Fri Sep  4 08:09:48 2009
+++ rtems/cpukit/sapi/src/exshutdown.c	Tue Aug 10 17:56:35 2010
@@ -18,6 +18,7 @@
 #include <rtems/system.h>
 #include <rtems/score/sysstate.h>
 #include <rtems/score/thread.h>
+#include <rtems/score/interr.h>
 
 /*
  *  rtems_shutdown_executive
@@ -35,8 +36,14 @@
    uint32_t   result
 )
 {
-  if ( !_System_state_Is_shutdown( _System_state_Get() ) ) {
+  if ( _System_state_Is_up( _System_state_Get() ) ) {
     _System_state_Set( SYSTEM_STATE_SHUTDOWN );
     _Thread_Stop_multitasking();
   }
+  _Internal_error_Occurred(
+    INTERNAL_ERROR_CORE,
+    true,
+    INTERNAL_ERROR_SHUTDOWN_WHEN_NOT_UP
+  );
+
 }

diff -u rtems/cpukit/score/include/rtems/score/interr.h:1.31 rtems/cpukit/score/include/rtems/score/interr.h:1.32
--- rtems/cpukit/score/include/rtems/score/interr.h:1.31	Wed Jul  7 09:06:52 2010
+++ rtems/cpukit/score/include/rtems/score/interr.h	Tue Aug 10 17:56:35 2010
@@ -69,7 +69,8 @@
   INTERNAL_ERROR_IMPLEMENTATION_KEY_CREATE_INCONSISTENCY,
   INTERNAL_ERROR_IMPLEMENTATION_BLOCKING_OPERATION_CANCEL,
   INTERNAL_ERROR_MUTEX_OBTAIN_FROM_BAD_STATE,
-  INTERNAL_ERROR_UNLIMITED_AND_MAXIMUM_IS_0
+  INTERNAL_ERROR_UNLIMITED_AND_MAXIMUM_IS_0,
+  INTERNAL_ERROR_SHUTDOWN_WHEN_NOT_UP
 } Internal_errors_Core_list;
 
 typedef uint32_t Internal_errors_t;


 *joel*:
2010-08-10	Joel Sherrill <joel.sherrill at oarcorp.com>

	PR 1658/cpukit
	* Makefile.am, configure.ac: If the system is not up, consider it a
	fatal error. This eliminated sp61 and added spfatal23.
	* spfatal23/.cvsignore, spfatal23/Makefile.am, spfatal23/spfatal23.doc,
	spfatal23/spfatal23.scn, spfatal23/testcase.h: New files.
	* sp61/.cvsignore, sp61/Makefile.am, sp61/init.c, sp61/sp61.doc,
	sp61/sp61.scn: Removed.

M  1.415  testsuites/sptests/ChangeLog
M  1.104  testsuites/sptests/Makefile.am
M  1.109  testsuites/sptests/configure.ac
R    1.1  testsuites/sptests/sp61/.cvsignore
R    1.1  testsuites/sptests/sp61/Makefile.am
R    1.3  testsuites/sptests/sp61/init.c
R    1.1  testsuites/sptests/sp61/sp61.doc
R    1.1  testsuites/sptests/sp61/sp61.scn
A    1.1  testsuites/sptests/spfatal23/.cvsignore
A    1.1  testsuites/sptests/spfatal23/Makefile.am
A    1.1  testsuites/sptests/spfatal23/spfatal23.doc
A    1.1  testsuites/sptests/spfatal23/spfatal23.scn
A    1.1  testsuites/sptests/spfatal23/testcase.h

diff -u rtems/testsuites/sptests/ChangeLog:1.414 rtems/testsuites/sptests/ChangeLog:1.415
--- rtems/testsuites/sptests/ChangeLog:1.414	Tue Aug 10 12:34:12 2010
+++ rtems/testsuites/sptests/ChangeLog	Tue Aug 10 17:57:59 2010
@@ -1,3 +1,13 @@
+2010-08-10	Joel Sherrill <joel.sherrill at oarcorp.com>
+
+	PR 1658/cpukit
+	* Makefile.am, configure.ac: If the system is not up, consider it a
+	fatal error. This eliminated sp61 and added spfatal23.
+	* spfatal23/.cvsignore, spfatal23/Makefile.am, spfatal23/spfatal23.doc,
+	spfatal23/spfatal23.scn, spfatal23/testcase.h: New files.
+	* sp61/.cvsignore, sp61/Makefile.am, sp61/init.c, sp61/sp61.doc,
+	sp61/sp61.scn: Removed.
+
 2010-08-10	Bharath Suri <bharath.s.jois at gmail.com>
 
 	PR 1663/testing

diff -u rtems/testsuites/sptests/Makefile.am:1.103 rtems/testsuites/sptests/Makefile.am:1.104
--- rtems/testsuites/sptests/Makefile.am:1.103	Thu Jul 29 17:28:51 2010
+++ rtems/testsuites/sptests/Makefile.am	Tue Aug 10 17:57:59 2010
@@ -13,7 +13,7 @@
     sp30 sp31 sp32 sp33 sp34 sp35      sp37 sp38 sp39 \
     sp40 sp41 sp42 sp43 sp44 sp45 sp46 sp47 sp48 sp49 \
     sp50 sp51 sp52 sp53 sp54 sp55 sp56 sp57 sp58 sp59 \
-    sp60 sp61 sp62 sp63 sp64 sp65 sp66 sp67 sp68 sp69 \
+    sp60      sp62 sp63 sp64 sp65 sp66 sp67 sp68 sp69 \
     sp70 sp71 sp72 \
     spassoc01 spchain spclockget spcoverage spobjgetnext \
     spnotepad01 spprintk spprivenv01 spsize spstkalloc spthreadq01 \
@@ -22,7 +22,7 @@
     spfatal01 spfatal02 spfatal03 spfatal04 spfatal05 spfatal06 spfatal07 \
     spfatal08 spfatal09 spfatal10 spfatal11 spfatal12 spfatal13 spfatal14 \
     spfatal15 spfatal16 spfatal17 spfatal18 spfatal19 spfatal20 spfatal21 \
-    spfatal22 \
+    spfatal22 spfatal23 \
     spfifo01 spfifo02 spfifo03 spfifo04 spfifo05 \
     spintrcritical01 spintrcritical02 spintrcritical03 spintrcritical04 \
     spintrcritical05 spintrcritical06 spintrcritical07 spintrcritical08 \

diff -u rtems/testsuites/sptests/configure.ac:1.108 rtems/testsuites/sptests/configure.ac:1.109
--- rtems/testsuites/sptests/configure.ac:1.108	Thu Jul 29 17:28:51 2010
+++ rtems/testsuites/sptests/configure.ac	Tue Aug 10 17:58:00 2010
@@ -86,7 +86,6 @@
 sp58/Makefile
 sp59/Makefile
 sp60/Makefile
-sp61/Makefile
 sp62/Makefile
 sp63/Makefile
 sp64/Makefile
@@ -127,6 +126,7 @@
 spfatal20/Makefile
 spfatal21/Makefile
 spfatal22/Makefile
+spfatal23/Makefile
 spfifo01/Makefile
 spfifo02/Makefile
 spfifo03/Makefile

diff -u /dev/null rtems/testsuites/sptests/spfatal23/.cvsignore:1.1
--- /dev/null	Tue Aug 10 18:11:10 2010
+++ rtems/testsuites/sptests/spfatal23/.cvsignore	Tue Aug 10 17:58:00 2010
@@ -0,0 +1,2 @@
+Makefile
+Makefile.in

diff -u /dev/null rtems/testsuites/sptests/spfatal23/Makefile.am:1.1
--- /dev/null	Tue Aug 10 18:11:10 2010
+++ rtems/testsuites/sptests/spfatal23/Makefile.am	Tue Aug 10 17:58:00 2010
@@ -0,0 +1,25 @@
+##
+## $Id$
+##
+
+rtems_tests_PROGRAMS = spfatal23
+spfatal23_SOURCES = ../spfatal_support/init.c \
+    ../spfatal_support/system.h testcase.h
+
+dist_rtems_tests_DATA = spfatal23.scn
+dist_rtems_tests_DATA += spfatal23.doc
+
+include $(RTEMS_ROOT)/make/custom/@RTEMS_BSP at .cfg
+include $(top_srcdir)/../automake/compile.am
+include $(top_srcdir)/../automake/leaf.am
+
+AM_CPPFLAGS += -I$(top_srcdir)/../support/include
+
+LINK_OBJS = $(spfatal23_OBJECTS) $(spfatal23_LDADD)
+LINK_LIBS = $(spfatal23_LDLIBS)
+
+spfatal23$(EXEEXT): $(spfatal23_OBJECTS) $(spfatal23_DEPENDENCIES)
+	@rm -f spfatal23$(EXEEXT)
+	$(make-exe)
+
+include $(top_srcdir)/../automake/local.am

diff -u /dev/null rtems/testsuites/sptests/spfatal23/spfatal23.doc:1.1
--- /dev/null	Tue Aug 10 18:11:10 2010
+++ rtems/testsuites/sptests/spfatal23/spfatal23.doc	Tue Aug 10 17:58:00 2010
@@ -0,0 +1,24 @@
+#
+#  $Id$
+#
+#  COPYRIGHT (c) 1989-2010.
+#  On-Line Applications Research Corporation (OAR).
+#
+#  The license and distribution terms for this file may be
+#  found in the file LICENSE in this distribution or at
+#  http://www.rtems.com/license/LICENSE.
+#
+
+This file describes the directives and concepts tested by this test set.
+
+test set name:  spfatal23
+
+directives:
+
+  _System_state_Set
+  rtems_shutdown_executive
+
+concepts:
+
++ Ensure that shutting RTEMS down when the system state indicates it is
+  already shutdown is a fatal error.

diff -u /dev/null rtems/testsuites/sptests/spfatal23/spfatal23.scn:1.1
--- /dev/null	Tue Aug 10 18:11:10 2010
+++ rtems/testsuites/sptests/spfatal23/spfatal23.scn	Tue Aug 10 17:58:00 2010
@@ -0,0 +1,3 @@
+*** TEST FATAL SHUTDOWN WHEN NOT UP ***
+Fatal error (SHUTDOWN WHEN NOT UP) hit
+*** END OF TEST ***

diff -u /dev/null rtems/testsuites/sptests/spfatal23/testcase.h:1.1
--- /dev/null	Tue Aug 10 18:11:10 2010
+++ rtems/testsuites/sptests/spfatal23/testcase.h	Tue Aug 10 17:58:00 2010
@@ -0,0 +1,24 @@
+/*
+ *  COPYRIGHT (c) 1989-2010.
+ *  On-Line Applications Research Corporation (OAR).
+ *
+ *  The license and distribution terms for this file may be
+ *  found in the file LICENSE in this distribution or at
+ *  http://www.rtems.com/license/LICENSE.
+ *
+ *  $Id$
+ */
+
+#define __RTEMS_VIOLATE_KERNEL_VISIBILITY__
+#define FATAL_ERROR_TEST_NAME            "SHUTDOWN WHEN NOT UP"
+#define FATAL_ERROR_DESCRIPTION          "SHUTDOWN WHEN NOT UP"
+#define FATAL_ERROR_EXPECTED_SOURCE      INTERNAL_ERROR_CORE
+#define FATAL_ERROR_EXPECTED_IS_INTERNAL TRUE
+#define FATAL_ERROR_EXPECTED_ERROR       INTERNAL_ERROR_SHUTDOWN_WHEN_NOT_UP
+
+void force_error()
+{
+  _System_state_Set( SYSTEM_STATE_SHUTDOWN );
+  rtems_shutdown_executive( 0 );
+  /* we will not run this far */
+}



--

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/20100810/a2406403/attachment-0001.html>


More information about the vc mailing list