Battery status unavailable after suspend / resume on some HP laptops

[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.

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?

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

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

Aditionally, my distro is Ubuntu 9.10

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

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

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

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.

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.

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

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

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

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.

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

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

Patched kernel works for me too on a dv7 laptop.

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

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

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!

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...