From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <ldv@alt-linux.org>
Date: Tue, 12 Feb 2002 18:12:17 +0300
From: "Dmitry V. Levin" <ldv@alt-linux.org>
To: ALT Linux Sisyphus mailing list <sisyphus@altlinux.ru>
Subject: Re: [sisyphus] =?koi8-r?B?7sHT1NLPysvBINfJ?=
	=?koi8-r?B?ztTP1y3TycTAy8/XINDSySDawdDV08vF?=
Message-ID: <20020212151216.GA26056@ldv.office.alt-linux.org>
Mail-Followup-To: ALT Linux Sisyphus mailing list <sisyphus@altlinux.ru>
References: <20020212225815.6766a521.DIMMeach@NewMail.ru>
Mime-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha1;
	protocol="application/pgp-signature"; boundary="nFreZHaLTZJo0R7j"
Content-Disposition: inline
In-Reply-To: <20020212225815.6766a521.DIMMeach@NewMail.ru>
X-fingerprint: 9658 398D 181B 1200 8FC5  26B8 F6F8 846B C1E2 3429
Sender: sisyphus-admin@altlinux.ru
Errors-To: sisyphus-admin@altlinux.ru
X-BeenThere: sisyphus@altlinux.ru
X-Mailman-Version: 2.0
Precedence: bulk
Reply-To: sisyphus@altlinux.ru
List-Help: <mailto:sisyphus-request@altlinux.ru?subject=help>
List-Post: <mailto:sisyphus@altlinux.ru>
List-Subscribe: <http://altlinux.ru/mailman/listinfo/sisyphus>,
	<mailto:sisyphus-request@altlinux.ru?subject=subscribe>
List-Id: <sisyphus.altlinux.ru>
List-Unsubscribe: <http://altlinux.ru/mailman/listinfo/sisyphus>,
	<mailto:sisyphus-request@altlinux.ru?subject=unsubscribe>
List-Archive: <http://altlinux.ru/pipermail/sisyphus/>
Archived-At: <http://lore.altlinux.org/sisyphus/20020212151216.GA26056@ldv.office.alt-linux.org/>
List-Archive: <http://lore.altlinux.org/sisyphus/>

--nFreZHaLTZJo0R7j
Content-Type: text/plain; charset=koi8-r
Content-Disposition: inline
Content-Transfer-Encoding: 8bit

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.

--nFreZHaLTZJo0R7j
Content-Type: application/pgp-signature
Content-Disposition: inline

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.0.6 (GNU/Linux)
Comment: For info see http://www.gnupg.org

iD8DBQE8aTDQ9viEa8HiNCkRAouXAJ9DtvULMVVMe++VVRmhFthEgrYVnwCfRhu9
ucDXnVRSXLm6DCTp2kAK0po=
=/MxH
-----END PGP SIGNATURE-----

--nFreZHaLTZJo0R7j--