fwts: aspm: add some more advice on warning messages

Bug #1055430 reported by Colin Ian King
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
fwts (Ubuntu)
Fix Released
Medium
Alex Hung

Bug Description

I was running the fwts aspm on some new kit and I got the following results:

PCIe ASPM check.
----------------------------------------------------------------------------------------------------
Test 1 of 2: PCIe ASPM ACPI test.
PCIE ASPM is not controlled by Linux kernel.

ADVICE: BIOS reports that Linux kernel should not modify ASPM settings that BIOS configured. It can
be intentional because hardware vendors identified some capability bugs between the motherboard and
the add-on cards.

Test 2 of 2: PCIe ASPM registers test.
WARNING: Test 2, RP 00h:1Ch.01h L0s not enabled.
WARNING: Test 2, Device 02h:00h.00h L0s not enabled.
PASSED: Test 2, PCIE aspm setting matched was matched.

Is it possible for the warning messages to have a little more explanation using fwts_advice() as I'm not sure what the warning message really means w/o looking it up in the documentation.

Thanks!

Changed in fwts (Ubuntu):
importance: Undecided → Medium
assignee: nobody → Alex Hung (alexhung)
Alex Hung (alexhung)
Changed in fwts (Ubuntu):
status: New → In Progress
Revision history for this message
Alex Hung (alexhung) wrote :

Good point. Some information will help users to easily understand what the warrnings and their impacts are.

Changed in fwts (Ubuntu):
status: In Progress → Fix Committed
status: Fix Committed → In Progress
Alex Hung (alexhung)
Changed in fwts (Ubuntu):
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (6.3 KiB)

This bug was fixed in the package fwts - 0.26.01-1

---------------
fwts (0.26.01-1) raring; urgency=low

  [Alex Hung]
  * pcie: aspm: add more information for L0s and L1 warnings (LP: #1055430)
  * hotkey: remove redundant framework parameter from hotkey_find_keyboard
  * bios: hdaaudio: fix user pin configurations (user_pin_configs) is not checked.

  [Colin Ian King]
  * lib: acpi: add ACPI 5.0 GIC and GICD structs to MADT
  * acpi: acpidump: Dump out the new MADT GIC and GICD structures
  * acpi: acpitables: Add checks for MADT GIC and GICD
  * acpi: acpitables: fix typo in LAPICX2 APIC NMI Flags tag
  * lib: acpi: Add support for the new ACPI 5.0 PCCT
  * acpi: acpidump: Add support for ACPI 5.0 PCCT
  * acpi: apcidump: Add ACPI 5.0 support for boot flag CMOS RTC present bit
  * acpi: method: re-order tests based on location in specification
  * acpi: method: re-order method test functions
  * acpi: method: tidy source; 80 columns wide formatting
  * acpi: method: add some more comments
  * acpi: method: clean up spacing in for loops
  * acpi: method: add in all ACPI control method and objects
  * acpi: method: add DDN, HID, HRV, PLD, SUB, STR checks
  * acpi: method: add checks for EDL, RMV, STA
  * acpi: method: add BCT test
  * acpi: method: add checks for GCP, GRT, GWS
  * acpi: method: Add check for GSB
  * acpi: method: Add check for SEG
  * acpi: method: Add PRx, SxD, SxW checks
  * acpi: method: fix typo SWP --> SWS
  * acpi: method: Add _S0_ .. _S5_, _SWS checks
  * acpi: method: Add CPC, CSD, PCT, PDL, PPC, TDL, TSD and TSS checks
  * bios: smbios: fix typo Stucture ==> Structure
  * cpu: virt fix typo extentions ==> extensions
  * cpu: virt: fix typo originall ==> originally
  * lib: fwts_formatting: fix typo upto ==> up to
  * lib: fwts_log_html: fix typo convertion ==> conversion
  * lib: fwts_checkeuid: fix typo priviledges ==> privileges
  * example: blank test: fix typos ommitted ==> omitted
  * acpi: wakealarm: fix typo existance ==> existence
  * acpi: acpitables: fix typo managment ==> management
  * acpi: acpidump: print format using inttypes
  * acpi: acpitables: print format using inttypes
  * acpi: checksum: print format using inttypes
  * acpi: fadt: print format using inttypes
  * acpi: method: print format using inttypes
  * acpi: s3power: print format using inttypes
  * bios: bios32: print format using inttypes
  * bios: ebda_region: print format using inttypes
  * bios: ebdadump: print format using inttypes
  * bios: mtrr: print format using inttypes
  * bios: mpcheck: print format using inttypes
  * cpu: msr: print format using inttypes
  * dmi: dmi_decode: print format using inttypes
  * hpet: hpet_check: print format using inttypes
  * lib: fwts_memorymap: print format using inttypes
  * uefi: csm: print format using inttypes
  * uefi: uefidump: print format using inttypes and whitespace tidy
  * lib: fwts_acpid: fix checking of return from read()
  * lib: fwts_acpi_method: fix for loop index type
  * lib: fwts_dump: fix loop variable type to match table length type
  * lib: fwts_framework: Make string length type a size_t rather than an int
  * lib: fwts_framework: remove redundant parameter from fwts_fra...

Read more...

Changed in fwts (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.