diff options
author | Nick Bofferding <bofferdn@us.ibm.com> | 2018-03-05 12:07:23 -0600 |
---|---|---|
committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2018-03-07 15:02:55 -0500 |
commit | bd1cd3c7d1fb4e38ebaf0885a298f2a44e165917 (patch) | |
tree | e0c980e34fcd1433846751e1d51456a3e407bfd8 /src/usr | |
parent | 74156401d2fb37e05e5ae0fc50906e321756a354 (diff) | |
download | talos-hostboot-bd1cd3c7d1fb4e38ebaf0885a298f2a44e165917.tar.gz talos-hostboot-bd1cd3c7d1fb4e38ebaf0885a298f2a44e165917.zip |
Secure Boot: Detach secure PNOR provider task
Once secure PNOR provider task starts, immediately detach it so that, should
it die, Hostboot will terminate; otherwise, Hostboot can hang until terminated
by a service processor boot timeout.
Change-Id: I586db0bb56e7e1eedb1183cda34e4bda4bc03bbd
CQ: SW419735
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/55048
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: ILYA SMIRNOV <ismirno@us.ibm.com>
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Diffstat (limited to 'src/usr')
-rw-r--r-- | src/usr/pnor/spnorrp.C | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/src/usr/pnor/spnorrp.C b/src/usr/pnor/spnorrp.C index d2a907e71..b55e3877d 100644 --- a/src/usr/pnor/spnorrp.C +++ b/src/usr/pnor/spnorrp.C @@ -40,6 +40,7 @@ #include <secureboot/containerheader.H> #include <secureboot/trustedbootif.H> #include <secureboot/header.H> +#include <sys/task.h> extern trace_desc_t* g_trac_pnor; @@ -72,6 +73,9 @@ using namespace PNOR; */ void* secure_wait_for_message( void* unused ) { + // Mark task as an independent daemon so if it crashes, Hostboot will + // terminate + (void)task_detach(); TRACUCOMP(g_trac_pnor, "wait_for_message> " ); Singleton<SPnorRP>::instance().waitForMessage(); return NULL; |