Control: tags -1 + moreinfo

Hi Ian,

On Tue, Jul 15, 2025 at 02:36:57PM +0100, Ian Jackson wrote:
> Package: src:linux
> Version: 6.12.35-1
> Severity: normal
> File: /boot/vmlinuz-6.12.35+deb13-amd64
> X-Debbugs-Cc: debian-am...@lists.debian.org
> User: debian-am...@lists.debian.org
> Usertags: amd64
> 
> Control: affects chiark-utils-bin 8.0.0
> 
> My battery monitor applet broke because:
> 
> $ uname -av
> Linux zealot 6.12.35+deb13-amd64 #1 SMP PREEMPT_DYNAMIC Debian 6.12.35-1 
> (2025-07-03) x86_64 GNU/Linux
> $ egrep . /sys/class/power_supply/*/uevent
> ...
> /sys/class/power_supply/BAT1/uevent:POWER_SUPPLY_STATUS=Discharging
> ...
> /sys/class/power_supply/BAT1/uevent:POWER_SUPPLY_CURRENT_NOW=-484000
> ...
> 
> Full event output here:
> https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1109342#15
> 
> 
> Whereas previously:
> 
> $ uname -av
> Linux zealot 6.12.33+deb13-amd64 #1 SMP PREEMPT_DYNAMIC Debian 6.12.33-1 
> (2025-06-19) x86_64 GNU/Linux
> $ egrep . /sys/class/power_supply/*/uevent
> ...
> /sys/class/power_supply/BAT1/uevent:POWER_SUPPLY_STATUS=Discharging
> ...
> /sys/class/power_supply/BAT1/uevent:POWER_SUPPLY_CURRENT_NOW=504000
> ...
> 
> Full event output here:
> https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1109342#10
> 
> 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.
> 
> In the meantime I will adjust my code to take abs(), since STATUS can
> be used to find out the sense of current flow.
> 
> Thanks for your attention.

I suspect this is a deliberate change maybe e8cd4a8d5b88 ("ACPI:
battery: negate current when discharging"), can you try with that
commit reverted?

If that is sucessfull we would appreciate if you can take it to
upstream involved people to ask if the side effect you are seeing is
considered a regression bug or to be kept deliberately.

Can you please do that?

Regards,
Salvatore

Reply via email to