! Please note that this is a snapshot of our old Bugzilla server, which is read only since May 29, 2020. Please go to gitlab.xfce.org for our new server !
power-manager sending notifications too often when battery is recharging
Status:
CLOSED: FIXED
Priority:
Medium
Severity:
normal
Product:
Xfce4-power-manager
Component:
General

Comments

Description Unnamed_Hero 2009-05-01 20:01:36 CEST 
I've made several tests - it seems, that message are shown when battery recharges every new percent of it's capacity, or more often (sometimes every 15 seconds) - it's very annoying.
Power manager - svn 7299, rc1, beta1 are affected too.
notification-daemon-xfce-0.3.7
libnotify-0.4.5

But one more thing - is these notifications depends on current acpi implementation of my laptop? (buggy for my model, frankly speaking)
Comment 1 Ali Abdallah editbugs 2009-05-01 20:09:05 CEST 
(In reply to comment #0)
> I've made several tests - it seems, that message are shown when battery
> recharges every new percent of it's capacity, or more often (sometimes every 15
> seconds) - it's very annoying.
> Power manager - svn 7299, rc1, beta1 are affected too.
> notification-daemon-xfce-0.3.7
> libnotify-0.4.5
> 
> But one more thing - is these notifications depends on current acpi
> implementation of my laptop? (buggy for my model, frankly speaking)

Weired,

Could you please configure the RC1 with --enable-debug=full then run the power manager from the terminal with the option --no-daemon, charge your battery and attach everything you see on the terminal, thanks.
Comment 2 Unnamed_Hero 2009-05-02 04:24:14 CEST 
svn 7301

building with 
  ./autogen.sh \
    --sysconfdir=/etc/xfce \
    --prefix=/usr \
    --disable-static \
    --enable-debug=full
Comment 3 Unnamed_Hero 2009-05-02 04:28:55 CEST 
svn 7301

building with 

make clean
  ./autogen.sh \
    --sysconfdir=/etc/xfce \
    --prefix=/usr \
    --disable-static \
    --enable-debug=full

failed with

