Re: [devel] [git update] packages/startup: tags/0.9.8.17-alt2

[Sergey Vlasov <vsu@altlinux.ru>]

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

On Tue, Mar 04, 2008 at 03:24:37PM +0300, Dmitry V. Levin wrote:
> On Tue, Mar 04, 2008 at 04:29:20AM +0300, Alexey Borovskoy wrote:
> > Update of /people/alb/packages/startup.git
> [...]
> > commit 806db97e10b92c271dd9b6120697b7a9c9c4ec0b
> > Author: Alexey Borovskoy <alb@altlinux>
> > Date:   Tue Jan 8 13:55:20 2008 +1200
> > 
> >     Fix LVM startup when / on MD device.
> > 
> > Full diff since `0.9.8.17-alt1' follows:
> > diff --git a/startup/rc.d/rc.sysinit b/startup/rc.d/rc.sysinit
> > index c11f197..24fd217 100755
> > --- a/startup/rc.d/rc.sysinit
> > +++ b/startup/rc.d/rc.sysinit
> > @@ -263,12 +263,12 @@ fi
> >  
> >  # Load device-mapper, create its device file
> >  if [ -e /proc/devices -a -e /proc/misc ]; then
> > -	rm -f /dev/mapper/control
> >  	modprobe dm-mod >/dev/null 2>&1
> >  	if [ -z "$RUN_UDEV" ]; then
> >  		MAJOR="$(sed -ne 's/^ *\([0-9]\+\) \+misc$/\1/p' /proc/devices)"
> >  		MINOR="$(sed -ne 's/^ *\([0-9]\+\) \+device-mapper$/\1/p' /proc/misc)"
> >  		if [ -n "$MAJOR" -a -n "$MINOR" ]; then
> > +	                rm -f /dev/mapper/control
> >  			mkdir -p -m755 /dev/mapper
> >  			action "Creating device-mapper character device ($MAJOR, $MINOR):" \
> >  				mknod --mode=600 /dev/mapper/control c "$MAJOR" "$MINOR"
> 
> Можно пояснить, что именно исправляет это изменение?

Если модуль dm-mod в этот момент уже был загружен (например, из
initrd), и при этом udevd был запущен из rc.sysinit (RUN_UDEV=1), в
результате выполнения этого кода файл устройства /dev/mapper/control
удалялся, что приводило к ошибкам при последующем запуске lvm.
Конечно, имеющаяся запись changelog никуда не годится - MD не имеет
никакого отношения к данной проблеме.

В новом варианте, правда, всё равно остаётся race - в случае
RUN_UDEV=1 файл устройства должен создавать udevd, но ожидание его
создания не производится.

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