Summary: | ethtool support somewhat broken? | ||
---|---|---|---|
Product: | Sisyphus | Reporter: | Michael Shigorin <mike> |
Component: | etcnet | Assignee: | Andrew Kornilov <hiddenman> |
Status: | CLOSED NOTABUG | QA Contact: | qa-sisyphus |
Severity: | normal | ||
Priority: | P2 | CC: | dottedmag, inger, ldv, rider, sem, shaba, vseleznv |
Version: | unstable | ||
Hardware: | all | ||
OS: | Linux |
Description
Michael Shigorin
2008-09-01 13:55:57 MSD
Делал я: http://racktables.org/trac/changeset/979 Миша, запиши в одну строчку "autoneg off speed NNN duplex DDD" и вопрос исчерпается. Насчёт "ize" я учту на будущее. (In reply to comment #1) > Делал я: http://racktables.org/trac/changeset/979 Спасибо :-) > Миша, запиши в одну строчку "autoneg off speed NNN duplex DDD" > и вопрос исчерпается. Руками-то он давно бы исчерпался... из альтератора (точнее, libshell::shell-config) неудобно работать, когда всё уже в куче. Багу на etcnet решил повесить не сразу, а сравнив синтаксис этого файла с соседними. Везде по значению на строчку, и IMHO человеку тоже будет не сильно с руки разбираться, почему тут иначе. Тем более что вариант "уже одна строчка" получается частным случаем варианта "значение на строчку" как для ethtool. PS: если ты считаешь неправильным ради однородного синтаксиса устраивать прыжки в ширину в /etc/net -- закрой как NOTABUG/WONTFIX, сделаю руками в одну строчку. Собсно как это выглядит сейчас: backend3/net-eth: http://tinyurl.com/5syfwd /bin/shell-config: http://tinyurl.com/5p99we Семантика строчек в файлах не разнородна, она везде одна и та же. Разнятся программы, которые обрабатывают строчки. В частности, iwconfig обрабатывает более одного параметра, поэтому ему можно таким же образом скармливать их помногу в одной строке файла. shell-config я не понял, что такое и зачем, поэтому во внимание не принимаю. По части управления ethtool в alterator-net-eth/backend3/net-eth позволю себе в очередной раз дать всё тот же избитый экспертный прогноз. Ты даёшь пользователю возможность щёлкать выключателем, смысл которого он часто до конца не понимает. Последствия этого предсказуемы: он будет им щёлкать, когда ему станет скучно, и на ровном месте появятся дополнительные 5-10 процентов обращений за поддержкой из-за "ошибок на порту". Пока что ты можешь сделать так, чтобы их не было. В современном мире autoneg работает, а количество портов на 100 и 1000 примерно равно, поэтому ошибкой будет насильно ставить любую из этих скоростей. Выбрось порцию ethtool из этого модуля совсем, а вместо неё добавь две строки в руководство пользователя. (In reply to comment #5) > В современном мире autoneg работает К сожалению, в данном случае запрос поступил от заказчика, которому в несовременном Крыму ethernet-провайдер настоятельно рекомендует прибивать 10... Ладно, спасибо и на том. (In reply to comment #6) > несовременном Крыму ethernet-провайдер настоятельно рекомендует прибивать 10... Как раз замечательный пример для опций, упоминаемых только в README. Этот конкретный запрос реализуется одним echo, причём вышенаписанное менее актуальным не делает. Let's agree to disagree. (In reply to comment #8) > Let's agree to disagree. Да я-то с тобой и так согласен :-) > всё тот же избитый экспертный прогноз BTW практика довольно проста: обычные пользователи не лазят по табам, даже если это нужно. Нащёлкать... надо будет глянуть на маке, на винде IIRC параметрами линка управлять дают. По-хорошему бы вынести такой код в какой alterator-ethtool, который по умолчанию не ставить, а если поставлен -- врезался бы в интерфейс alterator-net-eth. Не знаю только, возможно ли такое занедорого у нас сейчас. |