Skip to content

Commit

Permalink
Add SECUREBOOT::enabled() state to printk
Browse files Browse the repository at this point in the history
This commit traces the setting of SECUREBOOT::enabled() to the kernel
printk buffer for debug purposes.  It also updates a trace to display
the register value at the end.

Change-Id: I3b9abe54b86e4a8c2399c09074bd41f2f7ae1c8b
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/41977
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Reviewed-by: Stephen M. Cprek <smcprek@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Marshall J. Wilks <mjwilks@us.ibm.com>
Reviewed-by: Nicholas E. Bofferding <bofferdn@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
  • Loading branch information
mabaiocchi authored and dcrowell77 committed Jun 19, 2017
1 parent 331c3aa commit 4bef50f
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions src/usr/secureboot/base/settings.C
Expand Up @@ -33,6 +33,7 @@
#include <secureboot/settings.H>
#include <config.h>
#include <console/consoleif.H>
#include <kernel/console.H>

// SECUREBOOT : General driver traces
#include "../common/securetrace.H"
Expand Down Expand Up @@ -67,6 +68,7 @@ namespace SECUREBOOT
static_cast<uint64_t>(ProcSecurity::SabBit)));

SB_INF("getEnabled() state:%i",iv_enabled);
printk("SECUREBOOT::enabled() state:%i\n", iv_enabled);

// Report if secure boot is disabled
#ifdef CONFIG_SECUREBOOT
Expand Down Expand Up @@ -158,8 +160,8 @@ namespace SECUREBOOT
auto l_errl = readSecurityRegister(i_pProc,
static_cast<uint64_t>(ProcSecurity::SwitchRegister),
o_regValue);
SB_INF("getSecuritySwitch() err:%i reg:%.16llX huid:%.8X",
!!l_errl, o_regValue, get_huid(i_pProc));
SB_INF("getSecuritySwitch() err_rc:%.4X huid:%.8X reg:%.16llX",
ERRL_GETRC_SAFE(l_errl), get_huid(i_pProc), o_regValue);

return l_errl;
}
Expand Down

0 comments on commit 4bef50f

Please sign in to comment.