#3 Reduce load on ACPI monitoring for linux

open
nobody
None
5
2003-05-02
2003-05-02
Mike Hardy
No

This patch alters the battery status applet to do a
couple of things:

1) It will always use its own apmlib. This may be
controversial for other architectures that can't use
the bundled apmlib, but I didn't know how to do the
configure stuff well enough to only turn it on for
Linux and ACPI

2) The apm_info struct has been extended to have some
extra members where it can store battery information

3) the acpi read function will read the battery
information only once, and store it in the extra
members of the apm_info struct (that saves .7 seconds
per refresh on my laptop). It should probably be a user
preference to update on some slower period, or to have
a manual update for it in case people change batteries.
It works for my normal case though

4) the status applet will only refresh every 30 seconds
instead of every second. This last part may be the most
controversial. This should probably be a user
preference, but its good enough for me

Bottom line is, there are some questionable *policy*
decisions that I've made in the patch, but if you are
on Linux, and are using ACPI, this will make the applet
infinitely more well-behaved than it is by default.

Discussion

  • Mike Hardy

    Mike Hardy - 2003-05-02

    Logged In: YES
    user_id=564701

    I was told my patch was reversed, sorry. It should be the
    right direction now.

     

Log in to post a comment.

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:

JavaScript is required for this form.





No, thanks