← Back to team overview

kernel-packages team mailing list archive

[Bug 1175475] Re: [HP Pavilion Slimline S5] fwts_test reports Critical /High falures for hibernation/suspend test

 

More detail information,
This is caused by the wmi driver(hp-wmi.c) brings the input data 160bits
struct bios_args {
u32 signature;
u32 command;
u32 commandtype;
u32 datasize;
u32 data; --- 4 bytes data
};
into the the acpi method HWMC as Arg1,
    Method (HWMC, 2, NotSerialized)
    {
        CreateDWordField (Arg1, Zero, SGIN)
        CreateDWordField (Arg1, 0x04, COMD)
        CreateDWordField (Arg1, 0x08, CMDT)
        CreateDWordField (Arg1, 0x0C, DSZI)
        CreateByteField (Arg1, 0x10, D008)
        CreateByteField (Arg1, 0x11, D009)
        CreateByteField (Arg1, 0x12, D010)
        CreateField (Arg1, 0x80, 0x0400, D128) --- 128 bytes
        .....
     }

the error occurred because the mismatch between the 4bytes "data" provided, but the firmware 128 bytes "D128" used.
checking with the HP wmi spec, several wmi command type used input date 128 bytes (such as type 36h, Set Peak Shift Times, type 37h Set Battery Charging Times). But this is not used by the hp-wmi.c. It is not reasonable to CreateField for 128bytes at the method beginning, in stead, it should be createfielded while it is used.
So I suggest the firmware should move the
        CreateField (Arg1, 0x80, 0x0400, D128)
into the place that D128 is used, such as the type 36h, 37h.
                        If (LEqual (CMDT, 0x36))
                        {
move to here ---- CreateField (Arg1, 0x80, 0x0400, D128)
                            Store (\_SB.WMID.SPST (D128), Local2)
                            Store (Zero, RETC)
                        }

                        If (LEqual (CMDT, 0x37))
                        {
move to here --- CreateField (Arg1, 0x80, 0x0400, D128)
                            Store (\_SB.WMID.SBCT (D128), Local2)
                            Store (Zero, RETC)
                        }

but actually the D128 isn't really used by (type 36h, type 37h) and other type.
                If (LEqual (CMDT, 0x36))
                {
                    Store (\_SB.WMID.GPST (), Local2)
                    Store (Zero, RETC)
                }

                If (LEqual (CMDT, 0x37))
                {
                    Store (\_SB.WMID.GBCT (), Local2)
                    Store (Zero, RETC)
                }

So directly remove 
        CreateField (Arg1, 0x80, 0x0400, D128) in the firmware is suggested.

-- 
You received this bug notification because you are a member of Kernel
Packages, which is subscribed to linux in Ubuntu.
https://bugs.launchpad.net/bugs/1175475

Title:
  [HP Pavilion Slimline S5] fwts_test reports Critical /High falures for
  hibernation/suspend test

Status in “linux” package in Ubuntu:
  Confirmed

Bug description:
  CID: 201205-11052 (HP Pavilion Slimline S5)

  Ubuntu 12.04.2 LTS

  suspend_single.log, suspend_30_cycles.log, hibernate-single.log and
  hibernate_30_cycles.log are attached.

  ProblemType: Bug
  DistroRelease: Ubuntu 12.04
  Package: linux-image-3.5.0-23-generic 3.5.0-23.35~precise1
  ProcVersionSignature: Ubuntu 3.5.0-23.35~precise1-generic 3.5.7.2
  Uname: Linux 3.5.0-23-generic x86_64
  AlsaVersion: Advanced Linux Sound Architecture Driver Version 1.0.25.
  AplayDevices:
   **** List of PLAYBACK Hardware Devices ****
   card 0: PCH [HDA Intel PCH], device 0: STAC92xx Analog [STAC92xx Analog]
     Subdevices: 1/1
     Subdevice #0: subdevice #0
  ApportVersion: 2.0.1-0ubuntu17.1
  Architecture: amd64
  ArecordDevices:
   **** List of CAPTURE Hardware Devices ****
   card 0: PCH [HDA Intel PCH], device 0: STAC92xx Analog [STAC92xx Analog]
     Subdevices: 2/2
     Subdevice #0: subdevice #0
     Subdevice #1: subdevice #1
  AudioDevicesInUse:
   USER        PID ACCESS COMMAND
   /dev/snd/controlC0:  ubuntu     1547 F.... pulseaudio
  CRDA: Error: command ['iw', 'reg', 'get'] failed with exit code 1: nl80211 not found.
  Card0.Amixer.info:
   Card hw:0 'PCH'/'HDA Intel PCH at 0xf7d00000 irq 50'
     Mixer name	: 'IDT 92HD73E1X5'
     Components	: 'HDA:111d7676,103c2ada,00100103'
     Controls      : 39
     Simple ctrls  : 21
  Date: Thu May  2 02:39:35 2013
  HibernationDevice: RESUME=UUID=0059822b-2011-4dbc-aea6-3e1aff45462e
  InstallationMedia: Ubuntu 12.04.2 LTS "Precise Pangolin" - Release amd64 (20130213)
  IwConfig:
   eth0      no wireless extensions.
   
   lo        no wireless extensions.
  MarkForUpload: True
  ProcEnviron:
   TERM=xterm
   PATH=(custom, no user)
   LANG=en_US.UTF-8
   SHELL=/bin/bash
  ProcFB: 0 inteldrmfb
  ProcKernelCmdLine: BOOT_IMAGE=/boot/vmlinuz-3.5.0-23-generic root=UUID=c6420b34-c860-4df2-99fd-9fa1a0b893b7 ro quiet splash initcall_debug vt.handoff=7
  RelatedPackageVersions:
   linux-restricted-modules-3.5.0-23-generic N/A
   linux-backports-modules-3.5.0-23-generic  N/A
   linux-firmware                            1.79.1
  RfKill:
   
  SourcePackage: linux
  UpgradeStatus: No upgrade log present (probably fresh install)
  dmi.bios.date: 03/29/2013
  dmi.bios.vendor: AMI
  dmi.bios.version: 00.05
  dmi.board.name: 2ADA
  dmi.board.vendor: Hewlett-Packard
  dmi.board.version: PVT
  dmi.chassis.type: 3
  dmi.chassis.vendor: Hewlett-Packard
  dmi.modalias: dmi:bvnAMI:bvr00.05:bd03/29/2013:svnHewlett-Packard:pn:pvr:rvnHewlett-Packard:rn2ADA:rvrPVT:cvnHewlett-Packard:ct3:cvr:
  dmi.sys.vendor: Hewlett-Packard

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1175475/+subscriptions