Handle comments from pre-verify and runtime lid loading commits

Change-Id: I224079808493c062f04b7c3a59d45128a8f2e699
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/49875
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com>
Reviewed-by: Nicholas E. Bofferding <bofferdn@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: Daniel M. Crowell <dcrowell@us.ibm.com>

1 files changed, 7 insertions, 4 deletions

diff --git a/src/usr/runtime/test/testpreverifiedlidmgr.H b/src/usr/runtime/test/testpreverifiedlidmgr.H
index 26879574e..47ba6c61b 100644
--- a/src/usr/runtime/test/testpreverifiedlidmgr.H
+++ b/src/usr/runtime/test/testpreverifiedlidmgr.H
@@ -98,14 +98,17 @@ class PreVerifiedLidMgrTest : public CxxTest::TestSuite
             break;
         }
 
-        // Each section has 2 lids each (Header, Content) except the RINGOVD
-        // section. It only has 1 or is inhibited in secure mode
+        // Each section has 2 lids each (Header, Content)
+        // Note: even the RINGOVD section adds a Header element, although it is
+        //       INVALID_LID
+        // See runtime_utils.C for full list of PNOR sections and utillidpnor.C
+        //     for the mappings (PnorToLidsMap)
         size_t l_numSections = RUNTIME::preVerifiedPnorSections.size();
-        // See utillidpnor.C for more info on num of lids
         size_t l_expectedLids = (2 * l_numSections);
         if (SECUREBOOT::enabled())
         {
-            // RINGOVD not permitted in secure mode
+            // RINGOVD not permitted in secure mode. Meaning the Header and
+            // Content lid will be missing.
             l_expectedLids -= 2;
         }