Menu

#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

    Battery applet ACPI load reduction

     
  • 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.

     
  • Mike Hardy

    Mike Hardy - 2003-05-02

    Reduce load, especially on ACPI monitoring

     

Log in to post a comment.

Want the latest updates on software, tech news, and AI?
Get latest updates about software, tech news, and AI from SourceForge directly in your inbox once a month.