From 4bef50f8d4c06d24013821943bd9fbf24edef288 Mon Sep 17 00:00:00 2001 From: Mike Baiocchi Date: Fri, 16 Jun 2017 10:19:54 -0500 Subject: [PATCH] Add SECUREBOOT::enabled() state to printk 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 Tested-by: Jenkins OP Build CI Reviewed-by: Stephen M. Cprek Tested-by: FSP CI Jenkins Reviewed-by: Marshall J. Wilks Reviewed-by: Nicholas E. Bofferding Reviewed-by: Daniel M. Crowell --- src/usr/secureboot/base/settings.C | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/usr/secureboot/base/settings.C b/src/usr/secureboot/base/settings.C index ec86fe650f9..28c2728cc12 100644 --- a/src/usr/secureboot/base/settings.C +++ b/src/usr/secureboot/base/settings.C @@ -33,6 +33,7 @@ #include #include #include +#include // SECUREBOOT : General driver traces #include "../common/securetrace.H" @@ -67,6 +68,7 @@ namespace SECUREBOOT static_cast(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 @@ -158,8 +160,8 @@ namespace SECUREBOOT auto l_errl = readSecurityRegister(i_pProc, static_cast(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; }