gcc -DHAVE_CONFIG_H -I. -I..    -I.. -I../libxfpm -DLOCALEDIR=\"/usr/share/locale\" -DG_LOG_DOMAIN=\"xfce4-power-manager\" -I/usr/include/gtk-2.0 -I/usr/lib/gtk-2.0/include -I/usr/include/atk-1.0 -I/usr/include/cairo -I/usr/include/pango-1.0 -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include -I/usr/include/pixman-1 -I/usr/include/freetype2 -I/usr/include/libpng12   -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include   -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include   -pthread -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include   -I/usr/include/dbus-1.0 -I/usr/lib/dbus-1.0/include   -I/usr/include/dbus-1.0 -I/usr/lib/dbus-1.0/include -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include   -I/usr/include/xfce4 -I/usr/include/gtk-2.0 -I/usr/lib/gtk-2.0/include -I/usr/include/pango-1.0 -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include -I/usr/include/cairo -I/usr/include/pixman-1 -I/usr/include/freetype2 -I/usr/include/libpng12 -I/usr/include/atk-1.0   -I/usr/include/xfce4 -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include   -I/usr/include/xfce4/xfconf-0 -I/usr/include/dbus-1.0 -I/usr/lib/dbus-1.0/include -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include   -I/usr/include/gtk-2.0 -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include -I/usr/include/dbus-1.0 -I/usr/lib/dbus-1.0/include -I/usr/lib/gtk-2.0/include -I/usr/include/atk-1.0 -I/usr/include/cairo -I/usr/include/pango-1.0 -I/usr/include/pixman-1 -I/usr/include/freetype2 -I/usr/include/libpng12                 	 -O2 -march=i486 -mtune=i686 -g3 -Werror -Wall -DXFCE_DISABLE_DEPRECATED -MT xfce4_power_manager-xfpm-battery.o -MD -MP -MF .deps/xfce4_power_manager-xfpm-battery.Tpo -c -o xfce4_power_manager-xfpm-battery.o `test -f 'xfpm-battery.c' || echo './'`xfpm-battery.c
cc1: warnings being treated as errors
xfpm-battery.c: В функции ‘xfpm_battery_refresh’
xfpm-battery.c:388: предупреждение: ‘est_time’ may be used uninitialized in this function
xfpm-battery.c:388: замечание: ‘est_time’ was declared here
make[3]: *** [xfce4_power_manager-xfpm-battery.o] Ошибка 1
make[3]: Leaving directory `/tmp/uh/build/!desktop/xfce/plugins/powermanager/xfce4-power-manager/src'
make[2]: *** [all] Ошибка 2
make[2]: Leaving directory `/tmp/uh/build/!desktop/xfce/plugins/powermanager/xfce4-power-manager/src'
make[1]: *** [all-recursive] Ошибка 1
make[1]: Leaving directory `/tmp/uh/build/!desktop/xfce/plugins/powermanager/xfce4-power-manager'
make: *** [all] Ошибка 2

В функции = in function
предупреждение = warning
замечание = notice 


Without --enable-debug=full - all is OK
Comment 4 Ali Abdallah editbugs 2009-05-02 08:28:05 CEST 
(In reply to comment #3)

try r7302 with debug_full.
Comment 5 Unnamed_Hero 2009-05-02 10:36:43 CEST 
Compile is done, thanks!

Now, the output:

xfce4-power-manager --no-daemon
TRACE[xfpm-main.c:71] xfpm_start(): Starting the power manager

TRACE[xfpm-adapter.c:100] xfpm_adapter_set_device(): Found AC Adapter with udi=/org/freedesktop/Hal/devices/computer_power_supply_ac_adapter_ADP1

TRACE[xfpm-adapter.c:154] xfpm_adapter_init(): System is identified as a laptop

** (xfce4-power-manager:9301): WARNING **: Metadata for error domain "xfpm-error-quark" already registered

TRACE[xfpm-cpu.c:256] xfpm_cpu_get_available_governors(): found CPU governor conservative
TRACE[xfpm-cpu.c:256] xfpm_cpu_get_available_governors(): found CPU governor ondemand
TRACE[xfpm-cpu.c:256] xfpm_cpu_get_available_governors(): found CPU governor powersave
TRACE[xfpm-cpu.c:256] xfpm_cpu_get_available_governors(): found CPU governor userspace
TRACE[xfpm-cpu.c:256] xfpm_cpu_get_available_governors(): found CPU governor performance
TRACE[xfpm-supply.c:375] xfpm_supply_add_battery(): New battery found /org/freedesktop/Hal/devices/computer_power_supply_battery_BAT1
TRACE[xfpm-battery.c:163] xfpm_battery_refresh_icon(): battery state : ((XfpmBatteryState) BATTERY_FULLY_CHARGED)
TRACE[xfpm-battery.c:326] xfpm_battery_refresh_common(): Emitting signal battery state changed

(xfce4-power-manager:9301): xfce4-power-manager-CRITICAL **: could not map keysym 1008ff2a to keycode


(xfce4-power-manager:9301): xfce4-power-manager-CRITICAL **: could not map keysym 1008ff2f to keycode


(xfce4-power-manager:9301): xfce4-power-manager-CRITICAL **: could not map keysym 1008ff02 to keycode

TRACE[xfpm-brightness-hal.c:471] xfpm_brightness_hal_init(): No lcd brightness control found in the system

(xfce4-power-manager:9301): GLib-GObject-WARNING **: invalid (NULL) pointer instance

(xfce4-power-manager:9301): GLib-GObject-CRITICAL **: g_signal_handler_is_connected: assertion `G_TYPE_CHECK_INSTANCE (instance)' failed

(xfce4-power-manager:9301): GLib-GObject-WARNING **: invalid (NULL) pointer instance

(xfce4-power-manager:9301): GLib-GObject-CRITICAL **: g_signal_handler_is_connected: assertion `G_TYPE_CHECK_INSTANCE (instance)' failed

(xfce4-power-manager:9301): GLib-GObject-WARNING **: invalid (NULL) pointer instance

(xfce4-power-manager:9301): GLib-GObject-CRITICAL **: g_signal_handler_is_connected: assertion `G_TYPE_CHECK_INSTANCE (instance)' failed

(xfce4-power-manager:9301): GLib-GObject-WARNING **: invalid (NULL) pointer instance

