From: Alexey Starikovskiy <aystarik@gmail.com>
To: Ash Milsted <ashmilsted@gmail.com>
Cc: linux-acpi@vger.kernel.org
Subject: Re: 2.6.24-rc1 acpi battery driver -> sysfs interface does not update correctly
Date: Thu, 25 Oct 2007 07:08:53 +0400 [thread overview]
Message-ID: <472008C5.8030906@gmail.com> (raw)
In-Reply-To: <3e7b9f4b0710241537i1d3db590u2220900e30ae0d1b@mail.gmail.com>
Ash Milsted wrote:
> On 25/10/2007, Ash Milsted <thatistosayiseenem@gawab.com> wrote:
>> On 24/10/2007, Alexey Starikovskiy <aystarik@gmail.com> wrote:
>>> Ash Milsted wrote:
>>>> On 24/10/2007, Alexey Starikovskiy <aystarik@gmail.com> wrote:
>>>>> This one is not easy. According to code, it should send twice as many notifications now.
>>>>> One over netlink, and one as power_supply class device...
>>>>> Could you log all the uevents?
>>>>>
>>>> Forgive my ignorance, but I'm not entirely sure how to capture
>>>> uevents. If "udevmonitor --kernel" is sufficient then it appears there
>>>> are no events associated with the battery discharging or charging,
>>>> whereas those associated with (un)plugging do appear. To clarify, I've
>>>> seen cat /sys/.../charge_now change without udevmonitor picking up a
>>>> single kernel event.
>>> Could you try to insert prink() into acpi_battery_notify() in drivers/acpi/battery.c?
>>> This is the only place which could send uevent. If you don't see this printk in
>>> dmesg, then HAL uses some other method to get battery state changes.
>>>
>> I can confirm that this function is *not* called as the battery
>> (dis)charges (having seen charge_now fall), but is called on
>> power-plug events.
> Sorry... I mean it appears to be called on module-insertion and then
> *never* again (not on plug events).
>
Could you please confirm, that there were no battery discharge uevents before, e.g in 2.6.23?
next prev parent reply other threads:[~2007-10-25 3:09 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-10-24 18:14 2.6.24-rc1 acpi battery driver -> sysfs interface does not update correctly Ash Milsted
2007-10-24 18:53 ` Alexey Starikovskiy
2007-10-24 19:52 ` Ash Milsted
2007-10-24 20:23 ` Ash Milsted
2007-10-24 20:32 ` Alexey Starikovskiy
2007-10-24 21:17 ` Ash Milsted
2007-10-24 21:49 ` Alexey Starikovskiy
2007-10-24 22:28 ` Ash Milsted
2007-10-24 22:37 ` Ash Milsted
2007-10-25 3:08 ` Alexey Starikovskiy [this message]
2007-10-25 9:31 ` Ash Milsted
2007-10-27 16:52 ` Ash Milsted
2007-10-27 17:22 ` Alexey Starikovskiy
2007-10-28 12:06 ` Ash Milsted
[not found] ` <3e7b9f4b0710280524u1898296ase70f70c9cd03ced0@mail.gmail.com>
2007-10-28 12:33 ` [PATCH] ACPI: battery: Support for non-spec name for LiIon technology Alexey Starikovskiy
2007-10-28 12:40 ` 2.6.24-rc1 acpi battery driver -> sysfs interface does not update correctly Alexey Starikovskiy
2007-10-28 12:58 ` Alexey Starikovskiy
2007-10-28 14:03 ` Ash Milsted
2007-10-28 14:27 ` Alexey Starikovskiy
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
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=472008C5.8030906@gmail.com \
--to=aystarik@gmail.com \
--cc=ashmilsted@gmail.com \
--cc=linux-acpi@vger.kernel.org \
/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.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.