[PATCH v2 7/7] syslinux: add syslinux parser support
Brett Grandbois
brett.grandbois at opengear.com
Fri Feb 9 14:26:41 AEDT 2018
Signed-off-by: Brett Grandbois <brett.grandbois at opengear.com>
---
discover/Makefile.am | 3 +-
discover/syslinux-parser.c | 484 +++++++++++++++++++++++++++
test/parser/Makefile.am | 13 +-
test/parser/data/syslinux-include-nest-1.cfg | 7 +
test/parser/data/syslinux-include-nest-2.cfg | 6 +
test/parser/data/syslinux-include-root.cfg | 18 +
test/parser/test-syslinux-explicit.c | 41 +++
test/parser/test-syslinux-global-append.c | 56 ++++
test/parser/test-syslinux-nested-config.c | 41 +++
test/parser/test-syslinux-single-yocto.c | 36 ++
10 files changed, 702 insertions(+), 3 deletions(-)
create mode 100644 discover/syslinux-parser.c
create mode 100644 test/parser/data/syslinux-include-nest-1.cfg
create mode 100644 test/parser/data/syslinux-include-nest-2.cfg
create mode 100644 test/parser/data/syslinux-include-root.cfg
create mode 100644 test/parser/test-syslinux-explicit.c
create mode 100644 test/parser/test-syslinux-global-append.c
create mode 100644 test/parser/test-syslinux-nested-config.c
create mode 100644 test/parser/test-syslinux-single-yocto.c
diff --git a/discover/Makefile.am b/discover/Makefile.am
index 4a6cbd0..ef4c602 100644
--- a/discover/Makefile.am
+++ b/discover/Makefile.am
@@ -52,7 +52,8 @@ discover_pb_discover_SOURCES = \
discover/user-event.h \
discover/kboot-parser.c \
discover/yaboot-parser.c \
- discover/pxe-parser.c
+ discover/pxe-parser.c \
+ discover/syslinux-parser.c
discover_pb_discover_LDADD = \
discover/grub2/grub2-parser.ro \
diff --git a/discover/syslinux-parser.c b/discover/syslinux-parser.c
new file mode 100644
index 0000000..8aef9c3
--- /dev/null
+++ b/discover/syslinux-parser.c
@@ -0,0 +1,484 @@
+#if defined(HAVE_CONFIG_H)
+#include "config.h"
+#endif
+
+#include <assert.h>
+#include <stdlib.h>
+#include <string.h>
+#include <i18n/i18n.h>
+#include <libgen.h>
+
+#include "log/log.h"
+#include "list/list.h"
+#include "file/file.h"
+#include "talloc/talloc.h"
+#include "types/types.h"
+#include "parser-conf.h"
+#include "parser-utils.h"
+#include "resource.h"
+#include "paths.h"
+
+struct syslinux_boot_option {
+ char *label;
+ char *image;
+ char *append;
+ char *initrd;
+ struct list_item list;
+};
+
+/* by spec 16 is allowed */
+#define INCLUDE_NEST_LIMIT 16
+
+struct syslinux_options {
+ struct list processed_options;
+ struct syslinux_boot_option *current_option;
+ int include_nest_level;
+ char *cfg_dir;
+};
+
+
+static const char *const syslinux_conf_files[] = {
+ "/boot/syslinux/syslinux.cfg",
+ "/syslinux/syslinux.cfg",
+ "/syslinux.cfg",
+ "/BOOT/SYSLINUX/SYSLINUX.CFG",
+ "/SYSLINUX/SYSLINUX.CFG",
+ "/SYSLINUX.CFG",
+ NULL
+};
+
+static const char *const syslinux_kernel_unsupported_extensions[] = {
+ ".0", /* eventually support PXE here? */
+ ".bin",
+ ".bs",
+ ".bss",
+ ".c32",
+ ".cbt",
+ ".com",
+ ".img",
+ NULL
+};
+
+static const char *const syslinux_ignored_names[] = {
+ "config",
+ "sysapend",
+ "localboot",
+ "ui",
+ "prompt",
+ "noescape",
+ "nocomplete",
+ "allowoptions",
+ "timeout",
+ "totaltimeout",
+ "ontimeout",
+ "onerror",
+ "serial",
+ "nohalt",
+ "console",
+ "font",
+ "kbdmap",
+ "say",
+ "display",
+ "f1",
+ "f2",
+ "f3",
+ "f4",
+ "f5"
+ "f6",
+ "f7",
+ "f8",
+ "f9",
+ "f10",
+ "f11",
+ "f12",
+ NULL
+};
+
+static const char *const syslinux_unsupported_boot_names[] = {
+ "boot",
+ "bss",
+ "pxe",
+ "fdimage",
+ "comboot",
+ "com32",
+ NULL
+};
+
+static struct conf_global_option syslinux_global_options[] = {
+ { .name = "default" },
+ { .name = "implicit" },
+ { .name = "append" },
+ { .name = NULL }
+};
+
+
+static void finish_boot_option(struct syslinux_options *state,
+ bool free_if_unused)
+{
+ /*
+ * in the normal this function signals a new image block which means
+ * move the current block to the list of processed items
+ * the special case is a label before an image block which we need to
+ * know whether to keep it for further processing or junk it
+ */
+ if (state->current_option) {
+ if (state->current_option->image) {
+ list_add(&state->processed_options,
+ &state->current_option->list);
+ state->current_option = NULL;
+ } else if (free_if_unused) {
+ talloc_free(state->current_option);
+ state->current_option = NULL;
+ }
+ }
+}
+
+static bool start_new_option(struct syslinux_options *state)
+{
+ bool ret = false;
+
+ finish_boot_option(state, false);
+ if (!state->current_option)
+ state->current_option = talloc_zero(state, struct syslinux_boot_option);
+
+ if (state->current_option)
+ ret = true;
+
+ return ret;
+}
+
+static void syslinux_process_pair(struct conf_context *conf, const char *name, char *value)
+{
+ struct syslinux_options *state = conf->parser_info;
+ char *buf, *pos, *path;
+ int len, rc;
+
+ /* ignore bare values */
+ if (!name)
+ return;
+
+ if (conf_param_in_list(syslinux_ignored_names, name))
+ return;
+
+ /* a new boot entry needs to terminate any prior one */
+ if (conf_param_in_list(syslinux_unsupported_boot_names, name)) {
+ finish_boot_option(state, true);
+ return;
+ }
+
+ if (streq(name, "label")) {
+ finish_boot_option(state, true);
+ state->current_option = talloc_zero(state,
+ struct syslinux_boot_option);
+ if (state->current_option)
+ state->current_option->label = talloc_strdup(state, value);
+ return;
+ }
+
+ if (streq(name, "linux")) {
+
+ if (start_new_option(state)) {
+ state->current_option->image = talloc_strdup(state, value);
+ if (!state->current_option->image) {
+ talloc_free(state->current_option);
+ state->current_option = NULL;
+ }
+ }
+
+ return;
+ }
+
+ if (streq(name, "kernel")) {
+
+ if (start_new_option(state)) {
+ /*
+ * by spec a linux image can not have any of these
+ * extensions, it can have no extension or anything not
+ * in this list
+ */
+ pos = strrchr(value, '.');
+ if (!pos ||
+ !conf_param_in_list(syslinux_kernel_unsupported_extensions, pos)) {
+ state->current_option->image = talloc_strdup(state, value);
+ if (!state->current_option->image) {
+ talloc_free(state->current_option);
+ state->current_option = NULL;
+ }
+ } else /* clean up any possible trailing label */
+ finish_boot_option(state, true);
+ }
+ return;
+ }
+
+
+
+ /* APPEND can be global and/or local so need special handling */
+ if (streq(name, "append")) {
+ if (state->current_option) {
+ /* by spec only take last if multiple APPENDs */
+ if (state->current_option->append)
+ talloc_free(state->current_option->append);
+ state->current_option->append = talloc_strdup(state, value);
+ if (!state->current_option->append) {
+ talloc_free(state->current_option);
+ state->current_option = NULL;
+ }
+ } else {
+ finish_boot_option(state, true);
+ conf_set_global_option(conf, name, value);
+ }
+ return;
+ }
+
+ /* now the general globals */
+ if (conf_set_global_option(conf, name, value)) {
+ finish_boot_option(state, true);
+ return;
+ }
+
+ if (streq(name, "initrd")) {
+ if (state->current_option) {
+ state->current_option->initrd = talloc_strdup(state, value);
+ if (!state->current_option->initrd) {
+ talloc_free(state->current_option);
+ state->current_option = NULL;
+ }
+ }
+ return;
+ }
+
+ if (streq(name, "include")) {
+ if (state->include_nest_level < INCLUDE_NEST_LIMIT) {
+ state->include_nest_level++;
+
+ /* if absolute in as-is */
+ if (value[0] == '/')
+ path = talloc_strdup(state, value);
+ else /* otherwise relative to the root config file */
+ path = join_paths(state, state->cfg_dir, value);
+
+ rc = parser_request_file(conf->dc, conf->dc->device, path, &buf, &len);
+ if (!rc) {
+ conf_parse_buf(conf, buf, len);
+
+ device_handler_status_dev_info(conf->dc->handler, conf->dc->device,
+ _("Parsed nested syslinux configuration from %s"), value);
+ talloc_free(buf);
+ } else {
+ device_handler_status_dev_info(conf->dc->handler, conf->dc->device,
+ _("Failed to parse nested syslinux configuration from %s"), value);
+ }
+
+ talloc_free(path);
+
+ state->include_nest_level--;
+ } else {
+ device_handler_status_dev_err(conf->dc->handler, conf->dc->device,
+ _("Nested syslinux INCLUDE exceeds limit...ignored"));
+ }
+ return;
+ }
+
+ pb_debug("%s: unknown name: %s\n", __func__, name);
+}
+
+static void syslinux_finalize(struct conf_context *conf)
+{
+ struct syslinux_options *state = conf->parser_info;
+ struct syslinux_boot_option *syslinux_opt;
+ struct discover_context *dc = conf->dc;
+ struct discover_boot_option *d_opt;
+ bool implicit_image = true;
+ char *args_sigfile_default;
+ const char *global_default;
+ const char *global_append;
+ struct boot_option *opt;
+ const char *image;
+ const char *label;
+
+ /* clean up any lingering boot entries */
+ finish_boot_option(state, true);
+
+ global_append = conf_get_global_option(conf, "append");
+ global_default = conf_get_global_option(conf, "default");
+
+ /*
+ * by spec '0' means disable
+ * note we set the default to '1' (which is by spec) in syslinux_parse
+ */
+ if (conf_get_global_option(conf, "implicit"), "0")
+ implicit_image = false;
+
+ list_for_each_entry(&state->processed_options, syslinux_opt, list) {
+ /* need a valid image */
+ if (!syslinux_opt->image)
+ continue;
+
+ image = syslinux_opt->image;
+ label = syslinux_opt->label;
+
+ /* if implicit is disabled we must have a label */
+ if (!label && !implicit_image)
+ continue;
+
+ d_opt = discover_boot_option_create(dc, dc->device);
+ if (!d_opt)
+ continue;
+ if (!d_opt->option)
+ goto fail;
+
+ opt = d_opt->option;
+
+ if (syslinux_opt->append) {
+ /* '-' can signal do not use global APPEND */
+ if (!strcmp(syslinux_opt->append, "-"))
+ opt->boot_args = talloc_strdup(opt, "");
+ else
+ opt->boot_args = talloc_asprintf(opt, "%s %s",
+ global_append,
+ syslinux_opt->append);
+ } else
+ opt->boot_args = talloc_strdup(opt, global_append);
+
+ if (!opt->boot_args)
+ goto fail;
+
+ opt->id = talloc_asprintf(opt, "%s#%s", dc->device->device->id, image);
+
+ if (!opt->id)
+ goto fail;
+
+ if (label) {
+ opt->name = talloc_strdup(opt, label);
+ if (!strcmp(label, global_default))
+ opt->is_default = true;
+
+ opt->description = talloc_asprintf(opt, "(%s) %s", label, image);
+ } else {
+ opt->name = talloc_strdup(opt, image);
+ opt->description = talloc_strdup(opt, image);
+ }
+
+ if (!opt->name)
+ goto fail;
+
+ d_opt->boot_image = create_devpath_resource(d_opt, dc->device, image);
+
+ if(!d_opt->boot_image)
+ goto fail;
+
+ if (syslinux_opt->initrd) {
+ d_opt->initrd = create_devpath_resource(d_opt, dc->device,
+ syslinux_opt->initrd);
+ opt->description = talloc_asprintf_append(opt->description,
+ " initrd=%s",
+ syslinux_opt->initrd);
+
+ if (!d_opt->initrd)
+ goto fail;
+ }
+
+ opt->description = talloc_asprintf_append(opt->description,
+ " args=%s",
+ opt->boot_args);
+
+ if (!opt->description)
+ goto fail;
+
+ args_sigfile_default = talloc_asprintf(d_opt, "%s.cmdline.sig",
+ image);
+
+ if (!args_sigfile_default)
+ goto fail;
+
+ d_opt->args_sig_file = create_devpath_resource(d_opt, dc->device,
+ args_sigfile_default);
+
+ if (!d_opt->args_sig_file)
+ goto fail;
+
+ talloc_free(args_sigfile_default);
+
+ conf_strip_str(opt->boot_args);
+ conf_strip_str(opt->description);
+
+ discover_context_add_boot_option(dc, d_opt);
+ continue;
+fail:
+ talloc_free(d_opt);
+ }
+}
+
+static int syslinux_parse(struct discover_context *dc)
+{
+ struct syslinux_options *state;
+ const char * const *filename;
+ struct conf_context *conf;
+ char *cfg_dir;
+ int len, rc;
+ char *buf;
+
+ /* Support block device boot only at present */
+ if (dc->event)
+ return -1;
+
+ conf = talloc_zero(dc, struct conf_context);
+
+ if (!conf)
+ return -1;
+
+ conf->dc = dc;
+ conf->global_options = syslinux_global_options,
+ conf_init_global_options(conf);
+ conf->get_pair = conf_get_pair_space;
+ conf->process_pair = syslinux_process_pair;
+
+ conf->parser_info = state = talloc_zero(conf, struct syslinux_options);
+ list_init(&state->processed_options);
+
+ /*
+ * set the global defaults
+ * by spec 'default' defaults to 'linux' and
+ * 'implicit' defaults to '1', we also just set
+ * and empty string in 'append' to make it easier
+ * in syslinux_finish
+ */
+ conf_set_global_option(conf, "default", "linux");
+ conf_set_global_option(conf, "implicit", "1");
+ conf_set_global_option(conf, "append", "");
+
+ for (filename = syslinux_conf_files; *filename; filename++) {
+ rc = parser_request_file(dc, dc->device, *filename, &buf, &len);
+ if (rc)
+ continue;
+
+ /*
+ * save location of root config file for possible
+ * INCLUDE directives later
+ *
+ * dirname can overwrite so need local copy to work on
+ */
+ cfg_dir = talloc_strdup(conf, *filename);
+ state->cfg_dir = talloc_strdup(state, dirname(cfg_dir));
+ talloc_free(cfg_dir);
+
+ conf_parse_buf(conf, buf, len);
+ device_handler_status_dev_info(dc->handler, dc->device,
+ _("Parsed syslinux configuration from %s"),
+ *filename);
+ talloc_free(buf);
+
+ syslinux_finalize(conf);
+ }
+
+ talloc_free(conf);
+ return 0;
+}
+
+static struct parser syslinux_parser = {
+ .name = "syslinux",
+ .parse = syslinux_parse,
+ .resolve_resource = resolve_devpath_resource,
+};
+
+register_parser(syslinux_parser);
diff --git a/test/parser/Makefile.am b/test/parser/Makefile.am
index 31300f0..a0795db 100644
--- a/test/parser/Makefile.am
+++ b/test/parser/Makefile.am
@@ -73,7 +73,12 @@ parser_TESTS = \
test/parser/test-pxe-discover-bootfile-relative-conffile \
test/parser/test-pxe-discover-bootfile-absolute-conffile \
test/parser/test-pxe-discover-bootfile-async-file \
- test/parser/test-unresolved-remove
+ test/parser/test-unresolved-remove \
+ test/parser/test-syslinux-single-yocto \
+ test/parser/test-syslinux-global-append \
+ test/parser/test-syslinux-explicit \
+ test/parser/test-syslinux-nested-config
+
TESTS += $(parser_TESTS)
check_PROGRAMS += $(parser_TESTS) test/parser/libtest.ro
@@ -82,7 +87,10 @@ check_DATA += \
test/parser/data/grub2-f18-ppc64.conf \
test/parser/data/grub2-f20-ppc.conf \
test/parser/data/grub2-ubuntu-13_04-x86.conf \
- test/parser/data/yaboot-rh8-ppc64.conf
+ test/parser/data/yaboot-rh8-ppc64.conf \
+ test/parser/data/syslinux-include-root.cfg \
+ test/parser/data/syslinux-include-nest-1.cfg \
+ test/parser/data/syslinux-include-nest-2.cfg
$(parser_TESTS): AM_CPPFLAGS += \
-I$(top_srcdir)/discover \
@@ -107,6 +115,7 @@ test_parser_libtest_ro_SOURCES = \
discover/yaboot-parser.c \
discover/kboot-parser.c \
discover/pxe-parser.c \
+ discover/syslinux-parser.c \
discover/platform.c \
discover/resource.c \
discover/paths.c \
diff --git a/test/parser/data/syslinux-include-nest-1.cfg b/test/parser/data/syslinux-include-nest-1.cfg
new file mode 100644
index 0000000..65e680e
--- /dev/null
+++ b/test/parser/data/syslinux-include-nest-1.cfg
@@ -0,0 +1,7 @@
+LABEL com32
+COM32 /boot/com32.c32
+
+INCLUDE syslinux-include-nest-2.cfg
+
+LABEL bss
+KERNEL /boot/test.bss
diff --git a/test/parser/data/syslinux-include-nest-2.cfg b/test/parser/data/syslinux-include-nest-2.cfg
new file mode 100644
index 0000000..26d7cff
--- /dev/null
+++ b/test/parser/data/syslinux-include-nest-2.cfg
@@ -0,0 +1,6 @@
+LABEL boot
+KERNEL /bzImage-boot
+INITRD /initrd-boot
+APPEND root=/dev/sda
+
+DEFAULT boot
diff --git a/test/parser/data/syslinux-include-root.cfg b/test/parser/data/syslinux-include-root.cfg
new file mode 100644
index 0000000..834360c
--- /dev/null
+++ b/test/parser/data/syslinux-include-root.cfg
@@ -0,0 +1,18 @@
+APPEND console=ttyS0
+
+LABEL floppy
+FDIMAGE /boot/floppy.img
+
+LABEL backup
+KERNEL /backup/vmlinuz
+APPEND root=/dev/sdb
+INITRD /boot/initrd
+
+LABEL comboot
+KERNEL /boot/comboot.com
+
+INCLUDE /syslinux-include-nest-1.cfg
+
+LABEL linux
+LINUX /boot/bzImage
+APPEND root=/dev/sdc
diff --git a/test/parser/test-syslinux-explicit.c b/test/parser/test-syslinux-explicit.c
new file mode 100644
index 0000000..5d23f50
--- /dev/null
+++ b/test/parser/test-syslinux-explicit.c
@@ -0,0 +1,41 @@
+/* test a standard yocto syslinux wic cfg */
+
+#include "parser-test.h"
+
+#if 0 /* PARSER_EMBEDDED_CONFIG */
+
+
+DEFAULT boot
+
+KERNEL /vmlinuz
+APPEND console=tty0
+
+LABEL backup
+KERNEL /backup/vmlinuz
+APPEND root=/dev/sdb
+INITRD /boot/initrd
+
+IMPLICIT 0
+
+#endif
+
+void run_test(struct parser_test *test)
+{
+ struct discover_boot_option *opt;
+ struct discover_context *ctx;
+
+ test_read_conf_embedded(test, "/boot/syslinux/syslinux.cfg");
+
+ test_run_parser(test, "syslinux");
+
+ ctx = test->ctx;
+
+ check_boot_option_count(ctx, 1);
+
+ opt = get_boot_option(ctx, 0);
+
+ check_name(opt, "backup");
+ check_resolved_local_resource(opt->boot_image, ctx->device, "/backup/vmlinuz");
+ check_args(opt, " root=/dev/sdb");
+ check_resolved_local_resource(opt->initrd, ctx->device, "/boot/initrd");
+}
diff --git a/test/parser/test-syslinux-global-append.c b/test/parser/test-syslinux-global-append.c
new file mode 100644
index 0000000..18af99a
--- /dev/null
+++ b/test/parser/test-syslinux-global-append.c
@@ -0,0 +1,56 @@
+
+#include "parser-test.h"
+
+#if 0 /* PARSER_EMBEDDED_CONFIG */
+
+APPEND console=ttyS0
+
+LABEL linux
+LINUX /vmlinuz
+APPEND console=tty0
+
+LABEL backup
+KERNEL /backup/vmlinuz
+APPEND root=/dev/sdb
+INITRD /boot/initrd
+
+LABEL hyphen
+KERNEL /test/vmlinuz
+APPEND -
+
+#endif
+
+void run_test(struct parser_test *test)
+{
+ struct discover_boot_option *opt;
+ struct discover_context *ctx;
+
+ test_read_conf_embedded(test, "/syslinux/syslinux.cfg");
+
+ test_run_parser(test, "syslinux");
+
+ ctx = test->ctx;
+
+ check_boot_option_count(ctx, 3);
+ opt = get_boot_option(ctx, 2);
+
+ check_name(opt, "linux");
+ check_resolved_local_resource(opt->boot_image, ctx->device, "/vmlinuz");
+ check_is_default(opt);
+ check_args(opt, "console=ttyS0 console=tty0");
+ check_not_present_resource(opt->initrd);
+
+ opt = get_boot_option(ctx, 1);
+
+ check_name(opt, "backup");
+ check_resolved_local_resource(opt->boot_image, ctx->device, "/backup/vmlinuz");
+ check_args(opt, "console=ttyS0 root=/dev/sdb");
+ check_resolved_local_resource(opt->initrd, ctx->device, "/boot/initrd");
+
+ opt = get_boot_option(ctx, 0);
+
+ check_name(opt, "hyphen");
+ check_resolved_local_resource(opt->boot_image, ctx->device, "/test/vmlinuz");
+ check_args(opt, "");
+ check_not_present_resource(opt->initrd);
+}
diff --git a/test/parser/test-syslinux-nested-config.c b/test/parser/test-syslinux-nested-config.c
new file mode 100644
index 0000000..73c4774
--- /dev/null
+++ b/test/parser/test-syslinux-nested-config.c
@@ -0,0 +1,41 @@
+
+#include "parser-test.h"
+
+
+void run_test(struct parser_test *test)
+{
+ struct discover_boot_option *opt;
+ struct discover_context *ctx;
+
+ test_read_conf_file(test, "syslinux-include-root.cfg", "/boot/syslinux/syslinux.cfg");
+ test_read_conf_file(test, "syslinux-include-nest-1.cfg", "/syslinux-include-nest-1.cfg");
+ test_read_conf_file(test, "syslinux-include-nest-2.cfg", "/boot/syslinux/syslinux-include-nest-2.cfg");
+
+ test_run_parser(test, "syslinux");
+
+ ctx = test->ctx;
+
+ check_boot_option_count(ctx, 3);
+
+ opt = get_boot_option(ctx, 1);
+
+ check_name(opt, "boot");
+ check_resolved_local_resource(opt->boot_image, ctx->device, "/bzImage-boot");
+ check_is_default(opt);
+ check_args(opt, "console=ttyS0 root=/dev/sda");
+ check_resolved_local_resource(opt->initrd, ctx->device, "/initrd-boot");
+
+ opt = get_boot_option(ctx, 2);
+
+ check_name(opt, "backup");
+ check_resolved_local_resource(opt->boot_image, ctx->device, "/backup/vmlinuz");
+ check_args(opt, "console=ttyS0 root=/dev/sdb");
+ check_resolved_local_resource(opt->initrd, ctx->device, "/boot/initrd");
+
+ opt = get_boot_option(ctx, 0);
+
+ check_name(opt, "linux");
+ check_resolved_local_resource(opt->boot_image, ctx->device, "/boot/bzImage");
+ check_args(opt, "console=ttyS0 root=/dev/sdc");
+ check_not_present_resource(opt->initrd);
+}
diff --git a/test/parser/test-syslinux-single-yocto.c b/test/parser/test-syslinux-single-yocto.c
new file mode 100644
index 0000000..e5e084d
--- /dev/null
+++ b/test/parser/test-syslinux-single-yocto.c
@@ -0,0 +1,36 @@
+/* test a standard yocto syslinux wic cfg */
+
+#include "parser-test.h"
+
+#if 0 /* PARSER_EMBEDDED_CONFIG */
+PROMPT 0
+TIMEOUT 0
+
+ALLOWOPTIONS 1
+SERIAL 0 115200
+
+DEFAULT boot
+LABEL boot
+KERNEL /vmlinuz
+APPEND console=ttyS0,115200n8 console=tty0
+#endif
+
+void run_test(struct parser_test *test)
+{
+ struct discover_boot_option *opt;
+ struct discover_context *ctx;
+
+ test_read_conf_embedded(test, "/syslinux.cfg");
+
+ test_run_parser(test, "syslinux");
+
+ ctx = test->ctx;
+
+ check_boot_option_count(ctx, 1);
+ opt = get_boot_option(ctx, 0);
+
+ check_name(opt, "boot");
+ check_resolved_local_resource(opt->boot_image, ctx->device, "/vmlinuz");
+ check_is_default(opt);
+ check_args(opt, " console=ttyS0,115200n8 console=tty0");
+}
--
2.7.4
More information about the Petitboot
mailing list