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

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

[Вадим Илларионов]

[-- Attachment #1: Type: text/plain, Size: 1243 bytes --]

В скрипте /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
   вместо того, чтоб искать его вручную, пихать затем в конфиг?
4) СтОит ли вносить модификатор
	&& [ "$PARAM" -ge 0 ]
   если в конфиг-файле стоИт либо PARAM= либо не стоит? ;)
   А ежели вдруг ПОНАДОБИЛОСЬ глючному винту отрубить DMA/LBA/ИТД?
5) Не оформить ли маловразумительное "Setting parameters for hd$i"
   как "Setting parameters for Primary|Secondary|etc Master|Slave"?
   Мелочь, а приятно.

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


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

P.S. Теперь точно - прилагается.

[-- Attachment #2: idetune --]
[-- Type: application/octet-stream, Size: 1989 bytes --]

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

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 1903 bytes --]

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.

[-- Attachment #2: Type: application/pgp-signature, Size: 232 bytes --]

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

[Вадим Илларионов]

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, но это надо тестировать.
:)

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

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

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 1158 bytes --]

On Wed, Feb 13, 2002 at 01:27:05AM +0800, Вадим Илларионов wrote:
> > > 4) СтОит ли вносить модификатор
> > > 	&& [ "$PARAM" -ge 0 ]
> > >    если в конфиг-файле стоИт либо PARAM= либо не стоит? ;)
> > >    А ежели вдруг ПОНАДОБИЛОСЬ глючному винту отрубить DMA/LBA/ИТД?
> > 
> > Стоит. Посмотрите код внимательнее.
> 
> Может быть, я проглядел, и существует опасность подстановки отрицательных,
> завышенных или нецелочисленных значений? То есть, будут ли они восприняты?
> В противном случае, если значение существует [ -n "$PARAM" ], этого вполне
> достаточно, чтобы его подставить. А вот если его необходимо отключить,
> силком выставив в нуль, && [ "$PARAM" -ge 0 ] подобного не допустит.

[ "PARAM" -ge 0 ] 2>/dev/null
- это просто проверка на целочисленность значения.


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.

[-- Attachment #2: Type: application/pgp-signature, Size: 232 bytes --]