Battery status unavailable after suspend / resume on some HP laptops

Bug #453963 reported by Gadi Cohen
58
This bug affects 10 people
Affects Status Importance Assigned to Milestone
linux (Ubuntu)
Medium
Unassigned
Declined for Jaunty by Stefan Bader
Karmic
Medium
Unassigned

Bug Description

SRU Justification:

Impact: On some HP laptops, battery status is not available after suspend.

Fix: Upstream fixes pulled in via 2.6.31.12 stable update

---

This is actually already worked around (for some models) in linux-2.6.31-git14, but a backport for the current Ubuntu kernels would be great.

http://bugzilla.kernel.org/show_bug.cgi?id=13745
http://patchwork.kernel.org/patch/35062/

The above patch is specifically for the "Hewlett-Packard HP G7000 Notebook PC". In my case, the DMI_PRODUCT_NAME is "HP Pavilion dv4 Notebook PC".

As per https://bugzilla.redhat.com/show_bug.cgi?id=490684, the follow models are also affected:
* HP Pavilion dv7-1174ca
* HP Pavillion-dv7t 1000

I'd imagine it affects a number of other notebooks in HP Pavilions range.

ProblemType: Bug
Architecture: amd64
Dependencies:

DistroRelease: Ubuntu 9.04
NonfreeKernelModules: nvidia
Package: linux-image None [modified: /var/lib/dpkg/info/linux-image.list]
ProcEnviron:
 PATH=(custom, user)
 LANG=en_ZA.UTF-8
 SHELL=/bin/bash
SourcePackage: linux-meta
Uname: Linux 2.6.28-15-generic x86_64
UnreportableReason: This is not a genuine Ubuntu package

Gadi Cohen (gadicc)
Changed in linux-meta (Ubuntu):
assignee: nobody → Ubuntu Kernel ACPI Team (ubuntu-kernel-acpi)
Revision history for this message
Leann Ogasawara (leannogasawara) wrote :

[This is an automated message. Apologies if it has reached you inappropriately.]

This bug was reported against the linux-meta package when it likely should have been reported against the linux package instead. We are automatically transitioning this to the linux kernel package so that the appropriate teams are notified and made aware of this issue.

If this bug really is a bug in the linux-meta package you can move it back to linux-meta and set the Status to Confirmed, or contact us on the #ubuntu-kernel channel on the FreeNode IRC server. Thanks.

affects: linux-meta (Ubuntu) → linux (Ubuntu)
Revision history for this message
Doug Brunner (madengineer10) wrote :

This bug still exists with Kubuntu 9.10 and kernel 2.6.31-14.48. As the original poster says, a backport of the patch would be great; could it be included in the 2.6.31-15 kernel build?

Revision history for this message
Doug Brunner (madengineer10) wrote :

Additional: the bug is occurring for me on the same laptop (HP Pavilion dv4 Notebook PC).

Revision history for this message
juancherru (juancherru) wrote :

I can confirm this is happening to me on a Compaq Cq40-505 (Notebook PC).

Revision history for this message
juancherru (juancherru) wrote :

Aditionally, my distro is Ubuntu 9.10

Revision history for this message
dannymac64 (danny-mcpherson) wrote :

This affects my HP Pavilion dv3-2155mx as well.

Revision history for this message
daniel1904 (dan-orrego) wrote :

this bug affects to me too dv4-1428.
Distro: Ubuntu 9.10.

Revision history for this message
Gadi Cohen (gadicc) wrote :

Guys to expediate some action on this issue please provide the exact output of:

    "sudo dmidecode -s system-product-name"

It should look something like this:

$ sudo dmidecode -s system-product-name
HP Pavilion dv4 Notebook PC

If your laptop isn't branded as an HP (I saw a Compaq user above), please also give the output of "sudo dmidecode -s system-manufacturer".

Then I'll comit to making a patch.

Also, FYI (last entry http://bugzilla.kernel.org/show_bug.cgi?id=13449)
* drivers/acpi/sleep.c DMI for dv4 and CQ40 shipped in Linux 2.6.32-rc7

Revision history for this message
Gadi Cohen (gadicc) wrote :

Ok nevermind guys; all the models mentioned above have already been handled for the latest kernels.

So, attached is a patch against the latest karmic kernel, Ubuntu-2.6.31-16.51 (from git).
Note: this kernel already implements the patch for the HP G7000.
The patch adds the fix for the HP dv3, dv4 dv7 and Compaq CQ40.

Revision history for this message
Gadi Cohen (gadicc) wrote :

Ok, I realize we already got a new kernel from the updates, and I haven't checked the security implications of remaining on the current kernel, but anyway, for those interested, here are the .deb files of the last kernel with the above patch applied.

http://turtle.wastelands.net/dragon/stuff/

Note: you should really really never run kernels from strangers :)

