diff mbox series

[3/3] fwts-test: sync up with new fwts_acpi_reserved_zero_check

Message ID 1504744458-559-3-git-send-email-alex.hung@canonical.com
State Accepted
Headers show
Series [1/3] lib: fwts_acpi_tables: add a new function to check Reserved field | expand

Commit Message

Alex Hung Sept. 7, 2017, 12:34 a.m. UTC
Signed-off-by: Alex Hung <alex.hung@canonical.com>
---
 fwts-test/asf-0001/asf-0002.log   | 11 +++++++----
 fwts-test/erst-0001/erst-0002.log |  4 ++--
 fwts-test/hmat-0001/hmat-0002.log | 18 +++++++++---------
 fwts-test/iort-0001/iort-0002.log | 16 ++++++++--------
 fwts-test/lpit-0001/lpit-0002.log |  6 +++---
 fwts-test/mpst-0001/mpst-0002.log | 20 +++++++++++---------
 fwts-test/msdm-0001/msdm-0002.log |  8 ++++----
 fwts-test/nfit-0001/nfit-0002.log |  8 ++++----
 fwts-test/pmtt-0001/pmtt-0002.log | 22 +++++++++++-----------
 fwts-test/pptt-0001/pptt-0002.log | 10 +++++-----
 fwts-test/spcr-0001/spcr-0002.log |  4 ++--
 fwts-test/tpm2-0001/tpm2-0002.log |  2 +-
 12 files changed, 67 insertions(+), 62 deletions(-)

Comments

