[PATCH v4 4/7] eficonfig: add direct menu entry access in change boot order
Masahisa Kojima
masahisa.kojima at linaro.org
Mon Oct 24 06:48:01 CEST 2022
This commit adds the direct menu entry access
in change boot order menu.
To call eficonfig_handle_direct_accessor() from change boot order
menu, refactoring is required to use 'eficonfig_entry' structure
in change boot order menu processing.
Signed-off-by: Masahisa Kojima <masahisa.kojima at linaro.org>
---
Newly added in v4
cmd/eficonfig.c | 165 ++++++++++++++++++++++++++++++------------------
1 file changed, 102 insertions(+), 63 deletions(-)
diff --git a/cmd/eficonfig.c b/cmd/eficonfig.c
index 56d9268f9f..8c131849be 100644
--- a/cmd/eficonfig.c
+++ b/cmd/eficonfig.c
@@ -94,20 +94,14 @@ struct eficonfig_boot_selection_data {
};
/**
- * struct eficonfig_boot_order - structure to be used to update BootOrder variable
+ * struct eficonfig_boot_order_data - structure to be used to update BootOrder variable
*
- * @num: index in the menu entry
- * @description: pointer to the description string
* @boot_index: boot option index
* @active: flag to include the boot option into BootOrder variable
- * @list: list structure
*/
-struct eficonfig_boot_order {
- u32 num;
- u16 *description;
+struct eficonfig_boot_order_data {
u32 boot_index;
bool active;
- struct list_head list;
};
/**
@@ -1924,7 +1918,7 @@ static void eficonfig_display_change_boot_order(struct efimenu *efi_menu)
{
bool reverse;
struct list_head *pos, *n;
- struct eficonfig_boot_order *entry;
+ struct eficonfig_entry *entry;
printf(ANSI_CLEAR_CONSOLE ANSI_CURSOR_POSITION
"\n ** Change Boot Order **\n"
@@ -1940,7 +1934,7 @@ static void eficonfig_display_change_boot_order(struct efimenu *efi_menu)
/* draw boot option list */
list_for_each_safe(pos, n, &efi_menu->list) {
- entry = list_entry(pos, struct eficonfig_boot_order, list);
+ entry = list_entry(pos, struct eficonfig_entry, list);
reverse = (entry->num == efi_menu->active);
printf(ANSI_CURSOR_POSITION, entry->num + 4, 7);
@@ -1949,13 +1943,34 @@ static void eficonfig_display_change_boot_order(struct efimenu *efi_menu)
puts(ANSI_COLOR_REVERSE);
if (entry->num < efi_menu->count - 2) {
- if (entry->active)
+ if (((struct eficonfig_boot_order_data *)entry->data)->active)
printf("[*] ");
else
printf("[ ] ");
}
- printf("%ls", entry->description);
+ if (reverse && efi_menu->direct_access_mode) {
+ size_t len = u16_strlen(efi_menu->accessor_str);
+ char *accessor_str, *p;
+
+ accessor_str = calloc(1, utf16_utf8_strlen(efi_menu->accessor_str) + 1);
+ if (!accessor_str) {
+ printf("%s", entry->title);
+ return;
+ }
+ p = accessor_str;
+ utf16_utf8_strncpy(&p, efi_menu->accessor_str, EFICONFIG_ACCESSOR_STR_MAX);
+ len = strlen(accessor_str);
+ if (!strncasecmp(accessor_str, entry->title, len)) {
+ printf("%.*s" ANSI_COLOR_RESET "%s", (int)len, entry->title,
+ &entry->title[len]);
+ } else {
+ printf("%s", entry->title);
+ }
+ free(accessor_str);
+ } else {
+ printf("%s", entry->title);
+ }
if (reverse)
puts(ANSI_COLOR_RESET);
@@ -1972,12 +1987,18 @@ static efi_status_t eficonfig_choice_change_boot_order(struct efimenu *efi_menu)
{
int esc = 0;
struct list_head *pos, *n;
- struct eficonfig_boot_order *tmp;
enum bootmenu_key key = KEY_NONE;
- struct eficonfig_boot_order *entry;
+ struct eficonfig_entry *entry, *tmp;
if (efi_menu->direct_access_mode) {
- eficonfig_handle_direct_accessor(efi_menu);
+ if (eficonfig_handle_direct_accessor(efi_menu)) {
+ /* If the selected entry is "Save" or "Quit", complete selection */
+ if (efi_menu->active == efi_menu->count - 2)
+ return EFI_SUCCESS;
+
+ if (efi_menu->active == efi_menu->count - 1)
+ return EFI_ABORTED;
+ }
return EFI_NOT_READY;
}
@@ -1988,11 +2009,11 @@ static efi_status_t eficonfig_choice_change_boot_order(struct efimenu *efi_menu)
case KEY_PLUS:
if (efi_menu->active > 0) {
list_for_each_safe(pos, n, &efi_menu->list) {
- entry = list_entry(pos, struct eficonfig_boot_order, list);
+ entry = list_entry(pos, struct eficonfig_entry, list);
if (entry->num == efi_menu->active)
break;
}
- tmp = list_entry(pos->prev, struct eficonfig_boot_order, list);
+ tmp = list_entry(pos->prev, struct eficonfig_entry, list);
entry->num--;
tmp->num++;
list_del(&tmp->list);
@@ -2006,11 +2027,11 @@ static efi_status_t eficonfig_choice_change_boot_order(struct efimenu *efi_menu)
case KEY_MINUS:
if (efi_menu->active < efi_menu->count - 3) {
list_for_each_safe(pos, n, &efi_menu->list) {
- entry = list_entry(pos, struct eficonfig_boot_order, list);
+ entry = list_entry(pos, struct eficonfig_entry, list);
if (entry->num == efi_menu->active)
break;
}
- tmp = list_entry(pos->next, struct eficonfig_boot_order, list);
+ tmp = list_entry(pos->next, struct eficonfig_entry, list);
entry->num++;
tmp->num--;
list_del(&entry->list);
@@ -2036,9 +2057,11 @@ static efi_status_t eficonfig_choice_change_boot_order(struct efimenu *efi_menu)
case KEY_SPACE:
if (efi_menu->active < efi_menu->count - 2) {
list_for_each_safe(pos, n, &efi_menu->list) {
- entry = list_entry(pos, struct eficonfig_boot_order, list);
+ entry = list_entry(pos, struct eficonfig_entry, list);
if (entry->num == efi_menu->active) {
- entry->active = entry->active ? false : true;
+ struct eficonfig_boot_order_data *data = entry->data;
+
+ data->active = data->active ? false : true;
return EFI_NOT_READY;
}
}
@@ -2047,6 +2070,8 @@ static efi_status_t eficonfig_choice_change_boot_order(struct efimenu *efi_menu)
case KEY_QUIT:
return EFI_ABORTED;
case KEY_AMPERSAND:
+ memset(efi_menu->accessor_str, 0,
+ EFICONFIG_ACCESSOR_STR_MAX * sizeof(u16));
efi_menu->direct_access_mode = true;
return EFI_NOT_READY;
default:
@@ -2067,12 +2092,13 @@ static efi_status_t eficonfig_choice_change_boot_order(struct efimenu *efi_menu)
static efi_status_t eficonfig_add_change_boot_order_entry(struct efimenu *efi_menu,
u32 boot_index, bool active)
{
+ char *title, *p;
efi_status_t ret;
efi_uintn_t size;
void *load_option;
struct efi_load_option lo;
u16 varname[] = u"Boot####";
- struct eficonfig_boot_order *entry;
+ struct eficonfig_boot_order_data *data;
efi_create_indexed_name(varname, sizeof(varname), "Boot", boot_index);
load_option = efi_get_var(varname, &efi_global_variable_guid, &size);
@@ -2080,31 +2106,38 @@ static efi_status_t eficonfig_add_change_boot_order_entry(struct efimenu *efi_me
return EFI_SUCCESS;
ret = efi_deserialize_load_option(&lo, load_option, &size);
- if (ret != EFI_SUCCESS) {
- free(load_option);
- return ret;
+ if (ret != EFI_SUCCESS)
+ goto out;
+
+ data = calloc(1, sizeof(struct eficonfig_boot_order_data));
+ if (!data) {
+ ret = EFI_OUT_OF_RESOURCES;
+ goto out;
}
- entry = calloc(1, sizeof(struct eficonfig_boot_order));
- if (!entry) {
- free(load_option);
- return EFI_OUT_OF_RESOURCES;
+ title = calloc(1, utf16_utf8_strlen(lo.label) + 1);
+ if (!title) {
+ free(data);
+ ret = EFI_OUT_OF_RESOURCES;
+ goto out;
}
+ p = title;
+ utf16_utf8_strcpy(&p, lo.label);
- entry->description = u16_strdup(lo.label);
- if (!entry->description) {
- free(load_option);
- free(entry);
- return EFI_OUT_OF_RESOURCES;
+ data->boot_index = boot_index;
+ data->active = active;
+
+ ret = eficonfig_append_menu_entry(efi_menu, title, NULL, data);
+ if (ret != EFI_SUCCESS) {
+ free(data);
+ free(title);
+ goto out;
}
- entry->num = efi_menu->count++;
- entry->boot_index = boot_index;
- entry->active = active;
- list_add_tail(&entry->list, &efi_menu->list);
+out:
free(load_option);
- return EFI_SUCCESS;
+ return ret;
}
/**
@@ -2119,8 +2152,8 @@ static efi_status_t eficonfig_create_change_boot_order_entry(struct efimenu *efi
u16 *bootorder, efi_uintn_t num)
{
u32 i;
+ char *title;
efi_status_t ret;
- struct eficonfig_boot_order *entry;
/* list the load option in the order of BootOrder variable */
for (i = 0; i < num; i++) {
@@ -2147,27 +2180,25 @@ static efi_status_t eficonfig_create_change_boot_order_entry(struct efimenu *efi
}
/* add "Save" and "Quit" entries */
- entry = calloc(1, sizeof(struct eficonfig_boot_order));
- if (!entry)
+ title = strdup("Save");
+ if (!title) {
+ ret = EFI_OUT_OF_RESOURCES;
goto out;
+ }
- entry->num = efi_menu->count++;
- entry->description = u16_strdup(u"Save");
- list_add_tail(&entry->list, &efi_menu->list);
-
- entry = calloc(1, sizeof(struct eficonfig_boot_order));
- if (!entry)
+ ret = eficonfig_append_menu_entry(efi_menu, title, NULL, NULL);
+ if (ret != EFI_SUCCESS)
goto out;
- entry->num = efi_menu->count++;
- entry->description = u16_strdup(u"Quit");
- list_add_tail(&entry->list, &efi_menu->list);
+ ret = eficonfig_append_quit_entry(efi_menu);
+ if (ret != EFI_SUCCESS)
+ goto out;
efi_menu->active = 0;
return EFI_SUCCESS;
out:
- return EFI_OUT_OF_RESOURCES;
+ return ret;
}
/**
@@ -2183,13 +2214,17 @@ static efi_status_t eficonfig_process_change_boot_order(void *data)
efi_status_t ret;
efi_uintn_t num, size;
struct list_head *pos, *n;
- struct eficonfig_boot_order *entry;
+ struct eficonfig_entry *entry;
struct efimenu *efi_menu;
efi_menu = calloc(1, sizeof(struct efimenu));
if (!efi_menu)
return EFI_OUT_OF_RESOURCES;
+ efi_menu->accessor_str = calloc(1, 32);
+ if (!efi_menu->accessor_str)
+ return EFI_OUT_OF_RESOURCES;
+
bootorder = efi_get_var(u"BootOrder", &efi_global_variable_guid, &size);
INIT_LIST_HEAD(&efi_menu->list);
@@ -2214,9 +2249,16 @@ static efi_status_t eficonfig_process_change_boot_order(void *data)
/* create new BootOrder */
count = 0;
list_for_each_safe(pos, n, &efi_menu->list) {
- entry = list_entry(pos, struct eficonfig_boot_order, list);
- if (entry->active)
- new_bootorder[count++] = entry->boot_index;
+ struct eficonfig_boot_order_data *data;
+
+ entry = list_entry(pos, struct eficonfig_entry, list);
+ /* exit the loop when iteration reaches "Save" */
+ if (!strncmp(entry->title, "Save", strlen("Save")))
+ break;
+
+ data = entry->data;
+ if (data->active)
+ new_bootorder[count++] = data->boot_index;
}
size = count * sizeof(u16);
@@ -2235,15 +2277,12 @@ static efi_status_t eficonfig_process_change_boot_order(void *data)
}
}
out:
+ free(bootorder);
list_for_each_safe(pos, n, &efi_menu->list) {
- entry = list_entry(pos, struct eficonfig_boot_order, list);
- list_del(&entry->list);
- free(entry->description);
- free(entry);
+ entry = list_entry(pos, struct eficonfig_entry, list);
+ free(entry->data);
}
-
- free(bootorder);
- free(efi_menu);
+ eficonfig_destroy(efi_menu);
/* to stay the parent menu */
ret = (ret == EFI_ABORTED) ? EFI_NOT_READY : ret;
--
2.17.1
More information about the U-Boot
mailing list