diff options
author | Michael Brown | 2009-03-12 20:41:40 +0100 |
---|---|---|
committer | Michael Brown | 2009-03-13 03:06:30 +0100 |
commit | 1266d7902bf7f2534ee279671d48613ef9b2434c (patch) | |
tree | a1a5b188148d983fa962a887476259768f1751d4 /src/include/gpxe/settings.h | |
parent | [tcp] Avoid setting PSH flag when SYN flag is set (diff) | |
download | ipxe-1266d7902bf7f2534ee279671d48613ef9b2434c.tar.gz ipxe-1266d7902bf7f2534ee279671d48613ef9b2434c.tar.xz ipxe-1266d7902bf7f2534ee279671d48613ef9b2434c.zip |
[tables] Redefine methods for accessing linker tables
Intel's C compiler (icc) chokes on the zero-length arrays that we
currently use as part of the mechanism for accessing linker table
entries. Abstract away the zero-length arrays, to make a port to icc
easier.
Introduce macros such as for_each_table_entry() to simplify the common
case of iterating over all entries in a linker table.
Represent table names as #defined string constants rather than
unquoted literals; this avoids visual confusion between table names
and C variable or type names, and also allows us to force a
compilation error in the event of incorrect table names.
Diffstat (limited to 'src/include/gpxe/settings.h')
-rw-r--r-- | src/include/gpxe/settings.h | 15 |
1 files changed, 12 insertions, 3 deletions
diff --git a/src/include/gpxe/settings.h b/src/include/gpxe/settings.h index fe9c8082..4f7f98ef 100644 --- a/src/include/gpxe/settings.h +++ b/src/include/gpxe/settings.h @@ -36,8 +36,11 @@ struct setting { unsigned int tag; }; +/** Configuration setting table */ +#define SETTINGS "settings" + /** Declare a configuration setting */ -#define __setting __table ( struct setting, settings, 01 ) +#define __setting __table ( struct setting, SETTINGS, 01 ) /** Settings block operations */ struct settings_operations { @@ -123,9 +126,12 @@ struct setting_type { char *buf, size_t len ); }; +/** Configuration setting type table */ +#define SETTING_TYPES "setting_types" + /** Declare a configuration setting type */ #define __setting_type \ - __table ( struct setting_type, setting_types, 01 ) + __table ( struct setting_type, SETTING_TYPES, 01 ) /** * A settings applicator @@ -139,9 +145,12 @@ struct settings_applicator { int ( * apply ) ( void ); }; +/** Settings applicator table */ +#define SETTINGS_APPLICATORS "settings_applicators" + /** Declare a settings applicator */ #define __settings_applicator \ - __table ( struct settings_applicator, settings_applicators, 01 ) + __table ( struct settings_applicator, SETTINGS_APPLICATORS, 01 ) /** * A simple settings block |