On Thu, Sep 07, 2017 at 12:03:43AM +0200, Konstantin Lepikhov wrote:
> Hi Dmitry!
> 
> On 09/06/2017, at 04:37:35 PM you wrote:
> 
> > 
> > содержимое ucode_handled дальше только проверяется на непустоту,
> > можно записывать туда что-то более простое, например, ucode_handled=1
> > 
> ok
> 
> > > +	set +f
> > 
> > set +f дальше не используется и, видимо, не нужен.
> ну как бы readlink может обломаться, если у нас нет /boot/vmlinuz.

Это же +f, а не +e.

> Я не знаю, эта проверка есть в handle_kernel, я ее добавил по аналогии.
> 
> > 
> > > +	f="$(readlink -e /boot/vmlinuz)"
> > > +	if [ -n "$f" -a -n "$kernel_versions_handled" ]; then
> > > +		VERSION=${f#$VMLINUZ_PREFIX-}
> > > +		case "$kernel_version_handled" in
> > > +			"* $VERSION*")
> > > +			ucode_handled=
> > > +			;;
> > > +		esac
> > > +	fi
> > 
> > Это лучше делать всего один раз после обработки цикла по файлам.
> > Хотя почему это исключение сделано именно для /boot/vmlinuz, неочевидно.
> я тут не очень понял логику, которая ниже по коду - если у нас выставлен
> INITRD_AUTOUPDATE=all это значит что будет обновлятся initramfs для всех
> ядер, или только тех, что попали в обновление?

Судя по всему, для всех (commit 0.4.10-alt3~2), хотя зачем понадобилась
такая странная логика, я не знаю.

> Почему там еще раз проверяется является ли файл /boot/vmlinuz?

Чтобы не обновить дважды (commit 0.4.10-alt3^0).


-- 
ldv