Rather, here is info on how to build your own kernel:
https://help.ubuntu.com/community/Kernel/Compile

Btw - any word from the Kernel ACPI Team? The bug's been up for almost 2 months and we haven't had any official word yet.

Thanks.

Revision history for this message
Iain Buclaw (iainb) wrote :

Quick question on this bug, if you don't mind.

Regarding this part of the patch:

 .callback = init_set_sci_en_on_resume,
 .ident = "Hewlett-Packard Pavilion dv4",
 .matches = {
  DMI_MATCH(DMI_SYS_VENDOR, "Hewlett-Packard"),
  DMI_MATCH(DMI_PRODUCT_NAME, "HP Pavilion dv4"),
  },
 },

If dmidecode -s system-product-name returned: "HP Pavillion dv4 Notebook PC"
Would the above code catch that? Or will dv4 Laptops with that product name - instead of "HP Pavilion dv4" still be affected?

Regards
Iain

Revision history for this message
Iain Buclaw (iainb) wrote :

Sorry - seemed to have answered my own question.

Had a look at the function dmi_matches() and it uses strstr() to match strings - which is all OK.

Thanks anyway.
Regards
Iain

Revision history for this message
Gadi Cohen (gadicc) wrote : Re: [Bug 453963] Re: Battery status unavailable after suspend / resume on some HP laptops

Sure, np...

I never even noticed that discrepancy before, but yeah it does still
match... my system-product-name is also "HP Pavilion dv4 Notebook PC"
and the patch works great.

Feel free to 'confirm' that the bug affects you too on the bug report
:) And if the patch helps.

Thanks

Gadi

Iain Bucław wrote:

> Quick question on this bug, if you don't mind.
>
> Regarding this part of the patch:
>
> .callback = init_set_sci_en_on_resume,
> .ident = "Hewlett-Packard Pavilion dv4",
> .matches = {
> DMI_MATCH(DMI_SYS_VENDOR, "Hewlett-Packard"),
> DMI_MATCH(DMI_PRODUCT_NAME, "HP Pavilion dv4"),
> },
> },
>
>
> If dmidecode -s system-product-name returned: "HP Pavillion dv4 Notebook PC"
> Would the above code catch that? Or will dv4 Laptops with that product name - instead of "HP Pavilion dv4" still be affected?
>
> Regards
> Iain
>
>

--
Gadi Cohen aka Kinslayer <email address hidden> www.wastelands.net
Freelance admin/coding/design HABONIM DROR linux/fantasy enthusiast
KeyID 0x93F26EF5: 256A 1FC7 AA2B 6A8F 1D9B 6A5A 4403 F34B 93F2 6EF5

Revision history for this message
Gadi Cohen (gadicc) wrote :

More updates from me:

* Added support for HP-Compaq C700, which is also now in kernel 2.6.32.
* Patch + debs for kernel 2.6.31-17 at http://turtle.wastelands.net/dragon/stuff/

I also submitted this latest patch directly to the kernel-team mailing list, and hope it will be in the next release, or at least that we'll get an official response from someone.

Revision history for this message
Gadi Cohen (gadicc) wrote :

The changes were accepted and should be included in 2.6.31.12 (still unreleased).
From there it will take some time until it eventually makes it into 2.6.31-something for us in Ubuntu.
But eventually it will come down in the regular system updates. Yay!

Here's the thread from the kernelteam mailing list, for those interested:
https://lists.ubuntu.com/archives/kernel-team/2010-January/008294.html

Surbhi Palande (csurbhi)
Changed in linux (Ubuntu):
importance: Undecided → Medium
status: New → Triaged
Revision history for this message
Linus C Unneback (linusu) wrote :

I can confirm that the debs for kernel 2.6.31-17 at http://turtle.wastelands.net/dragon/stuff/ works on my HP pavillon dv7-1265eo.

$ sudo dmidecode -s system-product-name
HP Pavilion dv7 Notebook PC

Revision history for this message
rockfx01 (tim-filmchicago) wrote :

Patched kernel works for me too on a dv7 laptop.

Revision history for this message
Stefan Bader (smb) wrote :

For Lucid these fixes fixes seem to be in since 2.6.32 was released.

