Bug 26948

Summary:

Индикатор батареи показывает всегда 100%

Product:

Sisyphus

Reporter:

Aleksey Borisenkov <grizlik78>

Component:

lxde-lxpanel

Assignee:

Radik Usupov <radik>

Status:

CLOSED FIXED

QA Contact:

qa-sisyphus

Severity:

normal

Priority:

CC:

antohami, mike, snejok

Version:

unstable

Hardware:

all

OS:

Linux

Attachments:

Description	Flags
возможный вариант исправления	none
менее интрузивный вариант	none

Description Aleksey Borisenkov 2012-02-16 00:04:36 MSK

В очередной раз на моём eeepc сломался индикатор батареи. Апстрим код всё переписывает и переписывает, а нормально написать не может :)
В моём случае проблема из-за того, что у меня отсутствуют файлы
/sys/class/power_supply/BAT0/energy_*
зато есть
$ ls /sys/class/power_supply/BAT0/charge_*
/sys/class/power_supply/BAT0/charge_full
/sys/class/power_supply/BAT0/charge_now
/sys/class/power_supply/BAT0/charge_full_design

Comment 1 Aleksey Borisenkov 2012-02-16 00:09:50 MSK

Created attachment 5322 [details]
возможный вариант исправления

Данный патч вроде бы не должен ломать индикатор в системах, где energy_now всё-таки присутствует, так как выше по коду есть пересчёт из energy в charge. Но можно ещё и проверку добавить на предмет
(b->energy_now == -1 || b->energy_full == -1)
Если выполняется, то использовать charge, если нет, то energy

Comment 2 Radik Usupov 2012-02-20 17:45:47 MSK

Это после последнего обновления поломалось?
Я там вот этот патч применил, который исправил ошибку с заморозкой панели: http://git.altlinux.org/people/radik/packages/lxde-lxpanel.git?p=lxde-lxpanel.git;a=commit;h=0c1fb08a70ae10d4e959988752fa2e4320dd2faa

Ваше исправление точно ничего не сломает? Ато я LXDEsktop релизить хочу..

Comment 3 Aleksey Borisenkov 2012-02-20 18:12:35 MSK

Не должно, но я сегодня попозже напишу патч, с которым при наличии energy_* будет работать точно как в апстриме, а при отсутствии исправляет ошибку.

Comment 4 Aleksey Borisenkov 2012-02-20 20:34:46 MSK

Created attachment 5324 [details]
менее интрузивный вариант

Да, сломалось с версией 0.5.8-alt4, но не из-за самого этого фикса, а из-за сопутствующего ему мерджа.
Проявляется у меня на eeepc 1000, и, кажется, на эйсере на каком-то, завтра на работе постараюсь убедиться. Больше проверить негде, ноутбуков больше нет.
Второй вариант исправления точно ничего не ломает, хотя первый мне кажется ближе к тому, что хотели сделать авторы.

Comment 5 Radik Usupov 2012-02-20 21:47:53 MSK

Алексей, а расскажите что значит "сломался индикатор батареи"?
Я к тому, что у меня на моем lenovo сейчас все работает. Буду тестить Ваш фикс в том числе на своем ноуте..

Comment 6 Aleksey Borisenkov 2012-02-20 22:01:42 MSK

(В ответ на комментарий №5)
> Алексей, а расскажите что значит "сломался индикатор батареи"?
То, что написано в названии бага :) Он всегда показывает 100% независимо от заряда батареи. При этом gkrellm, например, заряд показывает правильно.

> Я к тому, что у меня на моем lenovo сейчас все работает. Буду тестить Ваш фикс
> в том числе на своем ноуте..
Это означает, что на lenovo есть файлы
/sys/sys/class/power_supply/BAT0/energy_now
и
/class/power_supply/BAT0/energy_full.
На eeepc их нет (но есть соответствующие charge_*). На асере не помню, кажется тоже не было, завтра посмотрю.
Вероятно у авторов тоже есть эти файлы, раз у них всё работает :)

Comment 7 Aleksey Borisenkov 2012-02-20 22:03:09 MSK

тьфу, скопировалось криво
/sys/class/power_supply/BAT0/energy_now
и
/sys/class/power_supply/BAT0/energy_full
конечно же

Comment 8 Nick S. Grechukh 2012-02-20 22:04:33 MSK

вообще, выкинуть бы весь этот мусор и научиться брать из upower...

Comment 9 Aleksey Borisenkov 2012-02-21 15:54:33 MSK

Выкинуть бы хорошо, только я, например, за это сейчас не возьмусь.

Проверил 0.5.8-alt4 на Acer Aspire 2920. Там та же проблема, так что у меня нет ноутбуков, где бы нынешняя сизифная версия нормально работала.

Comment 10 Radik Usupov 2012-02-21 19:43:28 MSK

Спасибо.. Соберу lxpanel с Вашим патчем, попробую на себе. Если ничего не отвалится, соберу в сизиф и р6.

Comment 11 Repository Robot 2012-02-23 11:02:10 MSK

lxde-lxpanel-0.5.8-alt5 -> sisyphus:

* Thu Feb 23 2012 Radik Usupov <radik@altlinux> 0.5.8-alt5
- Added batt_percentage_fix.patch (by Alexey Borisenkov) (Closes: 26948)