summaryrefslogtreecommitdiffstats
path: root/discover/grub2
diff options
context:
space:
mode:
authorJeremy Kerr <jk@ozlabs.org>2014-12-11 09:42:58 +0800
committerJeremy Kerr <jk@ozlabs.org>2014-12-11 15:11:42 +0800
commite330e3f5adf78d3ba77217995a4bc3cd1fd16f4c (patch)
tree5fe430243b363f991692f5e0929a5cf9a270fa88 /discover/grub2
parent9c33c54f7b431074a7d0daddce34140044aaadf6 (diff)
downloadtalos-petitboot-e330e3f5adf78d3ba77217995a4bc3cd1fd16f4c.tar.gz
talos-petitboot-e330e3f5adf78d3ba77217995a4bc3cd1fd16f4c.zip
discover/grub2: Allow unset and invalid defaults
If the default environment variable is unset or invalid (i.e., references a non-existent boot option), then GRUB2 will fallback to the first boot option present. This is preventing petitboot from autobooting where no default is explicitly set, or is stale. This change adds this fallback behaviour to petitboot. Because we don't know if the first option will be a default at parse time (as no other options matched the default env var), we need to keep options in a list, and register them with the discover server once the parse is complete. Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
Diffstat (limited to 'discover/grub2')
-rw-r--r--discover/grub2/grub2.h1
-rw-r--r--discover/grub2/script.c35
2 files changed, 35 insertions, 1 deletions
diff --git a/discover/grub2/grub2.h b/discover/grub2/grub2.h
index 0a89324..68176fb 100644
--- a/discover/grub2/grub2.h
+++ b/discover/grub2/grub2.h
@@ -98,6 +98,7 @@ struct grub2_script {
struct discover_boot_option *opt;
const char *filename;
unsigned int n_options;
+ struct list options;
};
struct grub2_parser {
diff --git a/discover/grub2/script.c b/discover/grub2/script.c
index 36f73b2..f521681 100644
--- a/discover/grub2/script.c
+++ b/discover/grub2/script.c
@@ -364,7 +364,7 @@ int statement_menuentry_execute(struct grub2_script *script,
opt->option->is_default = option_is_default(script, opt, id);
- discover_context_add_boot_option(script->ctx, opt);
+ list_add_tail(&script->options, &opt->list);
script->n_options++;
script->opt = NULL;
@@ -463,11 +463,43 @@ void script_register_function(struct grub2_script *script,
list_add(&script->symtab, &entry->list);
}
+static void set_fallback_default(struct grub2_script *script)
+{
+ struct discover_boot_option *opt, *first = NULL;
+ bool have_default = false;
+
+ list_for_each_entry(&script->options, opt, list) {
+ if (!first)
+ first = opt;
+ have_default = have_default || opt->option->is_default;
+ }
+
+ if (!have_default && first) {
+ const char *env = script_env_get(script, "default");
+
+ pb_log("grub: no explicit default (env default=%s), "
+ "falling back to first option (%s)\n",
+ env ?: "unset", first->option->name);
+
+ first->option->is_default = true;
+ }
+}
void script_execute(struct grub2_script *script)
{
+ struct discover_boot_option *opt, *tmp;
+
init_env(script);
statements_execute(script, script->statements);
+
+ set_fallback_default(script);
+
+ list_for_each_entry_safe(&script->options, opt, tmp, list)
+ discover_context_add_boot_option(script->ctx, opt);
+
+ /* Our option list will be invalid, as we've added all options to the
+ * discover context */
+ list_init(&script->options);
}
struct grub2_script *create_script(struct grub2_parser *parser,
@@ -480,6 +512,7 @@ struct grub2_script *create_script(struct grub2_parser *parser,
script->ctx = ctx;
list_init(&script->symtab);
+ list_init(&script->options);
register_builtins(script);
return script;
OpenPOWER on IntegriCloud