Bug 24030 - Добавить: Requires: ifplugd ethtool iproute2
Summary: Добавить: Requires: ifplugd ethtool iproute2
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: alterator-hw-functions (show other bugs)
Version: unstable
Hardware: all Linux
: P3 normal
Assignee: Mikhail Efremov
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-09-06 22:54 MSD by Alexei V. Mezin
Modified: 2011-10-05 00:31 MSK (History)
6 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alexei V. Mezin 2010-09-06 22:54:13 MSD
Пишет "сетевой провод не подсоединен" даже когда все подсоединено, и ethtool показывает link beat detected. 

(воспроизводится даже в VirtualBox)

4.6-alt2.M51.1

Версия из Сизифа (4.8-alt1) работает корректно.
Comment 1 Michael Shigorin 2010-09-07 19:00:45 MSD
Пока под рукой(?) -- если попытаться обновить до сизифного, получается/работает?
Comment 2 Michael Shigorin 2010-09-07 19:02:06 MSD
> Версия из Сизифа (4.8-alt1) работает корректно.
...или это понимать именно как "поставил на бранч сизифный пакет, помогло"?
Comment 3 Alexei V. Mezin 2010-09-07 23:31:21 MSD
(В ответ на комментарий №1)
> Пока под рукой(?) -- если попытаться обновить до сизифного,
> получается/работает?

Все сложнее: даже если ничего не обновлять, то в _установленной_ системе работает.

Установил в VB, показывает "кабель подсоединен". Однако, в режиме livecd не работает, что в виртуалке, что на реальной машине.
Comment 4 Michael Shigorin 2010-09-08 11:05:36 MSD
А ethtool с livecd?

Вообще можно заглянуть в /usr/lib/alterator/backend3/net-eth и посмотреть, что происходит в этом случае.
Comment 5 Alexei V. Mezin 2010-09-09 21:18:42 MSD
(В ответ на комментарий №4)
> А ethtool с livecd?
> 

Проверил при загрузке с livecd в виртуалке. На реальном компе ситуация аналогичная: ethtool показывает Link detected: yes, но в алтераторе "провод отсоединен".
Comment 6 Lenar Shakirov 2011-01-20 12:10:27 MSK
На самом деле проблема в alterator-hw-functions:
http://git.altlinux.org/people/inger/packages/?p=alterator-hw-functions.git;a=blob;f=alterator-hw-functions;h=3293d5108ad3765dc940325dfa27593de6bd7dff;hb=HEAD#l70

В спеке не хватает Requires: ifplugd ethtool iproute2

Поэтому в LiveCD не попал ifplugd (/usr/sbin/ifplugstatus)

Автоматический поиск зависимостей задокументированно(?) зафейлился, т.к. путь к командам вынесен в переменные:
ifplugstatus_tool=/usr/sbin/ifplugstatus
ethtool_tool=/usr/sbin/ethtool
ip_tool=/sbin/ip

2slazav, 2inger, 2sem: было задумано избавить a-hw-func от "лишних" зависимостей?
Comment 7 Michael Shigorin 2011-01-21 01:50:34 MSK
Похоже на то... можно оформить нужное в пакаджлисты дистрибутива, который подразумевает неэкономию на памяти (и не оформлять в lxde-шный, например).
Comment 8 Lenar Shakirov 2011-10-01 23:27:55 MSK
Словил на Simply Linux 6.0.0  (Flounder) i586 build 2011-09-22:
alterator-net-eth говорит "провод отсоединен"

2sem: может таки добавим зависимость на ifplugd?

Либо второй вариант:
добавляем Requires: ethtool (в коде уже используется) и меняем использование ifplugstatus на ethtool eth0:
http://git.altlinux.org/people/snejok/packages/?p=alterator-hw-functions.git;a=commitdiff;h=9f6873508c27329edb9fbb8d4ba7136ce5d8cb66

Test only:
http://git.altlinux.org/tasks/56057/

Судя по всему до этой баги вообще никому нет дела...
Comment 9 Andrey Cherepanov 2011-10-01 23:46:02 MSK
(В ответ на комментарий №8)
> Судя по всему до этой баги вообще никому нет дела...
А вот унывать не надо! Напиши sem@ персонально. У нас на каждого столько багов приходится, что за всеми не уследишь!
Comment 10 Mikhail Efremov 2011-10-02 14:16:02 MSK
> Test only:
> http://git.altlinux.org/tasks/56057/

По-моему правильное изменение. Хотел заапрувить, но там и так
access to alterator-hw-functions ALLOWED for snejok: member of approved group

> Судя по всему до этой баги вообще никому нет дела...

Скорее времени нет.
Comment 11 Michael Shigorin 2011-10-03 20:35:29 MSK
(In reply to comment #8)
> добавляем Requires: ethtool (в коде уже используется) и меняем использование
> ifplugstatus на ethtool eth0:
> http://git.altlinux.org/people/snejok/packages/?p=alterator-hw-functions.git;a=commitdiff;h=9f6873508c27329edb9fbb8d4ba7136ce5d8cb66

Лучше только не "меняем", а "фолбэкаемся, если нет ifplugstatus": он умеет три существующих варианта опроса драйвера, а ethtool -- только один из.
Comment 12 Lenar Shakirov 2011-10-04 20:17:11 MSK
(В ответ на комментарий №11)
> Лучше только не "меняем", а "фолбэкаемся, если нет ifplugstatus": он умеет три
> существующих варианта опроса драйвера, а ethtool -- только один из.

Разумно.
Сделано, попробуйте:
http://git.altlinux.org/tasks/56207/logs/events.1.1.log

И взгляните на код, глаз не режет? :)
http://git.altlinux.org/tasks/56207/gears/100/git?p=git;a=commitdiff;h=a8085764404c72f05e72c6cb57aa5af92e750745
Comment 13 Michael Shigorin 2011-10-04 21:49:22 MSK
(In reply to comment #12)
> Сделано, попробуйте: http://git.altlinux.org/tasks/56207/
На e1000e под рукой сработало при отсутствии и наличии ifplugstatus.

> И взгляните на код, глаз не режет? :)
Не-а, разве что написал бы здесь >&/dev/null и всё :) (не стоит мороки)
Comment 14 Repository Robot 2011-10-04 23:30:51 MSK
alterator-hw-functions-0.7-alt6 -> sisyphus:

* Tue Oct 04 2011 Lenar Shakirov <snejok@altlinux> 0.7-alt6
- first try ifplugstatus, otherwise ethtool:
  * to detect status of cable (closes: #24030)
Comment 15 Michael Shigorin 2011-10-05 00:31:50 MSK
Спасибо!