Re: [sisyphus] Настройка винтов-сидюков при запуске

["Вадим Илларионов" <DIMMeach@NewMail.ru>]

On Tue, 12 Feb 2002 18:12:17 +0300
"Dmitry V. Levin" <ldv@alt-linux.org> wrote:

> > 1) Конструкция
> > 	SourceIfNotEmpty /etc/sysconfig/harddisks && sourced=1
> > 	SourceIfNotEmpty /etc/sysconfig/harddisk/hd$i && sourced=1
> >    пытается подставить параметры из двух файлов, буде таковые имеются.
> >    Не лучше ль для начал глянуть в harddisk/hd$i, а уж если такового
> >    не окажется, вставить общие для всех параметры из harddisks?
> 
> Нет, нужно прочитать общие настройки, затем переопредилить их специальными.

Да, возможно, эту часть (наименьшую по переделке) стОит оставить ASIS.
А в остальном, если в общие настройки внести специфичную для сидюка SPEED=XX,
инит-скрипт обработает её только для сидюка. То же о винтовой опции MULTIPLE_IO
с точностью до наоборот. Так что отделять имеет смысл только те опции,
с которыми глючат отдельные устройства.

> > 3) Не определить ли автоматом разнящийся у разных устройств параметр
> > 	-c$EIDE_32BIT
> >    вместо того, чтоб искать его вручную, пихать затем в конфиг?
> 
> Ok, но это надо тестировать.

Можно ведь и не определять. Или определять по нулям. :)
Например, для винтов, которые возражали бы против указания им $MULTIPLE_IO
равным MaxMultSect из показаний hdparm -i.

> > 4) СтОит ли вносить модификатор
> > 	&& [ "$PARAM" -ge 0 ]
> >    если в конфиг-файле стоИт либо PARAM= либо не стоит? ;)
> >    А ежели вдруг ПОНАДОБИЛОСЬ глючному винту отрубить DMA/LBA/ИТД?
> 
> Стоит. Посмотрите код внимательнее.

Может быть, я проглядел, и существует опасность подстановки отрицательных,
завышенных или нецелочисленных значений? То есть, будут ли они восприняты?
В противном случае, если значение существует [ -n "$PARAM" ], этого вполне
достаточно, чтобы его подставить. А вот если его необходимо отключить,
силком выставив в нуль, && [ "$PARAM" -ge 0 ] подобного не допустит.

> > 5) Не оформить ли маловразумительное "Setting parameters for hd$i"
> >    как "Setting parameters for Primary|Secondary|etc Master|Slave"?
> >    Мелочь, а приятно.
> 
> По моему, это лишнее.

Как и графический индикатор загрузки/выгрузки Мастера. ;)
Красота вообще вещь непрактичная.
 
> > Переработанный с учётом вышеизложенного, потерявший в весе, и тем не менее
> > набравший в функциональности скрипт прилагается.
> 
> Однако потеряна прежняя функциональность.

Цитирую:
> Ok, но это надо тестировать.
:)

Искренне свой,
	Дымыч.