unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: "T.V Raman" <raman@google.com>
To: contovob@tcd.ie
Cc: raman@google.com, monnier@iro.umontreal.ca, emacs-devel@gnu.org
Subject: Re: battery.el broken?
Date: Fri, 11 Jun 2021 07:30:09 -0700	[thread overview]
Message-ID: <24771.29553.344941.739549@retriever.mtv.corp.google.com> (raw)
In-Reply-To: <87o8cchdxa.fsf@tcd.ie>

yes, looks like some upower dbus  bug; separatley, for the longest
time the dbus api has been reporting AC power connect/disconnect
*much* after the event.
It's pretty broken, and not Emacs' fault.

the upower commandline utility gives the right value by the way

Basil L. Contovounesios writes:
 > "T.V Raman" <raman@google.com> writes:
 > 
 > > I am mostly convinced this is a dbus/upower bug since I too checked and
 > > nothing has changed on the Emacs side  on the machine, still running
 > > Emacs from Github@HEAD.
 > >
 > > Will now try some of the other status functions, I think the output
 > > stopped telling me time left likely a couple of days ago.
 > >
 > > Typing this with the machine fully charged and not plugged into the AC
 > > jack, but the upower output shown below appears to think it's on AC
 > > Power.
 > >
 > > battery-status-function == battery-upower
 > >
 > > (mapcar #'battery--upower-device-properties
 > > (battery--upower-devices)
 > > )
 > >
 > > ((("NativePath" . "AC") ("Vendor" . "") ("Model" . "") ("Serial" . "")
 > > ("UpdateTime" . 1622943939) ("Type" . 1) ("PowerSupply" . t) ("HasHistory")
 > > ("HasStatistics") ("Online") ("Energy" . 0.0) ("EnergyEmpty" . 0.0) ...)
 > > (("NativePath" . "BAT0") ("Vendor" . "SMP") ("Model" . "01AV431") ("Serial"
 > > . "5195") ("UpdateTime" . 1622944181) ("Type" . 2) ("PowerSupply" . t)
 > > ("HasHistory" . t) ("HasStatistics" . t) ("Online") ("Energy" . 51.36)
 > > ("EnergyEmpty" . 0.0) ...) (("NativePath" . "ucsi-source-psy-USBC000:001")
 > > ("Vendor" . "") ("Model" . "") ("Serial" . "") ("UpdateTime" . 1622943939)
 > > ("Type" . 1) ("PowerSupply" . t) ("HasHistory") ("HasStatistics") ("Online" . t)
 > > ("Energy" . 0.0) ("EnergyEmpty" . 0.0) ...) (("NativePath"
 > > . "ucsi-source-psy-USBC000:002") ("Vendor" . "") ("Model" . "") ("Serial" . "")
 > > ("UpdateTime" . 1622943940) ("Type" . 1) ("PowerSupply" . t) ("HasHistory")
 > > ("HasStatistics") ("Online") ("Energy" . 0.0) ("EnergyEmpty" . 0.0) ...))
 > 
 > Hm, did you always have those ucsi-source-psy entries?  Searching for
 > them online led me to the following relatively recent discussions on
 > USB-C power sources, UPower, and Linux:
 > 
 > https://forum.manjaro.org/t/upowerd-not-recognising-usb-c-path-charger-is-usb-c
 > https://gitlab.freedesktop.org/upower/upower/-/issues/126
 > https://bugs.launchpad.net/ubuntu/+source/upower/+bug/1891799
 > 
 > In particular I notice that the first of the two ucsi-source-psy entries
 > says ("Online" . t), which seems contradictory and is probably what
 > throws battery-upower off.
 > 
 > -- 
 > Basil

-- 

Thanks,

--Raman(I Search, I Find, I Misplace, I Research)
♉ Id: kg:/m/0285kf1  🦮

--

Thanks,

--Raman(I Search, I Find, I Misplace, I Research)
♉ Id: kg:/m/0285kf1  🦮



  reply	other threads:[~2021-06-11 14:30 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-06-05 13:55 battery.el broken? T.V Raman
2021-06-05 22:21 ` Basil L. Contovounesios
2021-06-05 22:23   ` Stefan Monnier
2021-06-06  1:52     ` T.V Raman
2021-06-11 10:51       ` Basil L. Contovounesios
2021-06-11 14:30         ` T.V Raman [this message]
2021-06-06  1:58     ` T.V Raman
2021-06-26  2:01 ` T.V Raman

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=24771.29553.344941.739549@retriever.mtv.corp.google.com \
    --to=raman@google.com \
    --cc=contovob@tcd.ie \
    --cc=emacs-devel@gnu.org \
    --cc=monnier@iro.umontreal.ca \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).