Bug 15273 - ifread uses hal-find-by-property(1) where ip(8) is enough
Summary: ifread uses hal-find-by-property(1) where ip(8) is enough
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: alterator-net-common (show other bugs)
Version: unstable
Hardware: all Linux
: P2 enhancement
Assignee: inger@altlinux.org
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-04-08 17:12 MSD by Michael Shigorin
Modified: 2009-04-14 17:27 MSD (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Shigorin 2008-04-08 17:12:26 MSD
Предлагаю с целью уменьшения неоправданных по весу зависимостей заменить в
alterator-net-common/ifread вызов hal-find-by-property заради MAC-адреса
интерфейса на:

ip li sh dev $iface | fgrep 'link/ether' | sed 's/^ *link\/ether //' | cut
--delimiter=' ' --fields=1

(пример взят из /etc/net/scripts/contrib/initconf)
Comment 1 Michael Shigorin 2008-06-21 14:21:34 MSD
Всё-таки настаиваю на том, чтобы отодрать отсюда hal.  Обоснование: попадает в install2 и Server.  Ни там, ни там обязательным являться не должен, как мне кажется.

Пожалуй, для M40 сделаю сам.
Comment 2 Dmitry V. Levin 2008-06-21 16:05:14 MSD
Тогда уж
sed -n 's|.*[[:space:]]link/ether[[:space:]]\+\(\([0-9a-f][0-9a-f]:\)\{5\}[0-9a-f][0-9a-f]\).*|\1|p'

2inger: Есть причина, по которой hal лучше чем ip?
Comment 3 Vladislav Zavjalov 2008-06-21 16:05:52 MSD
еще насчет hal в install2 -- сейчас его требует новый alterator-x11 (который использует libXiconfig)
Comment 4 Michael Shigorin 2008-06-22 19:34:11 MSD
(In reply to comment #3)
> еще насчет hal в install2 -- сейчас его требует новый alterator-x11
В Server он не нужен, хотя мне почему-то кажется, что hal стоит как можно позже тащить в инсталер (или хотя бы запускать после подключения свопа).  По крайней мере 4.0 нормально встаёт на машинках с 64M RAM.

PS: собсно если не ошибаюсь -- багу вешал как раз собирая Server 4.0.x.
Comment 5 inger@altlinux.org 2008-06-23 11:20:14 MSD
(In reply to comment #2)
> Тогда уж
> sed -n 's|.*[[:space:]]link/ether[[:space:]]\+\(\([0-9a-f][0-9a-f]:\)\{5\}[0-9a-f][0-9a-f]\).*|\1|p'
> 
> 2inger: Есть причина, по которой hal лучше чем ip?
Раньше хал решал две задачи: выдача имени сетевухи и отделение виртуальных интерфейсов от физических. В обоих случаях он был лучше чем ip.

Другое дело, что сейчас первая фича не используется ... но остаётся вторая.

> 

Comment 6 inger@altlinux.org 2008-06-23 11:21:31 MSD
(In reply to comment #4)
> (In reply to comment #3)
> > еще насчет hal в install2 -- сейчас его требует новый alterator-x11
> В Server он не нужен, хотя мне почему-то кажется, что hal стоит как можно позже
> тащить в инсталер (или хотя бы запускать после подключения свопа).  По
> крайней мере 4.0 нормально встаёт на машинках с 64M RAM.
> 
> PS: собсно если не ошибаюсь -- багу вешал как раз собирая Server 4.0.x.
А что hal потребляет так много памяти? Что-то мне не верится. Гораздо больше памяти ест xorg и qt-4.4.

Comment 7 Michael Shigorin 2008-06-24 17:36:46 MSD
(In reply to comment #5)
> > 2inger: Есть причина, по которой hal лучше чем ip?
> Раньше хал решал две задачи: выдача имени сетевухи и отделение виртуальных
> интерфейсов от физических. В обоих случаях он был лучше чем ip.
> Другое дело, что сейчас первая фича не используется ... но остаётся вторая.
А чем для второй лучше -- это про bug #13350 (vlan)?

(In reply to comment #6)
> А что hal потребляет так много памяти? Что-то мне не верится. 
> Гораздо больше памяти ест xorg и qt-4.4.
В M40 (а вариант alterator-net-* с использованием hal попал и в M40 -- уже после выпуска Server 4.0.1) -- qt 4.3.  Насчёт 4.4 тоже в какой-то момент подумал, что не много ли блоата напихали... но это будет мозолить к Server 4.1, если он вообще будет ;-)

Дело не столько в том, что "так много", сколько в том, что "в дополнение к" и при этом в отличие от -- не являясь необходимым.

Ещё в установленном Server болтается лишний сервис в итоге.

Меня это интересует постольку, поскольку думаю выпустить (неофициальный) Server 4.0.2 по производственным нуждам (собственно, сейчас использую свою опубликованную и анонсированную сборку, в которой и наблюдается эта проблема).
Comment 8 Michael Shigorin 2008-06-25 18:40:18 MSD
Также:

--- inger@
>> Извините, что встреваю, но в стабильном бранче намертво виснет   
>> alterator-net-eth =). Ни через веб, ни с помощью alterator-cmdline
>> от него добиться ничего невозможно. Это alterator-net-eth-0.9-alt1.
>  На самом деле это был alterator-net-eth-2.0-alt3.
>  Помог ручной перезапуск dbus-daemon и haldaemon после апгрейда.
>  Вероятно, этот модуль что-то от них хочет.
От хала берётся список интерфейсов через command-line утилитки.
--- http://lists.altlinux.org/pipermail/devel-conf/2008-May/005063.html
Comment 9 inger@altlinux.org 2009-04-14 17:27:31 MSD
теперь в alterator-hw-functions hal используется ровно в одном месте. Я не знаю простого способа исправить это место без использования hal ;)