deepblue/ipmitool
1
0
Fork 0
mirror of https://github.com/ipmitool/ipmitool.git synced 2025-05-10 18:47:22 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

chassis: bootparam/bootdev: Refactor for less magic

Browse Source 
Refactor the boot flags decoder:

* Add macros for boot flag bits, replace magic numbers in
  the `chassis bootparam get 5` and in `chassis bootdev`
  handlers.

  The macros are prefixed with BFx_ where x stands for the
  boot flags data byte as per IPMI 2.0 specification Table 28-14;

* Add decoding of remote/redirected media boot flags;

* Remove erroneous decoding of boot flags byte 3 bit 1 as
  Sleep button lockout whereas the bit is a part of console
  redirection setting;

* Fix console redirection settings reported under the 'BIOS verbosity'
  header and vice versa;

* Fix resetting of all other boot flags in the data byte when
  setting any of the bits in the same byte. This fixes inability
  to set both 'efiboot' and 'persistent' bits at the same time,
  and other similar cases.

Resolves ipmitool/ipmitool#163
Signed-off-by: Alexander Amelkin <alexander@amelkin.msk.ru>
This commit is contained in:
Alexander Amelkin 2019-11-06 11:40:15 +03:00 committed by Alexander Amelkin
parent 0e3e436eb6
commit 4b89f1b42d
1 changed files with 339 additions and 86 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

425
lib/ipmi_chassis.c
View File