Colin Ian King Sept. 8, 2017, 7:51 a.m. UTC | #1
On 07/09/17 01:34, Alex Hung wrote:
> Signed-off-by: Alex Hung <alex.hung@canonical.com>
> ---
>  fwts-test/asf-0001/asf-0002.log   | 11 +++++++----
>  fwts-test/erst-0001/erst-0002.log |  4 ++--
>  fwts-test/hmat-0001/hmat-0002.log | 18 +++++++++---------
>  fwts-test/iort-0001/iort-0002.log | 16 ++++++++--------
>  fwts-test/lpit-0001/lpit-0002.log |  6 +++---
>  fwts-test/mpst-0001/mpst-0002.log | 20 +++++++++++---------
>  fwts-test/msdm-0001/msdm-0002.log |  8 ++++----
>  fwts-test/nfit-0001/nfit-0002.log |  8 ++++----
>  fwts-test/pmtt-0001/pmtt-0002.log | 22 +++++++++++-----------
>  fwts-test/pptt-0001/pptt-0002.log | 10 +++++-----
>  fwts-test/spcr-0001/spcr-0002.log |  4 ++--
>  fwts-test/tpm2-0001/tpm2-0002.log |  2 +-
>  12 files changed, 67 insertions(+), 62 deletions(-)
> 
> diff --git a/fwts-test/asf-0001/asf-0002.log b/fwts-test/asf-0001/asf-0002.log
> index e8d25bc..b8212e7 100644
> --- a/fwts-test/asf-0001/asf-0002.log
> +++ b/fwts-test/asf-0001/asf-0002.log
> @@ -11,9 +11,12 @@ asf             range 0x02..0xff
>  asf             FAILED [HIGH] ASF!InfoFeatureFlagsReserved: Test 1, ASF!
>  asf             ASF_INFO Feature Flags is 0xff, however reserved bits [7
>  asf             :1] must be zero
> -asf             FAILED [HIGH] ASF!InfoReservedNonZero: Test 1, ASF!
> -asf             ASF_INFO Reserved fields must be zero, however one or more
> -asf             of them are non-zero
> +asf             FAILED [MEDIUM] ASF!ReservedNonZero: Test 1, ASF! ASF_INFO
> +asf             Reserved1 field must be zero, got 0x10 instead
> +asf             FAILED [MEDIUM] ASF!ReservedNonZero: Test 1, ASF! ASF_INFO
> +asf             Reserved2 field must be zero, got 0x20 instead
> +asf             FAILED [MEDIUM] ASF!ReservedNonZero: Test 1, ASF! ASF_INFO
> +asf             Reserved3 field must be zero, got 0x40 instead
>  asf             FAILED [HIGH] ASF!AlrtNumOfAlertsInvalid: Test 1, ASF!
>  asf             ASF_ALRT Number of Alerts field is 0x23 and must be in the
>  asf             range 0x01..0x08
> @@ -30,6 +33,6 @@ asf             long for the size given by the ASF! table. Expected at
>  asf             most 22 bytes.
>  asf             
>  asf             ==========================================================
> -asf             1 passed, 8 failed, 0 warning, 0 aborted, 0 skipped, 0
> +asf             1 passed, 10 failed, 0 warning, 0 aborted, 0 skipped, 0
>  asf             info only.
>  asf             ==========================================================
> diff --git a/fwts-test/erst-0001/erst-0002.log b/fwts-test/erst-0001/erst-0002.log
> index 29eefef..d9aecc3 100644
> --- a/fwts-test/erst-0001/erst-0002.log
> +++ b/fwts-test/erst-0001/erst-0002.log
> @@ -5,8 +5,8 @@ erst            ERST Error Record Serialization Table :
>  erst              Serialization Hdr. Size:  0x00000030
>  erst              Reserved:                 0x000000f0
>  erst              Instruction Entry Count:  0x00000010
> -erst            FAILED [LOW] ERSTReservedNonZero: Test 1, ERST Reserved
> -erst            field must be zero, got 0xf0 instead
> +erst            FAILED [MEDIUM] ERSTReservedNonZero: Test 1, ERST Reserved
> +erst            field must be zero, got 0x000000f0 instead
>  erst            FAILED [HIGH] ERSTIInvalidFlag: Test 1, ERST Serialization
>  erst            Flag 0xff is an invalid value, values allowed are
>  erst            0x00..0x01 for the WRITE_REGISTER and WRITE_REGISTER_VALUE
> diff --git a/fwts-test/hmat-0001/hmat-0002.log b/fwts-test/hmat-0001/hmat-0002.log
> index 2394c28..d90468b 100644
> --- a/fwts-test/hmat-0001/hmat-0002.log
> +++ b/fwts-test/hmat-0001/hmat-0002.log
> @@ -14,13 +14,13 @@ hmat                Memory Proximity Domain:        0x00000000
>  hmat                Reserved:                       0x00000003
>  hmat                System Phy Addr Range Base:     0x0000000000000000
>  hmat                System Phy Addr Range Length:   0x0000000000000000
> -hmat            FAILED [LOW] HMATReservedNonZero: Test 1, HMAT reserved
> +hmat            FAILED [MEDIUM] HMATReservedNonZero: Test 1, HMAT Reserved
>  hmat            field must be zero, got 0x0001 instead
>  hmat            FAILED [HIGH] HMATBadFlags: Test 1, HMAT Flags's
>  hmat            Bits[15..3] must be zero, got 0x000f instead
> -hmat            FAILED [LOW] HMATReservedNonZero: Test 1, HMAT reserved
> +hmat            FAILED [MEDIUM] HMATReservedNonZero: Test 1, HMAT Reserved
>  hmat            field must be zero, got 0x0002 instead
> -hmat            FAILED [LOW] HMATReservedNonZero: Test 1, HMAT reserved
> +hmat            FAILED [MEDIUM] HMATReservedNonZero: Test 1, HMAT Reserved
>  hmat            field must be zero, got 0x00000003 instead
>  hmat            
>  hmat              System Locality Latency and Bandwidth Information (Type 1):
> @@ -34,15 +34,15 @@ hmat                Number of Initiator PDs:        0x00000002
>  hmat                Number of Target PDs:           0x00000003
>  hmat                Reserved:                       0x00000006
>  hmat                Entry Base Unit:                0x0000000000000000
> -hmat            FAILED [LOW] HMATReservedNonZero: Test 1, HMAT reserved
> +hmat            FAILED [MEDIUM] HMATReservedNonZero: Test 1, HMAT Reserved
>  hmat            field must be zero, got 0x0004 instead
>  hmat            FAILED [CRITICAL] HMATBadFlags: Test 1, HMAT Flags's
>  hmat            Bits[7..5] must be zero, got 0x3f instead
>  hmat            FAILED [CRITICAL] HMATBadFDataType: Test 1, HMAT Data Type
>  hmat            must be 0..5, got 0x06 instead
> -hmat            FAILED [LOW] HMATReservedNonZero: Test 1, HMAT reserved
> +hmat            FAILED [MEDIUM] HMATReservedNonZero: Test 1, HMAT Reserved
>  hmat            field must be zero, got 0x0005 instead
> -hmat            FAILED [LOW] HMATReservedNonZero: Test 1, HMAT reserved
> +hmat            FAILED [MEDIUM] HMATReservedNonZero: Test 1, HMAT Reserved
>  hmat            field must be zero, got 0x00000006 instead
>  hmat            
>  hmat              Memory Side Cache Information (Type 2):
> @@ -55,14 +55,14 @@ hmat                Memory Side Cache Size:         0x0000000000000000
>  hmat                Cache Attributes:               0x00002333
>  hmat                Reserved:                       0x0009
>  hmat                Number of SMBIOS Handles:       0x0003
> -hmat            FAILED [LOW] HMATReservedNonZero: Test 1, HMAT reserved
> +hmat            FAILED [MEDIUM] HMATReservedNonZero: Test 1, HMAT Reserved
>  hmat            field must be zero, got 0x0007 instead
>  hmat            FAILED [CRITICAL] HMATBadCacheAttributeReserved: Test 1,
>  hmat            HMAT Cache Attribute reserved values are used, got
>  hmat            0x00002333 instead
> -hmat            FAILED [LOW] HMATReservedNonZero: Test 1, HMAT reserved
> +hmat            FAILED [MEDIUM] HMATReservedNonZero: Test 1, HMAT Reserved
>  hmat            field must be zero, got 0x00000008 instead
> -hmat            FAILED [LOW] HMATReservedNonZero: Test 1, HMAT reserved
> +hmat            FAILED [MEDIUM] HMATReservedNonZero: Test 1, HMAT Reserved
>  hmat            field must be zero, got 0x0009 instead
>  hmat            
>  hmat            
> diff --git a/fwts-test/iort-0001/iort-0002.log b/fwts-test/iort-0001/iort-0002.log
> index a726735..7a0746b 100644
> --- a/fwts-test/iort-0001/iort-0002.log
> +++ b/fwts-test/iort-0001/iort-0002.log
> @@ -20,8 +20,8 @@ iort            Either the Number of ID mappings is too large or the IORT
>  iort            table size or ITS Group Node is too small.
>  iort            FAILED [LOW] IORTNodeRevisionNonZero: Test 1, IORT Node
>  iort            Revision field is 0x03 and should be zero.
> -iort            FAILED [LOW] IORTNodeReservedNonZero: Test 1, IORT Node
> -iort            Reserved field is 0x0ffffffff and should be zero.
> +iort            FAILED [MEDIUM] IORTReservedNonZero: Test 1, IORT Node
> +iort            Reserved field must be zero, got 0xffffffff instead
>  iort            FAILED [LOW] IORTNodeMappingsCountNonZero: Test 1, IORT
>  iort            Node Number of ID Mappings field is 0x000000080 and should
>  iort            be zero for this node type.
> @@ -57,8 +57,8 @@ iort              Output Reference:         0x88888888
>  iort              Flags:                    0x00000001
>  iort            FAILED [LOW] IORTNodeRevisionNonZero: Test 1, IORT Node
>  iort            Revision field is 0x02 and should be zero.
> -iort            FAILED [LOW] IORTNodeReservedNonZero: Test 1, IORT Node
> -iort            Reserved field is 0x00000f001 and should be zero.
> +iort            FAILED [MEDIUM] IORTReservedNonZero: Test 1, IORT Node
> +iort            Reserved field must be zero, got 0x0000f001 instead
>  iort            FAILED [HIGH] IORTNamedComponentNodeFlagsNonZero: Test 1,
>  iort            IORT Named Component Node Flags is 0xffffffff and is
>  iort            reserved and should be zero.
> @@ -100,8 +100,8 @@ iort              Output Reference:         0x00000005
>  iort              Flags:                    0x00000001
>  iort            FAILED [LOW] IORTNodeRevisionNonZero: Test 1, IORT Node
>  iort            Revision field is 0x50 and should be zero.
> -iort            FAILED [LOW] IORTNodeReservedNonZero: Test 1, IORT Node
> -iort            Reserved field is 0x00000000f and should be zero.
> +iort            FAILED [MEDIUM] IORTReservedNonZero: Test 1, IORT Node
> +iort            Reserved field must be zero, got 0x0000000f instead
>  iort            FAILED [HIGH] IORTCPCIRootComplexAtsAttrInvalid: Test 1,
>  iort            IORT PCI Root Complex Node ATS Attribute is 0x3 and was
>  iort            expecting either 0 (root complex supports ATS) or 1 (root
> @@ -162,8 +162,8 @@ iort              GSIV:                     0xf0008000
>  iort              Interrupt Flags:          0x00000004
>  iort            FAILED [LOW] IORTNodeRevisionNonZero: Test 1, IORT Node
>  iort            Revision field is 0xa0 and should be zero.
> -iort            FAILED [LOW] IORTNodeReservedNonZero: Test 1, IORT Node
> -iort            Reserved field is 0x000008100 and should be zero.
> +iort            FAILED [MEDIUM] IORTReservedNonZero: Test 1, IORT Node
> +iort            Reserved field must be zero, got 0x00008100 instead
>  iort            FAILED [MEDIUM] IORTSMMUIntrFlagsReservedNonZero: Test 1,
>  iort            IORT SMMU_NsgIrpt Flags field reserved bits [31:1] should
>  iort            be all zero, got 0x2
> diff --git a/fwts-test/lpit-0001/lpit-0002.log b/fwts-test/lpit-0001/lpit-0002.log
> index aa3de10..d362acd 100644
> --- a/fwts-test/lpit-0001/lpit-0002.log
> +++ b/fwts-test/lpit-0001/lpit-0002.log
> @@ -23,9 +23,9 @@ lpit                Access Size             0x00
>  lpit                Address                 0x1020304050607080
>  lpit              Residency Counter Freq:   0x000000000000b0a0
>  lpit            
> -lpit            FAILED [LOW] LPITNativeCStateLpitReservedNonZero: Test 1,
> -lpit            Native C-state based LPI structure reserved field was
> -lpit            expected to be zero, got 0xffff instead
> +lpit            FAILED [MEDIUM] LPITReservedNonZero: Test 1, LPIT Native
> +lpit            C-state based LPI structure reserved field must be zero,
> +lpit            got 0xffff instead
>  lpit            FAILED [LOW] LPITNativeCStateLpitFlagsReserved: Test 1,
>  lpit            Some of the Native C-state based LPI structure flags bits
>  lpit            [31:2] are set, they are expected to be zero
> diff --git a/fwts-test/mpst-0001/mpst-0002.log b/fwts-test/mpst-0001/mpst-0002.log
> index adda211..37b216b 100644
> --- a/fwts-test/mpst-0001/mpst-0002.log
> +++ b/fwts-test/mpst-0001/mpst-0002.log
> @@ -4,11 +4,11 @@ mpst            Test 1 of 1: Validate MPST table.
>  mpst            MPST Table:
>  mpst              Communication Channel ID:        0x00
>  mpst              Reserved:                        0x00010000
> -mpst            FAILED [LOW] MPSTReservedNonZero: Test 1, MPST reserved
> +mpst            FAILED [MEDIUM] MPSTReservedNonZero: Test 1, MPST Reserved
>  mpst            field must be zero, got 0x00010000 instead
>  mpst              Memory Power Node Count:         0x0002
>  mpst              Reserved:                        0x0003
> -mpst            FAILED [LOW] MPSTReservedNonZero: Test 1, MPST reserved
> +mpst            FAILED [MEDIUM] MPSTReservedNonZero: Test 1, MPST Reserved
>  mpst            field must be zero, got 0x0003 instead
>  mpst              MPST Power Node:
>  mpst                Flags:                         0x00
> @@ -19,7 +19,7 @@ mpst                Base Address:                  0x00000000f0000000
>  mpst                Memory Length:                 0x00000000e0000000
>  mpst                Number of Power States:        0x00000002
>  mpst                Number of Physical Components: 0x00000003
> -mpst            FAILED [LOW] MPSTReservedNonZero: Test 1, MPST reserved
> +mpst            FAILED [MEDIUM] MPSTReservedNonZero: Test 1, MPST Reserved
>  mpst            field must be zero, got 0x04 instead
>  mpst            FAILED [HIGH] MPSTBadPowerNodeLength: Test 1, MPST Power
>  mpst            Node Length should be 0x0000002a, got 0x0000012a instead
> @@ -39,7 +39,7 @@ mpst                Base Address:                  0x00000000a0000000
>  mpst                Memory Length:                 0x00000000b0000000
>  mpst                Number of Power States:        0x00000001
>  mpst                Number of Physical Components: 0x00000001
> -mpst            FAILED [LOW] MPSTReservedNonZero: Test 1, MPST reserved
> +mpst            FAILED [MEDIUM] MPSTReservedNonZero: Test 1, MPST Reserved
>  mpst            field must be zero, got 0x05 instead
>  mpst            FAILED [HIGH] MPSTBadPowerNodeLength: Test 1, MPST Power
>  mpst            Node Length should be 0x00000024, got 0x00000124 instead
> @@ -48,7 +48,7 @@ mpst                Power State Information Index: 0x06
>  mpst                Physical Component Id:         0x03
>  mpst              Memory Characteristics Count:    0x0002
>  mpst              Reserved:                        0x0006
> -mpst            FAILED [LOW] MPSTReservedNonZero: Test 1, MPST reserved
> +mpst            FAILED [MEDIUM] MPSTReservedNonZero: Test 1, MPST Reserved
>  mpst            field must be zero, got 0x0006 instead
>  mpst              MPST Power Characteristics:
>  mpst                Power State Structure ID:      0x02
> @@ -63,8 +63,9 @@ mpst            Power Characteristics ID must be 1, got 0x02 instead
>  mpst            FAILED [HIGH] MPSTBadPowerCharacteristicsRevision: Test 1,
>  mpst            MPST Power Characteristics Revision must be 1, got 0x00
>  mpst            instead
> -mpst            FAILED [LOW] MPSTReservedNonZero: Test 1, MPST reserved
> -mpst            field must be zero, got 0x0000000000000007 instead
> +mpst            FAILED [MEDIUM] MPSTReservedNonZero: Test 1, MPST
> +mpst            Reserved2 field must be zero, got 0x0000000000000007
> +mpst            instead
>  mpst              MPST Power Characteristics:
>  mpst                Power State Structure ID:      0x03
>  mpst                Flags:                         0x00
> @@ -78,8 +79,9 @@ mpst            Power Characteristics ID must be 1, got 0x03 instead
>  mpst            FAILED [HIGH] MPSTBadPowerCharacteristicsRevision: Test 1,
>  mpst            MPST Power Characteristics Revision must be 1, got 0x00
>  mpst            instead
> -mpst            FAILED [LOW] MPSTReservedNonZero: Test 1, MPST reserved
> -mpst            field must be zero, got 0x0000000000000008 instead
> +mpst            FAILED [MEDIUM] MPSTReservedNonZero: Test 1, MPST
> +mpst            Reserved2 field must be zero, got 0x0000000000000008
> +mpst            instead
>  mpst            
>  mpst            
>  mpst            ==========================================================
> diff --git a/fwts-test/msdm-0001/msdm-0002.log b/fwts-test/msdm-0001/msdm-0002.log
> index 2f4953e..9fa7792 100644
> --- a/fwts-test/msdm-0001/msdm-0002.log
> +++ b/fwts-test/msdm-0001/msdm-0002.log
> @@ -5,10 +5,10 @@ msdm              Reserved:                 0xffffffff
>  msdm              Data Type:                0x00000001
>  msdm              Data Reserved:            0x00000020
>  msdm              Data Length:              0x0000001d
> -msdm            FAILED [HIGH] MSDMReservedNonZero: Test 1, MSDM Reserved
> -msdm            field should be zero, got 0xffffffff instead
> -msdm            FAILED [HIGH] MSDMDataReservedNonZero: Test 1, MSDM Data
> -msdm            Reserved field should be zero, got 0x00000020 instead
> +msdm            FAILED [MEDIUM] MSDMReservedNonZero: Test 1, MSDM Reserved
> +msdm            field must be zero, got 0xffffffff instead
> +msdm            FAILED [MEDIUM] MSDMReservedNonZero: Test 1, MSDM Data
> +msdm            Reserved field must be zero, got 0x00000020 instead
>  msdm              Data:                     'AH3NR*B8FDN-C1WTG-5CR9X-1AKMH'
>  msdm            FAILED [HIGH] MSDMDataLengthInvalid: Test 1, MSDM Data
>  msdm            field did not contain digits, uppercase letters and -
> diff --git a/fwts-test/nfit-0001/nfit-0002.log b/fwts-test/nfit-0001/nfit-0002.log
> index 47be360..02fba5a 100644
> --- a/fwts-test/nfit-0001/nfit-0002.log
> +++ b/fwts-test/nfit-0001/nfit-0002.log
> @@ -4,7 +4,7 @@ nfit            Test 1 of 1: NFIT NVDIMM Firmware Interface Table test.
>  nfit            NFIT NVDIMM Firmware Interface Table:
>  nfit              Reserved:                 0x05000000
>  nfit            
> -nfit            FAILED [LOW] NFITReservedNonZero: Test 1, NFIT reserved
> +nfit            FAILED [MEDIUM] NFITReservedNonZero: Test 1, NFIT Reserved
>  nfit            field must be zero, got 0x05000000 instead
>  nfit              NFIT Subtable:
>  nfit                Type:                                   0x0000
> @@ -24,7 +24,7 @@ nfit            Bits[15..2] must be zero, got 0x00000005 instead
>  nfit            FAILED [HIGH] NFITBadMemoryMappingAttribute: Test 1, NFIT
>  nfit            Memory Mapping Attribute must meet UEFI Spec, got
>  nfit            0x00000000000000ff instead
> -nfit            FAILED [LOW] NFITReservedNonZero: Test 1, NFIT reserved
> +nfit            FAILED [MEDIUM] NFITReservedNonZero: Test 1, NFIT Reserved
>  nfit            field must be zero, got 0x000000000000000f instead
>  nfit            
>  nfit              NFIT Subtable:
> @@ -103,7 +103,7 @@ nfit            FAILED [HIGH] NFITBadValidField: Test 1, NFIT Valid
>  nfit            Field's Bits[7..1] must be zero, got 0x03 instead
>  nfit            FAILED [HIGH] NFITBadFlags: Test 1, NFIT Flags's
>  nfit            Bits[15..1] must be zero, got 0x0003 instead
> -nfit            FAILED [LOW] NFITReservedNonZero: Test 1, NFIT reserved
> +nfit            FAILED [MEDIUM] NFITReservedNonZero: Test 1, NFIT Reserved
>  nfit            field must be zero, got 0x0000000000000f00 instead
>  nfit            
>  nfit              NFIT Subtable:
> @@ -126,7 +126,7 @@ nfit                Number of Flush Hint Addresses:         0x0002
>  nfit                Reserved:                               0x00000000000000f0
>  nfit                Flush Hint Address:                     0x0000000418000000
>  nfit                Flush Hint Address:                     0x0000000618000000
> -nfit            FAILED [LOW] NFITReservedNonZero: Test 1, NFIT reserved
> +nfit            FAILED [MEDIUM] NFITReservedNonZero: Test 1, NFIT Reserved
>  nfit            field must be zero, got 0x00000000000000f0 instead
>  nfit            
>  nfit            
> diff --git a/fwts-test/pmtt-0001/pmtt-0002.log b/fwts-test/pmtt-0001/pmtt-0002.log
> index c5e060d..2ce24a2 100644
> --- a/fwts-test/pmtt-0001/pmtt-0002.log
> +++ b/fwts-test/pmtt-0001/pmtt-0002.log
> @@ -3,7 +3,7 @@ pmtt            ----------------------------------------------------------
>  pmtt            Test 1 of 1: Validate PMTT table.
>  pmtt            PMTT Table:
>  pmtt              Reserved:                         0x02000000
> -pmtt            FAILED [LOW] PMTTReservedNonZero: Test 1, PMTT reserved
> +pmtt            FAILED [MEDIUM] PMTTReservedNonZero: Test 1, PMTT Reserved
>  pmtt            field must be zero, got 0x02000000 instead
>  pmtt              PMTT Subtable:
>  pmtt                Type:                           0x00
> @@ -11,13 +11,13 @@ pmtt                Reserved:                       0x01
>  pmtt                Length:                         0x0098
>  pmtt                Flags:                          0x0001
>  pmtt                Reserved:                       0x0003
> -pmtt            FAILED [LOW] PMTTReservedNonZero: Test 1, PMTT reserved
> -pmtt            field must be zero, got 0x01 instead
> -pmtt            FAILED [LOW] PMTTReservedNonZero: Test 1, PMTT reserved
> -pmtt            field must be zero, got 0x0003 instead
> +pmtt            FAILED [MEDIUM] PMTTReservedNonZero: Test 1, PMTT
> +pmtt            Reserved1 field must be zero, got 0x01 instead
> +pmtt            FAILED [MEDIUM] PMTTReservedNonZero: Test 1, PMTT
> +pmtt            Reserved2 field must be zero, got 0x0003 instead
>  pmtt                Socket Identifier:              0x0001
>  pmtt                Reserved:                       0x0400
> -pmtt            FAILED [LOW] PMTTReservedNonZero: Test 1, PMTT reserved
> +pmtt            FAILED [MEDIUM] PMTTReservedNonZero: Test 1, PMTT Reserved
>  pmtt            field must be zero, got 0x0400 instead
>  pmtt              PMTT Subtable:
>  pmtt                Type:                           0x01
> @@ -25,8 +25,8 @@ pmtt                Reserved:                       0x05
>  pmtt                Length:                         0x006c
>  pmtt                Flags:                          0x0005
>  pmtt                Reserved:                       0x0000
> -pmtt            FAILED [LOW] PMTTReservedNonZero: Test 1, PMTT reserved
> -pmtt            field must be zero, got 0x05 instead
> +pmtt            FAILED [MEDIUM] PMTTReservedNonZero: Test 1, PMTT
> +pmtt            Reserved1 field must be zero, got 0x05 instead
>  pmtt                Read Latency:                   0x0000f000
>  pmtt                Write latency:                  0x0000e000
>  pmtt                Read Bandwidth:                 0x0000d000
> @@ -35,7 +35,7 @@ pmtt                Optimal Access Unit:            0x0004
>  pmtt                Optimal Access Alignment:       0x0008
>  pmtt                Reserved:                       0x0600
>  pmtt                Number of Proximity Domains:    0x0004
> -pmtt            FAILED [LOW] PMTTReservedNonZero: Test 1, PMTT reserved
> +pmtt            FAILED [MEDIUM] PMTTReservedNonZero: Test 1, PMTT Reserved
>  pmtt            field must be zero, got 0x0600 instead
>  pmtt                Proximity Domain:               0x00001000
>  pmtt                Proximity Domain:               0x00002000
> @@ -47,8 +47,8 @@ pmtt                Reserved:                       0x07
>  pmtt                Length:                         0x0014
>  pmtt                Flags:                          0x0002
>  pmtt                Reserved:                       0x0000
> -pmtt            FAILED [LOW] PMTTReservedNonZero: Test 1, PMTT reserved
> -pmtt            field must be zero, got 0x07 instead
> +pmtt            FAILED [MEDIUM] PMTTReservedNonZero: Test 1, PMTT
> +pmtt            Reserved1 field must be zero, got 0x07 instead
>  pmtt                Physical Component Identifier:  0x0000
>  pmtt                Reserved:                       0x0000
>  pmtt                Size of DIMM:                   0xa0000000
> diff --git a/fwts-test/pptt-0001/pptt-0002.log b/fwts-test/pptt-0001/pptt-0002.log
> index 57c3391..4b3f2a6 100644
> --- a/fwts-test/pptt-0001/pptt-0002.log
> +++ b/fwts-test/pptt-0001/pptt-0002.log
> @@ -11,7 +11,7 @@ pptt                Parent:                         0x00000000
>  pptt                ACPI Processor ID:              0x00000000
>  pptt                Number of Private Resources:    0x00000001
>  pptt                Private Resources[0]:           0x00000000
> -pptt            FAILED [LOW] PPTTReservedNonZero: Test 1, PPTT reserved
> +pptt            FAILED [MEDIUM] PPTTReservedNonZero: Test 1, PPTT Reserved
>  pptt            field must be zero, got 0x0001 instead
>  pptt            
>  pptt              Cache Type Structure (Type 1):
> @@ -25,7 +25,7 @@ pptt                Number of sets:                 0x00000000
>  pptt                Associativity:                  0x00
>  pptt                Attributes:                     0x00
>  pptt                Line size:                      0x0000
> -pptt            FAILED [LOW] PPTTReservedNonZero: Test 1, PPTT reserved
> +pptt            FAILED [MEDIUM] PPTTReservedNonZero: Test 1, PPTT Reserved
>  pptt            field must be zero, got 0x0002 instead
>  pptt            
>  pptt              Processor hierarchy node structure (Type 0):
> @@ -38,7 +38,7 @@ pptt                ACPI Processor ID:              0x00000000
>  pptt                Number of Private Resources:    0x00000002
>  pptt                Private Resources[0]:           0x00000000
>  pptt                Private Resources[1]:           0x00000000
> -pptt            FAILED [LOW] PPTTReservedNonZero: Test 1, PPTT reserved
> +pptt            FAILED [MEDIUM] PPTTReservedNonZero: Test 1, PPTT Reserved
>  pptt            field must be zero, got 0x0003 instead
>  pptt            FAILED [HIGH] PPTTBadFlags: Test 1, PPTT Flags's
>  pptt            Bits[31..2] must be zero, got 0x00000007 instead
> @@ -54,7 +54,7 @@ pptt                Number of sets:                 0x00000000
>  pptt                Associativity:                  0xc0
>  pptt                Attributes:                     0x00
>  pptt                Line size:                      0x0000
> -pptt            FAILED [LOW] PPTTReservedNonZero: Test 1, PPTT reserved
> +pptt            FAILED [MEDIUM] PPTTReservedNonZero: Test 1, PPTT Reserved
>  pptt            field must be zero, got 0x0004 instead
>  pptt            FAILED [HIGH] PPTTBadFlags: Test 1, PPTT Flags's
>  pptt            Bits[31..7] must be zero, got 0x000000ff instead
> @@ -69,7 +69,7 @@ pptt                LEVEL_2_ID:                     0x0000000000000000
>  pptt                MAJOR_REV:                      0x0000
>  pptt                MINOR_REV:                      0x0000
>  pptt                SPIN_REV:                       0x0000
> -pptt            FAILED [LOW] PPTTReservedNonZero: Test 1, PPTT reserved
> +pptt            FAILED [MEDIUM] PPTTReservedNonZero: Test 1, PPTT Reserved
>  pptt            field must be zero, got 0x0005 instead
>  pptt            
>  pptt            
> diff --git a/fwts-test/spcr-0001/spcr-0002.log b/fwts-test/spcr-0001/spcr-0002.log
> index 437a4f9..e22fc40 100644
> --- a/fwts-test/spcr-0001/spcr-0002.log
> +++ b/fwts-test/spcr-0001/spcr-0002.log
> @@ -5,8 +5,8 @@ spcr            test.
>  spcr            Serial Interface: Reserved (Do not Use)
>  spcr            FAILED [HIGH] SPCRInterfaceReserved: Test 1, SPCR Serial
>  spcr            interface type 0x08 is a reserved interface
> -spcr            FAILED [LOW] SPCRReservedNonZero: Test 1, SPCR reserved
> -spcr            field must be zero, got 0xf02000 instead
> +spcr            FAILED [MEDIUM] SPCRReservedNonZero: Test 1, SPCR
> +spcr            Reserved1 field must be zero, got 0x000020f0 instead
>  spcr            Baud Rate:        Reserved
>  spcr            FAILED [HIGH] SPCRBaudRateReserved: Test 1, SPCR Serial
>  spcr            baud rate type 0x08 is a reserved baud rate
> diff --git a/fwts-test/tpm2-0001/tpm2-0002.log b/fwts-test/tpm2-0001/tpm2-0002.log
> index 8e8bad7..c93c153 100644
> --- a/fwts-test/tpm2-0001/tpm2-0002.log
> +++ b/fwts-test/tpm2-0001/tpm2-0002.log
> @@ -9,7 +9,7 @@ tpm2              Start Method:                    0x0000000c
>  tpm2            FAILED [HIGH] TPM2BadPlatformClass: Test 1, TPM2's
>  tpm2            platform class must be zero (client) or one (server), got
>  tpm2            0x4
> -tpm2            FAILED [LOW] TPM2ReservedNonZero: Test 1, TPM2 reserved
> +tpm2            FAILED [MEDIUM] TPM2ReservedNonZero: Test 1, TPM2 Reserved
>  tpm2            field must be zero, got 0x0003 instead
>  tpm2            FAILED [HIGH] TPM2BadStartMethod: Test 1, TPM2's Start
>  tpm2            Method must be between one to eleven, got 0xc
> 
Acked-by: Colin Ian King <colin.king@canonical.com>
Ivan Hu Sept. 21, 2017, 8:25 a.m. UTC | #2
On 09/07/2017 08:34 AM, Alex Hung wrote:
> Signed-off-by: Alex Hung <alex.hung@canonical.com>
> ---
>   fwts-test/asf-0001/asf-0002.log   | 11 +++++++----
>   fwts-test/erst-0001/erst-0002.log |  4 ++--
>   fwts-test/hmat-0001/hmat-0002.log | 18 +++++++++---------
>   fwts-test/iort-0001/iort-0002.log | 16 ++++++++--------
>   fwts-test/lpit-0001/lpit-0002.log |  6 +++---
>   fwts-test/mpst-0001/mpst-0002.log | 20 +++++++++++---------
>   fwts-test/msdm-0001/msdm-0002.log |  8 ++++----
>   fwts-test/nfit-0001/nfit-0002.log |  8 ++++----
>   fwts-test/pmtt-0001/pmtt-0002.log | 22 +++++++++++-----------
>   fwts-test/pptt-0001/pptt-0002.log | 10 +++++-----
>   fwts-test/spcr-0001/spcr-0002.log |  4 ++--
>   fwts-test/tpm2-0001/tpm2-0002.log |  2 +-
>   12 files changed, 67 insertions(+), 62 deletions(-)
> 
> diff --git a/fwts-test/asf-0001/asf-0002.log b/fwts-test/asf-0001/asf-0002.log
> index e8d25bc..b8212e7 100644
> --- a/fwts-test/asf-0001/asf-0002.log
> +++ b/fwts-test/asf-0001/asf-0002.log
> @@ -11,9 +11,12 @@ asf             range 0x02..0xff
>   asf             FAILED [HIGH] ASF!InfoFeatureFlagsReserved: Test 1, ASF!
>   asf             ASF_INFO Feature Flags is 0xff, however reserved bits [7
>   asf             :1] must be zero
> -asf             FAILED [HIGH] ASF!InfoReservedNonZero: Test 1, ASF!
> -asf             ASF_INFO Reserved fields must be zero, however one or more
> -asf             of them are non-zero
> +asf             FAILED [MEDIUM] ASF!ReservedNonZero: Test 1, ASF! ASF_INFO
> +asf             Reserved1 field must be zero, got 0x10 instead
> +asf             FAILED [MEDIUM] ASF!ReservedNonZero: Test 1, ASF! ASF_INFO
> +asf             Reserved2 field must be zero, got 0x20 instead
> +asf             FAILED [MEDIUM] ASF!ReservedNonZero: Test 1, ASF! ASF_INFO
> +asf             Reserved3 field must be zero, got 0x40 instead
>   asf             FAILED [HIGH] ASF!AlrtNumOfAlertsInvalid: Test 1, ASF!
>   asf             ASF_ALRT Number of Alerts field is 0x23 and must be in the
>   asf             range 0x01..0x08
> @@ -30,6 +33,6 @@ asf             long for the size given by the ASF! table. Expected at
>   asf             most 22 bytes.
>   asf
>   asf             ==========================================================
> -asf             1 passed, 8 failed, 0 warning, 0 aborted, 0 skipped, 0
> +asf             1 passed, 10 failed, 0 warning, 0 aborted, 0 skipped, 0
>   asf             info only.
>   asf             ==========================================================
> diff --git a/fwts-test/erst-0001/erst-0002.log b/fwts-test/erst-0001/erst-0002.log
> index 29eefef..d9aecc3 100644
> --- a/fwts-test/erst-0001/erst-0002.log
> +++ b/fwts-test/erst-0001/erst-0002.log
> @@ -5,8 +5,8 @@ erst            ERST Error Record Serialization Table :
>   erst              Serialization Hdr. Size:  0x00000030
>   erst              Reserved:                 0x000000f0
>   erst              Instruction Entry Count:  0x00000010
> -erst            FAILED [LOW] ERSTReservedNonZero: Test 1, ERST Reserved
> -erst            field must be zero, got 0xf0 instead
> +erst            FAILED [MEDIUM] ERSTReservedNonZero: Test 1, ERST Reserved
> +erst            field must be zero, got 0x000000f0 instead
>   erst            FAILED [HIGH] ERSTIInvalidFlag: Test 1, ERST Serialization
>   erst            Flag 0xff is an invalid value, values allowed are
>   erst            0x00..0x01 for the WRITE_REGISTER and WRITE_REGISTER_VALUE
> diff --git a/fwts-test/hmat-0001/hmat-0002.log b/fwts-test/hmat-0001/hmat-0002.log
> index 2394c28..d90468b 100644
> --- a/fwts-test/hmat-0001/hmat-0002.log
> +++ b/fwts-test/hmat-0001/hmat-0002.log
> @@ -14,13 +14,13 @@ hmat                Memory Proximity Domain:        0x00000000
>   hmat                Reserved:                       0x00000003
>   hmat                System Phy Addr Range Base:     0x0000000000000000
>   hmat                System Phy Addr Range Length:   0x0000000000000000
> -hmat            FAILED [LOW] HMATReservedNonZero: Test 1, HMAT reserved
> +hmat            FAILED [MEDIUM] HMATReservedNonZero: Test 1, HMAT Reserved
>   hmat            field must be zero, got 0x0001 instead
>   hmat            FAILED [HIGH] HMATBadFlags: Test 1, HMAT Flags's
>   hmat            Bits[15..3] must be zero, got 0x000f instead
> -hmat            FAILED [LOW] HMATReservedNonZero: Test 1, HMAT reserved
> +hmat            FAILED [MEDIUM] HMATReservedNonZero: Test 1, HMAT Reserved
>   hmat            field must be zero, got 0x0002 instead
> -hmat            FAILED [LOW] HMATReservedNonZero: Test 1, HMAT reserved
> +hmat            FAILED [MEDIUM] HMATReservedNonZero: Test 1, HMAT Reserved
>   hmat            field must be zero, got 0x00000003 instead
>   hmat
>   hmat              System Locality Latency and Bandwidth Information (Type 1):
> @@ -34,15 +34,15 @@ hmat                Number of Initiator PDs:        0x00000002
>   hmat                Number of Target PDs:           0x00000003
>   hmat                Reserved:                       0x00000006
>   hmat                Entry Base Unit:                0x0000000000000000
> -hmat            FAILED [LOW] HMATReservedNonZero: Test 1, HMAT reserved
> +hmat            FAILED [MEDIUM] HMATReservedNonZero: Test 1, HMAT Reserved
>   hmat            field must be zero, got 0x0004 instead
>   hmat            FAILED [CRITICAL] HMATBadFlags: Test 1, HMAT Flags's
>   hmat            Bits[7..5] must be zero, got 0x3f instead
>   hmat            FAILED [CRITICAL] HMATBadFDataType: Test 1, HMAT Data Type
>   hmat            must be 0..5, got 0x06 instead
> -hmat            FAILED [LOW] HMATReservedNonZero: Test 1, HMAT reserved
> +hmat            FAILED [MEDIUM] HMATReservedNonZero: Test 1, HMAT Reserved
>   hmat            field must be zero, got 0x0005 instead
> -hmat            FAILED [LOW] HMATReservedNonZero: Test 1, HMAT reserved
> +hmat            FAILED [MEDIUM] HMATReservedNonZero: Test 1, HMAT Reserved
>   hmat            field must be zero, got 0x00000006 instead
>   hmat
>   hmat              Memory Side Cache Information (Type 2):
> @@ -55,14 +55,14 @@ hmat                Memory Side Cache Size:         0x0000000000000000
>   hmat                Cache Attributes:               0x00002333
>   hmat                Reserved:                       0x0009
>   hmat                Number of SMBIOS Handles:       0x0003
> -hmat            FAILED [LOW] HMATReservedNonZero: Test 1, HMAT reserved
> +hmat            FAILED [MEDIUM] HMATReservedNonZero: Test 1, HMAT Reserved
>   hmat            field must be zero, got 0x0007 instead
>   hmat            FAILED [CRITICAL] HMATBadCacheAttributeReserved: Test 1,
>   hmat            HMAT Cache Attribute reserved values are used, got
>   hmat            0x00002333 instead
> -hmat            FAILED [LOW] HMATReservedNonZero: Test 1, HMAT reserved
> +hmat            FAILED [MEDIUM] HMATReservedNonZero: Test 1, HMAT Reserved
>   hmat            field must be zero, got 0x00000008 instead
> -hmat            FAILED [LOW] HMATReservedNonZero: Test 1, HMAT reserved
> +hmat            FAILED [MEDIUM] HMATReservedNonZero: Test 1, HMAT Reserved
>   hmat            field must be zero, got 0x0009 instead
>   hmat
>   hmat
> diff --git a/fwts-test/iort-0001/iort-0002.log b/fwts-test/iort-0001/iort-0002.log
> index a726735..7a0746b 100644
> --- a/fwts-test/iort-0001/iort-0002.log
> +++ b/fwts-test/iort-0001/iort-0002.log
> @@ -20,8 +20,8 @@ iort            Either the Number of ID mappings is too large or the IORT
>   iort            table size or ITS Group Node is too small.
>   iort            FAILED [LOW] IORTNodeRevisionNonZero: Test 1, IORT Node
>   iort            Revision field is 0x03 and should be zero.
> -iort            FAILED [LOW] IORTNodeReservedNonZero: Test 1, IORT Node
> -iort            Reserved field is 0x0ffffffff and should be zero.
> +iort            FAILED [MEDIUM] IORTReservedNonZero: Test 1, IORT Node
> +iort            Reserved field must be zero, got 0xffffffff instead
>   iort            FAILED [LOW] IORTNodeMappingsCountNonZero: Test 1, IORT
>   iort            Node Number of ID Mappings field is 0x000000080 and should
>   iort            be zero for this node type.
> @@ -57,8 +57,8 @@ iort              Output Reference:         0x88888888
>   iort              Flags:                    0x00000001
>   iort            FAILED [LOW] IORTNodeRevisionNonZero: Test 1, IORT Node
>   iort            Revision field is 0x02 and should be zero.
> -iort            FAILED [LOW] IORTNodeReservedNonZero: Test 1, IORT Node
> -iort            Reserved field is 0x00000f001 and should be zero.
> +iort            FAILED [MEDIUM] IORTReservedNonZero: Test 1, IORT Node
> +iort            Reserved field must be zero, got 0x0000f001 instead
>   iort            FAILED [HIGH] IORTNamedComponentNodeFlagsNonZero: Test 1,
>   iort            IORT Named Component Node Flags is 0xffffffff and is
>   iort            reserved and should be zero.
> @@ -100,8 +100,8 @@ iort              Output Reference:         0x00000005
>   iort              Flags:                    0x00000001
>   iort            FAILED [LOW] IORTNodeRevisionNonZero: Test 1, IORT Node
>   iort            Revision field is 0x50 and should be zero.
> -iort            FAILED [LOW] IORTNodeReservedNonZero: Test 1, IORT Node
> -iort            Reserved field is 0x00000000f and should be zero.
> +iort            FAILED [MEDIUM] IORTReservedNonZero: Test 1, IORT Node
> +iort            Reserved field must be zero, got 0x0000000f instead
>   iort            FAILED [HIGH] IORTCPCIRootComplexAtsAttrInvalid: Test 1,
>   iort            IORT PCI Root Complex Node ATS Attribute is 0x3 and was
>   iort            expecting either 0 (root complex supports ATS) or 1 (root
> @@ -162,8 +162,8 @@ iort              GSIV:                     0xf0008000
>   iort              Interrupt Flags:          0x00000004
>   iort            FAILED [LOW] IORTNodeRevisionNonZero: Test 1, IORT Node
>   iort            Revision field is 0xa0 and should be zero.
> -iort            FAILED [LOW] IORTNodeReservedNonZero: Test 1, IORT Node
> -iort            Reserved field is 0x000008100 and should be zero.
> +iort            FAILED [MEDIUM] IORTReservedNonZero: Test 1, IORT Node
> +iort            Reserved field must be zero, got 0x00008100 instead
>   iort            FAILED [MEDIUM] IORTSMMUIntrFlagsReservedNonZero: Test 1,
>   iort            IORT SMMU_NsgIrpt Flags field reserved bits [31:1] should
>   iort            be all zero, got 0x2
> diff --git a/fwts-test/lpit-0001/lpit-0002.log b/fwts-test/lpit-0001/lpit-0002.log
> index aa3de10..d362acd 100644
> --- a/fwts-test/lpit-0001/lpit-0002.log
> +++ b/fwts-test/lpit-0001/lpit-0002.log
> @@ -23,9 +23,9 @@ lpit                Access Size             0x00
>   lpit                Address                 0x1020304050607080
>   lpit              Residency Counter Freq:   0x000000000000b0a0
>   lpit
> -lpit            FAILED [LOW] LPITNativeCStateLpitReservedNonZero: Test 1,
> -lpit            Native C-state based LPI structure reserved field was
> -lpit            expected to be zero, got 0xffff instead
> +lpit            FAILED [MEDIUM] LPITReservedNonZero: Test 1, LPIT Native
> +lpit            C-state based LPI structure reserved field must be zero,
> +lpit            got 0xffff instead
>   lpit            FAILED [LOW] LPITNativeCStateLpitFlagsReserved: Test 1,
>   lpit            Some of the Native C-state based LPI structure flags bits
>   lpit            [31:2] are set, they are expected to be zero
> diff --git a/fwts-test/mpst-0001/mpst-0002.log b/fwts-test/mpst-0001/mpst-0002.log
> index adda211..37b216b 100644
> --- a/fwts-test/mpst-0001/mpst-0002.log
> +++ b/fwts-test/mpst-0001/mpst-0002.log
> @@ -4,11 +4,11 @@ mpst            Test 1 of 1: Validate MPST table.
>   mpst            MPST Table:
>   mpst              Communication Channel ID:        0x00
>   mpst              Reserved:                        0x00010000
> -mpst            FAILED [LOW] MPSTReservedNonZero: Test 1, MPST reserved
> +mpst            FAILED [MEDIUM] MPSTReservedNonZero: Test 1, MPST Reserved
>   mpst            field must be zero, got 0x00010000 instead
>   mpst              Memory Power Node Count:         0x0002
>   mpst              Reserved:                        0x0003
> -mpst            FAILED [LOW] MPSTReservedNonZero: Test 1, MPST reserved
> +mpst            FAILED [MEDIUM] MPSTReservedNonZero: Test 1, MPST Reserved
>   mpst            field must be zero, got 0x0003 instead
>   mpst              MPST Power Node:
>   mpst                Flags:                         0x00
> @@ -19,7 +19,7 @@ mpst                Base Address:                  0x00000000f0000000
>   mpst                Memory Length:                 0x00000000e0000000
>   mpst                Number of Power States:        0x00000002
>   mpst                Number of Physical Components: 0x00000003
> -mpst            FAILED [LOW] MPSTReservedNonZero: Test 1, MPST reserved
> +mpst            FAILED [MEDIUM] MPSTReservedNonZero: Test 1, MPST Reserved
>   mpst            field must be zero, got 0x04 instead
>   mpst            FAILED [HIGH] MPSTBadPowerNodeLength: Test 1, MPST Power
>   mpst            Node Length should be 0x0000002a, got 0x0000012a instead
> @@ -39,7 +39,7 @@ mpst                Base Address:                  0x00000000a0000000
>   mpst                Memory Length:                 0x00000000b0000000
>   mpst                Number of Power States:        0x00000001
>   mpst                Number of Physical Components: 0x00000001
> -mpst            FAILED [LOW] MPSTReservedNonZero: Test 1, MPST reserved
> +mpst            FAILED [MEDIUM] MPSTReservedNonZero: Test 1, MPST Reserved
>   mpst            field must be zero, got 0x05 instead
>   mpst            FAILED [HIGH] MPSTBadPowerNodeLength: Test 1, MPST Power
>   mpst            Node Length should be 0x00000024, got 0x00000124 instead
> @@ -48,7 +48,7 @@ mpst                Power State Information Index: 0x06
>   mpst                Physical Component Id:         0x03
>   mpst              Memory Characteristics Count:    0x0002
>   mpst              Reserved:                        0x0006
> -mpst            FAILED [LOW] MPSTReservedNonZero: Test 1, MPST reserved
> +mpst            FAILED [MEDIUM] MPSTReservedNonZero: Test 1, MPST Reserved
>   mpst            field must be zero, got 0x0006 instead
>   mpst              MPST Power Characteristics:
>   mpst                Power State Structure ID:      0x02
> @@ -63,8 +63,9 @@ mpst            Power Characteristics ID must be 1, got 0x02 instead
>   mpst            FAILED [HIGH] MPSTBadPowerCharacteristicsRevision: Test 1,
>   mpst            MPST Power Characteristics Revision must be 1, got 0x00
>   mpst            instead
> -mpst            FAILED [LOW] MPSTReservedNonZero: Test 1, MPST reserved
> -mpst            field must be zero, got 0x0000000000000007 instead
> +mpst            FAILED [MEDIUM] MPSTReservedNonZero: Test 1, MPST
> +mpst            Reserved2 field must be zero, got 0x0000000000000007
> +mpst            instead
>   mpst              MPST Power Characteristics:
>   mpst                Power State Structure ID:      0x03
>   mpst                Flags:                         0x00
> @@ -78,8 +79,9 @@ mpst            Power Characteristics ID must be 1, got 0x03 instead
>   mpst            FAILED [HIGH] MPSTBadPowerCharacteristicsRevision: Test 1,
>   mpst            MPST Power Characteristics Revision must be 1, got 0x00
>   mpst            instead
> -mpst            FAILED [LOW] MPSTReservedNonZero: Test 1, MPST reserved
> -mpst            field must be zero, got 0x0000000000000008 instead
> +mpst            FAILED [MEDIUM] MPSTReservedNonZero: Test 1, MPST
> +mpst            Reserved2 field must be zero, got 0x0000000000000008
> +mpst            instead
>   mpst
>   mpst
>   mpst            ==========================================================
> diff --git a/fwts-test/msdm-0001/msdm-0002.log b/fwts-test/msdm-0001/msdm-0002.log
> index 2f4953e..9fa7792 100644
> --- a/fwts-test/msdm-0001/msdm-0002.log
> +++ b/fwts-test/msdm-0001/msdm-0002.log
> @@ -5,10 +5,10 @@ msdm              Reserved:                 0xffffffff
>   msdm              Data Type:                0x00000001
>   msdm              Data Reserved:            0x00000020
>   msdm              Data Length:              0x0000001d
> -msdm            FAILED [HIGH] MSDMReservedNonZero: Test 1, MSDM Reserved
> -msdm            field should be zero, got 0xffffffff instead
> -msdm            FAILED [HIGH] MSDMDataReservedNonZero: Test 1, MSDM Data
> -msdm            Reserved field should be zero, got 0x00000020 instead
> +msdm            FAILED [MEDIUM] MSDMReservedNonZero: Test 1, MSDM Reserved
> +msdm            field must be zero, got 0xffffffff instead
> +msdm            FAILED [MEDIUM] MSDMReservedNonZero: Test 1, MSDM Data
> +msdm            Reserved field must be zero, got 0x00000020 instead
>   msdm              Data:                     'AH3NR*B8FDN-C1WTG-5CR9X-1AKMH'
>   msdm            FAILED [HIGH] MSDMDataLengthInvalid: Test 1, MSDM Data
>   msdm            field did not contain digits, uppercase letters and -
> diff --git a/fwts-test/nfit-0001/nfit-0002.log b/fwts-test/nfit-0001/nfit-0002.log
> index 47be360..02fba5a 100644
> --- a/fwts-test/nfit-0001/nfit-0002.log
> +++ b/fwts-test/nfit-0001/nfit-0002.log
> @@ -4,7 +4,7 @@ nfit            Test 1 of 1: NFIT NVDIMM Firmware Interface Table test.
>   nfit            NFIT NVDIMM Firmware Interface Table:
>   nfit              Reserved:                 0x05000000
>   nfit
> -nfit            FAILED [LOW] NFITReservedNonZero: Test 1, NFIT reserved
> +nfit            FAILED [MEDIUM] NFITReservedNonZero: Test 1, NFIT Reserved
>   nfit            field must be zero, got 0x05000000 instead
>   nfit              NFIT Subtable:
>   nfit                Type:                                   0x0000
> @@ -24,7 +24,7 @@ nfit            Bits[15..2] must be zero, got 0x00000005 instead
>   nfit            FAILED [HIGH] NFITBadMemoryMappingAttribute: Test 1, NFIT
>   nfit            Memory Mapping Attribute must meet UEFI Spec, got
>   nfit            0x00000000000000ff instead
> -nfit            FAILED [LOW] NFITReservedNonZero: Test 1, NFIT reserved
> +nfit            FAILED [MEDIUM] NFITReservedNonZero: Test 1, NFIT Reserved
>   nfit            field must be zero, got 0x000000000000000f instead
>   nfit
>   nfit              NFIT Subtable:
> @@ -103,7 +103,7 @@ nfit            FAILED [HIGH] NFITBadValidField: Test 1, NFIT Valid
>   nfit            Field's Bits[7..1] must be zero, got 0x03 instead
>   nfit            FAILED [HIGH] NFITBadFlags: Test 1, NFIT Flags's
>   nfit            Bits[15..1] must be zero, got 0x0003 instead
> -nfit            FAILED [LOW] NFITReservedNonZero: Test 1, NFIT reserved
> +nfit            FAILED [MEDIUM] NFITReservedNonZero: Test 1, NFIT Reserved
>   nfit            field must be zero, got 0x0000000000000f00 instead
>   nfit
>   nfit              NFIT Subtable:
> @@ -126,7 +126,7 @@ nfit                Number of Flush Hint Addresses:         0x0002
>   nfit                Reserved:                               0x00000000000000f0
>   nfit                Flush Hint Address:                     0x0000000418000000
>   nfit                Flush Hint Address:                     0x0000000618000000
> -nfit            FAILED [LOW] NFITReservedNonZero: Test 1, NFIT reserved
> +nfit            FAILED [MEDIUM] NFITReservedNonZero: Test 1, NFIT Reserved
>   nfit            field must be zero, got 0x00000000000000f0 instead
>   nfit
>   nfit
> diff --git a/fwts-test/pmtt-0001/pmtt-0002.log b/fwts-test/pmtt-0001/pmtt-0002.log
> index c5e060d..2ce24a2 100644
> --- a/fwts-test/pmtt-0001/pmtt-0002.log
> +++ b/fwts-test/pmtt-0001/pmtt-0002.log
> @@ -3,7 +3,7 @@ pmtt            ----------------------------------------------------------
>   pmtt            Test 1 of 1: Validate PMTT table.
>   pmtt            PMTT Table:
>   pmtt              Reserved:                         0x02000000
> -pmtt            FAILED [LOW] PMTTReservedNonZero: Test 1, PMTT reserved
> +pmtt            FAILED [MEDIUM] PMTTReservedNonZero: Test 1, PMTT Reserved
>   pmtt            field must be zero, got 0x02000000 instead
>   pmtt              PMTT Subtable:
>   pmtt                Type:                           0x00
> @@ -11,13 +11,13 @@ pmtt                Reserved:                       0x01
>   pmtt                Length:                         0x0098
>   pmtt                Flags:                          0x0001
>   pmtt                Reserved:                       0x0003
> -pmtt            FAILED [LOW] PMTTReservedNonZero: Test 1, PMTT reserved
> -pmtt            field must be zero, got 0x01 instead
> -pmtt            FAILED [LOW] PMTTReservedNonZero: Test 1, PMTT reserved
> -pmtt            field must be zero, got 0x0003 instead
> +pmtt            FAILED [MEDIUM] PMTTReservedNonZero: Test 1, PMTT
> +pmtt            Reserved1 field must be zero, got 0x01 instead
> +pmtt            FAILED [MEDIUM] PMTTReservedNonZero: Test 1, PMTT
> +pmtt            Reserved2 field must be zero, got 0x0003 instead
>   pmtt                Socket Identifier:              0x0001
>   pmtt                Reserved:                       0x0400
> -pmtt            FAILED [LOW] PMTTReservedNonZero: Test 1, PMTT reserved
> +pmtt            FAILED [MEDIUM] PMTTReservedNonZero: Test 1, PMTT Reserved
>   pmtt            field must be zero, got 0x0400 instead
>   pmtt              PMTT Subtable:
>   pmtt                Type:                           0x01
> @@ -25,8 +25,8 @@ pmtt                Reserved:                       0x05
>   pmtt                Length:                         0x006c
>   pmtt                Flags:                          0x0005
>   pmtt                Reserved:                       0x0000
> -pmtt            FAILED [LOW] PMTTReservedNonZero: Test 1, PMTT reserved
> -pmtt            field must be zero, got 0x05 instead
> +pmtt            FAILED [MEDIUM] PMTTReservedNonZero: Test 1, PMTT
> +pmtt            Reserved1 field must be zero, got 0x05 instead
>   pmtt                Read Latency:                   0x0000f000
>   pmtt                Write latency:                  0x0000e000
>   pmtt                Read Bandwidth:                 0x0000d000
> @@ -35,7 +35,7 @@ pmtt                Optimal Access Unit:            0x0004
>   pmtt                Optimal Access Alignment:       0x0008
>   pmtt                Reserved:                       0x0600
>   pmtt                Number of Proximity Domains:    0x0004
> -pmtt            FAILED [LOW] PMTTReservedNonZero: Test 1, PMTT reserved
> +pmtt            FAILED [MEDIUM] PMTTReservedNonZero: Test 1, PMTT Reserved
>   pmtt            field must be zero, got 0x0600 instead
>   pmtt                Proximity Domain:               0x00001000
>   pmtt                Proximity Domain:               0x00002000
> @@ -47,8 +47,8 @@ pmtt                Reserved:                       0x07
>   pmtt                Length:                         0x0014
>   pmtt                Flags:                          0x0002
>   pmtt                Reserved:                       0x0000
> -pmtt            FAILED [LOW] PMTTReservedNonZero: Test 1, PMTT reserved
> -pmtt            field must be zero, got 0x07 instead
> +pmtt            FAILED [MEDIUM] PMTTReservedNonZero: Test 1, PMTT
> +pmtt            Reserved1 field must be zero, got 0x07 instead
>   pmtt                Physical Component Identifier:  0x0000
>   pmtt                Reserved:                       0x0000
>   pmtt                Size of DIMM:                   0xa0000000
> diff --git a/fwts-test/pptt-0001/pptt-0002.log b/fwts-test/pptt-0001/pptt-0002.log
> index 57c3391..4b3f2a6 100644
> --- a/fwts-test/pptt-0001/pptt-0002.log
> +++ b/fwts-test/pptt-0001/pptt-0002.log
> @@ -11,7 +11,7 @@ pptt                Parent:                         0x00000000
>   pptt                ACPI Processor ID:              0x00000000
>   pptt                Number of Private Resources:    0x00000001
>   pptt                Private Resources[0]:           0x00000000
> -pptt            FAILED [LOW] PPTTReservedNonZero: Test 1, PPTT reserved
> +pptt            FAILED [MEDIUM] PPTTReservedNonZero: Test 1, PPTT Reserved
>   pptt            field must be zero, got 0x0001 instead
>   pptt
>   pptt              Cache Type Structure (Type 1):
> @@ -25,7 +25,7 @@ pptt                Number of sets:                 0x00000000
>   pptt                Associativity:                  0x00
>   pptt                Attributes:                     0x00
>   pptt                Line size:                      0x0000
> -pptt            FAILED [LOW] PPTTReservedNonZero: Test 1, PPTT reserved
> +pptt            FAILED [MEDIUM] PPTTReservedNonZero: Test 1, PPTT Reserved
>   pptt            field must be zero, got 0x0002 instead
>   pptt
>   pptt              Processor hierarchy node structure (Type 0):
> @@ -38,7 +38,7 @@ pptt                ACPI Processor ID:              0x00000000
>   pptt                Number of Private Resources:    0x00000002
>   pptt                Private Resources[0]:           0x00000000
>   pptt                Private Resources[1]:           0x00000000
> -pptt            FAILED [LOW] PPTTReservedNonZero: Test 1, PPTT reserved
> +pptt            FAILED [MEDIUM] PPTTReservedNonZero: Test 1, PPTT Reserved
>   pptt            field must be zero, got 0x0003 instead
>   pptt            FAILED [HIGH] PPTTBadFlags: Test 1, PPTT Flags's
>   pptt            Bits[31..2] must be zero, got 0x00000007 instead
> @@ -54,7 +54,7 @@ pptt                Number of sets:                 0x00000000
>   pptt                Associativity:                  0xc0
>   pptt                Attributes:                     0x00
>   pptt                Line size:                      0x0000
> -pptt            FAILED [LOW] PPTTReservedNonZero: Test 1, PPTT reserved
> +pptt            FAILED [MEDIUM] PPTTReservedNonZero: Test 1, PPTT Reserved
>   pptt            field must be zero, got 0x0004 instead
>   pptt            FAILED [HIGH] PPTTBadFlags: Test 1, PPTT Flags's
>   pptt            Bits[31..7] must be zero, got 0x000000ff instead
> @@ -69,7 +69,7 @@ pptt                LEVEL_2_ID:                     0x0000000000000000
>   pptt                MAJOR_REV:                      0x0000
>   pptt                MINOR_REV:                      0x0000
>   pptt                SPIN_REV:                       0x0000
> -pptt            FAILED [LOW] PPTTReservedNonZero: Test 1, PPTT reserved
> +pptt            FAILED [MEDIUM] PPTTReservedNonZero: Test 1, PPTT Reserved
>   pptt            field must be zero, got 0x0005 instead
>   pptt
>   pptt
> diff --git a/fwts-test/spcr-0001/spcr-0002.log b/fwts-test/spcr-0001/spcr-0002.log
> index 437a4f9..e22fc40 100644
> --- a/fwts-test/spcr-0001/spcr-0002.log
> +++ b/fwts-test/spcr-0001/spcr-0002.log
> @@ -5,8 +5,8 @@ spcr            test.
>   spcr            Serial Interface: Reserved (Do not Use)
>   spcr            FAILED [HIGH] SPCRInterfaceReserved: Test 1, SPCR Serial
>   spcr            interface type 0x08 is a reserved interface
> -spcr            FAILED [LOW] SPCRReservedNonZero: Test 1, SPCR reserved
> -spcr            field must be zero, got 0xf02000 instead
> +spcr            FAILED [MEDIUM] SPCRReservedNonZero: Test 1, SPCR
> +spcr            Reserved1 field must be zero, got 0x000020f0 instead
>   spcr            Baud Rate:        Reserved
>   spcr            FAILED [HIGH] SPCRBaudRateReserved: Test 1, SPCR Serial
>   spcr            baud rate type 0x08 is a reserved baud rate
> diff --git a/fwts-test/tpm2-0001/tpm2-0002.log b/fwts-test/tpm2-0001/tpm2-0002.log
> index 8e8bad7..c93c153 100644
> --- a/fwts-test/tpm2-0001/tpm2-0002.log
> +++ b/fwts-test/tpm2-0001/tpm2-0002.log
> @@ -9,7 +9,7 @@ tpm2              Start Method:                    0x0000000c
>   tpm2            FAILED [HIGH] TPM2BadPlatformClass: Test 1, TPM2's
>   tpm2            platform class must be zero (client) or one (server), got
>   tpm2            0x4
> -tpm2            FAILED [LOW] TPM2ReservedNonZero: Test 1, TPM2 reserved
> +tpm2            FAILED [MEDIUM] TPM2ReservedNonZero: Test 1, TPM2 Reserved
>   tpm2            field must be zero, got 0x0003 instead
>   tpm2            FAILED [HIGH] TPM2BadStartMethod: Test 1, TPM2's Start
>   tpm2            Method must be between one to eleven, got 0xc
> 

Acked-by: Ivan Hu <ivan.hu@canonical.com>
diff mbox series

Patch

diff --git a/fwts-test/asf-0001/asf-0002.log b/fwts-test/asf-0001/asf-0002.log
index e8d25bc..b8212e7 100644
--- a/fwts-test/asf-0001/asf-0002.log
+++ b/fwts-test/asf-0001/asf-0002.log
@@ -11,9 +11,12 @@  asf             range 0x02..0xff
 asf             FAILED [HIGH] ASF!InfoFeatureFlagsReserved: Test 1, ASF!
 asf             ASF_INFO Feature Flags is 0xff, however reserved bits [7
 asf             :1] must be zero
-asf             FAILED [HIGH] ASF!InfoReservedNonZero: Test 1, ASF!
-asf             ASF_INFO Reserved fields must be zero, however one or more
-asf             of them are non-zero
+asf             FAILED [MEDIUM] ASF!ReservedNonZero: Test 1, ASF! ASF_INFO
+asf             Reserved1 field must be zero, got 0x10 instead
+asf             FAILED [MEDIUM] ASF!ReservedNonZero: Test 1, ASF! ASF_INFO
+asf             Reserved2 field must be zero, got 0x20 instead
+asf             FAILED [MEDIUM] ASF!ReservedNonZero: Test 1, ASF! ASF_INFO
+asf             Reserved3 field must be zero, got 0x40 instead
 asf             FAILED [HIGH] ASF!AlrtNumOfAlertsInvalid: Test 1, ASF!
 asf             ASF_ALRT Number of Alerts field is 0x23 and must be in the
 asf             range 0x01..0x08
@@ -30,6 +33,6 @@  asf             long for the size given by the ASF! table. Expected at
 asf             most 22 bytes.
 asf             
 asf             ==========================================================
-asf             1 passed, 8 failed, 0 warning, 0 aborted, 0 skipped, 0
+asf             1 passed, 10 failed, 0 warning, 0 aborted, 0 skipped, 0
 asf             info only.
 asf             ==========================================================
diff --git a/fwts-test/erst-0001/erst-0002.log b/fwts-test/erst-0001/erst-0002.log
index 29eefef..d9aecc3 100644
--- a/fwts-test/erst-0001/erst-0002.log
+++ b/fwts-test/erst-0001/erst-0002.log
@@ -5,8 +5,8 @@  erst            ERST Error Record Serialization Table :
 erst              Serialization Hdr. Size:  0x00000030
 erst              Reserved:                 0x000000f0
 erst              Instruction Entry Count:  0x00000010
-erst            FAILED [LOW] ERSTReservedNonZero: Test 1, ERST Reserved
-erst            field must be zero, got 0xf0 instead
+erst            FAILED [MEDIUM] ERSTReservedNonZero: Test 1, ERST Reserved
+erst            field must be zero, got 0x000000f0 instead
 erst            FAILED [HIGH] ERSTIInvalidFlag: Test 1, ERST Serialization
 erst            Flag 0xff is an invalid value, values allowed are
 erst            0x00..0x01 for the WRITE_REGISTER and WRITE_REGISTER_VALUE
diff --git a/fwts-test/hmat-0001/hmat-0002.log b/fwts-test/hmat-0001/hmat-0002.log
index 2394c28..d90468b 100644
--- a/fwts-test/hmat-0001/hmat-0002.log
+++ b/fwts-test/hmat-0001/hmat-0002.log
@@ -14,13 +14,13 @@  hmat                Memory Proximity Domain:        0x00000000
 hmat                Reserved:                       0x00000003
 hmat                System Phy Addr Range Base:     0x0000000000000000
 hmat                System Phy Addr Range Length:   0x0000000000000000
-hmat            FAILED [LOW] HMATReservedNonZero: Test 1, HMAT reserved
+hmat            FAILED [MEDIUM] HMATReservedNonZero: Test 1, HMAT Reserved
 hmat            field must be zero, got 0x0001 instead
 hmat            FAILED [HIGH] HMATBadFlags: Test 1, HMAT Flags's
 hmat            Bits[15..3] must be zero, got 0x000f instead
-hmat            FAILED [LOW] HMATReservedNonZero: Test 1, HMAT reserved
+hmat            FAILED [MEDIUM] HMATReservedNonZero: Test 1, HMAT Reserved
 hmat            field must be zero, got 0x0002 instead
-hmat            FAILED [LOW] HMATReservedNonZero: Test 1, HMAT reserved
+hmat            FAILED [MEDIUM] HMATReservedNonZero: Test 1, HMAT Reserved
 hmat            field must be zero, got 0x00000003 instead
 hmat            
 hmat              System Locality Latency and Bandwidth Information (Type 1):
@@ -34,15 +34,15 @@  hmat                Number of Initiator PDs:        0x00000002
 hmat                Number of Target PDs:           0x00000003
 hmat                Reserved:                       0x00000006
 hmat                Entry Base Unit:                0x0000000000000000
-hmat            FAILED [LOW] HMATReservedNonZero: Test 1, HMAT reserved
+hmat            FAILED [MEDIUM] HMATReservedNonZero: Test 1, HMAT Reserved
 hmat            field must be zero, got 0x0004 instead
 hmat            FAILED [CRITICAL] HMATBadFlags: Test 1, HMAT Flags's
 hmat            Bits[7..5] must be zero, got 0x3f instead
 hmat            FAILED [CRITICAL] HMATBadFDataType: Test 1, HMAT Data Type
 hmat            must be 0..5, got 0x06 instead
-hmat            FAILED [LOW] HMATReservedNonZero: Test 1, HMAT reserved
+hmat            FAILED [MEDIUM] HMATReservedNonZero: Test 1, HMAT Reserved
 hmat            field must be zero, got 0x0005 instead
-hmat            FAILED [LOW] HMATReservedNonZero: Test 1, HMAT reserved
+hmat            FAILED [MEDIUM] HMATReservedNonZero: Test 1, HMAT Reserved
 hmat            field must be zero, got 0x00000006 instead
 hmat            
 hmat              Memory Side Cache Information (Type 2):
@@ -55,14 +55,14 @@  hmat                Memory Side Cache Size:         0x0000000000000000
 hmat                Cache Attributes:               0x00002333
 hmat                Reserved:                       0x0009
 hmat                Number of SMBIOS Handles:       0x0003
-hmat            FAILED [LOW] HMATReservedNonZero: Test 1, HMAT reserved
+hmat            FAILED [MEDIUM] HMATReservedNonZero: Test 1, HMAT Reserved
 hmat            field must be zero, got 0x0007 instead
 hmat            FAILED [CRITICAL] HMATBadCacheAttributeReserved: Test 1,
 hmat            HMAT Cache Attribute reserved values are used, got
 hmat            0x00002333 instead
-hmat            FAILED [LOW] HMATReservedNonZero: Test 1, HMAT reserved
+hmat            FAILED [MEDIUM] HMATReservedNonZero: Test 1, HMAT Reserved
 hmat            field must be zero, got 0x00000008 instead
-hmat            FAILED [LOW] HMATReservedNonZero: Test 1, HMAT reserved
+hmat            FAILED [MEDIUM] HMATReservedNonZero: Test 1, HMAT Reserved
 hmat            field must be zero, got 0x0009 instead
 hmat            
 hmat            
diff --git a/fwts-test/iort-0001/iort-0002.log b/fwts-test/iort-0001/iort-0002.log
index a726735..7a0746b 100644
--- a/fwts-test/iort-0001/iort-0002.log
+++ b/fwts-test/iort-0001/iort-0002.log
@@ -20,8 +20,8 @@  iort            Either the Number of ID mappings is too large or the IORT
 iort            table size or ITS Group Node is too small.
 iort            FAILED [LOW] IORTNodeRevisionNonZero: Test 1, IORT Node
 iort            Revision field is 0x03 and should be zero.
-iort            FAILED [LOW] IORTNodeReservedNonZero: Test 1, IORT Node
-iort            Reserved field is 0x0ffffffff and should be zero.
+iort            FAILED [MEDIUM] IORTReservedNonZero: Test 1, IORT Node
+iort            Reserved field must be zero, got 0xffffffff instead
 iort            FAILED [LOW] IORTNodeMappingsCountNonZero: Test 1, IORT
 iort            Node Number of ID Mappings field is 0x000000080 and should
 iort            be zero for this node type.
@@ -57,8 +57,8 @@  iort              Output Reference:         0x88888888
 iort              Flags:                    0x00000001
 iort            FAILED [LOW] IORTNodeRevisionNonZero: Test 1, IORT Node
 iort            Revision field is 0x02 and should be zero.
-iort            FAILED [LOW] IORTNodeReservedNonZero: Test 1, IORT Node
-iort            Reserved field is 0x00000f001 and should be zero.
+iort            FAILED [MEDIUM] IORTReservedNonZero: Test 1, IORT Node
+iort            Reserved field must be zero, got 0x0000f001 instead
 iort            FAILED [HIGH] IORTNamedComponentNodeFlagsNonZero: Test 1,
 iort            IORT Named Component Node Flags is 0xffffffff and is
 iort            reserved and should be zero.
@@ -100,8 +100,8 @@  iort              Output Reference:         0x00000005
 iort              Flags:                    0x00000001
 iort            FAILED [LOW] IORTNodeRevisionNonZero: Test 1, IORT Node
 iort            Revision field is 0x50 and should be zero.
-iort            FAILED [LOW] IORTNodeReservedNonZero: Test 1, IORT Node
-iort            Reserved field is 0x00000000f and should be zero.
+iort            FAILED [MEDIUM] IORTReservedNonZero: Test 1, IORT Node
+iort            Reserved field must be zero, got 0x0000000f instead
 iort            FAILED [HIGH] IORTCPCIRootComplexAtsAttrInvalid: Test 1,
 iort            IORT PCI Root Complex Node ATS Attribute is 0x3 and was
 iort            expecting either 0 (root complex supports ATS) or 1 (root
@@ -162,8 +162,8 @@  iort              GSIV:                     0xf0008000
 iort              Interrupt Flags:          0x00000004
 iort            FAILED [LOW] IORTNodeRevisionNonZero: Test 1, IORT Node
 iort            Revision field is 0xa0 and should be zero.
-iort            FAILED [LOW] IORTNodeReservedNonZero: Test 1, IORT Node
-iort            Reserved field is 0x000008100 and should be zero.
+iort            FAILED [MEDIUM] IORTReservedNonZero: Test 1, IORT Node
+iort            Reserved field must be zero, got 0x00008100 instead
 iort            FAILED [MEDIUM] IORTSMMUIntrFlagsReservedNonZero: Test 1,
 iort            IORT SMMU_NsgIrpt Flags field reserved bits [31:1] should
 iort            be all zero, got 0x2
diff --git a/fwts-test/lpit-0001/lpit-0002.log b/fwts-test/lpit-0001/lpit-0002.log
index aa3de10..d362acd 100644
--- a/fwts-test/lpit-0001/lpit-0002.log
+++ b/fwts-test/lpit-0001/lpit-0002.log
@@ -23,9 +23,9 @@  lpit                Access Size             0x00
 lpit                Address                 0x1020304050607080
 lpit              Residency Counter Freq:   0x000000000000b0a0
 lpit            
-lpit            FAILED [LOW] LPITNativeCStateLpitReservedNonZero: Test 1,
-lpit            Native C-state based LPI structure reserved field was
-lpit            expected to be zero, got 0xffff instead
+lpit            FAILED [MEDIUM] LPITReservedNonZero: Test 1, LPIT Native
+lpit            C-state based LPI structure reserved field must be zero,
+lpit            got 0xffff instead
 lpit            FAILED [LOW] LPITNativeCStateLpitFlagsReserved: Test 1,
 lpit            Some of the Native C-state based LPI structure flags bits
 lpit            [31:2] are set, they are expected to be zero
diff --git a/fwts-test/mpst-0001/mpst-0002.log b/fwts-test/mpst-0001/mpst-0002.log
index adda211..37b216b 100644
--- a/fwts-test/mpst-0001/mpst-0002.log
+++ b/fwts-test/mpst-0001/mpst-0002.log
@@ -4,11 +4,11 @@  mpst            Test 1 of 1: Validate MPST table.
 mpst            MPST Table:
 mpst              Communication Channel ID:        0x00
 mpst              Reserved:                        0x00010000
-mpst            FAILED [LOW] MPSTReservedNonZero: Test 1, MPST reserved
+mpst            FAILED [MEDIUM] MPSTReservedNonZero: Test 1, MPST Reserved
 mpst            field must be zero, got 0x00010000 instead
 mpst              Memory Power Node Count:         0x0002
 mpst              Reserved:                        0x0003
-mpst            FAILED [LOW] MPSTReservedNonZero: Test 1, MPST reserved
+mpst            FAILED [MEDIUM] MPSTReservedNonZero: Test 1, MPST Reserved
 mpst            field must be zero, got 0x0003 instead
 mpst              MPST Power Node:
 mpst                Flags:                         0x00
@@ -19,7 +19,7 @@  mpst                Base Address:                  0x00000000f0000000
 mpst                Memory Length:                 0x00000000e0000000
 mpst                Number of Power States:        0x00000002
 mpst                Number of Physical Components: 0x00000003
-mpst            FAILED [LOW] MPSTReservedNonZero: Test 1, MPST reserved
+mpst            FAILED [MEDIUM] MPSTReservedNonZero: Test 1, MPST Reserved
 mpst            field must be zero, got 0x04 instead
 mpst            FAILED [HIGH] MPSTBadPowerNodeLength: Test 1, MPST Power
 mpst            Node Length should be 0x0000002a, got 0x0000012a instead
@@ -39,7 +39,7 @@  mpst                Base Address:                  0x00000000a0000000
 mpst                Memory Length:                 0x00000000b0000000
 mpst                Number of Power States:        0x00000001
 mpst                Number of Physical Components: 0x00000001
-mpst            FAILED [LOW] MPSTReservedNonZero: Test 1, MPST reserved
+mpst            FAILED [MEDIUM] MPSTReservedNonZero: Test 1, MPST Reserved
 mpst            field must be zero, got 0x05 instead
 mpst            FAILED [HIGH] MPSTBadPowerNodeLength: Test 1, MPST Power
 mpst            Node Length should be 0x00000024, got 0x00000124 instead
@@ -48,7 +48,7 @@  mpst                Power State Information Index: 0x06
 mpst                Physical Component Id:         0x03
 mpst              Memory Characteristics Count:    0x0002
 mpst              Reserved:                        0x0006
-mpst            FAILED [LOW] MPSTReservedNonZero: Test 1, MPST reserved
+mpst            FAILED [MEDIUM] MPSTReservedNonZero: Test 1, MPST Reserved
 mpst            field must be zero, got 0x0006 instead
 mpst              MPST Power Characteristics:
 mpst                Power State Structure ID:      0x02
@@ -63,8 +63,9 @@  mpst            Power Characteristics ID must be 1, got 0x02 instead
 mpst            FAILED [HIGH] MPSTBadPowerCharacteristicsRevision: Test 1,
 mpst            MPST Power Characteristics Revision must be 1, got 0x00
 mpst            instead
-mpst            FAILED [LOW] MPSTReservedNonZero: Test 1, MPST reserved
-mpst            field must be zero, got 0x0000000000000007 instead
+mpst            FAILED [MEDIUM] MPSTReservedNonZero: Test 1, MPST
+mpst            Reserved2 field must be zero, got 0x0000000000000007
+mpst            instead
 mpst              MPST Power Characteristics:
 mpst                Power State Structure ID:      0x03
 mpst                Flags:                         0x00
@@ -78,8 +79,9 @@  mpst            Power Characteristics ID must be 1, got 0x03 instead
 mpst            FAILED [HIGH] MPSTBadPowerCharacteristicsRevision: Test 1,
 mpst            MPST Power Characteristics Revision must be 1, got 0x00
 mpst            instead
-mpst            FAILED [LOW] MPSTReservedNonZero: Test 1, MPST reserved
-mpst            field must be zero, got 0x0000000000000008 instead
+mpst            FAILED [MEDIUM] MPSTReservedNonZero: Test 1, MPST
+mpst            Reserved2 field must be zero, got 0x0000000000000008
+mpst            instead
 mpst            
 mpst            
 mpst            ==========================================================
diff --git a/fwts-test/msdm-0001/msdm-0002.log b/fwts-test/msdm-0001/msdm-0002.log
index 2f4953e..9fa7792 100644
--- a/fwts-test/msdm-0001/msdm-0002.log
+++ b/fwts-test/msdm-0001/msdm-0002.log
@@ -5,10 +5,10 @@  msdm              Reserved:                 0xffffffff
 msdm              Data Type:                0x00000001
 msdm              Data Reserved:            0x00000020
 msdm              Data Length:              0x0000001d
-msdm            FAILED [HIGH] MSDMReservedNonZero: Test 1, MSDM Reserved
-msdm            field should be zero, got 0xffffffff instead
-msdm            FAILED [HIGH] MSDMDataReservedNonZero: Test 1, MSDM Data
-msdm            Reserved field should be zero, got 0x00000020 instead
+msdm            FAILED [MEDIUM] MSDMReservedNonZero: Test 1, MSDM Reserved
+msdm            field must be zero, got 0xffffffff instead
+msdm            FAILED [MEDIUM] MSDMReservedNonZero: Test 1, MSDM Data
+msdm            Reserved field must be zero, got 0x00000020 instead
 msdm              Data:                     'AH3NR*B8FDN-C1WTG-5CR9X-1AKMH'
 msdm            FAILED [HIGH] MSDMDataLengthInvalid: Test 1, MSDM Data
 msdm            field did not contain digits, uppercase letters and -
diff --git a/fwts-test/nfit-0001/nfit-0002.log b/fwts-test/nfit-0001/nfit-0002.log
index 47be360..02fba5a 100644
--- a/fwts-test/nfit-0001/nfit-0002.log
+++ b/fwts-test/nfit-0001/nfit-0002.log
@@ -4,7 +4,7 @@  nfit            Test 1 of 1: NFIT NVDIMM Firmware Interface Table test.
 nfit            NFIT NVDIMM Firmware Interface Table:
 nfit              Reserved:                 0x05000000
 nfit            
-nfit            FAILED [LOW] NFITReservedNonZero: Test 1, NFIT reserved
+nfit            FAILED [MEDIUM] NFITReservedNonZero: Test 1, NFIT Reserved
 nfit            field must be zero, got 0x05000000 instead
 nfit              NFIT Subtable:
 nfit                Type:                                   0x0000
@@ -24,7 +24,7 @@  nfit            Bits[15..2] must be zero, got 0x00000005 instead
 nfit            FAILED [HIGH] NFITBadMemoryMappingAttribute: Test 1, NFIT
 nfit            Memory Mapping Attribute must meet UEFI Spec, got
 nfit            0x00000000000000ff instead
-nfit            FAILED [LOW] NFITReservedNonZero: Test 1, NFIT reserved
+nfit            FAILED [MEDIUM] NFITReservedNonZero: Test 1, NFIT Reserved
 nfit            field must be zero, got 0x000000000000000f instead
 nfit            
 nfit              NFIT Subtable:
@@ -103,7 +103,7 @@  nfit            FAILED [HIGH] NFITBadValidField: Test 1, NFIT Valid
 nfit            Field's Bits[7..1] must be zero, got 0x03 instead
 nfit            FAILED [HIGH] NFITBadFlags: Test 1, NFIT Flags's
 nfit            Bits[15..1] must be zero, got 0x0003 instead
-nfit            FAILED [LOW] NFITReservedNonZero: Test 1, NFIT reserved
+nfit            FAILED [MEDIUM] NFITReservedNonZero: Test 1, NFIT Reserved
 nfit            field must be zero, got 0x0000000000000f00 instead
 nfit            
 nfit              NFIT Subtable:
@@ -126,7 +126,7 @@  nfit                Number of Flush Hint Addresses:         0x0002
 nfit                Reserved:                               0x00000000000000f0
 nfit                Flush Hint Address:                     0x0000000418000000
 nfit                Flush Hint Address:                     0x0000000618000000
-nfit            FAILED [LOW] NFITReservedNonZero: Test 1, NFIT reserved
+nfit            FAILED [MEDIUM] NFITReservedNonZero: Test 1, NFIT Reserved
 nfit            field must be zero, got 0x00000000000000f0 instead
 nfit            
 nfit            
diff --git a/fwts-test/pmtt-0001/pmtt-0002.log b/fwts-test/pmtt-0001/pmtt-0002.log
index c5e060d..2ce24a2 100644
--- a/fwts-test/pmtt-0001/pmtt-0002.log
+++ b/fwts-test/pmtt-0001/pmtt-0002.log
@@ -3,7 +3,7 @@  pmtt            ----------------------------------------------------------
 pmtt            Test 1 of 1: Validate PMTT table.
 pmtt            PMTT Table:
 pmtt              Reserved:                         0x02000000
-pmtt            FAILED [LOW] PMTTReservedNonZero: Test 1, PMTT reserved
+pmtt            FAILED [MEDIUM] PMTTReservedNonZero: Test 1, PMTT Reserved
 pmtt            field must be zero, got 0x02000000 instead
 pmtt              PMTT Subtable:
 pmtt                Type:                           0x00
@@ -11,13 +11,13 @@  pmtt                Reserved:                       0x01
 pmtt                Length:                         0x0098
 pmtt                Flags:                          0x0001
 pmtt                Reserved:                       0x0003
-pmtt            FAILED [LOW] PMTTReservedNonZero: Test 1, PMTT reserved
-pmtt            field must be zero, got 0x01 instead
-pmtt            FAILED [LOW] PMTTReservedNonZero: Test 1, PMTT reserved
-pmtt            field must be zero, got 0x0003 instead
+pmtt            FAILED [MEDIUM] PMTTReservedNonZero: Test 1, PMTT
+pmtt            Reserved1 field must be zero, got 0x01 instead
+pmtt            FAILED [MEDIUM] PMTTReservedNonZero: Test 1, PMTT
+pmtt            Reserved2 field must be zero, got 0x0003 instead
 pmtt                Socket Identifier:              0x0001
 pmtt                Reserved:                       0x0400
-pmtt            FAILED [LOW] PMTTReservedNonZero: Test 1, PMTT reserved
+pmtt            FAILED [MEDIUM] PMTTReservedNonZero: Test 1, PMTT Reserved
 pmtt            field must be zero, got 0x0400 instead
 pmtt              PMTT Subtable:
 pmtt                Type:                           0x01
@@ -25,8 +25,8 @@  pmtt                Reserved:                       0x05
 pmtt                Length:                         0x006c
 pmtt                Flags:                          0x0005
 pmtt                Reserved:                       0x0000
-pmtt            FAILED [LOW] PMTTReservedNonZero: Test 1, PMTT reserved
-pmtt            field must be zero, got 0x05 instead
+pmtt            FAILED [MEDIUM] PMTTReservedNonZero: Test 1, PMTT
+pmtt            Reserved1 field must be zero, got 0x05 instead
 pmtt                Read Latency:                   0x0000f000
 pmtt                Write latency:                  0x0000e000
 pmtt                Read Bandwidth:                 0x0000d000
@@ -35,7 +35,7 @@  pmtt                Optimal Access Unit:            0x0004
 pmtt                Optimal Access Alignment:       0x0008
 pmtt                Reserved:                       0x0600
 pmtt                Number of Proximity Domains:    0x0004
-pmtt            FAILED [LOW] PMTTReservedNonZero: Test 1, PMTT reserved
+pmtt            FAILED [MEDIUM] PMTTReservedNonZero: Test 1, PMTT Reserved
 pmtt            field must be zero, got 0x0600 instead
 pmtt                Proximity Domain:               0x00001000
 pmtt                Proximity Domain:               0x00002000
@@ -47,8 +47,8 @@  pmtt                Reserved:                       0x07
 pmtt                Length:                         0x0014
 pmtt                Flags:                          0x0002
 pmtt                Reserved:                       0x0000
-pmtt            FAILED [LOW] PMTTReservedNonZero: Test 1, PMTT reserved
-pmtt            field must be zero, got 0x07 instead
+pmtt            FAILED [MEDIUM] PMTTReservedNonZero: Test 1, PMTT
+pmtt            Reserved1 field must be zero, got 0x07 instead
 pmtt                Physical Component Identifier:  0x0000
 pmtt                Reserved:                       0x0000
 pmtt                Size of DIMM:                   0xa0000000
diff --git a/fwts-test/pptt-0001/pptt-0002.log b/fwts-test/pptt-0001/pptt-0002.log
index 57c3391..4b3f2a6 100644
--- a/fwts-test/pptt-0001/pptt-0002.log
+++ b/fwts-test/pptt-0001/pptt-0002.log
@@ -11,7 +11,7 @@  pptt                Parent:                         0x00000000
 pptt                ACPI Processor ID:              0x00000000
 pptt                Number of Private Resources:    0x00000001
 pptt                Private Resources[0]:           0x00000000
-pptt            FAILED [LOW] PPTTReservedNonZero: Test 1, PPTT reserved
+pptt            FAILED [MEDIUM] PPTTReservedNonZero: Test 1, PPTT Reserved
 pptt            field must be zero, got 0x0001 instead
 pptt            
 pptt              Cache Type Structure (Type 1):
@@ -25,7 +25,7 @@  pptt                Number of sets:                 0x00000000
 pptt                Associativity:                  0x00
 pptt                Attributes:                     0x00
 pptt                Line size:                      0x0000
-pptt            FAILED [LOW] PPTTReservedNonZero: Test 1, PPTT reserved
+pptt            FAILED [MEDIUM] PPTTReservedNonZero: Test 1, PPTT Reserved
 pptt            field must be zero, got 0x0002 instead
 pptt            
 pptt              Processor hierarchy node structure (Type 0):
@@ -38,7 +38,7 @@  pptt                ACPI Processor ID:              0x00000000
 pptt                Number of Private Resources:    0x00000002
 pptt                Private Resources[0]:           0x00000000
 pptt                Private Resources[1]:           0x00000000
-pptt            FAILED [LOW] PPTTReservedNonZero: Test 1, PPTT reserved
+pptt            FAILED [MEDIUM] PPTTReservedNonZero: Test 1, PPTT Reserved
 pptt            field must be zero, got 0x0003 instead
 pptt            FAILED [HIGH] PPTTBadFlags: Test 1, PPTT Flags's
 pptt            Bits[31..2] must be zero, got 0x00000007 instead
@@ -54,7 +54,7 @@  pptt                Number of sets:                 0x00000000
 pptt                Associativity:                  0xc0
 pptt                Attributes:                     0x00
 pptt                Line size:                      0x0000
-pptt            FAILED [LOW] PPTTReservedNonZero: Test 1, PPTT reserved
+pptt            FAILED [MEDIUM] PPTTReservedNonZero: Test 1, PPTT Reserved
 pptt            field must be zero, got 0x0004 instead
 pptt            FAILED [HIGH] PPTTBadFlags: Test 1, PPTT Flags's
 pptt            Bits[31..7] must be zero, got 0x000000ff instead
@@ -69,7 +69,7 @@  pptt                LEVEL_2_ID:                     0x0000000000000000
 pptt                MAJOR_REV:                      0x0000
 pptt                MINOR_REV:                      0x0000
 pptt                SPIN_REV:                       0x0000
-pptt            FAILED [LOW] PPTTReservedNonZero: Test 1, PPTT reserved
+pptt            FAILED [MEDIUM] PPTTReservedNonZero: Test 1, PPTT Reserved
 pptt            field must be zero, got 0x0005 instead
 pptt            
 pptt            
diff --git a/fwts-test/spcr-0001/spcr-0002.log b/fwts-test/spcr-0001/spcr-0002.log
index 437a4f9..e22fc40 100644
--- a/fwts-test/spcr-0001/spcr-0002.log
+++ b/fwts-test/spcr-0001/spcr-0002.log
@@ -5,8 +5,8 @@  spcr            test.
 spcr            Serial Interface: Reserved (Do not Use)
 spcr            FAILED [HIGH] SPCRInterfaceReserved: Test 1, SPCR Serial
 spcr            interface type 0x08 is a reserved interface
-spcr            FAILED [LOW] SPCRReservedNonZero: Test 1, SPCR reserved
-spcr            field must be zero, got 0xf02000 instead
+spcr            FAILED [MEDIUM] SPCRReservedNonZero: Test 1, SPCR
+spcr            Reserved1 field must be zero, got 0x000020f0 instead
 spcr            Baud Rate:        Reserved
 spcr            FAILED [HIGH] SPCRBaudRateReserved: Test 1, SPCR Serial
 spcr            baud rate type 0x08 is a reserved baud rate
diff --git a/fwts-test/tpm2-0001/tpm2-0002.log b/fwts-test/tpm2-0001/tpm2-0002.log
index 8e8bad7..c93c153 100644
--- a/fwts-test/tpm2-0001/tpm2-0002.log
+++ b/fwts-test/tpm2-0001/tpm2-0002.log
@@ -9,7 +9,7 @@  tpm2              Start Method:                    0x0000000c
 tpm2            FAILED [HIGH] TPM2BadPlatformClass: Test 1, TPM2's
 tpm2            platform class must be zero (client) or one (server), got
 tpm2            0x4
-tpm2            FAILED [LOW] TPM2ReservedNonZero: Test 1, TPM2 reserved
+tpm2            FAILED [MEDIUM] TPM2ReservedNonZero: Test 1, TPM2 Reserved
 tpm2            field must be zero, got 0x0003 instead
 tpm2            FAILED [HIGH] TPM2BadStartMethod: Test 1, TPM2's Start
 tpm2            Method must be between one to eleven, got 0xc