1. 29 5月, 2014 1 次提交
  2. 07 5月, 2014 4 次提交
    • L
      ACPICA: Tables: Add new mechanism to skip NULL entries in RSDT and XSDT. · 0f929fbf
      Lv Zheng 提交于
      It is reported that there are buggy BIOSes in the world: AMI uses an XSDT
      compiler for early BIOSes, this compiler will generate XSDT with a NULL
      entry.  The affected BIOS versions are "AMI BIOS F2-F4".
      
      Original solution on Linux is to use an alternative heathy root table
      instead of the ill one.  This commit is:
        Commit: 671cc68d
        Subject: ACPICA: Back port and refine validation of the XSDT root table.
      
      This is an example of such XSDT dumped from B85-HD3 (AMI F3 BIOS):
      [000h 0000   4]                    Signature : "XSDT"    [Extended System Description Table]
      [004h 0004   4]                 Table Length : 00000074
      [008h 0008   1]                     Revision : 01
      [009h 0009   1]                     Checksum : 18
      [00Ah 0010   6]                       Oem ID : "ALASKA"
      [010h 0016   8]                 Oem Table ID : "A M I"
      [018h 0024   4]                 Oem Revision : 01072009
      [01Ch 0028   4]              Asl Compiler ID : "AMI "
      [020h 0032   4]        Asl Compiler Revision : 00010013
      
      [024h 0036   8]       ACPI Table Address   0 : 00000000BA5F8180
      [02Ch 0044   8]       ACPI Table Address   1 : 00000000BA5F8290
      [034h 0052   8]       ACPI Table Address   2 : 00000000BA5F8308
      [03Ch 0060   8]       ACPI Table Address   3 : 00000000BA5F8848
      [044h 0068   8]       ACPI Table Address   4 : 00000000BA5F9320
      [04Ch 0076   8]       ACPI Table Address   5 : 00000000BA5F9360
      [054h 0084   8]       ACPI Table Address   6 : 00000000BA5F9398
      [05Ch 0092   8]       ACPI Table Address   7 : 00000000BA5F9708
      [064h d100   8]       ACPI Table Address   8 : 00000000BA5FC9A8
      [06Ch 0108   8]       ACPI Table Address   9 : 0000000000000000
      
      But according to the bug report, the XSDT in fact is not broken. In the
      above XSDT, ACPI Table Address 1-8 contains the same value as RSDT.  The
      differences can only be seen on the following 2 entries:
      1. The first entry points to a FADT whose Revision is 5 while the first
         entry in RSDT points to a FADT whose Revision is 2.
         The FADT dumped from the address indicated by the first entry of XSDT:
          FACP @ 0x00000000BA5F8180
            0000: 46 41 43 50 0C 01 00 00<05>4B 41 4C 41 53 4B 41  FACP.....KALASKA
            ...
         The FADT dumped from the address indicated by the first entry of RSDT:
          FACP @ 0x00000000BA5ED0F0
            0000: 46 41 43 50 84 00 00 00<02>A7 41 4C 41 53 4B 41  FACP......ALASKA
            ...
      2. The last entry is a NULL terminator.
      According to the test result, the Revision 5 FADT is accessible.  Thus the
      original solution turns out to be a work around that is preventing the
      higher revision tables to be used for such platforms (they are all x86-64
      platforms, and should use XSDT and higher revision FADT).
      
      This patch offers a new solution, where a sanity check is performed before
      installing a table address from XSDT. If the entry is NULL, it is simply
      discarded.
      
      Note that, this patch doesn't remove the original solution, so for Linux
      kernel, this commit is actually a no-op, but it allows acpidump to be
      working on such platforms. By doing so, we allow another easy revertable
      commit to enable this feature so that when that commit is reverted, the
      useful sanity check will not be affected. Lv Zheng.
      
      References: https://bugzilla.kernel.org/show_bug.cgi?id=73911
      References: https://bugs.archlinux.org/task/39811Signed-off-by: NLv Zheng <lv.zheng@intel.com>
      Reported-and-tested-by: NBruce Chiarelli <mano155@gmail.com>
      Reported-and-tested-by: NSpyros Stathopoulos <spystath@gmail.com>
      Signed-off-by: NBob Moore <robert.moore@intel.com>
      Signed-off-by: NRafael J. Wysocki <rafael.j.wysocki@intel.com>
      0f929fbf
    • L
      ACPICA: acpidump: Add support to force using RSDT. · c7932267
      Lv Zheng 提交于
      This patch adds "-x" and "-x -x" options to disable XSDT for acpidump.
      
      The single "-x" can be used to stop using XSDT, RSDT will be forced to find
      static tables, note that XSDT will still be dumped. The double "-x" can
      stop dumping XSDT, which is useful when the XSDT address reported by RSDP
      is pointing to an invalid address.
      
      It is reported there are platforms having broken XSDT shipped, acpidump
      will stop working while accessing such XSDT. This patch adds new option so
      that users can force acpidump to dump tables listed in the RSDT. Lv Zheng.
      
      Buglink: https://bugzilla.kernel.org/show_bug.cgi?id=73911
      Buglink: https://bugs.archlinux.org/task/39811Signed-off-by: NLv Zheng <lv.zheng@intel.com>
      Reported-and-tested-by: NBruce Chiarelli <mano155@gmail.com>
      Reported-and-tested-by: NSpyros Stathopoulos <spystath@gmail.com>
      Signed-off-by: NBob Moore <robert.moore@intel.com>
      Signed-off-by: NRafael J. Wysocki <rafael.j.wysocki@intel.com>
      c7932267
    • L
      ACPICA: acpidump: Fix truncated RSDP signature validation. · d63f3790
      Lv Zheng 提交于
      This patch enforces a rule to always use ACPI_VALIDATE_RSDP_SIG for RSDP
      signatures passed from table header or ACPI_SIG_RSDP so that truncated
      string comparison can be avoided.  This could help to fix the issue that
      "RSD " matches but "RSD PTR " doesn't match.  Lv Zheng.
      Signed-off-by: NLv Zheng <lv.zheng@intel.com>
      Signed-off-by: NBob Moore <robert.moore@intel.com>
      Signed-off-by: NRafael J. Wysocki <rafael.j.wysocki@intel.com>
      d63f3790
    • L
      ACPICA: acpidump: Fix code issue in invoking fread in the loop. · d87a2b75
      Lv Zheng 提交于
      This patch fixes an issue that the while loop is not needed as fread()
      should return exact the bytes of expected.
      
      The patch is tested by runing diff against the output of "-c" mode and
      the normal mode, and only finds the following differences:
      1. table addresses: the "-c" mode will always fill 0x0000000000000000 for
                          the address.
      2. RSDP/RSDT/XSDT: there is no generation of such tables for "-c" mode.
      So the test result shows the fix is valid.  Lv Zheng.
      Signed-off-by: NLv Zheng <lv.zheng@intel.com>
      Signed-off-by: NBob Moore <robert.moore@intel.com>
      Signed-off-by: NRafael J. Wysocki <rafael.j.wysocki@intel.com>
      d87a2b75
  3. 28 4月, 2014 1 次提交
  4. 22 4月, 2014 1 次提交
  5. 21 4月, 2014 5 次提交
  6. 16 1月, 2014 1 次提交
    • L
      ACPICA: acpidump: Cleanup tools/power/acpi makefiles. · f677b30b
      Lv Zheng 提交于
      This patch cleans up old tools/power/acpi Makefile for further porting,
      make it compiled in a similar way as the other tools.  No functional
      changes.
      
      The CFLAGS is modified as follows:
      1. Previous cc flags:
         -Wall -Wstrict-prototypes -Wdeclaration-after-statement -Os -s \
         -D_LINUX -DDEFINE_ALTERNATE_TYPES -I../../../include
      2. Current cc flags:
         DEBUG=false:
         -D_LINUX -DDEFINE_ALTERNATE_TYPES -I../../../include -Wall \
         -Wstrict-prototypes -Wdeclaration-after-statement -Os \
         -fomit-frame-pointer
         Normal:
         -D_LINUX -DDEFINE_ALTERNATE_TYPES -I../../../include -Wall \
         -Wstrict-prototypes -Wdeclaration-after-statement -O1 -g -DDEBUG
      
      There is only one difference: -fomit-frame-pointer.
      Signed-off-by: NLv Zheng <lv.zheng@intel.com>
      Signed-off-by: NRafael J. Wysocki <rafael.j.wysocki@intel.com>
      f677b30b
  7. 06 2月, 2013 1 次提交
  8. 09 10月, 2012 1 次提交
  9. 25 9月, 2012 2 次提交
  10. 23 9月, 2012 6 次提交