On Tue, Feb 12, 2002 at 10:58:15PM +0800, Вадим Илларионов wrote:
> В скрипте /etc/rc.d/scripts/idetune имеется ряд серьёзных недоработок.
> 
> 1) Конструкция
> 	SourceIfNotEmpty /etc/sysconfig/harddisks && sourced=1
> 	SourceIfNotEmpty /etc/sysconfig/harddisk/hd$i && sourced=1
>    пытается подставить параметры из двух файлов, буде таковые имеются.
>    Не лучше ль для начал глянуть в harddisk/hd$i, а уж если такового
>    не окажется, вставить общие для всех параметры из harddisks?

Нет, нужно прочитать общие настройки, затем переопредилить их специальными.

> 2) В разных секциях перечисляются общие для винтов и сидюков параметры.
>    Не вынести ли специфичные для винта - в одну, сидюка - в другую,
>    общие же скомпоновать?

Возможно.

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

Ok, но это надо тестировать.

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

Стоит. Посмотрите код внимательнее.

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

По моему, это лишнее.

> Переработанный с учётом вышеизложенного, потерявший в весе, и тем не менее
> набравший в функциональности скрипт прилагается.

Однако потеряна прежняя функциональность.


Regards,
	Dmitry

+-------------------------------------------------------------------------+
Dmitry V. Levin     mailto://ldv@alt-linux.org
ALT Linux Team      http://www.altlinux.com/
Fandra Project      http://www.fandra.org/
+-------------------------------------------------------------------------+
UNIX is user friendly. It's just very selective about who its friends are.