cper-section-memory: Fix validation dependency for Extended field bits
Fixed incorrect dependency between validation bits 16, 17, and 18 for
the Extended field. Previously, cardSmbiosHandle (validation bit 16)
and moduleSmbiosHandle (validation bit 17) were incorrectly made
dependent on the Extended field validation (bit 18), but these are
independent components.
Validation bit 18 controls the Extended field containing row address
bits 16 and 17, while validation bits 16 and 17 control SMBIOS handle
fields. These SMBIOS handle fields are independent components that
should be validated separately from the Extended field's row address
bits.
Tested: Added memory-validation-bits unit test
Change-Id: I9461c71bf0b782bda74ed24c95b63c080f913b19
Signed-off-by: Peter Benitez <pbenitez@nvidia.com>
diff --git a/tests/ir-tests.c b/tests/ir-tests.c
index ad4ae3b..b9235ef 100644
--- a/tests/ir-tests.c
+++ b/tests/ir-tests.c
@@ -650,6 +650,12 @@
cper_example_section_ir_test("nvidia_cmet_info");
}
+//Memory section test for validation bits.
+void MemoryValidationBitsSectionTests_IRValid()
+{
+ cper_example_section_ir_test("memory-validation-bits");
+}
+
//Unknown section tests.
void UnknownSectionTests_IRValid(void)
{
@@ -717,6 +723,7 @@
NVIDIASectionTests_IRValid();
NVIDIASectionTests_BinaryEqual();
NVIDIACMETSectionTests_IRValid();
+ MemoryValidationBitsSectionTests_IRValid();
UnknownSectionTests_IRValid();
UnknownSectionTests_BinaryEqual();
CompileTimeAssertions_TwoWayConversion();