Changed in linux (Ubuntu):
assignee: Ubuntu Kernel ACPI Team (ubuntu-kernel-acpi) → nobody
status: Triaged → Fix Released
Changed in linux (Ubuntu Karmic):
importance: Undecided → Medium
status: New → Fix Committed
description: updated
Revision history for this message
Gadi Cohen (gadicc) wrote :

Stefan, thanks again for all your help with this. Looking forward to Lucid.

Revision history for this message
Martin Pitt (pitti) wrote : Please test proposed package

Accepted linux into karmic-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

tags: added: verification-needed
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (5.1 KiB)

This bug was fixed in the package linux - 2.6.31-21.59

---------------
linux (2.6.31-21.59) karmic-proposed; urgency=low

  [ Andy Whitcroft ]

  * [Config] generic-pae switch to M586TSC
    - LP: #519448

  [ Chris Wilson ]

  * (pre-stable) drm/i915: Increase fb alignment to 64k
    - LP: #404064

  [ Colin Ian King ]

  * Input: i8042 - bypass AUX IRQ delivery test on laptops
    - LP: #534448

  [ Jerone Young ]

  * SAUCE: Fix volume hotkeys for Dell Studio 1557
    - LP: #465250

  [ Mirsal Ennaime ]

  * SAUCE: aufs: Fix header files inclusion in debug.h
    - LP: #517151

  [ Stefan Bader ]

  * [Config] Enable all CGROUP configuration options
    - LP: #480739

  [ Surbhi Palande ]

  * Revert "[Upstream] acerhdf: Limit modalias matching to supported
    boards"
    - LP: #509730
  * [Config] ext3 defaults to ordered mode
    - LP: #510067

  [ Tim Gardner ]

  * [Config] Fix sub-flavours package conflicts
    - LP: #454827

  [ Upstream Kernel Changes ]

  * PCI/cardbus: Add a fixup hook and fix powerpc
    - LP: #455723
  * fnctl: f_modown should call write_lock_irqsave/restore
    - LP: #519436
  * ACPI: enable C2 and Turbo-mode on Nehalem notebooks on A/C
    - LP: #516325
  * tg3: Add 57788, remove 57720
    - LP: #515390
  * HID: ignore all recent SoundGraph iMON devices
    - LP: #488443
  * Input: ALPS - add interleaved protocol support (Dell E6x00 series)
    - LP: #296610
  * acerhdf: limit modalias matching to supported
    - LP: #509730
  * ASoC: Do not write to invalid registers on the wm9712.
    - LP: #509730
  * cifs: NULL out tcon, pSesInfo, and srvTcp pointers when chasing DFS
    referrals
    - LP: #509730
  * clockevents: Prevent clockevent_devices list corruption on cpu hotplug
    - LP: #509730
  * dma: at_hdmac: correct incompatible type for argument 1 of
    'spin_lock_bh'
    - LP: #509730
  * drivers/net/usb: Correct code taking the size of a pointer
    - LP: #509730
  * Libertas: fix buffer overflow in lbs_get_essid()
    - LP: #509730
  * md: Fix unfortunate interaction with evms
    - LP: #509730
  * pata_cmd64x: fix overclocking of UDMA0-2 modes
    - LP: #509730
  * pata_hpt3x2n: fix clock turnaround
    - LP: #509730
  * SCSI: fc class: fix fc_transport_init error handling
    - LP: #509730
  * sound: sgio2audio/pdaudiocf/usb-audio: initialize PCM buffer
    - LP: #509730
  * USB: emi62: fix crash when trying to load EMI 6|2 firmware
    - LP: #509730
  * USB: Fix a bug on appledisplay.c regarding signedness
    - LP: #509730
  * USB: musb: gadget_ep0: avoid SetupEnd interrupt
    - LP: #509730
  * USB: option: support hi speed for modem Haier CE100
    - LP: #490068, #509730
  * x86, cpuid: Add "volatile" to asm in native_cpuid()
    - LP: #509730
  * e100: Use pci pool to work around GFP_ATOMIC order 5 memory allocation
    failure
    - LP: #509730
  * e100: Fix broken cbs accounting due to missing memset.
    - LP: #509730
  * hostap: Revert a toxic part of the conversion to net_device_ops
    - LP: #509730
  * hwmon: (fschmd) Fix check on unsigned in watchdog_write()
    - LP: #509730
  * hwmon: (sht15) Off-by-one error in array index + incorrect constants
    - LP: #509730
  * i2c/tsl2550: Fix...

Read more...

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

Other bug subscribers

Remote bug watches

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