Use static array for parsers
authorGeoff Levand <geoff@infradead.org>
Sat, 17 Mar 2012 05:06:03 +0000 (22:06 -0700)
committerGeoff Levand <geoff@infradead.org>
Sat, 17 Mar 2012 05:16:41 +0000 (22:16 -0700)
Change the parser structure array implementation from using an
array in a seperate parsers section to a static array of pointers
in parser.c.  Parser priority is now set by the position in the
new parsers array.

Signed-off-by: Geoff Levand <geoff@infradead.org>
discover/kboot-parser.c
discover/native-parser.c
discover/parser-utils.h
discover/parser.c
discover/yaboot-parser.c

index 429ae096ddccc0573056052243e80faaf85de2e4..025f13b6afa1bd8e32606da013f6dc2a15f4f667 100644 (file)
@@ -153,4 +153,4 @@ static int kboot_parse(struct discover_context *dc)
        return rc;
 }
 
        return rc;
 }
 
-define_parser(kboot, 98, kboot_parse);
+define_parser(kboot, kboot_parse);
index a8429ec2c1675a9cb4a51d751246cf59ae0e64e2..08309d1b317ea243121f24aab078368d8c57e83e 100644 (file)
@@ -128,4 +128,4 @@ static int native_parse(const char *device)
        return 1;
 }
 
        return 1;
 }
 
-define_parser(native, 100, native_parse);
+define_parser(native, native_parse);
index 781867c4c22fe496f7d54f94991aff7b61e7597b..fe28b7bd74d7adfd100f011649fdbf27f17aca24 100644 (file)
@@ -8,16 +8,13 @@
 
 #define artwork_pathname(file) (PKG_SHARE_DIR "/artwork/" file)
 
 
 #define artwork_pathname(file) (PKG_SHARE_DIR "/artwork/" file)
 
-#define define_parser(__name, __priority, __parse_fn)                  \
+#define define_parser(__name, __parse_fn)                      \
        struct parser                                                   \
        struct parser                                                   \
-               __attribute__((unused, section("parsers")))             \
        __ ## __name ## _parser = {                                     \
                .name           = #__name,                              \
        __ ## __name ## _parser = {                                     \
                .name           = #__name,                              \
-               .priority       = __priority,                           \
                .parse          = __parse_fn,                           \
        };
 
                .parse          = __parse_fn,                           \
        };
 
-
 void device_add_boot_option(struct device *device,
                struct boot_option *boot_option);
 
 void device_add_boot_option(struct device *device,
                struct boot_option *boot_option);
 
index 8f2735ccc6553b3d00bdc575b5debdd7b7203a3a..c106dec0ae20c0a6210d317060cbe125f9656450 100644 (file)
@@ -8,39 +8,33 @@
 #include "parser.h"
 #include "parser-utils.h"
 
 #include "parser.h"
 #include "parser-utils.h"
 
-extern struct parser __start_parsers[], __stop_parsers[];
+struct parser __native_parser;
+struct parser __yaboot_parser;
+struct parser __kboot_parser;
+struct parser __grub2_parser;
+
+static const struct parser *const parsers[] = {
+//     &__native_parser,
+       &__yaboot_parser,
+       &__kboot_parser,
+       NULL
+};
 
 void iterate_parsers(struct discover_context *ctx)
 {
 
 void iterate_parsers(struct discover_context *ctx)
 {
-       struct parser *parser;
+       int i;
        unsigned int count = 0;
 
        pb_log("trying parsers for %s\n", ctx->device_path);
 
        unsigned int count = 0;
 
        pb_log("trying parsers for %s\n", ctx->device_path);
 
-       for (parser = __start_parsers; parser < __stop_parsers; parser++) {
-               pb_log("\ttrying parser '%s'\n", parser->name);
-               count += parser->parse(ctx);
+       for (i = 0; parsers[i]; i++) {
+               pb_log("\ttrying parser '%s'\n", parsers[i]->name);
+               count += parsers[i]->parse(ctx);
        }
        if (!count)
                pb_log("\tno boot_options found\n");
 }
 
        }
        if (!count)
                pb_log("\tno boot_options found\n");
 }
 
-static int compare_parsers(const void *a, const void *b)
-{
-       const struct parser *parser_a = a, *parser_b = b;
-
-       if (parser_a->priority > parser_b->priority)
-               return -1;
-
-       if (parser_a->priority < parser_b->priority)
-               return 1;
-
-       return 0;
-}
-
 void parser_init(void)
 {
 void parser_init(void)
 {
-       /* sort our parsers into descending priority order */
-       qsort(__start_parsers, __stop_parsers - __start_parsers,
-                       sizeof(struct parser), compare_parsers);
 }
 }
index 6101cd89f12ec8c0540b2701a7017cd2621f7de6..ace994581311108d4e8231d2c0eff046e48639d9 100644 (file)
@@ -318,4 +318,4 @@ static int yaboot_parse(struct discover_context *dc)
        return rc;
 }
 
        return rc;
 }
 
-define_parser(yaboot, 99, yaboot_parse);
+define_parser(yaboot, yaboot_parse);