[rtems commit] score: Statically initialize IO manager

Sebastian Huber sebh at rtems.org
Thu Apr 10 10:49:51 UTC 2014


Module:    rtems
Branch:    master
Commit:    263f4beceae0ec5602cb41e792c25ebe3ec49c97
Changeset: http://git.rtems.org/rtems/commit/?id=263f4beceae0ec5602cb41e792c25ebe3ec49c97

Author:    Sebastian Huber <sebastian.huber at embedded-brains.de>
Date:      Mon Apr  7 14:50:37 2014 +0200

score: Statically initialize IO manager

This simplifies the RTEMS initialization and helps to avoid a memory
overhead.  The workspace demands of the IO manager were not included in
the <rtems/confdefs.h> workspace size estimate.  This is also fixed as a
side-effect.

Update documentation and move "Specifying Application Defined Device
Driver Table" to the section end.  This sub-section is not that
important for the user.  Mentioning this at the beginning may lead to
confusion.

---

 cpukit/libmisc/monitor/mon-driver.c |    5 +--
 cpukit/sapi/include/confdefs.h      |   37 +++++++------------
 cpukit/sapi/include/rtems/config.h  |   13 -------
 cpukit/sapi/include/rtems/io.h      |   13 +------
 cpukit/sapi/src/exinit.c            |    3 --
 cpukit/sapi/src/io.c                |   56 ----------------------------
 doc/user/conf.t                     |   68 ++++++++++++++++++-----------------
 testsuites/libtests/block11/init.c  |    1 -
 testsuites/sptests/sp21/init.c      |    3 +-
 testsuites/sptests/spsize/size.c    |    3 --
 10 files changed, 54 insertions(+), 148 deletions(-)

