[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