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

Colin Ian King colin.king at canonical.com
Fri Sep 8 07:51:48 UTC 2017


On 07/09/17 01:34, Alex Hung wrote:
> Signed-off-by: Alex Hung <alex.hung at 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 at canonical.com>




More information about the fwts-devel mailing list