diff --git a/cpukit/libmisc/monitor/mon-driver.c b/cpukit/libmisc/monitor/mon-driver.c
index 70073c6..8abe281 100644
--- a/cpukit/libmisc/monitor/mon-driver.c
+++ b/cpukit/libmisc/monitor/mon-driver.c
@@ -63,10 +63,9 @@ rtems_monitor_driver_next(
     rtems_id              *next_id
 )
 {
-    const rtems_configuration_table *c = &Configuration;
     uint32_t   n = rtems_object_id_get_index(*next_id);
 
-    if (n >= c->number_of_device_drivers)
+    if (n >= _IO_Number_of_drivers)
         goto failed;
 
     _Thread_Disable_dispatch();
@@ -79,7 +78,7 @@ rtems_monitor_driver_next(
     canonical_driver->name = rtems_build_name('-', '-', '-', '-');
 
     *next_id += 1;
-    return (const void *) (c->Device_driver_table + n);
+    return (const void *) (&_IO_Driver_address_table[n]);
 
 failed:
     *next_id = RTEMS_OBJECT_ID_FINAL;
diff --git a/cpukit/sapi/include/confdefs.h b/cpukit/sapi/include/confdefs.h
index 5f64d73..53a8699 100644
--- a/cpukit/sapi/include/confdefs.h
+++ b/cpukit/sapi/include/confdefs.h
@@ -70,7 +70,6 @@ extern "C" {
 #endif
 
 extern rtems_initialization_tasks_table Initialization_tasks[];
-extern rtems_driver_address_table       Device_drivers[];
 #if defined(RTEMS_MULTIPROCESSING)
   extern rtems_multiprocessing_table      Multiprocessing_configuration;
 #endif
@@ -1202,8 +1201,18 @@ const rtems_libio_helper rtems_fs_init_helper =
 
 #ifndef CONFIGURE_HAS_OWN_DEVICE_DRIVER_TABLE
 
+/**
+ * This specifies the maximum number of device drivers that
+ * can be installed in the system at one time.  It must account
+ * for both the statically and dynamically installed drivers.
+ */
+#ifndef CONFIGURE_MAXIMUM_DRIVERS
+  #define CONFIGURE_MAXIMUM_DRIVERS
+#endif
+
 #ifdef CONFIGURE_INIT
-  rtems_driver_address_table Device_drivers[] = {
+  rtems_driver_address_table
+    _IO_Driver_address_table[ CONFIGURE_MAXIMUM_DRIVERS ] = {
     #ifdef CONFIGURE_BSP_PREREQUISITE_DRIVERS
       CONFIGURE_BSP_PREREQUISITE_DRIVERS,
     #endif
@@ -1254,27 +1263,12 @@ const rtems_libio_helper rtems_fs_init_helper =
       NULL_DRIVER_TABLE_ENTRY
     #endif
   };
-#endif
-
-#endif  /* CONFIGURE_HAS_OWN_DEVICE_DRIVER_TABLE */
 
-/*
- *  Default the number of drivers per node.  This value may be
- *  overridden by the user.
- */
-
-#define CONFIGURE_NUMBER_OF_DRIVERS \
-  RTEMS_ARRAY_SIZE(Device_drivers)
-
-/**
- * This specifies the maximum number of device drivers that
- * can be installed in the system at one time.  It must account
- * for both the statically and dynamically installed drivers.
- */
-#ifndef CONFIGURE_MAXIMUM_DRIVERS
-  #define CONFIGURE_MAXIMUM_DRIVERS CONFIGURE_NUMBER_OF_DRIVERS
+  const size_t _IO_Number_of_drivers =
+    RTEMS_ARRAY_SIZE( _IO_Driver_address_table );
 #endif
 
+#endif  /* CONFIGURE_HAS_OWN_DEVICE_DRIVER_TABLE */
 
 #ifdef CONFIGURE_APPLICATION_NEEDS_ATA_DRIVER
   /*
@@ -2475,9 +2469,6 @@ const rtems_libio_helper rtems_fs_init_helper =
         false,
       #endif
     #endif
-    CONFIGURE_MAXIMUM_DRIVERS,                /* maximum device drivers */
-    CONFIGURE_NUMBER_OF_DRIVERS,              /* static device drivers */
-    Device_drivers,                           /* pointer to driver table */
     CONFIGURE_NUMBER_OF_INITIAL_EXTENSIONS,   /* number of static extensions */
     CONFIGURE_INITIAL_EXTENSION_TABLE,        /* pointer to static extensions */
     #if defined(RTEMS_MULTIPROCESSING)
diff --git a/cpukit/sapi/include/rtems/config.h b/cpukit/sapi/include/rtems/config.h
index 0998f05..9a9d0a3 100644
--- a/cpukit/sapi/include/rtems/config.h
+++ b/cpukit/sapi/include/rtems/config.h
@@ -57,7 +57,6 @@ typedef void *posix_api_configuration_table;
 #include <rtems/rtems/config.h>
 
 #include <rtems/extension.h>
-#include <rtems/io.h>
 #if defined(RTEMS_MULTIPROCESSING)
 #include <rtems/score/mpci.h>
 #endif
@@ -242,9 +241,6 @@ typedef struct {
     bool                         smp_enabled;
   #endif
 
-  uint32_t                       maximum_drivers;
-  uint32_t                       number_of_device_drivers;
-  rtems_driver_address_table    *Device_driver_table;
   uint32_t                       number_of_initial_extensions;
   const rtems_extensions_table  *User_extension_table;
   #if defined(RTEMS_MULTIPROCESSING)
@@ -335,15 +331,6 @@ extern const rtems_configuration_table Configuration;
 #define rtems_configuration_get_do_zero_of_workspace() \
    (Configuration.do_zero_of_workspace)
 
-#define rtems_configuration_get_maximum_drivers() \
-        (Configuration.maximum_drivers)
-
-#define rtems_configuration_get_number_of_device_drivers() \
-        (Configuration.number_of_device_drivers)
-
-#define rtems_configuration_get_device_driver_table() \
-        (Configuration.Device_driver_table)
-
 #define rtems_configuration_get_number_of_initial_extensions() \
         (Configuration.number_of_initial_extensions)
 
diff --git a/cpukit/sapi/include/rtems/io.h b/cpukit/sapi/include/rtems/io.h
index 799a305..ce750b0 100644
--- a/cpukit/sapi/include/rtems/io.h
+++ b/cpukit/sapi/include/rtems/io.h
@@ -250,18 +250,9 @@ rtems_status_code rtems_io_lookup_name(
     rtems_driver_name_t  *device_info
 ) RTEMS_COMPILER_DEPRECATED_ATTRIBUTE;
 
-SAPI_IO_EXTERN uint32_t _IO_Number_of_drivers;
+extern const size_t _IO_Number_of_drivers;
 
-SAPI_IO_EXTERN rtems_driver_address_table *_IO_Driver_address_table;
-
-/**
- * @brief Initialization of device drivers.
- *
- * NOTE: The IO manager has been extended to support runtime driver
- * registration. The driver table is now allocated in the
- * workspace.
- */
-void _IO_Manager_initialization( void );
+extern rtems_driver_address_table _IO_Driver_address_table[];
 
 /**
  * @brief Initialization of all device drivers.
diff --git a/cpukit/sapi/src/exinit.c b/cpukit/sapi/src/exinit.c
index 45c6cf6..235ba77 100644
--- a/cpukit/sapi/src/exinit.c
+++ b/cpukit/sapi/src/exinit.c
@@ -32,7 +32,6 @@
 #include <rtems/debug.h>
 #include <rtems/extensionimpl.h>
 #include <rtems/init.h>
-#include <rtems/io.h>
 #include <rtems/score/sysstate.h>
 
 #include <rtems/score/apiext.h>
@@ -142,8 +141,6 @@ void rtems_initialize_data_structures(void)
 
   _Extension_Manager_initialization();
 
-  _IO_Manager_initialization();
-
   _POSIX_API_Initialize();
 
   _System_state_Set( SYSTEM_STATE_BEFORE_MULTITASKING );
diff --git a/cpukit/sapi/src/io.c b/cpukit/sapi/src/io.c
index 99da902..c14f5e7 100644
--- a/cpukit/sapi/src/io.c
+++ b/cpukit/sapi/src/io.c
@@ -19,63 +19,7 @@
 #include "config.h"
 #endif
 
-#include <rtems/system.h>
-#include <rtems/config.h>
 #include <rtems/io.h>
-#include <rtems/score/isr.h>
-#include <rtems/score/thread.h>
-#include <rtems/score/wkspace.h>
-
-#include <string.h>
-
-void _IO_Manager_initialization(void)
-{
-  uint32_t                    index;
-  rtems_driver_address_table *driver_table;
-  uint32_t                    drivers_in_table;
-  uint32_t                    number_of_drivers;
-
-  driver_table      = rtems_configuration_get_device_driver_table();
-  drivers_in_table  = rtems_configuration_get_number_of_device_drivers();
-  number_of_drivers = rtems_configuration_get_maximum_drivers();
-
-  /*
-   *  If the user claims there are less drivers than are actually in
-   *  the table, then let's just go with the table's count.
-   */
-  if ( number_of_drivers <= drivers_in_table )
-    number_of_drivers = drivers_in_table;
-
-  /*
-   *  If the maximum number of driver is the same as the number in the
-   *  table, then we do not have to copy the driver table.  They can't
-   *  register any dynamically.
-   */
-  if ( number_of_drivers == drivers_in_table ) {
-    _IO_Driver_address_table = driver_table;
-    _IO_Number_of_drivers = number_of_drivers;
-    return;
-  }
-
-  /*
-   *  The application requested extra slots in the driver table, so we
-   *  have to allocate a new driver table and copy theirs to it.
-   */
-
-  _IO_Driver_address_table = (rtems_driver_address_table *)
-      _Workspace_Allocate_or_fatal_error(
-        sizeof( rtems_driver_address_table ) * ( number_of_drivers )
-      );
-  _IO_Number_of_drivers = number_of_drivers;
-
-  memset(
-    _IO_Driver_address_table, 0,
-    sizeof( rtems_driver_address_table ) * ( number_of_drivers )
-  );
-
-  for ( index = 0 ; index < drivers_in_table ; index++ )
-    _IO_Driver_address_table[index] = driver_table[index];
-}
 
 void _IO_Initialize_all_drivers( void )
 {
diff --git a/doc/user/conf.t b/doc/user/conf.t
index 68b6246..c0bb35a 100644
--- a/doc/user/conf.t
+++ b/doc/user/conf.t
@@ -3836,39 +3836,6 @@ custom device drivers.
 Note that network device drivers are not configured in the Device Driver Table.
 
 @c
- at c === CONFIGURE_HAS_OWN_DEVICE_DRIVER_TABLE ===
- at c
- at subsection Specifying Application Defined Device Driver Table
-
- at findex CONFIGURE_HAS_OWN_DEVICE_DRIVER_TABLE
-
- at table @b
- at item CONSTANT:
- at code{CONFIGURE_HAS_OWN_DEVICE_DRIVER_TABLE}
-
- at item DATA TYPE:
-List of device driver initializers (@code{rtems_driver_address_table}).
-
- at item RANGE:
-Undefined or array of device drivers.
-
- at item DEFAULT VALUE:
-This is not defined by default, indicating the @code{<rtems/confdefs.h>}
-is providing the device driver table.
-
- at end table
-
- at subheading DESCRIPTION:
- at code{CONFIGURE_HAS_OWN_DEVICE_DRIVER_TABLE} is defined if the application
-wishes to provide their own Device Driver Table.
-
-The table must be an array of @code{rtems_driver_address_table} entries
-named @code{Device_drivers}.
-
- at subheading NOTES:
-It is expected that there the application would only rarely need to do this.
-
- at c
 @c === CONFIGURE_MAXIMUM_DRIVERS ===
 @c
 @subsection Specifying the Maximum Number of Device Drivers
@@ -4297,6 +4264,41 @@ device driver.
 This device driver is supported by all BSPs.
 
 @c
+ at c === CONFIGURE_HAS_OWN_DEVICE_DRIVER_TABLE ===
+ at c
+ at subsection Specifying Application Defined Device Driver Table
+
+ at findex CONFIGURE_HAS_OWN_DEVICE_DRIVER_TABLE
+
+ at table @b
+ at item CONSTANT:
+ at code{CONFIGURE_HAS_OWN_DEVICE_DRIVER_TABLE}
+
+ at item DATA TYPE:
+Boolean feature macro.
+
+ at item RANGE:
+Defined or undefined.
+
+ at item DEFAULT VALUE:
+This is not defined by default, indicating the @code{<rtems/confdefs.h>}
+is providing the device driver table.
+
+ at end table
+
+ at subheading DESCRIPTION:
+ at code{CONFIGURE_HAS_OWN_DEVICE_DRIVER_TABLE} is defined if the application
+wishes to provide their own Device Driver Table.
+
+The table must be an array of @code{rtems_driver_address_table} entries named
+ at code{_IO_Driver_address_table}.  The application must also provide a const
+variable @code{_IO_Number_of_drivers} of type @code{size_t} indicating the
+number of entries in the @code{_IO_Driver_address_table}.
+
+ at subheading NOTES:
+It is expected that there the application would only rarely need to do this.
+
+ at c
 @c === Multiprocessing Configuration ===
 @c
 @section Multiprocessing Configuration
diff --git a/testsuites/libtests/block11/init.c b/testsuites/libtests/block11/init.c
index e144f0c..0704dad 100644
--- a/testsuites/libtests/block11/init.c
+++ b/testsuites/libtests/block11/init.c
@@ -434,7 +434,6 @@ static rtems_task Init(rtems_task_argument argument)
 #define CONFIGURE_LIBIO_MAXIMUM_FILE_DESCRIPTORS 5
 
 #define CONFIGURE_MAXIMUM_TASKS 1
-#define CONFIGURE_MAXIMUM_DRIVERS 2
 
 #define CONFIGURE_INITIAL_EXTENSIONS RTEMS_TEST_INITIAL_EXTENSION
 
diff --git a/testsuites/sptests/sp21/init.c b/testsuites/sptests/sp21/init.c
index 62a48b8..0eb453c 100644
--- a/testsuites/sptests/sp21/init.c
+++ b/testsuites/sptests/sp21/init.c
@@ -29,8 +29,7 @@ void do_test_zero_driver(void);
 #define STUB_DRIVER_MAJOR     0x2
 #define ZERO_DRIVER_MAJOR     0x3
 #define NO_DRIVER_MAJOR       0x4
-#define INVALID_DRIVER_MAJOR  \
-     (rtems_configuration_get_number_of_device_drivers() + 1)
+#define INVALID_DRIVER_MAJOR  UINT32_MAX
 
 /* driver entries to use with io_register */
 rtems_driver_address_table GoodDriver = DEVNULL_DRIVER_TABLE_ENTRY;
diff --git a/testsuites/sptests/spsize/size.c b/testsuites/sptests/spsize/size.c
index 6acdaa6..a6e6047 100644
--- a/testsuites/sptests/spsize/size.c
+++ b/testsuites/sptests/spsize/size.c
@@ -290,9 +290,6 @@ uninitialized =
 
 /*intr.h*/      0                                         +
 
-/*io.h*/        (sizeof _IO_Number_of_drivers)            +
-                (sizeof _IO_Driver_address_table)         +
-
 /*isr.h*/       (sizeof _ISR_Nest_level)                  +
 #if (CPU_SIMPLE_VECTORED_INTERRUPTS == TRUE)
                 (sizeof _ISR_Vector_table)                +




More information about the vc mailing list