Skip to content

Commit

Permalink
Commit errors for VPD writes as visible logs
Browse files Browse the repository at this point in the history
The basic path should be working now so we need to stop hiding
error logs we generate in the VPD write message path.

Change-Id: Iaa53d7a2e2d2444981d62f71d80ed9f8b844a0d1
RTC: 180490
Backport: fips910
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/52846
Tested-by: Jenkins Server <pfd-jenkins+hostboot@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: Christian R. Geddes <crgeddes@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
  • Loading branch information
dcrowell77 committed Feb 7, 2018
1 parent 4a33c32 commit caaafc0
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions src/usr/vpd/runtime/rt_vpd.C
Expand Up @@ -516,11 +516,10 @@ errlHndl_t sendMboxWriteMsg ( size_t i_numBytes,
if( l_resp_fw_msg ) { free(l_resp_fw_msg); }
l_req_fw_msg = l_resp_fw_msg = nullptr;

// Commit any errors we get in here because we'd like the main
// function to complete even if we can't send the data out
if (l_err)
{
// @fixme-RTC:180490 - Temporarily commit until FSP code is implemented
// Just commit the log for now until FSP code is implemented
l_err->setSev(ERRORLOG::ERRL_SEV_INFORMATIONAL);
errlCommit( l_err, VPD_COMP_ID );
}

Expand Down

0 comments on commit caaafc0

Please sign in to comment.