@ -53,6 +53,96 @@
#define CHASSIS_BOOT_MBOX_MAX_BLOCK 0xFF #define CHASSIS_BOOT_MBOX_MAX_BLOCK 0xFF
#define CHASSIS_BOOT_MBOX_MAX_BLOCKS (CHASSIS_BOOT_MBOX_MAX_BLOCK + 1) #define CHASSIS_BOOT_MBOX_MAX_BLOCKS (CHASSIS_BOOT_MBOX_MAX_BLOCK + 1)
/* Get/Set system boot option boot flags bit definitions */
/* Boot flags byte 1 bits */
#define BF1_VALID_SHIFT 7
#define BF1_INVALID 0
#define BF1_VALID (1 << BF1_VALID_SHIFT)
#define BF1_VALID_MASK BF1_VALID
#define BF1_PERSIST_SHIFT 6
#define BF1_ONCE 0
#define BF1_PERSIST (1 << BF1_PERSIST_SHIFT)
#define BF1_PERSIST_MASK BF1_PERSIST
#define BF1_BOOT_TYPE_SHIFT 5
#define BF1_BOOT_TYPE_LEGACY 0
#define BF1_BOOT_TYPE_EFI (1 << BF1_BOOT_TYPE_SHIFT)
#define BF1_BOOT_TYPE_MASK BF1_BOOT_TYPE_EFI
/* Boot flags byte 2 bits */
#define BF2_CMOS_CLEAR_SHIFT 7
#define BF2_CMOS_CLEAR (1 << BF2_CMOS_CLEAR_SHIFT)
#define BF2_CMOS_CLEAR_MASK BF2_CMOS_CLEAR
#define BF2_KEYLOCK_SHIFT 6
#define BF2_KEYLOCK (1 << BF2_KEYLOCK_SHIFT)
#define BF2_KEYLOCK_MASK BF2_KEYLOCK
#define BF2_BOOTDEV_SHIFT 2
#define BF2_BOOTDEV_DEFAULT (0 << BF2_BOOTDEV_SHIFT)
#define BF2_BOOTDEV_PXE (1 << BF2_BOOTDEV_SHIFT)
#define BF2_BOOTDEV_HDD (2 << BF2_BOOTDEV_SHIFT)
#define BF2_BOOTDEV_HDD_SAFE (3 << BF2_BOOTDEV_SHIFT)
#define BF2_BOOTDEV_DIAG_PART (4 << BF2_BOOTDEV_SHIFT)
#define BF2_BOOTDEV_CDROM (5 << BF2_BOOTDEV_SHIFT)
#define BF2_BOOTDEV_SETUP (6 << BF2_BOOTDEV_SHIFT)
#define BF2_BOOTDEV_REMOTE_FDD (7 << BF2_BOOTDEV_SHIFT)
#define BF2_BOOTDEV_REMOTE_CDROM (8 << BF2_BOOTDEV_SHIFT)
#define BF2_BOOTDEV_REMOTE_PRIMARY_MEDIA (9 << BF2_BOOTDEV_SHIFT)
#define BF2_BOOTDEV_REMOTE_HDD (11 << BF2_BOOTDEV_SHIFT)
#define BF2_BOOTDEV_FDD (15 << BF2_BOOTDEV_SHIFT)
#define BF2_BOOTDEV_MASK (0xF << BF2_BOOTDEV_SHIFT)
#define BF2_BLANK_SCREEN_SHIFT 1
#define BF2_BLANK_SCREEN (1 << BF2_BLANK_SCREEN_SHIFT)
#define BF2_BLANK_SCREEN_MASK BF2_BLANK_SCREEN
#define BF2_RESET_LOCKOUT_SHIFT 0
#define BF2_RESET_LOCKOUT (1 << BF2_RESET_LOCKOUT_SHIFT)
#define BF2_RESET_LOCKOUT_MASK BF2_RESET_LOCKOUT
/* Boot flags byte 3 bits */
#define BF3_POWER_LOCKOUT_SHIFT 7
#define BF3_POWER_LOCKOUT (1 << BF3_POWER_LOCKOUT_SHIFT)
#define BF3_POWER_LOCKOUT_MASK BF3_POWER_LOCKOUT
#define BF3_VERBOSITY_SHIFT 5
#define BF3_VERBOSITY_DEFAULT (0 << BF3_VERBOSITY_SHIFT)
#define BF3_VERBOSITY_QUIET (1 << BF3_VERBOSITY_SHIFT)
#define BF3_VERBOSITY_VERBOSE (2 << BF3_VERBOSITY_SHIFT)
#define BF3_VERBOSITY_MASK (3 << BF3_VERBOSITY_SHIFT)
#define BF3_EVENT_TRAPS_SHIFT 4
#define BF3_EVENT_TRAPS (1 << BF3_EVENT_TRAPS_SHIFT)
#define BF3_EVENT_TRAPS_MASK BF3_EVENT_TRAPS
#define BF3_PASSWD_BYPASS_SHIFT 3
#define BF3_PASSWD_BYPASS (1 << BF3_PASSWD_BYPASS_SHIFT)
#define BF3_PASSWD_BYPASS_MASK BF3_PASSWD_BYPASS
#define BF3_SLEEP_LOCKOUT_SHIFT 2
#define BF3_SLEEP_LOCKOUT (1 << BF3_SLEEP_LOCKOUT_SHIFT)
#define BF3_SLEEP_LOCKOUT_MASK BF3_SLEEP_LOCKOUT
#define BF3_CONSOLE_REDIR_SHIFT 0
#define BF3_CONSOLE_REDIR_DEFAULT (0 << BF3_CONSOLE_REDIR_SHIFT)
#define BF3_CONSOLE_REDIR_SUPPRESS (1 << BF3_CONSOLE_REDIR_SHIFT)
#define BF3_CONSOLE_REDIR_ENABLE (2 << BF3_CONSOLE_REDIR_SHIFT)
#define BF3_CONSOLE_REDIR_MASK (3 << BF3_CONSOLE_REDIR_SHIFT)
/* Boot flags byte 4 bits */
#define BF4_SHARED_MODE_SHIFT 3
#define BF4_SHARED_MODE (1 << BF4_SHARED_MODE_SHIFT)
#define BF4_SHARED_MODE_MASK BF4_SHARED_MODE
#define BF4_BIOS_MUX_SHIFT 0
#define BF4_BIOS_MUX_DEFAULT (0 << BF4_BIOS_MUX_SHIFT)
#define BF4_BIOS_MUX_BMC (1 << BF4_BIOS_MUX_SHIFT)
#define BF4_BIOS_MUX_SYSTEM (2 << BF4_BIOS_MUX_SHIFT)
#define BF4_BIOS_MUX_MASK (7 << BF4_BIOS_MUX_SHIFT)
typedef struct { typedef struct {
uint8_t iana[CHASSIS_BOOT_MBOX_IANA_SZ]; uint8_t iana[CHASSIS_BOOT_MBOX_IANA_SZ];
uint8_t data[CHASSIS_BOOT_MBOX_BLOCK0_SZ]; uint8_t data[CHASSIS_BOOT_MBOX_BLOCK0_SZ];
@ -776,79 +866,147 @@ ipmi_chassis_get_bootparam(struct ipmi_intf * intf,
{ {
printf( " Boot Flags :\n"); printf( " Boot Flags :\n");
if((rsp->data[2]&0x80) == 0x80) if(rsp->data[2] & BF1_VALID)
printf(" - Boot Flag Valid\n"); printf(" - Boot Flag Valid\n");
else else
printf(" - Boot Flag Invalid\n"); printf(" - Boot Flag Invalid\n");
if((rsp->data[2]&0x40) == 0x40) if(rsp->data[2] & BF1_PERSIST)
printf(" - Options apply to all future boots\n"); printf(" - Options apply to all future boots\n");
else else
printf(" - Options apply to only next boot\n"); printf(" - Options apply to only next boot\n");
if((rsp->data[2]&0x20) == 0x20) if(rsp->data[2] & BF1_BOOT_TYPE_EFI)
printf(" - BIOS EFI boot \n"); printf(" - BIOS EFI boot \n");
else else
printf(" - BIOS PC Compatible (legacy) boot \n"); printf(" - BIOS PC Compatible (legacy) boot \n");
if((rsp->data[3]&0x80) == 0x80) if(rsp->data[3] & BF2_CMOS_CLEAR)
printf(" - CMOS Clear\n"); printf(" - CMOS Clear\n");
if((rsp->data[3]&0x40) == 0x40) if(rsp->data[3] & BF2_KEYLOCK)
printf(" - Lock Keyboard\n"); printf(" - Lock Keyboard\n");
printf(" - Boot Device Selector : "); printf(" - Boot Device Selector : ");
switch( ((rsp->data[3]>>2)&0x0f)) switch(rsp->data[3] & BF2_BOOTDEV_MASK)
{ {
case 0: printf("No override\n"); break; case BF2_BOOTDEV_DEFAULT:
case 1: printf("Force PXE\n"); break; printf("No override\n");
case 2: printf("Force Boot from default Hard-Drive\n"); break; break;
case 3: printf("Force Boot from default Hard-Drive, request Safe-Mode\n"); break; case BF2_BOOTDEV_PXE:
case 4: printf("Force Boot from Diagnostic Partition\n"); break; printf("Force PXE\n");
case 5: printf("Force Boot from CD/DVD\n"); break; break;
case 6: printf("Force Boot into BIOS Setup\n"); break; case BF2_BOOTDEV_HDD:
case 15: printf("Force Boot from Floppy/primary removable media\n"); break; printf("Force Boot from default Hard-Drive\n");
default: printf("Flag error\n"); break; break;
case BF2_BOOTDEV_HDD_SAFE:
printf("Force Boot from default Hard-Drive, "
"request Safe-Mode\n");
break;
case BF2_BOOTDEV_DIAG_PART:
printf("Force Boot from Diagnostic Partition\n");
break;
case BF2_BOOTDEV_CDROM:
printf("Force Boot from CD/DVD\n");
break;
case BF2_BOOTDEV_SETUP:
printf("Force Boot into BIOS Setup\n");
break;
case BF2_BOOTDEV_REMOTE_FDD:
printf("Force Boot from remotely connected "
"Floppy/primary removable media\n");
break;
case BF2_BOOTDEV_REMOTE_CDROM:
printf("Force Boot from remotely connected "
"CD/DVD\n");
break;
case BF2_BOOTDEV_REMOTE_PRIMARY_MEDIA:
printf("Force Boot from primary remote media\n");
break;
case BF2_BOOTDEV_REMOTE_HDD:
printf("Force Boot from remotely connected "
"Hard-Drive\n");
break;
case BF2_BOOTDEV_FDD:
printf("Force Boot from Floppy/primary "
"removable media\n");
break;
default:
printf("Flag error\n");
break;
} }
if((rsp->data[3]&0x02) == 0x02) if(rsp->data[3] & BF2_BLANK_SCREEN)
printf(" - Screen blank\n"); printf(" - Screen blank\n");
if((rsp->data[3]&0x01) == 0x01) if(rsp->data[3] & BF2_RESET_LOCKOUT)
printf(" - Lock out Reset buttons\n"); printf(" - Lock out Reset buttons\n");
if((rsp->data[4]&0x80) == 0x80) if(rsp->data[4] & BF3_POWER_LOCKOUT)
printf(" - Lock out (power off/sleep request) vi Power Button\n"); printf(" - Lock out (power off/sleep "
printf(" - Console Redirection control : "); "request) via Power Button\n");
switch( ((rsp->data[4]>>5)&0x03))
{
case 0: printf("System Default\n"); break;
case 1: printf("Request Quiet Display\n"); break;
case 2: printf("Request Verbose Display\n"); break;
default: printf("Flag error\n"); break;
}
if((rsp->data[4]&0x10) == 0x10)
printf(" - Force progress event traps\n");
if((rsp->data[4]&0x08) == 0x08)
printf(" - User password bypass\n");
if((rsp->data[4]&0x04) == 0x04)
printf(" - Lock Out Sleep Button\n");
if((rsp->data[4]&0x02) == 0x02)
printf(" - Lock Out Sleep Button\n");
printf(" - BIOS verbosity : "); printf(" - BIOS verbosity : ");
switch( ((rsp->data[4]>>0)&0x03)) switch(rsp->data[4] & BF3_VERBOSITY_MASK)
{ {
case 0: printf("Console redirection occurs per BIOS configuration setting (default)\n"); break; case BF3_VERBOSITY_DEFAULT:
case 1: printf("Suppress (skip) console redirection if enabled\n"); break; printf("System Default\n");
case 2: printf("Request console redirection be enabled\n"); break; break;
default: printf("Flag error\n"); break; case BF3_VERBOSITY_QUIET:
printf("Request Quiet Display\n");
break;
case BF3_VERBOSITY_VERBOSE:
printf("Request Verbose Display\n");
break;
default:
printf("Flag error\n");
break;
}
if(rsp->data[4] & BF3_EVENT_TRAPS)
printf(" - Force progress event traps\n");
if(rsp->data[4] & BF3_PASSWD_BYPASS)
printf(" - User password bypass\n");
if(rsp->data[4] & BF3_SLEEP_LOCKOUT)
printf(" - Lock Out Sleep Button\n");
printf(" - Console Redirection control : ");
switch(rsp->data[4] & BF3_CONSOLE_REDIR_MASK)
{
case BF3_CONSOLE_REDIR_DEFAULT:
printf(
"Console redirection occurs per BIOS "
"configuration setting (default)\n");
break;
case BF3_CONSOLE_REDIR_SUPPRESS:
printf("Suppress (skip) console redirection "
"if enabled\n");
break;
case BF3_CONSOLE_REDIR_ENABLE:
printf("Request console redirection be "
"enabled\n");
break;
default:
printf("Flag error\n");
break;
} }
if((rsp->data[5]&0x08) == 0x08) if(rsp->data[5] & BF4_SHARED_MODE)
printf(" - BIOS Shared Mode Override\n"); printf(" - BIOS Shared Mode Override\n");
printf(" - BIOS Mux Control Override : "); printf(" - BIOS Mux Control Override : ");
switch( ((rsp->data[5]>>0)&0x07)) switch (rsp->data[5] & BF4_BIOS_MUX_MASK) {
{ case BF4_BIOS_MUX_DEFAULT:
case 0: printf("BIOS uses recommended setting of the mux at the end of POST\n"); break; printf("BIOS uses recommended setting of the "
case 1: printf("Requests BIOS to force mux to BMC at conclusion of POST/start of OS boot\n"); break; "mux at the end of POST\n");
case 2: printf("Requests BIOS to force mux to system at conclusion of POST/start of OS boot\n"); break; break;
default: printf("Flag error\n"); break; case BF4_BIOS_MUX_BMC:
printf(
"Requests BIOS to force mux to BMC at "
"conclusion of POST/start of OS boot\n");
break;
case BF4_BIOS_MUX_SYSTEM:
printf(
"Requests BIOS to force mux to system "
"at conclusion of POST/start of "
"OS boot\n");
break;
default:
printf("Flag error\n");
break;
} }
} }
break; break;
@ -1706,49 +1864,144 @@ ipmi_chassis_main(struct ipmi_intf * intf, int argc, char ** argv)
unsigned char flags[5]; unsigned char flags[5];
static struct { static struct {
char *name; char *name;
int i; off_t offset;
#define BF1_OFFSET 0
#define BF2_OFFSET 1
#define BF3_OFFSET 2
#define BF4_OFFSET 3
unsigned char mask; unsigned char mask;
unsigned char value; unsigned char value;
char *desc; char *desc;
} options[] = { } *op, options[] = {
/* data 1 */ /* data 1 */
{"valid", 0, (1<<7), (1<<7), {
"Boot flags valid"}, "valid",
{"persistent", 0, (1<<6), (1<<6), BF1_OFFSET,
"Changes are persistent for all future boots"}, BF1_VALID_MASK,
{"efiboot", 0, (1<<5), (1<<5), BF1_VALID,
"Extensible Firmware Interface Boot (EFI)"}, "Boot flags valid"
},
{
"persistent",
BF1_OFFSET,
BF1_PERSIST_MASK,
BF1_PERSIST,
"Changes are persistent for "
"all future boots"
},
{
"efiboot",
BF1_OFFSET,
BF1_BOOT_TYPE_MASK,
BF1_BOOT_TYPE_EFI,
"Extensible Firmware Interface "
"Boot (EFI)"
},
/* data 2 */ /* data 2 */
{"clear-cmos", 1, (1<<7), (1<<7), {
"CMOS clear"}, "clear-cmos",
{"lockkbd", 1, (1<<6), (1<<6), BF2_OFFSET,
"Lock Keyboard"}, BF2_CMOS_CLEAR_MASK,
BF2_CMOS_CLEAR,
"CMOS clear"
},
{
"lockkbd",
BF2_OFFSET,
BF2_KEYLOCK_MASK,
BF2_KEYLOCK,
"Lock Keyboard"
},
/* data2[5:2] is parsed elsewhere */ /* data2[5:2] is parsed elsewhere */
{"screenblank", 1, (1<<1), (1<<1), {
"Screen Blank"}, "screenblank",
{"lockoutreset", 1, (1<<0), (1<<0), BF2_OFFSET,
"Lock out Resetbuttons"}, BF2_BLANK_SCREEN_MASK,
BF2_BLANK_SCREEN,
"Screen Blank"
},
{
"lockoutreset",
BF2_OFFSET,
BF2_RESET_LOCKOUT_MASK,
BF2_RESET_LOCKOUT,
"Lock out Reset buttons"
},
/* data 3 */ /* data 3 */
{"lockout_power", 2, (1<<7), (1<<7), {
"Lock out (power off/sleep request) via Power Button"}, "lockout_power",
{"verbose=default", 2, (3<<5), (0<<5), BF3_OFFSET,
"Request quiet BIOS display"}, BF3_POWER_LOCKOUT_MASK,
{"verbose=no", 2, (3<<5), (1<<5), BF3_POWER_LOCKOUT,
"Request quiet BIOS display"}, "Lock out (power off/sleep "
{"verbose=yes", 2, (3<<5), (2<<5), "request) via Power Button"
"Request verbose BIOS display"}, },
{"force_pet", 2, (1<<4), (1<<4), {
"Force progress event traps"}, "verbose=default",
{"upw_bypass", 2, (1<<3), (1<<3), BF3_OFFSET,
"User password bypass"}, BF3_VERBOSITY_MASK,
{"lockout_sleep", 2, (1<<2), (1<<2), BF3_VERBOSITY_DEFAULT,
"Log Out Sleep Button"}, "Request quiet BIOS display"
{"cons_redirect=default", 2, (3<<0), (0<<0), },
"Console redirection occurs per BIOS configuration setting"}, {
{"cons_redirect=skip", 2, (3<<0), (1<<0), "verbose=no",
"Suppress (skip) console redirection if enabled"}, BF3_OFFSET,
{"cons_redirect=enable", 2, (3<<0), (2<<0), BF3_VERBOSITY_MASK,
"Suppress (skip) console redirection if enabled"}, BF3_VERBOSITY_QUIET,
"Request quiet BIOS display"
},
{
"verbose=yes",
BF3_OFFSET,
BF3_VERBOSITY_MASK,
BF3_VERBOSITY_VERBOSE,
"Request verbose BIOS display"
},
{
"force_pet",
BF3_OFFSET,
BF3_EVENT_TRAPS_MASK,
BF3_EVENT_TRAPS,
"Force progress event traps"
},
{
"upw_bypass",
BF3_OFFSET,
BF3_PASSWD_BYPASS_MASK,
BF3_PASSWD_BYPASS,
"User password bypass"
},
{
"lockout_sleep",
BF3_OFFSET,
BF3_SLEEP_LOCKOUT_MASK,
BF3_SLEEP_LOCKOUT,
"Log Out Sleep Button"
},
{
"cons_redirect=default",
BF3_OFFSET,
BF3_CONSOLE_REDIR_MASK,
BF3_CONSOLE_REDIR_DEFAULT,
"Console redirection occurs per "
"BIOS configuration setting"
},
{
"cons_redirect=skip",
BF3_OFFSET,
BF3_CONSOLE_REDIR_MASK,
BF3_CONSOLE_REDIR_SUPPRESS,
"Suppress (skip) console "
"redirection if enabled"
},
{
"cons_redirect=enable",
BF3_OFFSET,
BF3_CONSOLE_REDIR_MASK,
BF3_CONSOLE_REDIR_ENABLE,
"Suppress (skip) console "
"redirection if enabled"
},
/* data 4 */ /* data 4 */
/* data4[7:4] reserved */ /* data4[7:4] reserved */
/* data4[3] BIOS Shared Mode Override, not implemented here */ /* data4[3] BIOS Shared Mode Override, not implemented here */
@ -1757,7 +2010,7 @@ ipmi_chassis_main(struct ipmi_intf * intf, int argc, char ** argv)
/* data5 reserved */ /* data5 reserved */
{NULL} /* End marker */ {NULL} /* End marker */
}, *op; };
memset(&flags[0], 0, sizeof(flags)); memset(&flags[0], 0, sizeof(flags));
token = strtok_r(argv[2] + 8, ",", &saveptr); token = strtok_r(argv[2] + 8, ",", &saveptr);
@ -1768,8 +2021,8 @@ ipmi_chassis_main(struct ipmi_intf * intf, int argc, char ** argv)
} }
for (op = options; op->name; ++op) { for (op = options; op->name; ++op) {
if (strcmp(token, op->name) == 0) { if (strcmp(token, op->name) == 0) {
flags[op->i] &= op->mask; flags[op->offset] &= ~(op->mask);
flags[op->i] |= op->value; flags[op->offset] |= op->value;
break; break;
} }
} }