26948 2012-02-16 00:04:36 +0400 Индикатор батареи показывает всегда 100% 2012-02-24 19:44:09 +0400 1 1 4 Development Sisyphus lxde-lxpanel unstable all Linux CLOSED FIXED P3 normal --- 1 grizlik78 radik antohami mike snejok qa-sisyphus oldest_to_newest 128963 0 grizlik78 2012-02-16 00:04:36 +0400 В очередной раз на моём 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 128964 1 5322 grizlik78 2012-02-16 00:09:50 +0400 Created attachment 5322 возможный вариант исправления Данный патч вроде бы не должен ломать индикатор в системах, где energy_now всё-таки присутствует, так как выше по коду есть пересчёт из energy в charge. Но можно ещё и проверку добавить на предмет (b->energy_now == -1 || b->energy_full == -1) Если выполняется, то использовать charge, если нет, то energy 129098 2 radik 2012-02-20 17:45:47 +0400 Это после последнего обновления поломалось? Я там вот этот патч применил, который исправил ошибку с заморозкой панели: http://git.altlinux.org/people/radik/packages/lxde-lxpanel.git?p=lxde-lxpanel.git;a=commit;h=0c1fb08a70ae10d4e959988752fa2e4320dd2faa Ваше исправление точно ничего не сломает? Ато я LXDEsktop релизить хочу.. 129102 3 grizlik78 2012-02-20 18:12:35 +0400 Не должно, но я сегодня попозже напишу патч, с которым при наличии energy_* будет работать точно как в апстриме, а при отсутствии исправляет ошибку. 129105 4 5324 grizlik78 2012-02-20 20:34:46 +0400 Created attachment 5324 менее интрузивный вариант Да, сломалось с версией 0.5.8-alt4, но не из-за самого этого фикса, а из-за сопутствующего ему мерджа. Проявляется у меня на eeepc 1000, и, кажется, на эйсере на каком-то, завтра на работе постараюсь убедиться. Больше проверить негде, ноутбуков больше нет. Второй вариант исправления точно ничего не ломает, хотя первый мне кажется ближе к тому, что хотели сделать авторы. 129106 5 radik 2012-02-20 21:47:53 +0400 Алексей, а расскажите что значит "сломался индикатор батареи"? Я к тому, что у меня на моем lenovo сейчас все работает. Буду тестить Ваш фикс в том числе на своем ноуте.. 129107 6 grizlik78 2012-02-20 22:01:42 +0400 (В ответ на комментарий №5) > Алексей, а расскажите что значит "сломался индикатор батареи"? То, что написано в названии бага :) Он всегда показывает 100% независимо от заряда батареи. При этом gkrellm, например, заряд показывает правильно. > Я к тому, что у меня на моем lenovo сейчас все работает. Буду тестить Ваш фикс > в том числе на своем ноуте.. Это означает, что на lenovo есть файлы /sys/sys/class/power_supply/BAT0/energy_now и /class/power_supply/BAT0/energy_full. На eeepc их нет (но есть соответствующие charge_*). На асере не помню, кажется тоже не было, завтра посмотрю. Вероятно у авторов тоже есть эти файлы, раз у них всё работает :) 129108 7 grizlik78 2012-02-20 22:03:09 +0400 тьфу, скопировалось криво /sys/class/power_supply/BAT0/energy_now и /sys/class/power_supply/BAT0/energy_full конечно же 129109 8 gns 2012-02-20 22:04:33 +0400 вообще, выкинуть бы весь этот мусор и научиться брать из upower... 129126 9 grizlik78 2012-02-21 15:54:33 +0400 Выкинуть бы хорошо, только я, например, за это сейчас не возьмусь. Проверил 0.5.8-alt4 на Acer Aspire 2920. Там та же проблема, так что у меня нет ноутбуков, где бы нынешняя сизифная версия нормально работала. 129140 10 radik 2012-02-21 19:43:28 +0400 Спасибо.. Соберу lxpanel с Вашим патчем, попробую на себе. Если ничего не отвалится, соберу в сизиф и р6. 129193 11 repository-robot 2012-02-23 11:02:10 +0400 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) 5322 2012-02-16 00:09:50 +0400 2012-02-16 00:09:50 +0400 возможный вариант исправления battery_100_percents.patch text/plain 539 grizlik78 ZGlmZiAtLWdpdCBhL2x4cGFuZWwvc3JjL3BsdWdpbnMvYmF0dC9iYXR0X3N5cy5jIGIvbHhwYW5l bC9zcmMvcGx1Z2lucy9iYXR0L2JhdHRfc3lzLmMKaW5kZXggNjI0NDVmZS4uY2M0ZjBlMiAxMDA2 NDQKLS0tIGEvbHhwYW5lbC9zcmMvcGx1Z2lucy9iYXR0L2JhdHRfc3lzLmMKKysrIGIvbHhwYW5l bC9zcmMvcGx1Z2lucy9iYXR0L2JhdHRfc3lzLmMKQEAgLTIxNiw3ICsyMTYsNyBAQCB2b2lkIGJh dHRlcnlfdXBkYXRlKGJhdHRlcnkgKmIpCiAgICAgaWYgKGItPmNoYXJnZV9mdWxsIDwgTUlOX0NB UEFDSVRZKQogCWItPnBlcmNlbnRhZ2UgPSAwOwogICAgIGVsc2UgewotCWludCBwcm9taWxsZSA9 IChiLT5lbmVyZ3lfbm93ICogMTAwMCkgLyBiLT5lbmVyZ3lfZnVsbDsKKwlpbnQgcHJvbWlsbGUg PSAoYi0+Y2hhcmdlX25vdyAqIDEwMDApIC8gYi0+Y2hhcmdlX2Z1bGw7CiAJYi0+cGVyY2VudGFn ZSA9IChwcm9taWxsZSArIDUpIC8gMTA7IC8qIHJvdW5kIHByb3Blcmx5ICovCiAgICAgfQogICAg IGlmIChiLT5wZXJjZW50YWdlID4gMTAwKQo= 5324 2012-02-20 20:34:46 +0400 2012-02-20 20:34:46 +0400 менее интрузивный вариант batt_percentage_fix.patch text/plain 550 grizlik78 ZGlmZiAtLWdpdCBhL2x4cGFuZWwvc3JjL3BsdWdpbnMvYmF0dC9iYXR0X3N5cy5jIGIvbHhwYW5l bC9zcmMvcGx1Z2lucy9iYXR0L2JhdHRfc3lzLmMKaW5kZXggNjI0NDVmZS4uN2I3ZGVmNSAxMDA2 NDQKLS0tIGEvbHhwYW5lbC9zcmMvcGx1Z2lucy9iYXR0L2JhdHRfc3lzLmMKKysrIGIvbHhwYW5l bC9zcmMvcGx1Z2lucy9iYXR0L2JhdHRfc3lzLmMKQEAgLTIxNyw2ICsyMTcsOCBAQCB2b2lkIGJh dHRlcnlfdXBkYXRlKGJhdHRlcnkgKmIpCiAJYi0+cGVyY2VudGFnZSA9IDA7CiAgICAgZWxzZSB7 CiAJaW50IHByb21pbGxlID0gKGItPmVuZXJneV9ub3cgKiAxMDAwKSAvIGItPmVuZXJneV9mdWxs OworCWlmIChiLT5lbmVyZ3lfbm93ID09IC0xIHx8IGItPmVuZXJneV9mdWxsID09IC0xKQorCSAg ICBwcm9taWxsZSA9IChiLT5jaGFyZ2Vfbm93ICogMTAwMCkgLyBiLT5jaGFyZ2VfZnVsbDsKIAli LT5wZXJjZW50YWdlID0gKHByb21pbGxlICsgNSkgLyAxMDsgLyogcm91bmQgcHJvcGVybHkgKi8K ICAgICB9CiAgICAgaWYgKGItPnBlcmNlbnRhZ2UgPiAxMDApCg==