Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.debian.bugs.dist > #1253169

Bug#1109344: /boot/vmlinuz-6.12.35+deb13-amd64: uevent field power_supply/BAT1/uevent:POWER_SUPPLY_CURRENT_NOW now -ve

Cross-posted to 2 groups.

Show all headers | View raw

Hi Bastian, hi Ian,

On Thu, Jul 17, 2025 at 10:35:39PM +0200, Bastian Blank wrote:
> Control: tags -1 wontfix
> Control: close -1
> 
> On Tue, Jul 15, 2025 at 02:36:57PM +0100, Ian Jackson wrote:
> > Note that the previous value was positive, and the new value is
> > negative.  In both cases the laptop was not charging.
> > The negative value makes semantic sense, but I wonder if this change
> > was deliberate.  It seems like precisely the kind of bug-for-bug
> > compatibility thing that Linux normally tries to retain.
> 
> The documentation of Linux specifies the allowed values as:
> 
> | Valid values: Represented in microamps. Negative values are used for
> | discharging batteries, positive values for charging batteries and for
> | USB IBUS current.
> 
> Implementations for other power supplies, current macs for example,
> implement this correctly with negative values for discharging.  So the
> kernel already returns positive or negative numbers depending on the
> exact hardware used.
> 
> After careful consideration, I'm closing this as not a bug.

While preparing the 6.12.39 import (for a future trixie upload) I
noticed that there was a revert of that commit now in 6.12.39, with
the following explanation:

    Revert commit 234f71555019 ("ACPI: battery: negate current when
    discharging") breaks not one but several userspace implementations
    of battery monitoring: Steam and MangoHud. Perhaps it breaks more,
    but those are the two that have been tested.

So apparently more were affected :-/

Regards,
Salvatore

Back to linux.debian.bugs.dist | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

Bug#1109344: /boot/vmlinuz-6.12.35+deb13-amd64: uevent field power_supply/BAT1/uevent:POWER_SUPPLY_CURRENT_NOW now -ve Ian Jackson <ijackson@chiark.greenend.org.uk> - 2025-07-15 15:40 +0200
  Bug#1109344: /boot/vmlinuz-6.12.35+deb13-amd64: uevent field power_supply/BAT1/uevent:POWER_SUPPLY_CURRENT_NOW now -ve Salvatore Bonaccorso <carnil@debian.org> - 2025-07-16 09:30 +0200
    Bug#1109344: /boot/vmlinuz-6.12.35+deb13-amd64: uevent field power_supply/BAT1/uevent:POWER_SUPPLY_CURRENT_NOW now -ve Ian Jackson <ijackson@chiark.greenend.org.uk> - 2025-07-16 17:20 +0200
      Bug#1109344: /boot/vmlinuz-6.12.35+deb13-amd64: uevent field power_supply/BAT1/uevent:POWER_SUPPLY_CURRENT_NOW now -ve Salvatore Bonaccorso <carnil@debian.org> - 2025-07-17 14:40 +0200
        Bug#1109344: /boot/vmlinuz-6.12.35+deb13-amd64: uevent field power_supply/BAT1/uevent:POWER_SUPPLY_CURRENT_NOW now -ve Ian Jackson <ijackson@chiark.greenend.org.uk> - 2025-07-17 15:00 +0200
    Bug#1109344: /boot/vmlinuz-6.12.35+deb13-amd64: uevent field power_supply/BAT1/uevent:POWER_SUPPLY_CURRENT_NOW now -ve Bastian Blank <waldi@debian.org> - 2025-07-17 21:50 +0200
  Bug#1109344: /boot/vmlinuz-6.12.35+deb13-amd64: uevent field power_supply/BAT1/uevent:POWER_SUPPLY_CURRENT_NOW now -ve Bastian Blank <waldi@debian.org> - 2025-07-17 22:40 +0200
    Bug#1109344: /boot/vmlinuz-6.12.35+deb13-amd64: uevent field power_supply/BAT1/uevent:POWER_SUPPLY_CURRENT_NOW now -ve Salvatore Bonaccorso <carnil@debian.org> - 2025-07-18 15:30 +0200

csiph-web