(xfce4-power-manager:9301): GLib-GObject-CRITICAL **: g_signal_handler_is_connected: assertion `G_TYPE_CHECK_INSTANCE (instance)' failed

(xfce4-power-manager:9301): GLib-GObject-WARNING **: invalid (NULL) pointer instance

(xfce4-power-manager:9301): GLib-GObject-CRITICAL **: g_signal_handler_is_connected: assertion `G_TYPE_CHECK_INSTANCE (instance)' failed

(xfce4-power-manager:9301): GLib-GObject-WARNING **: invalid (NULL) pointer instance

(xfce4-power-manager:9301): GLib-GObject-CRITICAL **: g_signal_handler_is_connected: assertion `G_TYPE_CHECK_INSTANCE (instance)' failed
TRACE[xfpm-engine.c:337] xfpm_engine_set_inactivity_timeouts(): setting inactivity sleep timeout on ac to never
TRACE[xfpm-engine.c:341] xfpm_engine_set_inactivity_timeouts(): setting inactivity sleep timeout on battery to never
TRACE[xfpm-battery.c:471] xfpm_battery_device_changed_cb(): start
TRACE[xfpm-battery.c:163] xfpm_battery_refresh_icon(): battery state : ((XfpmBatteryState) BATTERY_IS_DISCHARGING)
TRACE[xfpm-battery.c:471] xfpm_battery_device_changed_cb(): start
TRACE[xfpm-battery.c:163] xfpm_battery_refresh_icon(): battery state : ((XfpmBatteryState) BATTERY_IS_DISCHARGING)
TRACE[xfpm-battery.c:471] xfpm_battery_device_changed_cb(): start
TRACE[xfpm-battery.c:163] xfpm_battery_refresh_icon(): battery state : ((XfpmBatteryState) BATTERY_IS_DISCHARGING)
TRACE[xfpm-battery.c:326] xfpm_battery_refresh_common(): Emitting signal battery state changed
TRACE[xfpm-battery.c:471] xfpm_battery_device_changed_cb(): start
TRACE[xfpm-battery.c:163] xfpm_battery_refresh_icon(): battery state : ((XfpmBatteryState) BATTERY_IS_CHARGING)
TRACE[xfpm-battery.c:471] xfpm_battery_device_changed_cb(): start
TRACE[xfpm-battery.c:163] xfpm_battery_refresh_icon(): battery state : ((XfpmBatteryState) BATTERY_IS_CHARGING)
TRACE[xfpm-battery.c:471] xfpm_battery_device_changed_cb(): start
TRACE[xfpm-battery.c:163] xfpm_battery_refresh_icon(): battery state : ((XfpmBatteryState) BATTERY_IS_CHARGING)
TRACE[xfpm-cpu.c:199] xfpm_cpu_set_governor(): Settings cpu governor to ondemand
TRACE[xfpm-battery.c:163] xfpm_battery_refresh_icon(): battery state : ((XfpmBatteryState) BATTERY_IS_CHARGING)
TRACE[xfpm-battery.c:471] xfpm_battery_device_changed_cb(): start
TRACE[xfpm-battery.c:163] xfpm_battery_refresh_icon(): battery state : ((XfpmBatteryState) BATTERY_IS_CHARGING)
TRACE[xfpm-battery.c:326] xfpm_battery_refresh_common(): Emitting signal battery state changed
TRACE[xfpm-battery.c:471] xfpm_battery_device_changed_cb(): start
TRACE[xfpm-battery.c:163] xfpm_battery_refresh_icon(): battery state : ((XfpmBatteryState) BATTERY_NOT_FULLY_CHARGED)
TRACE[xfpm-battery.c:471] xfpm_battery_device_changed_cb(): start
TRACE[xfpm-battery.c:163] xfpm_battery_refresh_icon(): battery state : ((XfpmBatteryState) BATTERY_NOT_FULLY_CHARGED)
TRACE[xfpm-battery.c:471] xfpm_battery_device_changed_cb(): start
TRACE[xfpm-battery.c:163] xfpm_battery_refresh_icon(): battery state : ((XfpmBatteryState) BATTERY_NOT_FULLY_CHARGED)
TRACE[xfpm-battery.c:471] xfpm_battery_device_changed_cb(): start
TRACE[xfpm-battery.c:163] xfpm_battery_refresh_icon(): battery state : ((XfpmBatteryState) BATTERY_NOT_FULLY_CHARGED)
TRACE[xfpm-battery.c:471] xfpm_battery_device_changed_cb(): start
TRACE[xfpm-battery.c:163] xfpm_battery_refresh_icon(): battery state : ((XfpmBatteryState) BATTERY_NOT_FULLY_CHARGED)
TRACE[xfpm-battery.c:471] xfpm_battery_device_changed_cb(): start
TRACE[xfpm-battery.c:163] xfpm_battery_refresh_icon(): battery state : ((XfpmBatteryState) BATTERY_NOT_FULLY_CHARGED)
TRACE[xfpm-battery.c:471] xfpm_battery_device_changed_cb(): start
TRACE[xfpm-battery.c:163] xfpm_battery_refresh_icon(): battery state : ((XfpmBatteryState) BATTERY_NOT_FULLY_CHARGED)
TRACE[xfpm-battery.c:471] xfpm_battery_device_changed_cb(): start
TRACE[xfpm-battery.c:163] xfpm_battery_refresh_icon(): battery state : ((XfpmBatteryState) BATTERY_NOT_FULLY_CHARGED)
TRACE[xfpm-battery.c:471] xfpm_battery_device_changed_cb(): start
TRACE[xfpm-battery.c:163] xfpm_battery_refresh_icon(): battery state : ((XfpmBatteryState) BATTERY_NOT_FULLY_CHARGED)
TRACE[xfpm-battery.c:326] xfpm_battery_refresh_common(): Emitting signal battery state changed
TRACE[xfpm-battery.c:471] xfpm_battery_device_changed_cb(): start
TRACE[xfpm-battery.c:163] xfpm_battery_refresh_icon(): battery state : ((XfpmBatteryState) BATTERY_IS_CHARGING)
TRACE[xfpm-battery.c:471] xfpm_battery_device_changed_cb(): start
TRACE[xfpm-battery.c:163] xfpm_battery_refresh_icon(): battery state : ((XfpmBatteryState) BATTERY_IS_CHARGING)
TRACE[xfpm-battery.c:471] xfpm_battery_device_changed_cb(): start
TRACE[xfpm-battery.c:163] xfpm_battery_refresh_icon(): battery state : ((XfpmBatteryState) BATTERY_IS_CHARGING)
TRACE[xfpm-battery.c:326] xfpm_battery_refresh_common(): Emitting signal battery state changed
TRACE[xfpm-battery.c:471] xfpm_battery_device_changed_cb(): start
TRACE[xfpm-battery.c:163] xfpm_battery_refresh_icon(): battery state : ((XfpmBatteryState) BATTERY_NOT_FULLY_CHARGED)
TRACE[xfpm-battery.c:471] xfpm_battery_device_changed_cb(): start
TRACE[xfpm-battery.c:163] xfpm_battery_refresh_icon(): battery state : ((XfpmBatteryState) BATTERY_NOT_FULLY_CHARGED)
TRACE[xfpm-battery.c:471] xfpm_battery_device_changed_cb(): start
TRACE[xfpm-battery.c:163] xfpm_battery_refresh_icon(): battery state : ((XfpmBatteryState) BATTERY_NOT_FULLY_CHARGED)
TRACE[xfpm-battery.c:471] xfpm_battery_device_changed_cb(): start
TRACE[xfpm-battery.c:163] xfpm_battery_refresh_icon(): battery state : ((XfpmBatteryState) BATTERY_NOT_FULLY_CHARGED)
TRACE[xfpm-battery.c:471] xfpm_battery_device_changed_cb(): start
TRACE[xfpm-battery.c:163] xfpm_battery_refresh_icon(): battery state : ((XfpmBatteryState) BATTERY_NOT_FULLY_CHARGED)
TRACE[xfpm-battery.c:471] xfpm_battery_device_changed_cb(): start
TRACE[xfpm-battery.c:163] xfpm_battery_refresh_icon(): battery state : ((XfpmBatteryState) BATTERY_NOT_FULLY_CHARGED)
TRACE[xfpm-battery.c:471] xfpm_battery_device_changed_cb(): start
TRACE[xfpm-battery.c:163] xfpm_battery_refresh_icon(): battery state : ((XfpmBatteryState) BATTERY_NOT_FULLY_CHARGED)
TRACE[xfpm-battery.c:471] xfpm_battery_device_changed_cb(): start
TRACE[xfpm-battery.c:163] xfpm_battery_refresh_icon(): battery state : ((XfpmBatteryState) BATTERY_NOT_FULLY_CHARGED)
TRACE[xfpm-battery.c:326] xfpm_battery_refresh_common(): Emitting signal battery state changed
TRACE[xfpm-battery.c:471] xfpm_battery_device_changed_cb(): start
TRACE[xfpm-battery.c:163] xfpm_battery_refresh_icon(): battery state : ((XfpmBatteryState) BATTERY_IS_CHARGING)
TRACE[xfpm-battery.c:471] xfpm_battery_device_changed_cb(): start
TRACE[xfpm-battery.c:163] xfpm_battery_refresh_icon(): battery state : ((XfpmBatteryState) BATTERY_IS_CHARGING)
TRACE[xfpm-battery.c:471] xfpm_battery_device_changed_cb(): start
TRACE[xfpm-battery.c:163] xfpm_battery_refresh_icon(): battery state : ((XfpmBatteryState) BATTERY_IS_CHARGING)
TRACE[xfpm-battery.c:326] xfpm_battery_refresh_common(): Emitting signal battery state changed
TRACE[xfpm-battery.c:471] xfpm_battery_device_changed_cb(): start
TRACE[xfpm-battery.c:163] xfpm_battery_refresh_icon(): battery state : ((XfpmBatteryState) BATTERY_NOT_FULLY_CHARGED)
TRACE[xfpm-battery.c:471] xfpm_battery_device_changed_cb(): start
TRACE[xfpm-battery.c:163] xfpm_battery_refresh_icon(): battery state : ((XfpmBatteryState) BATTERY_NOT_FULLY_CHARGED)
TRACE[xfpm-battery.c:471] xfpm_battery_device_changed_cb(): start
TRACE[xfpm-battery.c:163] xfpm_battery_refresh_icon(): battery state : ((XfpmBatteryState) BATTERY_NOT_FULLY_CHARGED)


and so on.
Comment 6 Ali Abdallah editbugs 2009-05-02 10:57:02 CEST 
(In reply to comment #5)

Please try out the latest svn revision and let me know if it reduces the notifications.
Comment 7 Unnamed_Hero 2009-05-02 14:21:16 CEST 
Yes, they are reduced.

But this weird behavior is caused by invalid ACPI implementation for my laptop in linux kernel. There are several bugs - for example http://bugzilla.kernel.org/show_bug.cgi?id=12011. 

So, I've made some observations.
In a normal state my  /proc/acpi/battery/BAT1/state contains

present:                 yes
capacity state:          ok
charging state:          charging
present rate:            827 mA
remaining capacity:      1142 mAh
present voltage:         11980 mV

When a notification is appeared - it contains

present:                 yes
capacity state:          ok
charging state:          charged
present rate:            0 mA
remaining capacity:      1159 mAh
present voltage:         11961 mV

next moment it contains valid "charging" state.
So, I think it is a my local problem with a solution to switch off notifications, and do not spend your time.

Thank for your fast replies! Thank for your great job!
Comment 8 Ali Abdallah editbugs 2009-05-03 09:42:06 CEST 
(In reply to comment #7)
Interesting case, please could you please post the output of lshal, what i see from your previous post that when your battery is "physically discharging" the message coming from ACPI says that the battery is charging. weired.
Comment 9 Unnamed_Hero 2009-05-03 10:24:02 CEST 
Oh... I was starting 'power-manager --no-daemon' on battery power and then plug in power cable, so this is normal.
Comment 10 Ali Abdallah editbugs 2009-05-15 20:24:13 CEST 
Can i close this bug! do you still have problems?
Comment 11 Unnamed_Hero 2009-05-15 20:34:03 CEST 
(In reply to comment #10)
> Can i close this bug! do you still have problems?

Yes, you can close it.
Comment 12 Ali Abdallah editbugs 2009-07-02 14:11:23 CEST 
Closing old bugs to keep things organized.

Bug #5323

Reported by:
Unnamed_Hero
Reported on: 2009-05-01
Last modified on: 2009-07-14

People

Assignee:
Ali Abdallah
CC List:
0 users

Version

Version:
Unspecified
Target Milestone:
---

Attachments

Additional information